fix: add ability for course instructors to create libraries same as c…

…ourse staff

cms/djangoapps/contentstore/views/library.py

@@ -83,8 +83,9 @@ def user_can_create_library(user, org=None):
  is_course_creator = get_course_creator_status(user) == 'granted'
  has_org_staff_role = OrgStaffRole().get_orgs_for_user(user).exists()
  has_course_staff_role = UserBasedRole(user=user, role=CourseStaffRole.ROLE).courses_with_role().exists()
+ has_course_admin_role = UserBasedRole(user=user, role=CourseInstructorRole.ROLE).courses_with_role().exists()
 
- return is_course_creator or has_org_staff_role or has_course_staff_role
+ return is_course_creator or has_org_staff_role or has_course_staff_role or has_course_admin_role
  else:
  # EDUCATOR-1924: DISABLE_LIBRARY_CREATION overrides DISABLE_COURSE_CREATION, if present.
  disable_library_creation = settings.FEATURES.get('DISABLE_LIBRARY_CREATION', None)

cms/djangoapps/contentstore/views/tests/test_library.py

@@ -19,7 +19,7 @@
 from cms.djangoapps.contentstore.tests.utils import AjaxEnabledTestClient, CourseTestCase, parse_json
 from cms.djangoapps.contentstore.utils import reverse_course_url, reverse_library_url
 from cms.djangoapps.course_creators.views import add_user_with_status_granted as grant_course_creator_status
-from common.djangoapps.student.roles import LibraryUserRole, CourseStaffRole
+from common.djangoapps.student.roles import LibraryUserRole, CourseStaffRole, CourseInstructorRole
 from xmodule.modulestore.tests.factories import LibraryFactory # lint-amnesty, pylint: disable=wrong-import-order
 from cms.djangoapps.course_creators.models import CourseCreator
 
@@ -101,6 +101,14 @@ def test_library_creator_status_with_course_staff_role_for_enabled_creator_group
  auth.add_users(self.user, CourseStaffRole(self.course.id), nostaff_user)
  self.assertEqual(user_can_create_library(nostaff_user), True)
 
+ # When creator groups are enabled, course instructor members can create libraries
+ @mock.patch("cms.djangoapps.contentstore.views.library.LIBRARIES_ENABLED", True)
+ def test_library_creator_status_with_course_instructor_role_for_enabled_creator_group_setting(self):
+ _, nostaff_user = self.create_non_staff_authed_user_client()
+ with mock.patch.dict('django.conf.settings.FEATURES', {"ENABLE_CREATOR_GROUP": True}):
+ auth.add_users(self.user, CourseInstructorRole(self.course.id), nostaff_user)
+ self.assertEqual(user_can_create_library(nostaff_user), True)
+
  @ddt.data(
  (False, False, True),
  (False, True, False),