Commit c609b163 by Chris Dodge

change proctoring eligibility to be true unless we explicitly failed a icrv

parent bf5de42f
...@@ -771,7 +771,7 @@ def _check_eligibility_of_prerequisites(credit_state): ...@@ -771,7 +771,7 @@ def _check_eligibility_of_prerequisites(credit_state):
# then make sure those has a 'satisfied' status # then make sure those has a 'satisfied' status
for requirement in credit_state['credit_requirement_status']: for requirement in credit_state['credit_requirement_status']:
if requirement['namespace'] == 'reverification': if requirement['namespace'] == 'reverification':
if requirement['status'] != 'satisfied': if requirement['status'] == 'failed':
return False return False
return True return True
......
...@@ -650,14 +650,14 @@ class ProctoredExamApiTests(LoggedInTestCase): ...@@ -650,14 +650,14 @@ class ProctoredExamApiTests(LoggedInTestCase):
self.assertIsNone(rendered_response) self.assertIsNone(rendered_response)
@ddt.data( @ddt.data(
('reverification', None, False, True, ProctoredExamStudentAttemptStatus.declined), ('reverification', None, True, True, False),
('reverification', 'failed', False, False, ProctoredExamStudentAttemptStatus.declined), ('reverification', 'failed', False, False, True),
('reverification', 'satisfied', True, True, None), ('reverification', 'satisfied', True, True, False),
('grade', 'failed', True, False, None) ('grade', 'failed', True, False, False)
) )
@ddt.unpack @ddt.unpack
def test_prereq_scarios(self, namespace, req_status, show_proctored, def test_prereq_scenarios(self, namespace, req_status, show_proctored,
pre_create_attempt, mark_as_declined): pre_create_attempt, mark_as_declined):
""" """
This test asserts that proctoring will not be displayed under the following This test asserts that proctoring will not be displayed under the following
conditions: conditions:
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment