From eafea504aca2218da0ad82e9077ca1ba006b1ed3 Mon Sep 17 00:00:00 2001 From: cdeery Date: Thu, 5 Aug 2021 12:35:54 -0400 Subject: [PATCH] fix: changes suggested in code review Cleaned up the nits. --- lms/djangoapps/courseware/tests/test_masquerade.py | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/lms/djangoapps/courseware/tests/test_masquerade.py b/lms/djangoapps/courseware/tests/test_masquerade.py index c0b730f034..42568e6af9 100644 --- a/lms/djangoapps/courseware/tests/test_masquerade.py +++ b/lms/djangoapps/courseware/tests/test_masquerade.py @@ -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)