From 2757ac4fcb23bfa970691c515e6e351a06bcd7f7 Mon Sep 17 00:00:00 2001 From: Julia Hansbrough Date: Mon, 9 Jun 2014 17:30:22 +0000 Subject: [PATCH] Opaque-keys: response to code review --- cms/djangoapps/contentstore/tests/test_permissions.py | 5 ++--- 1 file changed, 2 insertions(+), 3 deletions(-) diff --git a/cms/djangoapps/contentstore/tests/test_permissions.py b/cms/djangoapps/contentstore/tests/test_permissions.py index dd4903d74c..2534cd8c86 100644 --- a/cms/djangoapps/contentstore/tests/test_permissions.py +++ b/cms/djangoapps/contentstore/tests/test_permissions.py @@ -99,10 +99,8 @@ class TestCourseAccess(ModuleStoreTestCase): user_by_role[role] = [] # Org-based roles are created via org name, rather than course_key if (role is OrgStaffRole) or (role is OrgInstructorRole): - # pylint: disable=protected-access group = role(self.course_key.org) else: - # pylint: disable=protected-access group = role(self.course_key) # NOTE: this loop breaks the roles.py abstraction by purposely assigning # users to one of each possible groupname in order to test that has_course_access @@ -125,7 +123,8 @@ class TestCourseAccess(ModuleStoreTestCase): auth.add_users( self.user, role(copy_course_key.org), - *role(self.course_key.org).users_with_role()) + *role(self.course_key.org).users_with_role() + ) else: auth.add_users( self.user,