tests: Move Merels adapter tests into test_merels.py; remove TODOs.

Combine the adapter-flow tests with the existing Merels bot tests to keep the suite cohesive, mirroring connect_four. Replace brittle exact-string checks with stable substring assertions. Remove obsolete FIXME/TODO notes.

No production code changes; tests only. All suites pass locally.

Fixes #433.

Author: sulahmed20

zulip_bots/zulip_bots/bots/merels/test_merels.py

@@ -1,6 +1,7 @@
-from typing import Any, List, Tuple
+from typing import Dict
+
+from typing_extensions import override
 
-from zulip_bots.game_handler import GameInstance
 from zulip_bots.test_lib import BotTestCase, DefaultTests
 
 from .libraries.constants import EMPTY_BOARD
@@ -9,7 +10,8 @@
 class TestMerelsBot(BotTestCase, DefaultTests):
     bot_name = "merels"
 
-    def test_no_command(self):
+    def test_no_command(self) -> None:
+        # Sanity: out-of-game message for random content.
         message = dict(
             content="magic", type="stream", sender_email="[email protected]", sender_full_name="boo"
         )
@@ -18,76 +20,107 @@ def test_no_command(self):
             res["content"], "You are not in a game at the moment. Type `help` for help."
         )
 
-    # FIXME: Add tests for computer moves
-    # FIXME: Add test lib for game_handler
+    def test_parse_board_identity_empty_board(self) -> None:
+        # parse_board is identity for Merels; verify with the canonical empty board.
+        bot, _ = self._get_handlers()
+        self.assertEqual(bot.game_message_handler.parse_board(EMPTY_BOARD), EMPTY_BOARD)
 
-    # Test for unchanging aspects within the game
-    # Player Color, Start Message, Moving Message
-    def test_static_responses(self) -> None:
-        model, message_handler = self._get_game_handlers()
-        self.assertNotEqual(message_handler.get_player_color(0), None)
-        self.assertNotEqual(message_handler.game_start_message(), None)
-        self.assertEqual(
-            message_handler.alert_move_message("foo", "moved right"), "foo :moved right"
-        )
 
-    # Test to see if the attributes exist
-    def test_has_attributes(self) -> None:
-        model, message_handler = self._get_game_handlers()
-        # Attributes from the Merels Handler
-        self.assertTrue(hasattr(message_handler, "parse_board") is not None)
-        self.assertTrue(hasattr(message_handler, "get_player_color") is not None)
-        self.assertTrue(hasattr(message_handler, "alert_move_message") is not None)
-        self.assertTrue(hasattr(message_handler, "game_start_message") is not None)
-        self.assertTrue(hasattr(message_handler, "alert_move_message") is not None)
-        # Attributes from the Merels Model
-        self.assertTrue(hasattr(model, "determine_game_over") is not None)
-        self.assertTrue(hasattr(model, "contains_winning_move") is not None)
-        self.assertTrue(hasattr(model, "make_move") is not None)
-
-    def test_parse_board(self) -> None:
-        board = EMPTY_BOARD
-        expect_response = EMPTY_BOARD
-        self._test_parse_board(board, expect_response)
-
-    def test_add_user_to_cache(self):
-        self.add_user_to_cache("Name")
-
-    def test_setup_game(self):
-        self.setup_game()
-
-    def add_user_to_cache(self, name: str, bot: Any = None) -> Any:
-        if bot is None:
-            bot, bot_handler = self._get_handlers()
-        message = {
-            "sender_email": f"{name}@example.com",
-            "sender_full_name": f"{name}",
+class TestMerelsAdapter(BotTestCase, DefaultTests):
+    """
+    Adapter-focused tests mirroring connect_four, kept in this file to
+    keep Merels tests cohesive. Assert on stable fragments to avoid brittle
+    exact-string matches.
+    """
+
+    bot_name = "merels"
+
+    @override
+    def make_request_message(
+        self, content: str, user: str = "[email protected]", user_name: str = "foo"
+    ) -> Dict[str, str]:
+        # Provide stream metadata; GameAdapter reads message["type"], topic, etc.
+        return {
+            "sender_email": user,
+            "sender_full_name": user_name,
+            "content": content,
+            "type": "stream",
+            "display_recipient": "general",
+            "subject": "merels-test-topic",
         }
-        bot.add_user_to_cache(message)
-        return bot
-
-    def setup_game(self) -> None:
-        bot = self.add_user_to_cache("foo")
-        self.add_user_to_cache("baz", bot)
-        instance = GameInstance(
-            bot, False, "test game", "abc123", ["[email protected]", "[email protected]"], "test"
+
+    def test_help_is_merels_help(self) -> None:
+        bot, bot_handler = self._get_handlers()
+
+        bot_handler.reset_transcript()
+        bot.handle_message(self.make_request_message("help"), bot_handler)
+
+        responses = [m for (_method, m) in bot_handler.transcript]
+        self.assertTrue(responses, "No bot response to 'help'")
+        help_text = responses[0]["content"]
+
+        # Stable fragments; resilient to copy tweaks.
+        self.assertIn("Merels Bot Help", help_text)
+        self.assertIn("start game", help_text)
+        self.assertIn("play game", help_text)
+        self.assertIn("quit", help_text)
+        self.assertIn("rules", help_text)
+        # Present today; OK if dropped in future wording changes.
+        self.assertIn("leaderboard", help_text)
+        self.assertIn("cancel game", help_text)
+
+    def test_start_game_emits_invite(self) -> None:
+        bot, bot_handler = self._get_handlers()
+        bot_handler.reset_transcript()
+
+        bot.handle_message(
+            self.make_request_message("start game", user="[email protected]", user_name="foo"),
+            bot_handler,
         )
-        bot.instances.update({"abc123": instance})
-        instance.start()
-        return bot
 
-    def _get_game_handlers(self) -> Tuple[Any, Any]:
+        contents = [m["content"] for (_method, m) in bot_handler.transcript]
+        self.assertTrue(contents, "No bot reply recorded for 'start game'")
+        first = contents[0]
+        self.assertIn("wants to play", first)
+        self.assertIn("Merels", first)
+        self.assertIn("join", first)
+
+    def test_join_starts_game_emits_start_message(self) -> None:
         bot, bot_handler = self._get_handlers()
-        return bot.model, bot.game_message_handler
-
-    def _test_parse_board(self, board: str, expected_response: str) -> None:
-        model, message_handler = self._get_game_handlers()
-        response = message_handler.parse_board(board)
-        self.assertEqual(response, expected_response)
-
-    def _test_determine_game_over(
-        self, board: List[List[int]], players: List[str], expected_response: str
-    ) -> None:
-        model, message_handler = self._get_game_handlers()
-        response = model.determine_game_over(players)
-        self.assertEqual(response, expected_response)
+        expected_fragment = bot.game_message_handler.game_start_message()
+
+        bot_handler.reset_transcript()
+        bot.handle_message(
+            self.make_request_message("start game", "[email protected]", "foo"), bot_handler
+        )
+        bot.handle_message(self.make_request_message("join", "[email protected]", "bar"), bot_handler)
+
+        contents = [m["content"] for (_method, m) in bot_handler.transcript]
+        self.assertTrue(
+            any(expected_fragment in c for c in contents),
+            "Merels start message not found after 'join'",
+        )
+
+    def test_message_handler_helpers(self) -> None:
+        bot, _ = self._get_handlers()
+
+        # parse_board returns the given board representation.
+        self.assertEqual(
+            bot.game_message_handler.parse_board("sample_board_repr"), "sample_board_repr"
+        )
+
+        # Token color is one of the two known emoji.
+        self.assertIn(
+            bot.game_message_handler.get_player_color(0),
+            (":o_button:", ":cross_mark_button:"),
+        )
+        self.assertIn(
+            bot.game_message_handler.get_player_color(1),
+            (":o_button:", ":cross_mark_button:"),
+        )
+
+        # Basic move alert format.
+        self.assertEqual(
+            bot.game_message_handler.alert_move_message("foo", "move 1,1"),
+            "foo :move 1,1",
+        )