名単体テスト 禁断の傀儡（モック）

2025-05-14
シネマ de LT会〜あなたのナレッジ大上映〜
https://aeon.connpass.com/event/352070/

Transcript

1. 名単体テスト 名単体テスト ～ 禁断の ～ 禁断の傀儡 傀儡 ～ ～ モ ッ ク モ ッ ク 2025.5.14　シネマ de L T会＠イオンシネマ シアタス調布 2025.5.14　シネマ de LT会＠イオンシネマ シアタス調布 https:/ /aeon.connpass.com/event/352070/ https://aeon.connpass.com/event/352070/
2. 俺の名前は岩本隆史 俺の名前は岩本隆史 ここ シアタス調布の常連さ ここ シアタス調布の常連さ
3. ENECHANGEって会社で ENECHANGEって会社で インフラを見てるのが表の顔 インフラを見てるのが表の顔
4. 裏では Slackbotを開発してる 裏では Slackbotを開発してる 刑事じゃねえけど「 刑事じゃねえけど「Collmbo Collmbo」ってヤツさ 」ってヤツさ コ ロ ン ボ コ ロ ン ボ
5. ある日 俺はAIエージェントに ある日 俺はAIエージェントに Collmboの単体テストを書かせてみた Collmboの単体テストを書かせてみた
6. だが 生成されたテストに だが 生成されたテストに 霧のような不安を覚えちまった 霧のような不安を覚えちまった
7. @patch("app.bolt_listeners.find_parent_message", return_value=None) @patch("app.bolt_listeners.is_this_app_mentioned") def test_is_child_message_and_mentioned_no_channel_id( mock_is_mentioned, mock_find_parent, mock_client, mock_context ): mock_context.channel_id = None assert is_child_message_and_mentioned( mock_client, mock_context, "12345") is False mock_find_parent.assert_not_called() mock_is_mentioned.assert_not_called() ―― モックだらけじゃねえか ―― モックだらけじゃねえか こんなもん 受け入れていいのか？ こんなもん 受け入れていいのか？
8. そんとき俺は ある本が そんとき俺は ある本が 棚に眠ってんのを思い出した 棚に眠ってんのを思い出した
10. 読み進めて 俺は 読み進めて 俺は ようやく理解できた ようやく理解できた
11. 「モックを単体テストで使うのは悪手」 「モックを単体テストで使うのは悪手」 ってことが ―― ってことが ――
12. モックって奴は モックって奴は 実装を知りすぎてんだ 実装を知りすぎてんだ
13. だから 実装を少し変えるだけで だから 実装を少し変えるだけで 簡単に壊れちまう 簡単に壊れちまう
14. 実装を変えやすくするのが 実装を変えやすくするのが 単体テストの目的だってのによ 単体テストの目的だってのによ
15. bmi :: Float -> Float -> Float bmi weight height = weight / height ^ 2 なら モックのいらない実装に変えりゃいい なら モックのいらない実装に変えりゃいい 理想は「副作用のない純粋関数」さ 理想は「副作用のない純粋関数」さ
16. def build_slack_user_prefixed_text(reply: dict, text: str) -> str: user_identifier = reply.get("user", reply.get("username")) return f"<@{user_identifier}>: {text}" 光の見えた俺は Collmboに 光の見えた俺は Collmboに 純粋関数を増やしてった 純粋関数を増やしてった
17. @pytest.mark.parametrize( "reply, text, expected", [ ({"user": "U123"}, "hello", "<@U123>: hello"), ({"username": "someone"}, "hi", "<@someone>: hi"), ({}, "yo", "<@None>: yo"), ], ) def test_build_slack_user_prefixed_text(reply, text, expected): assert build_slack_user_prefixed_text(reply, text) == expected AIの書いたテストも 今なら AIの書いたテストも 今なら 自信を持って受け入れられるぜ 自信を持って受け入れられるぜ
18. もう俺は モックにゃ頼らねえ もう俺は モックにゃ頼らねえ あくまで単体テストでは ―― だけどな あくまで単体テストでは ―― だけどな
19. 名単体テスト 名単体テスト ～ 禁断の ～ 禁断の傀儡 傀儡 ～ ～ モ ッ ク モ ッ ク END END
20. 速 報
21. 名統合テスト 名統合テスト ～ 抗えぬ ～ 抗えぬ傀儡 傀儡 ～ ～ モ ッ ク モ ッ ク 時期未定 時期未定