Merge pull request #257 from edx/hasnain-naveed/PHX-239-admin-error

PHX-239 / Removed the "not reviewed" status from the model

edx_proctoring/admin.py

@@ -386,7 +386,6 @@ class ProctoredExamAttemptForm(forms.ModelForm):
         (ProctoredExamStudentAttemptStatus.second_review_required, _('Second Review Required')),
         (ProctoredExamStudentAttemptStatus.verified, _('Verified')),
         (ProctoredExamStudentAttemptStatus.rejected, _('Rejected')),
-        (ProctoredExamStudentAttemptStatus.not_reviewed, _('Not Reviewed')),
         (ProctoredExamStudentAttemptStatus.error, _('Error')),
     ]
 

edx_proctoring/models.py

@@ -159,9 +159,6 @@ class ProctoredExamStudentAttemptStatus(object):
     # the exam has been rejected
     rejected = 'rejected'
 
-    # the exam was not reviewed
-    not_reviewed = 'not_reviewed'
-
     # the exam is believed to be in error
     error = 'error'
 
@@ -180,7 +177,7 @@ class ProctoredExamStudentAttemptStatus(object):
         """
         return status in [
             cls.declined, cls.timed_out, cls.submitted, cls.second_review_required,
-            cls.verified, cls.rejected, cls.not_reviewed, cls.error
+            cls.verified, cls.rejected, cls.error
         ]
 
     @classmethod
@@ -199,8 +196,7 @@ class ProctoredExamStudentAttemptStatus(object):
         Returns a boolean if the passed in to_status calls for an update to the credit requirement status.
         """
         return to_status in [
-            cls.verified, cls.rejected, cls.declined, cls.not_reviewed,
-            cls.submitted, cls.error
+            cls.verified, cls.rejected, cls.declined, cls.submitted, cls.error
         ]
 
     @classmethod

edx_proctoring/static/proctoring/js/views/proctored_exam_attempt_view.js

@@ -18,7 +18,6 @@ var edx = edx || {};
         submitted: gettext('Submitted'),
         verified: gettext('Verified'),
         rejected: gettext('Rejected'),
-        not_reviewed: gettext('Not reviewed'),
         error: gettext('Error')
     };
     var viewHelper = {

edx_proctoring/tests/test_api.py

@@ -2089,7 +2089,6 @@ class ProctoredExamApiTests(LoggedInTestCase):
         (ProctoredExamStudentAttemptStatus.submitted, ProctoredExamStudentAttemptStatus.ready_to_start),
         (ProctoredExamStudentAttemptStatus.verified, ProctoredExamStudentAttemptStatus.started),
         (ProctoredExamStudentAttemptStatus.rejected, ProctoredExamStudentAttemptStatus.started),
-        (ProctoredExamStudentAttemptStatus.not_reviewed, ProctoredExamStudentAttemptStatus.started),
         (ProctoredExamStudentAttemptStatus.error, ProctoredExamStudentAttemptStatus.started),
         (ProctoredExamStudentAttemptStatus.submitted, ProctoredExamStudentAttemptStatus.error),
     )
@@ -2560,7 +2559,6 @@ class ProctoredExamApiTests(LoggedInTestCase):
         ProctoredExamStudentAttemptStatus.ready_to_submit,
         ProctoredExamStudentAttemptStatus.declined,
         ProctoredExamStudentAttemptStatus.timed_out,
-        ProctoredExamStudentAttemptStatus.not_reviewed,
         ProctoredExamStudentAttemptStatus.error
     )
     @patch.dict('settings.PROCTORING_SETTINGS', {'ALLOW_TIMED_OUT_STATE': True})