🔧 chore: remove extra jira logs (#93192)

Author: iamrajjoshi

src/sentry/middleware/integrations/parsers/jira.py

@@ -61,17 +61,11 @@ def get_response(self):
         regions = self.get_regions_from_organizations()
 
         if len(regions) == 0:
-            logger.info("%s.no_regions", self.provider, extra={"path": self.request.path})
             return self.get_default_missing_integration_response()
 
         if len(regions) > 1:
-            # Since Jira is region_restricted (see JiraIntegrationProvider) we can just pick the
-            # first region to forward along to.
-            logger.info(
-                "%s.too_many_regions",
-                self.provider,
-                extra={"path": self.request.path, "regions": regions},
-            )
+            # This shouldn't happen because we block multi region install at the install time.
+            raise ValueError("Jira integration is installed in multiple regions")
 
         if self.view_class in self.immediate_response_region_classes:
             try:

tests/sentry/middleware/integrations/parsers/test_jira.py

@@ -2,6 +2,7 @@
 
 from unittest.mock import patch
 
+import pytest
 import responses
 from django.http import HttpRequest, HttpResponse
 from django.test import RequestFactory, override_settings
@@ -201,11 +202,8 @@ def test_get_response_multiple_regions(self):
 
         with patch.object(parser, "get_integration_from_request") as method:
             method.return_value = integration
-            response = parser.get_response()
+            # assert ValueError is raised if the integration is not valid
+            with pytest.raises(ValueError):
+                parser.get_response()
 
-        # Response should go to first region
-        assert isinstance(response, HttpResponse)
-        assert response.status_code == status.HTTP_200_OK
-        assert response.content == b"region response"
-        assert len(responses.calls) == 1
         assert_no_webhook_payloads()