feat: Adding chat messages to the DB

learning_assistant/models.py

@@ -35,7 +35,15 @@ class LearningAssistantMessage(TimeStampedModel):
     .. pii_retirement: third_party
     """
 
+    USER_ROLE = 'user'
+    ASSISTANT_ROLE = 'assistant'
+
+    Roles = (
+        (USER_ROLE, USER_ROLE),
+        (ASSISTANT_ROLE, ASSISTANT_ROLE),
+    )
+
     course_id = CourseKeyField(max_length=255, db_index=True)
     user = models.ForeignKey(USER_MODEL, db_index=True, on_delete=models.CASCADE)
-    role = models.CharField(max_length=64)
+    role = models.CharField(choices=Roles, max_length=64)
     content = models.TextField()

learning_assistant/serializers.py

@@ -2,6 +2,7 @@
 Serializers for the learning-assistant API.
 """
 from rest_framework import serializers
+from learning_assistant.models import LearningAssistantMessage
 
 
 class MessageSerializer(serializers.Serializer):  # pylint: disable=abstract-method
@@ -16,7 +17,7 @@ def validate_role(self, value):
         """
         Validate that role is one of two acceptable values.
         """
-        valid_roles = ['user', 'assistant']
+        valid_roles = [LearningAssistantMessage.USER_ROLE, LearningAssistantMessage.ASSISTANT_ROLE]
         if value not in valid_roles:
             raise serializers.ValidationError('Must be valid role.')
         return value

learning_assistant/views.py

@@ -4,6 +4,7 @@
 import logging
 
 from django.conf import settings
+from django.contrib.auth import get_user_model
 from edx_rest_framework_extensions.auth.jwt.authentication import JwtAuthentication
 from opaque_keys import InvalidKeyError
 from opaque_keys.edx.keys import CourseKey
@@ -12,7 +13,6 @@
 from rest_framework.permissions import IsAuthenticated
 from rest_framework.response import Response
 from rest_framework.views import APIView
-
 try:
     from common.djangoapps.course_modes.models import CourseMode
     from common.djangoapps.student.models import CourseEnrollment
@@ -21,10 +21,12 @@
     pass
 
 from learning_assistant.api import get_course_id, learning_assistant_enabled, render_prompt_template
+from learning_assistant.models import LearningAssistantMessage
 from learning_assistant.serializers import MessageSerializer
 from learning_assistant.utils import get_chat_response, user_role_is_staff
 
 log = logging.getLogger(__name__)
+User = get_user_model()
 
 
 class CourseChatView(APIView):
@@ -76,6 +78,15 @@ def post(self, request, course_run_id):
         unit_id = request.query_params.get('unit_id')
 
         message_list = request.data
+
+        # Check that the last message in the list corresponds to a user
+        new_user_message = message_list[-1]
+        if new_user_message['role'] != LearningAssistantMessage.USER_ROLE:
+            return Response(
+                status=http_status.HTTP_400_BAD_REQUEST,
+                data={'detail': "Expects user role on last message."}
+            )
+
         serializer = MessageSerializer(data=message_list, many=True)
 
         # serializer will not be valid in the case that the message list contains any roles other than
@@ -103,6 +114,22 @@ def post(self, request, course_run_id):
         )
         status_code, message = get_chat_response(prompt_template, message_list)
 
+        user = User.objects.get(id=request.user.id)  # Based on the previous code, user exists.
+
+        # Save the user message to the database.
+        LearningAssistantMessage.objects.create(
+            user=user,
+            role=LearningAssistantMessage.USER_ROLE,
+            content=new_user_message['content'],
+        )
+
+        # Save the assistant response to the database.
+        LearningAssistantMessage.objects.create(
+            user=user,
+            role=LearningAssistantMessage.ASSISTANT_ROLE,
+            content=message['content'],
+        )
+
         return Response(status=status_code, data=message)
 
 

tests/test_views.py

@@ -13,6 +13,7 @@
 from django.test import TestCase, override_settings
 from django.test.client import Client
 from django.urls import reverse
+from learning_assistant.models import LearningAssistantMessage
 
 User = get_user_model()
 
@@ -170,16 +171,29 @@ def test_chat_response_default(
 
         test_data = [
             {'role': 'user', 'content': 'What is 2+2?'},
-            {'role': 'assistant', 'content': 'It is 4'}
+            {'role': 'assistant', 'content': 'It is 4'},
+            {'role': 'user', 'content': 'And what else?'},
         ]
 
         response = self.client.post(
             reverse('chat', kwargs={'course_run_id': self.course_id})+f'?unit_id={test_unit_id}',
             data=json.dumps(test_data),
             content_type='application/json'
         )
+
         self.assertEqual(response.status_code, 200)
 
+        last_rows = LearningAssistantMessage.objects.all().order_by('-created').values()[:2][::-1]
+
+        user_msg = last_rows[0]
+        assistant_msg = last_rows[1]
+
+        self.assertEqual(user_msg['role'], LearningAssistantMessage.USER_ROLE)
+        self.assertEqual(user_msg['content'], test_data[2]['content'])
+
+        self.assertEqual(assistant_msg['role'], LearningAssistantMessage.ASSISTANT_ROLE)
+        self.assertEqual(assistant_msg['content'], 'Something else')
+
         render_args = mock_render.call_args.args
         self.assertIn(test_unit_id, render_args)
         self.assertIn('This is the default template', render_args)