Skip to content

Commit 20ba3ee

Browse files
fix: removal of temporary saml toggle
1 parent fca4501 commit 20ba3ee

File tree

1 file changed

+6
-13
lines changed
  • common/djangoapps/third_party_auth/management/commands/tests

1 file changed

+6
-13
lines changed

common/djangoapps/third_party_auth/management/commands/tests/test_saml.py

Lines changed: 6 additions & 13 deletions
Original file line numberDiff line numberDiff line change
@@ -64,21 +64,13 @@ def setUp(self):
6464
self.site = Site.objects.get_current()
6565
self.other_site = Site.objects.create(domain='other.example.com', name='Other Site')
6666

67-
# Create a disabled SAML configuration that won't be processed
68-
# This uses a different site to avoid signal handler interference with test providers
69-
self.disabled_site = Site.objects.create(domain='disabled.fake', name='disabled.fake')
67+
# We are creating SAMLConfiguration instance here so that there is always at-least one
68+
# disabled saml configuration instance, this is done to verify that disabled configurations are
69+
# not processed.
7070
self.saml_config = SAMLConfigurationFactory.create(
7171
enabled=False,
72-
site=self.disabled_site
73-
)
74-
75-
# Create initial provider config with saml_configuration set to avoid orphaned provider
76-
# Using a site/slug combination that won't match the test configurations created later
77-
provider_saml_config = SAMLConfigurationFactory.create(
78-
enabled=True,
7972
site__domain='testserver.fake',
80-
site__name='testserver.fake',
81-
slug='test-shib'
73+
site__name='testserver.fake'
8274
)
8375
self.provider_config = SAMLProviderConfigFactory.create(
8476
site__domain='testserver.fake',
@@ -87,9 +79,10 @@ def setUp(self):
8779
name='TestShib College',
8880
entity_id='https://idp.testshib.org/idp/shibboleth',
8981
metadata_source='https://www.testshib.org/metadata/testshib-providers.xml',
90-
saml_configuration=provider_saml_config
9182
)
9283

84+
85+
9386
def _setup_test_configs_for_run_checks(self):
9487
"""
9588
Helper method to create SAML configurations for run-checks tests.

0 commit comments

Comments
 (0)