Andal.LND/edx-platform
6
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

test: test case for duplicate user in multiple providers

Browse Source
This commit is contained in:
Zach Hancock
2021-07-12 11:10:55 -04:00
committed by Matt Hughes
parent aa2bf9c063
commit 6ba12c71d3
2 changed files with 11 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files

10
lms/djangoapps/program_enrollments/api/reading.py
View File

@@ -405,6 +405,8 @@ def get_users_by_external_keys_and_org_key(external_user_keys, org_key):
"""
saml_providers = get_saml_providers_by_org_key(org_key)
found_users_by_external_keys = dict()
# if the same external id exists in multiple providers (for this organization)
# it is expected both providers return the same user
for saml_provider in saml_providers:
social_auth_uids = {
saml_provider.get_social_auth_uid(external_user_key)
@@ -478,7 +480,10 @@ def get_external_key_by_user_and_course(user, course_key):
def get_saml_providers_by_org_key(org_key):
"""
Returns a list of SAML providers associated with the provided org_key
Returns a list of SAML providers associated with the provided org_key.
In most cases an organization will only have one configured provider.
However, multiple may be returned during a migration between two active
providers.
Arguments:
org_key (str)
@@ -522,6 +527,9 @@ def get_org_key_for_program(program_uuid):
def get_saml_providers_for_organization(organization):
"""
Return currently configured SAML provider(s) for the given Organization.
In most cases an organization will only have one configured provider.
However, multiple may be returned during a migration between two active
providers.
Arguments:
organization: Organization

2
lms/djangoapps/program_enrollments/api/tests/test_reading.py
View File

@@ -603,11 +603,13 @@ class GetUsersByExternalKeysTests(CacheIsolationTestCase):
)
self.create_social_auth_entry(self.user_0, provider_1, 'ext-user-0')
self.create_social_auth_entry(self.user_1, provider_1, 'ext-user-1')
self.create_social_auth_entry(self.user_1, provider_2, 'ext-user-1')
self.create_social_auth_entry(self.user_2, provider_2, 'ext-user-2')
requested_keys = {'ext-user-1', 'ext-user-2', 'ext-user-3'}
actual = get_users_by_external_keys(self.program_uuid, requested_keys)
# ext-user-0 not requested, ext-user-3 doesn't exist,
# ext-user-2 is authorized with secondary provider
# ext-user-1 has an entry in both providers
expected = {
'ext-user-1': self.user_1,
'ext-user-2': self.user_2,