Fixed some typos in the templates.

edx_proctoring/templates/proctoring/seq_proctored_exam_entrance.html

@@ -2,24 +2,24 @@
 <div class="sequence proctored-exam entrance" data-exam-id="{{exam_id}}">
   <h3>
     {% blocktrans %}
-      Would you Like to take {{ display_name }} as Proctored Exam?
+      Would you like to take {{ display_name }} as a Proctored Exam?
     {% endblocktrans %}
   </h3>
   <p>
     {% blocktrans %}
-      Since you're enrolled in this course as a verified student, you have the option to take this exam
-      with online proctoring. Online proctoring is one requirement towards being eligible for credit.
+      Since you're enrolled in this course as a verified student, you have the option to take this exam with
+      online proctoring. Online proctoring is one requirement towards being eligible for credit.
     {% endblocktrans %}
   </p>
   <div class="gated-sequence">
     <span><i class="fa fa-lock"></i></span>
     <a class="start-timed-exam" data-ajax-url="{{enter_exam_endpoint}}" data-exam-id="{{exam_id}}" data-attempt-proctored=true>
-      {% trans "Yes, take this exam as a proctored exam (and be eligible for credit)" %}
+      {% trans "Yes, take this as a proctored exam (and be eligible for credit)" %}
     </a>
     <p>
       {% blocktrans %}
-        You will need to <strong>download and install edX-approved software </strong>for the online proctoring
-        of your exam. After successful installation, you will be <strong>guided through setting up your
+        You will need to <strong>download and install edX-approved software </strong>for the online proctoring of
+        your exam. After successful installation, you will be <strong>guided through setting up your
         proctored session and begin the exam immediately </strong>afterwards.</p>
       {% endblocktrans %}
     <i class="fa fa-arrow-circle-right start-timed-exam" data-ajax-url="{{enter_exam_endpoint}}" data-exam-id="{{exam_id}}" data-attempt-proctored=true></i>
@@ -27,12 +27,12 @@
   <div class="gated-sequence">
     <span><i class="fa fa-unlock"></i></span>
     <a class="start-timed-exam" data-ajax-url="{{enter_exam_endpoint}}" data-exam-id="{{exam_id}}" data-attempt-proctored=false>
-      {% trans "No, take this exam as an open exam (and not be eligible for credit)" %}
+      {% trans "No, take this as an open exam (and not be eligible for credit)" %}
     </a>
     <p>
       {% blocktrans %}
-        You may proceed and begin the exam at your leisure, but <strong>you will not be able to apply for college
-        credit </strong>upon completing the exam or this course in general.
+        You may proceed and begin the exam at your leisure, but <strong>you will not be able to apply for
+        college credit </strong>upon completing the exam or this course in general.
       {% endblocktrans %}
     </p>
       <i class="fa fa-arrow-circle-right start-timed-exam" data-ajax-url="{{enter_exam_endpoint}}" data-exam-id="{{exam_id}}" data-attempt-proctored=false></i>

edx_proctoring/templates/proctoring/seq_proctored_exam_instructions.html

@@ -7,8 +7,8 @@
   </h3>
   <p>
     {% blocktrans %}
-      you have chosen to take {{display_name}} as a proctored exam. You should be redirected to a new window
-      with installation and setup instructions. You can also <a href="#">open the installation</a>
+      You've chosen to take {{display_name}} as a proctored exam. You should be directed to a new window
+      with installation and setup instructions. You can also <a href="#">open the installation window directly.</a>
     {% endblocktrans %}
   </p>
   <div class="proctored-exam-message">
@@ -20,16 +20,16 @@
     <h1> {{exam_code}}</h1>
     <p>
       {% blocktrans %}
-        Please do not share this code. It can only be used once and it tied to your edX Account.
+        Please do not share this code. It can only be used once and it tied to your edX account.
       {% endblocktrans %}
     </p>
   </div>
 </div>
 <div class="footer-sequence border-b-0 padding-b-0">
-  <span> {% trans "Note: Once you complete your installation and set up, your timed exam will begin." %} </span>
+  <span> {% trans "Note: Once you complete installation and set up, your timed exam will begin." %} </span>
     <p>
     {% blocktrans %}
-      Please be prepared to start the exam and follow all the guidelines of an edX proctored exam.
+      Please be prepared to start the exam and follow all of the guidelines of an edX proctored exam.
     {% endblocktrans %}
   </p>
   <p class="proctored-exam-option">

edx_proctoring/tests/test_api.py

@@ -15,22 +15,23 @@ from edx_proctoring.api import (
     get_active_exams_for_user,
     get_exam_attempt,
     create_exam_attempt,
-    get_student_view)
+    get_student_view,
+)
 from edx_proctoring.exceptions import (
     ProctoredExamAlreadyExists,
     ProctoredExamNotFoundException,
     StudentExamAttemptAlreadyExistsException,
     StudentExamAttemptDoesNotExistsException,
-    StudentExamAttemptedAlreadyStarted
+    StudentExamAttemptedAlreadyStarted,
 )
 from edx_proctoring.models import (
     ProctoredExam,
     ProctoredExamStudentAllowance,
-    ProctoredExamStudentAttempt
+    ProctoredExamStudentAttempt,
 )
 
 from .utils import (
-    LoggedInTestCase
+    LoggedInTestCase,
 )
 
 
@@ -54,6 +55,12 @@ class ProctoredExamApiTests(LoggedInTestCase):
         self.external_id = 'test_external_id'
         self.proctored_exam_id = self._create_proctored_exam()
 
+        # Messages for get_student_view
+        self.start_an_exam_msg = 'Would you like to take %s as a Proctored Exam?'
+        self.timed_exam_msg = '%s is a Timed Exam'
+        self.exam_time_expired_msg = 'you did not submit your exam before the time allotted expired'
+        self.chose_proctored_exam_msg = 'You\'ve chosen to take %s as a proctored exam'
+
     def _create_proctored_exam(self):
         """
         Calls the api's create_exam to create an exam object.
@@ -201,7 +208,7 @@ class ProctoredExamApiTests(LoggedInTestCase):
 
     def test_create_an_exam_attempt(self):
         """
-        Start an exam attempt.
+        Create an unstarted exam attempt.
         """
         attempt_id = create_exam_attempt(self.proctored_exam_id, self.user_id, '')
         self.assertGreater(attempt_id, 0)
@@ -217,7 +224,7 @@ class ProctoredExamApiTests(LoggedInTestCase):
 
     def test_get_exam_attempt(self):
         """
-        Test to get the already made exam attempt.
+        Test to get the existing exam attempt.
         """
         self._create_unstarted_exam_attempt()
         exam_attempt = get_exam_attempt(self.proctored_exam_id, self.user_id)
@@ -298,7 +305,8 @@ class ProctoredExamApiTests(LoggedInTestCase):
 
     def test_get_student_view(self):
         """
-        Test for get_student_view
+        Test for get_student_view promting the user to take the exam
+        as a timed exam or a proctored exam.
         """
         rendered_response = get_student_view(
             user_id=self.user_id,
@@ -311,7 +319,7 @@ class ProctoredExamApiTests(LoggedInTestCase):
             }
         )
         self.assertIn('data-exam-id="%d"' % self.proctored_exam_id, rendered_response)
-        self.assertIn('Would you Like to take %s as Proctored Exam?' % self.exam_name, rendered_response)
+        self.assertIn(self.start_an_exam_msg % self.exam_name, rendered_response)
 
     def test_getstudentview_timedexam(self):
         """
@@ -328,8 +336,8 @@ class ProctoredExamApiTests(LoggedInTestCase):
             }
         )
         self.assertNotIn('data-exam-id="%d"' % self.proctored_exam_id, rendered_response)
-        self.assertIn('%s is a Timed Exam' % self.exam_name, rendered_response)
-        self.assertNotIn('Would you Like to take %s as Proctored Exam?' % self.exam_name, rendered_response)
+        self.assertIn(self.timed_exam_msg % self.exam_name, rendered_response)
+        self.assertNotIn(self.start_an_exam_msg % self.exam_name, rendered_response)
 
     def test_getstudentview_unstartedxm(self):
         """
@@ -348,7 +356,7 @@ class ProctoredExamApiTests(LoggedInTestCase):
                 'default_time_limit_mins': 90
             }
         )
-        self.assertIn('you have chosen to take %s as a proctored exam' % self.exam_name, rendered_response)
+        self.assertIn(self.chose_proctored_exam_msg % self.exam_name, rendered_response)
 
     def test_getstudentview_startedxam(self):
         """
@@ -390,4 +398,4 @@ class ProctoredExamApiTests(LoggedInTestCase):
                 'default_time_limit_mins': 90
             }
         )
-        self.assertIn('you did not submit your exam before the time allotted expired', rendered_response)
+        self.assertIn(self.exam_time_expired_msg, rendered_response)