Skip to content

Commit 3ba8698

Browse files
committed
fix: Don't include OpenEdxEventsTestMixin twice.
This mixin is already included via one of the other mixins on this test class so including it again was messing with the MRO for the test classes.
1 parent 45b4ad7 commit 3ba8698

1 file changed

Lines changed: 2 additions & 2 deletions

File tree

openedx/core/djangoapps/user_authn/views/tests/test_register.py

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -2687,7 +2687,7 @@ def test_expired_pipeline(self):
26872687

26882688
@skipUnless(settings.FEATURES.get("ENABLE_THIRD_PARTY_AUTH"), "third party auth not enabled")
26892689
class TestFacebookRegistrationView(
2690-
OpenEdxEventsTestMixin, ThirdPartyRegistrationTestMixin, ThirdPartyOAuthTestMixinFacebook, TransactionTestCase
2690+
ThirdPartyRegistrationTestMixin, ThirdPartyOAuthTestMixinFacebook, TransactionTestCase
26912691
):
26922692
"""Tests the User API registration endpoint with Facebook authentication."""
26932693

@@ -2708,7 +2708,7 @@ def test_social_auth_exception(self):
27082708

27092709
@skipUnless(settings.FEATURES.get("ENABLE_THIRD_PARTY_AUTH"), "third party auth not enabled")
27102710
class TestGoogleRegistrationView(
2711-
OpenEdxEventsTestMixin, ThirdPartyRegistrationTestMixin, ThirdPartyOAuthTestMixinGoogle, TransactionTestCase
2711+
ThirdPartyRegistrationTestMixin, ThirdPartyOAuthTestMixinGoogle, TransactionTestCase
27122712
):
27132713
"""Tests the User API registration endpoint with Google authentication."""
27142714

0 commit comments

Comments
 (0)