resolve weird special message in fix_make_board

Author: alexmojaki

core/chapters/c11_tic_tac_toe_project.py

@@ -1768,18 +1768,18 @@ def make_board(size):
         def generate_inputs(cls):
             return dict(size=randint(4, 12))
 
-        # TODO
-        class special_messages_:
+        class special_messages:
             class not_separate:
-                text = "The sublists in the result are not all separate objects."
+                text = "However, the sublists in the result are not all separate objects."
                 program = "pass\ndef make_board(size): return [[' '] * size] * size"
+                expected_exact_match = False
 
         @classmethod
         def check_result(cls, func, inputs, expected_result):
             test_result, func_result = check_result(func, inputs, expected_result)
             if test_result["passed"] and len(func_result) != len(set(map(id, func_result))):
                 test_result["passed"] = False
-                test_result["message"] += "\n\n" + cls.special_messages_.not_separate.text
+                test_result["message"] += "\n\n" + cls.special_messages.not_separate.text
             return test_result
 
     final_text = """

tests/golden_files/en/test_transcript.json

@@ -6665,6 +6665,19 @@
         },
         "step": "naive_make_board"
     },
+    {
+        "page": "Making the Board",
+        "program": [
+            "pass",
+            "def make_board(size): return [[' '] * size] * size"
+        ],
+        "response": {
+            "message": "<p>Given these values:</p>\n<pre><code class=\"codehilite\"><span><span class=\"n\">size</span> <span class=\"o\">=</span> <span class=\"mi\">2</span>\n</span></code></pre>\n<p>your code outputs:</p>\n<pre><code class=\"codehilite\"><span><span class=\"p\">[[</span><span class=\"s1\">' '</span><span class=\"p\">,</span> <span class=\"s1\">' '</span><span class=\"p\">],</span> <span class=\"p\">[</span><span class=\"s1\">' '</span><span class=\"p\">,</span> <span class=\"s1\">' '</span><span class=\"p\">]]</span>\n</span></code></pre>\n<p>which is correct!</p>\n<p>However, the sublists in the result are not all separate objects.</p>",
+            "passed": false,
+            "result": []
+        },
+        "step": "fix_make_board"
+    },
     {
         "get_solution": "program",
         "page": "Making the Board",

tests/test_steps.py

@@ -102,8 +102,13 @@ def normalise_response(response, is_message, substep):
         assert not message_sections
     else:
         section = message_sections[0]
-        assert section == dict(
-            type="messages",
-            messages=[highlighted_markdown(substep.text)],
-        )
+        assert set(section.keys()) == {"type", "messages"}
+        assert section["type"] == "messages"
+        assert len(section["messages"]) == 1
+        message = section["messages"][0]
+        expected = highlighted_markdown(substep.text)
+        if getattr(substep, "expected_exact_match", True):
+            assert expected == message
+        else:
+            assert expected in message
         response["message"] = only(section["messages"])

translations/english.po

@@ -27460,6 +27460,10 @@ msgstr "There are a few ways to copy a list in Python with a tiny bit of code."
 msgid "pages.MakingTheBoard.steps.fix_make_board.hints.8.text"
 msgstr "Making a new row each time can be done by just rearranging the code."
 
+#. https://futurecoder.io/course/#MakingTheBoard
+msgid "pages.MakingTheBoard.steps.fix_make_board.special_messages.not_separate.text"
+msgstr "However, the sublists in the result are not all separate objects."
+
 #. https://futurecoder.io/course/#MakingTheBoard
 #. 
 #.     # __code0__: