lint

Author: jeremydvoss

instrumentation/opentelemetry-instrumentation-fastapi/tests/test_fastapi_instrumentation.py

@@ -1095,10 +1095,6 @@ def test_instruments_with_fastapi_installed(self, mock_logger):
         mock_distro = Mock()
         mock_distro.load_instrumentor.return_value = None
         _load_instrumentors(mock_distro)
-        print(
-            "mock_distro.load_instrumentor.call_args_list: %s"
-            % mock_distro.load_instrumentor.call_args_list
-        )
         self.assertEqual(len(mock_distro.load_instrumentor.call_args_list), 1)
         (ep,) = mock_distro.load_instrumentor.call_args.args
         self.assertEqual(ep.name, "fastapi")
@@ -1131,7 +1127,6 @@ def test_instruments_with_old_fastapi_installed(
                 dependency_conflict,
             ),
         )
-        # print("mock_distro.load_instrumentor.call_args_list: %s" % mock_distro.load_instrumentor.call_args_list)
         # self.assertEqual(len(mock_distro.load_instrumentor.call_args_list), 1)
         # (ep,) = mock_distro.load_instrumentor.call_args.args
         # self.assertEqual(ep.name, "fastapi")
@@ -1167,7 +1162,6 @@ def test_instruments_without_fastapi_installed(
                 dependency_conflict,
             ),
         )
-        # print("mock_distro.load_instrumentor.call_args_list: %s" % mock_distro.load_instrumentor.call_args_list)
         # self.assertEqual(len(mock_distro.load_instrumentor.call_args_list), 1)
         # (ep,) = mock_distro.load_instrumentor.call_args.args
         # self.assertEqual(ep.name, "fastapi")

opentelemetry-instrumentation/tests/test_dependencies.py

@@ -130,7 +130,7 @@ def requires(self):
         def version_side_effect(package_name):
             if package_name == "foo":
                 raise PackageNotFoundError("foo not found")
-            elif package_name == "bar":
+            if package_name == "bar":
                 return "1.0.0"
             raise PackageNotFoundError(f"{package_name} not found")
 
@@ -190,9 +190,9 @@ def requires(self):
         def version_side_effect(package_name):
             if package_name == "foo":
                 return "1.2.0"
-            elif package_name == "bar":
+            if package_name == "bar":
                 raise PackageNotFoundError("bar not found")
-            elif package_name == "baz":
+            if package_name == "baz":
                 return "3.7.0"
             raise PackageNotFoundError(f"{package_name} not found")
 
@@ -226,9 +226,9 @@ def requires(self):
         def version_side_effect(package_name):
             if package_name == "foo":
                 raise PackageNotFoundError("foo not found")
-            elif package_name == "bar":
+            if package_name == "bar":
                 raise PackageNotFoundError("bar not found")
-            elif package_name == "baz":
+            if package_name == "baz":
                 return "3.7.0"
             raise PackageNotFoundError(f"{package_name} not found")
 
@@ -267,9 +267,9 @@ def requires(self):
         def version_side_effect(package_name):
             if package_name == "foo":
                 return "1.7.0"
-            elif package_name == "bar":
+            if package_name == "bar":
                 raise PackageNotFoundError("bar not found")
-            elif package_name == "baz":
+            if package_name == "baz":
                 raise PackageNotFoundError("baz not found")
             raise PackageNotFoundError(f"{package_name} not found")