1

ラッパークラスを作っています。
このラッパークラスはクラスAを簡単に使うためのものとします。
ラッパークラスのユニットテストをしたいです。

ラッパークラスを使う側はクラスAの使い方や中の実装を意識しないのでテストもクラスAを意識しないものになるはずです。
しかしそれを守りながら作っていたらラッパークラスのテストケースが複雑になってきてしまいました。
ラッパークラスのメソッドのテストで使うテストケースや条件の準備が膨れ上がってしまいました。

これはラッパークラスが責任を持ちすぎなように感じクラス(デリゲート)を分離しようとしましたがそのデリゲートを外から入れるという操作はしたくありません。

このデリゲートは別でテストをして、ラッパークラスをテストする時はデリゲートのモックを使うというようにすればテストも簡潔に済みそうですが、プライベートのデリゲートを外見の振る舞いのようにモックに付けるというのは何か間違っている気がします。

こういった場合に使われるパターンや考え方を教えていただきたいです。
そもそもラッパークラスをテストする必要自体がないのでしょうか?


追記

コメントありがとうございます。一緒に作っている人がいるのでコードは直接は載せられませんが詳しく書きます。
拙い説明なので足りないところがありましたら追記していきます。

似たようなwebhookリクエストを受け取って似たようなWeb上のAPIを呼ぶということを共通に行うためのラッパークラスを作っています。その際テストからコードを書くということの勉強をしていました。
ラッパークラスは使うAPIの種類をコンストラクタで指定してどのAPIのためのラッパーとして動くのかを決めます。

PHPとPHPUnitを使っています。

ラッパークラス

class APIWrapper {

  private $api;

  public function __construct($type) {
    switch($type) {
      // APIの種類ごとに初期化処理をします
      case 'hoge':
      $this->api = new HogeAPI($a, $b, $c);
      break;
      // ここ以外のラッパーのメソッドなどにも登場する分岐を
      // Hoge用の設定・処理、Fuga用の設定・処理とクラスに分けて
      // それらを別クラスに作ってもらい同じインタフェースで使うという方法を
      // 最終的には取っていきたいですが、これも中にあるものなのでラッパーではプライベートです
    }
    // 各APIのクラスの中でグローバル変数を使うことがあります、リクエストヘッダを見たりもします
    // このリクエストヘッダの設定やリクエストボディの設定がテスト時に厄介です
    if (!$this->api->validate()) {
      throw new UnexpectedValueException('ヘッダがおかしいです。');
    }
  }

}

テストクラス

class APIWrapperTest extends PHPUnit\Framework\TestCase {

  public function testInitialize() {
    // ここでリクエストヘッダやグローバル変数を設定します
    // リクエストヘッダを読み込んだりするクラスを別で用意すれば
    // それをどこかのコンストラクタで引数に渡すということもできそうですが最終的にはラッパーなので隠したいです
    $file_get_contents_rtv = 'file_get_contentsの戻り値を上書き';
    $_SERVER['X_HOGE'] = 'dummy';
    $wrapper = new APIWrapper();
  }

}
2
  • 具体的なコードは提示できませんか?たとえば「テストケースや条件の準備が膨れ上がって」いるとありますが、その原因は元のクラスにあるのかもしれませんしラッパークラスにあるのかもしれませんしテストにあるかもしれません。
    – suzukis
    Commented 2017年5月22日 7:53
  • @suzukis コメントありがとうございます。質問をしておいて申し訳ないのですがそのままは載せることが難しいのでやろうとしていることに近いことを追記しました。
    – i5irin
    Commented 2017年5月22日 12:05

0

この質問に回答するには、ログインする必要があります。

のタグが付いた他の質問を参照する。