fix: update tests and lint

edx_proctoring/api.py

@@ -1669,7 +1669,7 @@ def update_attempt_status(attempt_id, to_status,
         if email:
             try:
                 email.send()
-            except Exception as err:  # pylint: disable=broad-except
+            except Exception as err:
                 log.exception(
                     ('Exception occurred while trying to send proctoring attempt '
                      'status email for user_id=%(user_id)s in course_id=%(course_id)s -- %(err)s'),

edx_proctoring/backends/rest.py

@@ -274,7 +274,7 @@ def on_exam_saved(self, exam):
         try:
             response = self.session.post(url, json=exam)
             data = response.json()
-        except Exception as exc:  # pylint: disable=broad-except
+        except Exception as exc:
             if response:
                 if hasattr(exc, 'response') and exc.response is not None:
                     content = exc.response.content

edx_proctoring/backends/software_secure.py

@@ -409,7 +409,7 @@ def get_instructor_url(
             # reformat video url as per MST-871 findings
             reformatted_url = decrypted_video_url.replace('DirectLink-Generic', 'DirectLink-HTML5')
             return reformatted_url
-        except Exception as err:  # pylint: disable=broad-except
+        except Exception as err:
             log.exception(
                 'Could not decrypt video url for attempt_id=%(attempt_id)s '
                 'due to the following error: %(error_string)s',

edx_proctoring/rules.py

@@ -12,5 +12,5 @@ def is_in_reviewer_group(user, attempt):
     return user.groups.filter(name=backend_group).exists()
 
 
-# pylint: disable=unsupported-binary-operation
+# pylint: disable-next=unsupported-binary-operation
 rules.add_perm('edx_proctoring.can_review_attempt', is_in_reviewer_group | rules.is_staff)

edx_proctoring/tests/test_api.py

@@ -6,6 +6,7 @@
 """
 from datetime import datetime, timedelta
 from itertools import product
+import logging
 
 import ddt
 import pytz
@@ -2896,6 +2897,12 @@ def test_dashboard_availability(self):
         # backend with a dashboard
         self.assertTrue(is_backend_dashboard_available(self.course_id))
 
+    @patch('edx_proctoring.api.get_backend_provider')
+    def test_dashboard_availability_no_provider(self, mock_get_backend):
+        mock_get_backend.side_effect = NotImplementedError()
+        self.assertFalse(is_backend_dashboard_available(self.course_id))
+
+
     def test_does_provider_support_onboarding(self):
         self.assertTrue(does_backend_support_onboarding('test'))
         self.assertFalse(does_backend_support_onboarding('mock'))

edx_proctoring/views.py

@@ -640,7 +640,7 @@ class StudentOnboardingStatusView(ProctoredAPIView):
         * 'onboarding_past_due': Whether the onboarding exam is past due. All onboarding exams in the course must
           be past due in order for onboarding_past_due to be true.
     """
-    def get(self, request):
+    def get(self, request):  # pylint: disable=too-many-statements
         """
         HTTP GET handler. Returns the learner's onboarding status.
         """
@@ -679,7 +679,7 @@ def get(self, request):
         onboarding_exams = list(ProctoredExam.get_practice_proctored_exams_for_course(course_id).order_by('-created'))
         provider = None
         try:
-            provider = get_backend_provider(name=onboarding_exams[0].backend)
+            provider = get_backend_provider(name=onboarding_exams[0].backend) if onboarding_exams else None
         except NotImplementedError:
             logging.exception(
                 'No proctoring backend configured for backend=%(backend)s',
@@ -860,7 +860,7 @@ def get(self, request, course_id):
                            .order_by('-created').first())
         provider = None
         try:
-            provider = get_backend_provider(name=onboarding_exam.backend)
+            provider = get_backend_provider(name=onboarding_exam.backend) if onboarding_exam else None
         except NotImplementedError:
             logging.exception(
                 'No proctoring backend configured for backend=%(backend)s',