モックとスタブの落とし穴：ユニットテストにおける注意点

ユニットテストは、ソフトウェア開発において不可欠な品質保証の手段です。
しかし、テストの作成方法によっては、思わぬ落とし穴に陥る可能性があります。

特に、モックとスタブの使用には注意が必要です。
本記事では、モックとスタブの問題点を探り、より信頼性の高いテスト手法について考察します。

モックとスタブの主な問題点

モックとスタブを使用する際の主な問題点は以下の2つです。

実際の動作とかけ離れたテストになる可能性
テストが実装の詳細に密接に結びつく可能性

これらの問題点について、具体例を交えて詳しく見ていきましょう。

実際の動作とかけ離れたテスト

モックやスタブを使用する際、テスト作成者はそれらの振る舞いを定義する必要があります。
しかし、この過程で実際のクラスや関数の動作と異なる振る舞いを定義してしまう可能性があります。

例えば、以下のような注文処理システムを考えてみましょう。

class OrderProcessor:
    def __init__(self, inventory_checker):
        self.inventory_checker = inventory_checker

    def process_order(self, product_id, quantity):
        available = self.inventory_checker.check_availability(product_id)
        if available >= quantity:
            # 注文処理のロジック
            return "注文が成功しました"
        else:
            return "在庫不足です"

class InventoryChecker:
    def check_availability(self, product_id):
        # 実際の在庫チェックロジック
        pass

このシステムをテストする際、InventoryCheckerをモックして以下のようなテストを書くかもしれません。

import unittest
from unittest.mock import Mock

class TestOrderProcessor(unittest.TestCase):
    def test_process_order_success(self):
        mock_inventory = Mock()
        mock_inventory.check_availability.return_value = 10

        processor = OrderProcessor(mock_inventory)
        result = processor.process_order("PROD001", 5)

        self.assertEqual(result, "注文が成功しました")

    def test_process_order_failure(self):
        mock_inventory = Mock()
        mock_inventory.check_availability.return_value = 3

        processor = OrderProcessor(mock_inventory)
        result = processor.process_order("PROD001", 5)

        self.assertEqual(result, "在庫不足です")

これらのテストは一見問題なさそうに見えます。
しかし、実際のInventoryCheckerクラスが負の値を返す可能性がある場合（例えば、商品が存在しない場合に-1を返すなど）、このテストではそのシナリオをカバーできていません。

実装の詳細との密接な結びつき

モックやスタブを使用すると、テストが実装の詳細に密接に結びつく可能性があります。
これは、コードのリファクタリングを困難にする原因となります。

例えば、先ほどのOrderProcessorクラスを以下のようにリファクタリングしたとします。

class OrderProcessor:
    def __init__(self, inventory_service):
        self.inventory_service = inventory_service

    def process_order(self, product_id, quantity):
        try:
            self.inventory_service.reserve_stock(product_id, quantity)
            # 注文処理のロジック
            return "注文が成功しました"
        except InsufficientStockError:
            return "在庫不足です"

class InventoryService:
    def reserve_stock(self, product_id, quantity):
        # 在庫の予約処理
        # 在庫が不足している場合はInsufficientStockErrorを発生させる
        pass

このリファクタリングにより、在庫チェックと予約が一つのメソッドにまとまり、例外処理を使用するようになりました。
しかし、先ほどのテストケースはcheck_availabilityメソッドの呼び出しを期待しています。

そのため、このリファクタリング後のコードではテストが失敗してしまいます。