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

fix: changes suggested in code review

Browse Source 
Cleaned up the nits.
This commit is contained in:
cdeery
2021-08-05 12:35:54 -04:00
parent 35caeb4036
commit eafea504ac
1 changed files with 2 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files

4
lms/djangoapps/courseware/tests/test_masquerade.py
View File

@@ -176,7 +176,7 @@ class MasqueradeTestCase(SharedModuleStoreTestCase, LoginEnrollmentTestCase, Mas
"""
Verifies if learner masquerade option is available
Args:
exists: True to test if Learner is in options, False to test it is NOT
learner_option_expected: True to test if Learner is in options, False to test it is NOT
"""
response = self.get_available_masquerade_identities()
items = response.json()['available']
@@ -247,8 +247,8 @@ class TestMasqueradeOptionsForUserPartition(StaffMasqueradeTestCase):
"""
Check that 'Learner' option is NOT available if there are no groups or partitions
"""
@ patch.dict('django.conf.settings.FEATURES', {'ENABLE_ENROLLMENT_TRACK_USER_PARTITION': True})
@ patch.dict('django.conf.settings.FEATURES', {'ENABLE_MASQUERADE': True})
@ patch.dict('django.conf.settings.FEATURES', {'ENABLE_ENROLLMENT_TRACK_USER_PARTITION': True})
def test_masquerade_options_no_cohort(self):
self.verify_learner_masquerade_available(False)