Browse Source

fix(hc): Mark some endpoint tests stable (#52489)

These cases pass already without any additional changes needed,
including when running manually with `SENTRY_USE_SPLIT_DBS=1`.
Matt Duncan 1 year ago
parent
commit
2d77ba7456

+ 1 - 1
tests/sentry/api/endpoints/test_user_emails.py

@@ -5,7 +5,7 @@ from sentry.testutils import APITestCase
 from sentry.testutils.silo import control_silo_test
 from sentry.testutils.silo import control_silo_test
 
 
 
 
-@control_silo_test
+@control_silo_test(stable=True)
 class UserEmailsTest(APITestCase):
 class UserEmailsTest(APITestCase):
     def setUp(self):
     def setUp(self):
         super().setUp()
         super().setUp()

+ 1 - 1
tests/sentry/api/endpoints/test_user_emails_confirm.py

@@ -5,7 +5,7 @@ from sentry.testutils import APITestCase
 from sentry.testutils.silo import control_silo_test
 from sentry.testutils.silo import control_silo_test
 
 
 
 
-@control_silo_test
+@control_silo_test(stable=True)
 class UserEmailsConfirmTest(APITestCase):
 class UserEmailsConfirmTest(APITestCase):
     endpoint = "sentry-api-0-user-emails-confirm"
     endpoint = "sentry-api-0-user-emails-confirm"
     method = "post"
     method = "post"

+ 1 - 1
tests/sentry/api/endpoints/test_user_identity.py

@@ -3,7 +3,7 @@ from sentry.testutils import APITestCase
 from sentry.testutils.silo import control_silo_test
 from sentry.testutils.silo import control_silo_test
 
 
 
 
-@control_silo_test
+@control_silo_test(stable=True)
 class UserIdentityTest(APITestCase):
 class UserIdentityTest(APITestCase):
     endpoint = "sentry-api-0-user-identity"
     endpoint = "sentry-api-0-user-identity"
     method = "get"
     method = "get"

+ 3 - 3
tests/sentry/api/endpoints/test_user_identity_config.py

@@ -26,7 +26,7 @@ def mock_is_login_provider_effect(provider_key: str) -> bool:
     return provider_key in ("github", "vsts", "google")
     return provider_key in ("github", "vsts", "google")
 
 
 
 
-@control_silo_test
+@control_silo_test(stable=True)
 class UserIdentityConfigEndpointTest(UserIdentityConfigTest):
 class UserIdentityConfigEndpointTest(UserIdentityConfigTest):
     endpoint = "sentry-api-0-user-identity-config"
     endpoint = "sentry-api-0-user-identity-config"
     method = "get"
     method = "get"
@@ -150,7 +150,7 @@ class UserIdentityConfigEndpointTest(UserIdentityConfigTest):
         assert identity["status"] == "needed_for_org_auth"
         assert identity["status"] == "needed_for_org_auth"
 
 
 
 
-@control_silo_test
+@control_silo_test(stable=True)
 class UserIdentityConfigDetailsEndpointGetTest(UserIdentityConfigTest):
 class UserIdentityConfigDetailsEndpointGetTest(UserIdentityConfigTest):
     endpoint = "sentry-api-0-user-identity-config-details"
     endpoint = "sentry-api-0-user-identity-config-details"
     method = "get"
     method = "get"
@@ -193,7 +193,7 @@ class UserIdentityConfigDetailsEndpointGetTest(UserIdentityConfigTest):
         )
         )
 
 
 
 
-@control_silo_test
+@control_silo_test(stable=True)
 class UserIdentityConfigDetailsEndpointDeleteTest(UserIdentityConfigTest):
 class UserIdentityConfigDetailsEndpointDeleteTest(UserIdentityConfigTest):
     endpoint = "sentry-api-0-user-identity-config-details"
     endpoint = "sentry-api-0-user-identity-config-details"
     method = "delete"
     method = "delete"

+ 1 - 1
tests/sentry/api/endpoints/test_user_identity_details.py

@@ -3,7 +3,7 @@ from sentry.testutils import APITestCase
 from sentry.testutils.silo import control_silo_test
 from sentry.testutils.silo import control_silo_test
 
 
 
 
-@control_silo_test
+@control_silo_test(stable=True)
 class DeleteUserIdentityTest(APITestCase):
 class DeleteUserIdentityTest(APITestCase):
     endpoint = "sentry-api-0-user-identity-details"
     endpoint = "sentry-api-0-user-identity-details"
     method = "delete"
     method = "delete"

+ 1 - 1
tests/sentry/api/endpoints/test_user_ips.py

@@ -7,7 +7,7 @@ from sentry.testutils import APITestCase
 from sentry.testutils.silo import control_silo_test
 from sentry.testutils.silo import control_silo_test
 
 
 
 
-@control_silo_test
+@control_silo_test(stable=True)
 class UserEmailsTest(APITestCase):
 class UserEmailsTest(APITestCase):
     endpoint = "sentry-api-0-user-ips"
     endpoint = "sentry-api-0-user-ips"
 
 

+ 1 - 1
tests/sentry/api/endpoints/test_user_password.py

@@ -6,7 +6,7 @@ from sentry.testutils import APITestCase
 from sentry.testutils.silo import control_silo_test
 from sentry.testutils.silo import control_silo_test
 
 
 
 
-@control_silo_test
+@control_silo_test(stable=True)
 class UserPasswordTest(APITestCase):
 class UserPasswordTest(APITestCase):
     endpoint = "sentry-api-0-user-password"
     endpoint = "sentry-api-0-user-password"
     method = "put"
     method = "put"

+ 1 - 1
tests/sentry/api/endpoints/test_user_social_identities_index.py

@@ -3,7 +3,7 @@ from sentry.testutils.silo import control_silo_test
 from social_auth.models import UserSocialAuth
 from social_auth.models import UserSocialAuth
 
 
 
 
-@control_silo_test
+@control_silo_test(stable=True)
 class UserSocialIdentitiesIndexTest(APITestCase):
 class UserSocialIdentitiesIndexTest(APITestCase):
     endpoint = "sentry-api-0-user-social-identities-index"
     endpoint = "sentry-api-0-user-social-identities-index"
 
 

+ 1 - 1
tests/sentry/api/endpoints/test_user_social_identity_details.py

@@ -3,7 +3,7 @@ from sentry.testutils.silo import control_silo_test
 from social_auth.models import UserSocialAuth
 from social_auth.models import UserSocialAuth
 
 
 
 
-@control_silo_test
+@control_silo_test(stable=True)
 class UserSocialIdentityDetailsEndpointTest(APITestCase):
 class UserSocialIdentityDetailsEndpointTest(APITestCase):
     endpoint = "sentry-api-0-user-social-identity-details"
     endpoint = "sentry-api-0-user-social-identity-details"
     method = "delete"
     method = "delete"

+ 1 - 1
tests/sentry/api/endpoints/test_user_subscriptions.py

@@ -11,7 +11,7 @@ from sentry.testutils.silo import control_silo_test
     settings.SENTRY_NEWSLETTER != "sentry.newsletter.dummy.DummyNewsletter",
     settings.SENTRY_NEWSLETTER != "sentry.newsletter.dummy.DummyNewsletter",
     reason="Requires DummyNewsletter.",
     reason="Requires DummyNewsletter.",
 )
 )
-@control_silo_test
+@control_silo_test(stable=True)
 class UserSubscriptionsNewsletterTest(APITestCase):
 class UserSubscriptionsNewsletterTest(APITestCase):
     endpoint = "sentry-api-0-user-subscriptions"
     endpoint = "sentry-api-0-user-subscriptions"
     method = "put"
     method = "put"