fix: Corrected bug where .replace() was treated as an in-place method

Signed-off-by: Varsha GS <[email protected]>

Author: GSVarsha

.circleci/config.yml

@@ -220,7 +220,7 @@ jobs:
           requirements: "tests/requirements-gevent-starlette.txt"
       - run-tests-with-coverage-report:
           gevent: "true"
-          tests: "tests/frameworks/test_gevent.py tests/frameworks/test_starlette.py"
+          tests: "tests/frameworks/test_starlette.py tests/frameworks/test_gevent.py"
       - store-pytest-results
       - store-coverage-report
 

src/instana/__init__.py

@@ -83,7 +83,7 @@ def key_to_bool(k: str) -> bool:
         import inspect
 
         all_accepted_patch_all_args = inspect.getfullargspec(monkey.patch_all)[0]
-        provided_options.replace(" ", "").replace("--", "").split(",")
+        provided_options = provided_options.replace(" ", "").replace("--", "").split(",")
 
         provided_options = [
             k for k in provided_options if short_key(k) in all_accepted_patch_all_args

tests/conftest.py

@@ -28,6 +28,7 @@
 collect_ignore_glob = [
     "*collector/test_gcr*",
     "*agent/test_google*",
+    "*test_gevent_autotrace*"
 ]
 
 # ppc64le and s390x have limitations with some supported libraries.
@@ -53,7 +54,7 @@
 if not os.environ.get("GEVENT_STARLETTE_TEST"):
     collect_ignore_glob.extend(
         [
-            "*test_gevent*",
+            "*test_gevent.py",
             "*test_starlette*",
         ]
     )

tests/frameworks/test_starlette.py

@@ -29,7 +29,6 @@ def _resource(self) -> Generator[None, None, None]:
         # Clear all spans before a test run.
         self.recorder = tracer.span_processor
         self.recorder.clear_spans()
-        yield
 
     def test_vanilla_get(self) -> None:
         result = self.client.get("/")