fix: lint and other quality issues

enterprise/api_client/lms.py

@@ -128,7 +128,15 @@ def has_course_mode(self, course_run_id, mode):
         course_modes = self.get_course_modes(course_run_id)
         return any(course_mode for course_mode in course_modes if course_mode['slug'] == mode)
 
-    def enroll_user_in_course(self, username, course_id, mode, cohort=None, enterprise_uuid=None, force_enrollment=False):
+    def enroll_user_in_course(
+        self,
+        username,
+        course_id,
+        mode,
+        cohort=None,
+        enterprise_uuid=None,
+        force_enrollment=False,
+    ):
         """
         Call the enrollment API to enroll the user in the course specified by course_id.
 

enterprise/utils.py

@@ -1702,7 +1702,7 @@ def enroll_user(enterprise_customer, user, course_mode, *course_ids, **kwargs):
         *course_ids: An iterable containing any number of course IDs to eventually enroll the user in.
         kwargs: Contains optional params such as:
             - enrollment_client, if it's already been instantiated and should be passed in
-            - force_enrollment, if the course is "Invite Only" and the "force_enrollment" is needed 
+            - force_enrollment, if the course is "Invite Only" and the "force_enrollment" is needed
 
     Returns:
         Boolean: Whether or not enrollment succeeded for all courses specified

test_utils/fake_enrollment_api.py

@@ -150,7 +150,8 @@ def get_course_details(course_id):
         return None
 
 
-def enroll_user_in_course(user, course_id, mode, cohort=None, enterprise_uuid=None, force_enrollment=False):
+def enroll_user_in_course(user, course_id, mode, cohort=None, enterprise_uuid=None, force_enrollment=False):  # pylint: disable=unused-argument
+
     """
     Fake implementation.
     """

tests/test_admin/test_view.py

@@ -894,7 +894,16 @@ def test_post_existing_pending_record_with_another_enterprise_customer(self):
         self._test_post_existing_record_response(response)
         assert PendingEnterpriseCustomerUser.objects.filter(user_email=email).count() == 2
 
-    def _enroll_user_request(self, user, mode, course_id="", notify=True, reason="tests", discount=0.0, force_enrollment=False):
+    def _enroll_user_request(
+        self,
+        user,
+        mode,
+        course_id="",
+        notify=True,
+        reason="tests",
+        discount=0.0,
+        force_enrollment=False
+    ):
         """
         Perform post request to log in and submit the form to enroll a user.
         """

tests/test_enterprise/api/test_views.py

@@ -65,8 +65,8 @@
     update_program_with_enterprise_context,
 )
 from test_utils.factories import FAKER, EnterpriseCustomerUserFactory, PendingEnterpriseCustomerUserFactory, UserFactory
-from test_utils.fake_enterprise_api import get_default_branding_object
 from test_utils.fake_enrollment_api import get_course_details
+from test_utils.fake_enterprise_api import get_default_branding_object
 
 fake = Faker()
 
@@ -4098,10 +4098,10 @@ def test_bulk_enrollment_in_bulk_courses_pending_licenses(
 
         self.assertEqual(len(PendingEnrollment.objects.all()), 0)
 
-        with mock.patch("enterprise.models.EnrollmentApiClient.get_course_details", wraps=get_course_details) as mock_client:
-            # mock_client = mock.Mock(
-            #     get_course_details=get_course_details
-            # )
+        with mock.patch(
+            "enterprise.models.EnrollmentApiClient.get_course_details",
+            wraps=get_course_details
+        ):
             response = self.client.post(
                 settings.TEST_SERVER + ENTERPRISE_CUSTOMER_BULK_ENROLL_LEARNERS_IN_COURSES_ENDPOINT,
                 data=json.dumps(body),

tests/test_enterprise/api_client/test_lms.py

@@ -99,7 +99,8 @@ def test_enroll_user_in_course():
         'mode': mode,
         'cohort': cohort,
         'is_active': True,
-        'enterprise_uuid': 'None'
+        'enterprise_uuid': 'None',
+        'force_enrollment': False
     }
     responses.add(
         responses.POST,