As reported in #42712, our usage of called_with is incorrect, thus, it was not testing what we intended.

Unfortunately, I have not been able to fix it in any of the cases.

Author: armenzg

tests/sentry/tasks/test_derive_code_mappings.py

@@ -39,10 +39,6 @@ def generate_data(self, frames: List[Dict[str, Union[str, bool]]], platform: str
 class TestTaskBehavior(BaseDeriveCodeMappings):
     """Test task behavior that is not language specific."""
 
-    @pytest.fixture(autouse=True)
-    def inject_fixtures(self, caplog):
-        self._caplog = caplog
-
     def setUp(self):
         super().setUp()
         self.platform = "any"
@@ -57,11 +53,9 @@ def test_does_not_raise_installation_removed(self):
             side_effect=ApiError(
                 '{"message":"Not Found","documentation_url":"https://docs.github.com/rest/reference/apps#create-an-installation-access-token-for-an-app"}'
             ),
-        ), patch("sentry.integrations.utils.code_mapping.logger") as logger:
+        ):
             assert derive_code_mappings(self.project.id, self.event_data) is None
 
-        assert logger.warning.called_with("The org has uninstalled the Sentry App.")
-
     def test_raises_other_api_errors(self):
         with patch(
             "sentry.integrations.github.client.GitHubClientMixin.get_trees_for_org",
@@ -74,12 +68,10 @@ def test_unable_to_get_lock(self):
         with patch(
             "sentry.integrations.github.client.GitHubClientMixin.get_trees_for_org",
             side_effect=UnableToAcquireLock,
-        ), patch("sentry.integrations.utils.code_mapping.logger") as logger:
+        ):
             with pytest.raises(UnableToAcquireLock):
                 derive_code_mappings(self.project.id, self.event_data)
 
-            assert logger.warning.called_with("derive_code_mappings.getting_lock_failed")
-
 
 class TestJavascriptDeriveCodeMappings(BaseDeriveCodeMappings):
     def setUp(self):