Fix silent no-op on repository migration updates

Co-authored-by: Armen Zambrano G. <armenzg@users.noreply.github.com>

Author: cursoragent

src/sentry/integrations/api/endpoints/organization_repository_details.py

@@ -81,27 +81,26 @@ def put(self, request: Request, organization: Organization, repo_id) -> Response
                 update_kwargs["status"] = ObjectStatus.HIDDEN
             else:
                 raise NotImplementedError
-        if update_kwargs:
-            old_status = repo.status
-            with transaction.atomic(router.db_for_write(Repository)):
-                repo.update(**update_kwargs)
-                if (
-                    old_status == ObjectStatus.PENDING_DELETION
-                    and repo.status == ObjectStatus.ACTIVE
-                ):
-                    repo.reset_pending_deletion_field_names()
-                    repo.delete_pending_deletion_option()
-                elif repo.status == ObjectStatus.HIDDEN and old_status != repo.status:
-                    repository_cascade_delete_on_hide.apply_async(kwargs={"repo_id": repo.id})
-
-                    if repo.external_id and repo.provider:
-                        cleanup_seer_repository_preferences.apply_async(
-                            kwargs={
-                                "organization_id": repo.organization_id,
-                                "repo_external_id": repo.external_id,
-                                "repo_provider": repo.provider,
-                            }
-                        )
+        if not update_kwargs:
+            return Response({"detail": "No valid update fields were provided"}, status=400)
+
+        old_status = repo.status
+        with transaction.atomic(router.db_for_write(Repository)):
+            repo.update(**update_kwargs)
+            if old_status == ObjectStatus.PENDING_DELETION and repo.status == ObjectStatus.ACTIVE:
+                repo.reset_pending_deletion_field_names()
+                repo.delete_pending_deletion_option()
+            elif repo.status == ObjectStatus.HIDDEN and old_status != repo.status:
+                repository_cascade_delete_on_hide.apply_async(kwargs={"repo_id": repo.id})
+
+                if repo.external_id and repo.provider:
+                    cleanup_seer_repository_preferences.apply_async(
+                        kwargs={
+                            "organization_id": repo.organization_id,
+                            "repo_external_id": repo.external_id,
+                            "repo_provider": repo.provider,
+                        }
+                    )
 
         return Response(serialize(repo, request.user))
 

tests/sentry/integrations/api/endpoints/test_organization_repository_details.py

@@ -323,7 +323,7 @@ def test_put_hide_repo_with_commits(self, mock_cleanup_task: MagicMock) -> None:
             }
         )
 
-    def test_put_does_not_change_provider(self) -> None:
+    def test_put_rejects_integration_id(self) -> None:
         self.login_as(user=self.user)
 
         org = self.create_organization(owner=self.user, name="baz")
@@ -339,9 +339,10 @@ def test_put_does_not_change_provider(self) -> None:
         )
 
         url = reverse("sentry-api-0-organization-repository-details", args=[org.slug, repo.id])
-        response = self.client.put(url, data={"status": "visible", "integrationId": integration.id})
+        response = self.client.put(url, data={"integrationId": integration.id})
 
-        assert response.status_code == 200
+        assert response.status_code == 400
+        assert response.data["detail"] == "No valid update fields were provided"
 
         repo = Repository.objects.get(id=repo.id)
         assert repo.provider == "integrations:github"