chore: Updated the place where the list is reversed

learning_assistant/api.py

@@ -217,6 +217,6 @@ def get_message_history(courserun_key, user, message_count):
 
     Returns a number of messages equal to the message_count value.
     """
-    message_history = LearningAssistantMessage.objects.filter(
-        course_id=courserun_key, user=user).order_by('-created')[:message_count]
+    message_history = list(LearningAssistantMessage.objects.filter(
+        course_id=courserun_key, user=user).order_by('-created')[:message_count])[::-1]
     return message_history

learning_assistant/views.py

@@ -236,6 +236,6 @@ def get(self, request, course_run_id):
         user = request.user
 
         message_count = int(request.GET.get('message_count', 50))
-        message_history = reversed(list(get_message_history(courserun_key, user, message_count)))  # Reversing order
+        message_history = get_message_history(courserun_key, user, message_count)
         data = MessageSerializer(message_history, many=True).data
         return Response(status=http_status.HTTP_200_OK, data=data)

tests/test_api.py

@@ -365,8 +365,8 @@ def test_get_message_history(self):
 
         return_value = get_message_history(self.course_key, self.user, message_count)
 
-        expected_value = LearningAssistantMessage.objects.filter(
-            course_id=self.course_key, user=self.user).order_by('-created')[:message_count]
+        expected_value = list(LearningAssistantMessage.objects.filter(
+            course_id=self.course_key, user=self.user).order_by('-created')[:message_count])[::-1]
 
         # Ensure same number of entries
         self.assertEqual(len(return_value), len(expected_value))

tests/test_views.py

@@ -374,7 +374,7 @@ def test_learning_message_history_view_get(
         self.assertEqual(len(data), db_messages_count)
 
         # Ensure values are as expected
-        for i, expected in enumerate(db_messages):
-            self.assertEqual(expected.role, data[i]['role'])
-            self.assertEqual(expected.content, data[i]['content'])
-            self.assertEqual(expected.created.isoformat(), data[i]['timestamp'])
+        for i, message in enumerate(data):
+            self.assertEqual(message['role'], db_messages[i].role)
+            self.assertEqual(message['content'], db_messages[i].content)
+            self.assertEqual(message['timestamp'], db_messages[i].created.isoformat())