Commit e3080a52 by attiyaishaque

EDUCATOR-1256 Fix 500 error when user is not enrolled.

parent 47ce2865
...@@ -1896,7 +1896,7 @@ def get_student_view(user_id, course_id, content_id, ...@@ -1896,7 +1896,7 @@ def get_student_view(user_id, course_id, content_id,
# call service to get course end date. # call service to get course end date.
credit_state = credit_service.get_credit_state(user_id, course_id, return_course_info=True) credit_state = credit_service.get_credit_state(user_id, course_id, return_course_info=True)
course_end_date = credit_state.get('course_end_date', None) course_end_date = credit_state.get('course_end_date') if credit_state else None
exam_id = None exam_id = None
try: try:
......
...@@ -29,7 +29,7 @@ from edx_proctoring.models import ( ...@@ -29,7 +29,7 @@ from edx_proctoring.models import (
) )
from edx_proctoring.runtime import set_runtime_service from edx_proctoring.runtime import set_runtime_service
from .test_services import MockCreditServiceWithCourseEndDate from .test_services import MockCreditServiceWithCourseEndDate, MockCreditServiceNone
from .utils import ProctoredExamTestCase from .utils import ProctoredExamTestCase
...@@ -388,6 +388,24 @@ class ProctoredExamStudentViewTests(ProctoredExamTestCase): ...@@ -388,6 +388,24 @@ class ProctoredExamStudentViewTests(ProctoredExamTestCase):
) )
) )
def test_student_response_without_credit_state(self):
"""
Test that response is not None for users who are not enrolled.
"""
set_runtime_service('credit', MockCreditServiceNone())
rendered_response = get_student_view(
user_id=self.user_id,
course_id=self.course_id,
content_id=self.content_id,
context={
'is_proctored': True,
'display_name': self.exam_name,
'default_time_limit_mins': 90
},
user_role='student'
)
self.assertIsNotNone(rendered_response)
@ddt.data(False, True) @ddt.data(False, True)
def test_get_studentview_unstarted_exam(self, allow_proctoring_opt_out): def test_get_studentview_unstarted_exam(self, allow_proctoring_opt_out):
""" """
......
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