Browse Source

fix: register fly under fly.io (#52478)

Nathan Hsieh 1 year ago
parent
commit
aa8666c99d

+ 3 - 1
src/sentry/auth/providers/fly/apps.py

@@ -1,5 +1,7 @@
 from django.apps import AppConfig
 
+from sentry.auth.partnership_config import ChannelName
+
 
 class Config(AppConfig):
     name = "sentry.auth.providers.fly"
@@ -9,7 +11,7 @@ class Config(AppConfig):
 
         from .provider import FlyOAuth2Provider
 
-        auth.register("fly", FlyOAuth2Provider)
+        auth.register(ChannelName.FLY_IO.value, FlyOAuth2Provider)
 
         options.register(
             "auth-fly.client-id",

+ 2 - 1
tests/sentry/auth/providers/fly/test_provider.py

@@ -1,6 +1,7 @@
 import pytest
 
 from sentry.auth.exceptions import IdentityNotValid
+from sentry.auth.partnership_config import ChannelName
 from sentry.auth.provider import MigratingIdentityId
 from sentry.models import AuthIdentity, AuthProvider
 from sentry.testutils import TestCase
@@ -11,7 +12,7 @@ from sentry.testutils.silo import control_silo_test
 class FlyOAuth2ProviderTest(TestCase):
     def setUp(self):
         self.auth_provider: AuthProvider = AuthProvider.objects.create(
-            provider="fly", organization_id=self.organization.id
+            provider=ChannelName.FLY_IO.value, organization_id=self.organization.id
         )
         super().setUp()