test cases tweaks

edx_proctoring/api.py

@@ -87,7 +87,7 @@ def update_exam(exam_id, exam_name=None, time_limit_mins=None,
     if is_proctored is not None:
         proctored_exam.is_proctored = is_proctored
     if is_practice_exam is not None:
-        proctored_exam.is_practice_exam=is_practice_exam,
+        proctored_exam.is_practice_exam = is_practice_exam
     if external_id is not None:
         proctored_exam.external_id = external_id
     if is_active is not None:

edx_proctoring/models.py

@@ -121,7 +121,7 @@ class ProctoredExamStudentAttemptManager(models.Manager):
         Returns the Student Exam Attempts for the given course_id.
         """
 
-        return self.filter(proctored_exam__course_id=course_id)
+        return self.filter(proctored_exam__course_id=course_id).order_by('-created')
 
     def get_filtered_exam_attempts(self, course_id, search_by):
         """

edx_proctoring/serializers.py

@@ -29,6 +29,7 @@ class ProctoredExamSerializer(serializers.ModelSerializer):
     time_limit_mins = serializers.IntegerField(required=True)
 
     is_active = StrictBooleanField(required=True)
+    is_practice_exam = StrictBooleanField(required=False)
     is_proctored = StrictBooleanField(required=True)
 
     class Meta:
@@ -39,7 +40,7 @@ class ProctoredExamSerializer(serializers.ModelSerializer):
 
         fields = (
             "id", "course_id", "content_id", "external_id", "exam_name",
-            "time_limit_mins", "is_proctored", "is_active"
+            "time_limit_mins", "is_proctored", "is_practice_exam", "is_active"
         )
 
 

edx_proctoring/tests/test_api.py

@@ -473,8 +473,8 @@ class ProctoredExamApiTests(LoggedInTestCase):
         )
         filtered_attempts = get_filtered_exam_attempts(self.course_id, self.user.username)
         self.assertEqual(len(filtered_attempts), 2)
-        self.assertEqual(filtered_attempts[0]['id'], exam_attempt.id)
-        self.assertEqual(filtered_attempts[1]['id'], new_exam_attempt)
+        self.assertEqual(filtered_attempts[0]['id'], new_exam_attempt)
+        self.assertEqual(filtered_attempts[1]['id'], exam_attempt.id)
 
     def test_get_all_exam_attempts(self):
         """
@@ -493,8 +493,8 @@ class ProctoredExamApiTests(LoggedInTestCase):
         )
         all_exams = get_all_exam_attempts(self.course_id)
         self.assertEqual(len(all_exams), 2)
-        self.assertEqual(all_exams[0]['id'], exam_attempt.id)
-        self.assertEqual(all_exams[1]['id'], updated_exam_attempt_id)
+        self.assertEqual(all_exams[0]['id'], updated_exam_attempt_id)
+        self.assertEqual(all_exams[1]['id'], exam_attempt.id)
 
     def test_get_student_view(self):
         """