test: Add stack trace env config tests to options

Signed-off-by: Varsha GS <varsha.gs@ibm.com>

Author: GSVarsha

src/instana/options.py

@@ -140,11 +140,7 @@ def set_trace_configurations(self) -> None:
             try:
                 length = int(os.environ["INSTANA_STACK_TRACE_LENGTH"])
                 if length >= 1:
-                    self.stack_trace_length = min(length, 40)  # Enforce max of 40
-                    if length > 40:
-                        logger.warning(
-                            f"INSTANA_STACK_TRACE_LENGTH of {length} exceeds maximum of 40. Using 40."
-                        )
+                    self.stack_trace_length = length
                 else:
                     logger.warning(
                         "INSTANA_STACK_TRACE_LENGTH must be positive. Using default 30"

tests/test_options.py

@@ -637,4 +637,159 @@ def test_gcr_options_with_env_vars(self) -> None:
         assert self.gcr_options.log_level == logging.INFO
 
 
-# Made with Bob
+class TestStackTraceConfiguration:
+    """Test stack trace configuration options."""
+
+    @pytest.fixture(autouse=True)
+    def _resource(self) -> Generator[None, None, None]:
+        self.options = None
+        yield
+        if "tracing" in config.keys():
+            del config["tracing"]
+
+    def test_stack_trace_defaults(self) -> None:
+        """Test default stack trace configuration."""
+        self.options = BaseOptions()
+        
+        assert self.options.stack_trace_level == "all"
+        assert self.options.stack_trace_length == 30
+
+    @patch.dict(
+        os.environ,
+        {"INSTANA_STACK_TRACE": "error"},
+    )
+    def test_stack_trace_level_env_var_error(self) -> None:
+        """Test INSTANA_STACK_TRACE environment variable with 'error'."""
+        self.options = BaseOptions()
+        
+        assert self.options.stack_trace_level == "error"
+        assert self.options.stack_trace_length == 30  # Default
+
+    @patch.dict(
+        os.environ,
+        {"INSTANA_STACK_TRACE": "none"},
+    )
+    def test_stack_trace_level_env_var_none(self) -> None:
+        """Test INSTANA_STACK_TRACE environment variable with 'none'."""
+        self.options = BaseOptions()
+        
+        assert self.options.stack_trace_level == "none"
+        assert self.options.stack_trace_length == 30  # Default
+
+    @patch.dict(
+        os.environ,
+        {"INSTANA_STACK_TRACE": "all"},
+    )
+    def test_stack_trace_level_env_var_all(self) -> None:
+        """Test INSTANA_STACK_TRACE environment variable with 'all'."""
+        self.options = BaseOptions()
+        
+        assert self.options.stack_trace_level == "all"
+        assert self.options.stack_trace_length == 30  # Default
+
+    @patch.dict(
+        os.environ,
+        {"INSTANA_STACK_TRACE": "INVALID"},
+    )
+    def test_stack_trace_level_env_var_invalid(
+        self,
+        caplog: pytest.LogCaptureFixture,
+    ) -> None:
+        """Test INSTANA_STACK_TRACE with invalid value falls back to default."""
+        caplog.set_level(logging.WARNING, logger="instana")
+        self.options = BaseOptions()
+        
+        assert self.options.stack_trace_level == "all"  # Falls back to default
+        assert any(
+            "Invalid INSTANA_STACK_TRACE value" in message
+            for message in caplog.messages
+        )
+
+    @patch.dict(
+        os.environ,
+        {"INSTANA_STACK_TRACE_LENGTH": "25"},
+    )
+    def test_stack_trace_length_env_var(self) -> None:
+        """Test INSTANA_STACK_TRACE_LENGTH environment variable."""
+        self.options = BaseOptions()
+        
+        assert self.options.stack_trace_level == "all"  # Default
+        assert self.options.stack_trace_length == 25
+
+    @patch.dict(
+        os.environ,
+        {"INSTANA_STACK_TRACE_LENGTH": "0"},
+    )
+    def test_stack_trace_length_env_var_zero(
+        self,
+        caplog: pytest.LogCaptureFixture,
+    ) -> None:
+        """Test INSTANA_STACK_TRACE_LENGTH with zero value."""
+        caplog.set_level(logging.WARNING, logger="instana")
+        self.options = BaseOptions()
+        
+        assert self.options.stack_trace_length == 30  # Falls back to default
+        assert any(
+            "must be positive" in message for message in caplog.messages
+        )
+
+    @patch.dict(
+        os.environ,
+        {"INSTANA_STACK_TRACE_LENGTH": "-5"},
+    )
+    def test_stack_trace_length_env_var_negative(
+        self,
+        caplog: pytest.LogCaptureFixture,
+    ) -> None:
+        """Test INSTANA_STACK_TRACE_LENGTH with negative value."""
+        caplog.set_level(logging.WARNING, logger="instana")
+        self.options = BaseOptions()
+        
+        assert self.options.stack_trace_length == 30  # Falls back to default
+        assert any(
+            "must be positive" in message for message in caplog.messages
+        )
+
+    @patch.dict(
+        os.environ,
+        {"INSTANA_STACK_TRACE_LENGTH": "invalid"},
+    )
+    def test_stack_trace_length_env_var_invalid(
+        self,
+        caplog: pytest.LogCaptureFixture,
+    ) -> None:
+        """Test INSTANA_STACK_TRACE_LENGTH with invalid value."""
+        caplog.set_level(logging.WARNING, logger="instana")
+        self.options = BaseOptions()
+        
+        assert self.options.stack_trace_length == 30  # Falls back to default
+        assert any(
+            "Invalid INSTANA_STACK_TRACE_LENGTH" in message
+            for message in caplog.messages
+        )
+
+    @patch.dict(
+        os.environ,
+        {
+            "INSTANA_STACK_TRACE": "error",
+            "INSTANA_STACK_TRACE_LENGTH": "15",
+        },
+    )
+    def test_stack_trace_both_env_vars(self) -> None:
+        """Test both INSTANA_STACK_TRACE and INSTANA_STACK_TRACE_LENGTH."""
+        self.options = BaseOptions()
+        
+        assert self.options.stack_trace_level == "error"
+        assert self.options.stack_trace_length == 15
+
+    @patch.dict(
+        os.environ,
+        {
+            "INSTANA_STACK_TRACE": "ERROR",  # Test case insensitivity
+        },
+    )
+    def test_stack_trace_level_case_insensitive(self) -> None:
+        """Test INSTANA_STACK_TRACE is case insensitive."""
+        self.options = BaseOptions()
+        
+        assert self.options.stack_trace_level == "error"