Skip to content

Commit

Permalink
style: Fix a pylint and style violations.
Browse files Browse the repository at this point in the history
  • Loading branch information
feanil committed Jul 25, 2024
1 parent 62269f8 commit 8813e8b
Show file tree
Hide file tree
Showing 3 changed files with 4 additions and 3 deletions.
3 changes: 2 additions & 1 deletion cms/djangoapps/contentstore/utils.py
Original file line number Diff line number Diff line change
Expand Up @@ -1699,7 +1699,8 @@ def get_home_context(request, no_course=False):
'library_authoring_mfe_url': LIBRARY_AUTHORING_MICROFRONTEND_URL,
'taxonomy_list_mfe_url': get_taxonomy_list_url(),
'libraries': libraries,
'show_new_library_button': user_can_view_create_library_button(user) and not should_redirect_to_library_authoring_mfe(),
'show_new_library_button': user_can_view_create_library_button(user)
and not should_redirect_to_library_authoring_mfe(),
'user': user,
'request_course_creator_url': reverse('request_course_creator'),
'course_creator_status': _get_course_creator_status(user),
Expand Down
2 changes: 1 addition & 1 deletion cms/djangoapps/contentstore/views/library.py
Original file line number Diff line number Diff line change
Expand Up @@ -68,6 +68,7 @@ def should_redirect_to_library_authoring_mfe():
REDIRECT_TO_LIBRARY_AUTHORING_MICROFRONTEND.is_enabled()
)


def user_can_view_create_library_button(user):
"""
Helper method for displaying the visibilty of the create_library_button.
Expand All @@ -92,7 +93,6 @@ def user_can_view_create_library_button(user):
return not disable_course_creation



def user_can_create_library(user, org):
"""
Helper method for returning the library creation status for a particular user,
Expand Down
2 changes: 1 addition & 1 deletion cms/djangoapps/contentstore/views/tests/test_library.py
Original file line number Diff line number Diff line change
Expand Up @@ -111,7 +111,7 @@ def test_library_creator_status_with_course_staff_role_for_enabled_creator_group
self.assertEqual(user_can_create_library(nostaff_user, 'SomEOtherOrg'), False)

# When creator groups are enabled, course instructor members can create libraries
# but only in the org they are course staff for.
# but only in the org they are course staff for.
@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()
Expand Down

0 comments on commit 8813e8b

Please sign in to comment.