fix(bitbucket): Match GitHub's page_number_limit clamping behavior

Drop the < 1 guard so that zero/negative limits naturally return only
the first page (the while loop never executes), matching how GitHub's
_get_with_pagination handles the same case. Add tests for both the
zero-limit and excessive-limit clamping paths.

Author: jaydgoss

src/sentry/integrations/bitbucket/client.py

@@ -125,11 +125,7 @@ def _get_all_from_paginated(
         Bitbucket uses a ``next`` URL in the response body (not headers)
         to link to the next page.
         """
-        if (
-            page_number_limit is None
-            or page_number_limit < 1
-            or page_number_limit > self.page_number_limit
-        ):
+        if page_number_limit is None or page_number_limit > self.page_number_limit:
             page_number_limit = self.page_number_limit
 
         if params is None:

tests/sentry/integrations/bitbucket/test_integration.py

@@ -123,6 +123,49 @@ def test_get_repositories_respects_page_limit(self) -> None:
         ]
         assert len(responses.calls) == 1
 
+    @responses.activate
+    def test_get_repositories_zero_page_limit_returns_first_page(self) -> None:
+        """A page_number_limit of 0 still returns the first page but fetches no further."""
+        base_url = "https://api.bitbucket.org/2.0/repositories/sentryuser"
+        responses.add(
+            responses.GET,
+            base_url,
+            json={
+                "values": [{"full_name": "sentryuser/repo-1", "uuid": "{r1}"}],
+                "next": f"{base_url}?pagelen=100&page=2",
+            },
+        )
+        responses.add(
+            responses.GET,
+            f"{base_url}?pagelen=100&page=2",
+            json={"values": [{"full_name": "sentryuser/repo-2", "uuid": "{r2}"}]},
+        )
+
+        installation = self.integration.get_installation(self.organization.id)
+        result = installation.get_repositories(page_number_limit=0)
+        assert result == [
+            {"identifier": "sentryuser/repo-1", "name": "sentryuser/repo-1", "external_id": "{r1}"},
+        ]
+        assert len(responses.calls) == 1
+
+    @responses.activate
+    def test_get_repositories_clamps_excessive_page_limit(self) -> None:
+        """A page_number_limit above the class max is clamped to the default."""
+        base_url = "https://api.bitbucket.org/2.0/repositories/sentryuser"
+        responses.add(
+            responses.GET,
+            base_url,
+            json={"values": [{"full_name": "sentryuser/repo-1", "uuid": "{r1}"}]},
+        )
+
+        installation = self.integration.get_installation(self.organization.id)
+        client = installation.get_client()
+        result = installation.get_repositories(page_number_limit=client.page_number_limit + 100)
+        assert result == [
+            {"identifier": "sentryuser/repo-1", "name": "sentryuser/repo-1", "external_id": "{r1}"},
+        ]
+        assert len(responses.calls) == 1
+
     @responses.activate
     def test_get_repositories_exact_match(self) -> None:
         querystring = urlencode({"q": 'name="stuf"'})