Skip to content

Commit 5eba750

Browse files
committed
fix: Don't run tagger integration tests without GCP setup
1 parent 0629258 commit 5eba750

File tree

1 file changed

+7
-1
lines changed

1 file changed

+7
-1
lines changed

projects/policyengine-apis-integ/tests/tagger/test_cleanup.py

Lines changed: 7 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -10,11 +10,16 @@
1010

1111

1212
@pytest.mark.beta_only
13+
@pytest.mark.requires_gcp
1314
class TestCleanupEndpoint:
1415
"""
1516
Tests for the /cleanup endpoint.
1617
17-
These tests are marked beta_only because:
18+
These tests are marked:
19+
- requires_gcp: They need real GCP credentials (excluded from PR tests)
20+
- beta_only: They should only run on beta deployments (excluded from prod)
21+
22+
Rationale:
1823
1. They interact with real Cloud Run traffic configuration
1924
2. They modify the deployments manifest
2025
3. While designed to be safe, they should not run against production
@@ -81,6 +86,7 @@ def test_cleanup_rejects_invalid_keep_count(self, tagger_client: httpx.Client):
8186

8287

8388
@pytest.mark.beta_only
89+
@pytest.mark.requires_gcp
8490
class TestCleanupSafeguards:
8591
"""
8692
Tests to verify cleanup safeguards are working.

0 commit comments

Comments
 (0)