Updated OTHER submission type handling to replace content with value, adjusted related tests and logic in _extract_muEd_submission.

Author: m-messer

tests/mued_handling_test.py

@@ -192,12 +192,12 @@ def test_text_submission_extracts_text(self):
         self.assertEqual(self.captured["answer"], "hello")
         self.assertEqual(result[0]["awardedPoints"], True)  # type: ignore
 
-    def test_other_submission_extracts_content(self):
+    def test_other_submission_extracts_value(self):
         event = {
             "path": "/evaluate",
             "body": {
-                "submission": {"type": "OTHER", "content": {"content": "some text"}},
-                "task": {"title": "T", "referenceSolution": {"content": "some text"}},
+                "submission": {"type": "OTHER", "content": {"value": "some text"}},
+                "task": {"title": "T", "referenceSolution": {"value": "some text"}},
             },
         }
         result = handler(event)
@@ -399,11 +399,11 @@ def test_text_submission_extracts_text(self):
 
         self.assertEqual(self.captured["response"], "hello")
 
-    def test_other_submission_extracts_content(self):
+    def test_other_submission_extracts_value(self):
         event = {
             "path": "/evaluate",
             "body": {
-                "submission": {"type": "OTHER", "content": {"content": "some text"}},
+                "submission": {"type": "OTHER", "content": {"value": "some text"}},
                 "preSubmissionFeedback": {"enabled": True},
             },
         }

tests/mued_requests_test.py

@@ -51,7 +51,7 @@ def test_valid_minimal_request(self):
         validate.body(body, MuEdReqBodyValidators.EVALUATION)
 
     def test_valid_other_submission_type(self):
-        body = {"submission": {"type": "OTHER", "content": {"content": "some text"}}}
+        body = {"submission": {"type": "OTHER", "content": {"value": "some text"}}}
         validate.body(body, MuEdReqBodyValidators.EVALUATION)
 
     def test_valid_request_with_task(self):

tools/commands.py

@@ -142,7 +142,7 @@ def _extract_muEd_submission(body: JsonType):
     """Extract response, params, and content_key from a muEd request body."""
     submission = body["submission"]
     sub_type = submission.get("type", "OTHER")
-    _type_key = {"MATH": "expression", "TEXT": "text", "CODE": "code", "MODEL": "model", "OTHER": "content"}
+    _type_key = {"MATH": "expression", "TEXT": "text", "CODE": "code", "MODEL": "model"}
     content_key = _type_key.get(sub_type, "value")
     response = submission.get("content", {}).get(content_key)
     params = body.get("configuration", {}).get("params", {})