レガシーなコードに対してのphpunit の書き方

Question

下記のようなコードに対してユニットテストを書こうとしています。
①どのようにしたらテストコードが書けますか？
②かけない場合、どのようにコードを変更すればよいですか？

対象のソースコード)

    <?
    class Sample_Model_Service_A
    {
    private $_result
    private $_options
    private $_someValue
    
    	public function __construct($params, $ids, $data) {
    		$this->_options = Sample_Model_Service_B::getOption($data);
    	}

    	private function setSomeValue() { 
    	// $someValueを$_optionsをもとに整形を行う処理
    		$this->_someValue= $someValue;
    	}
    
    	private function setResult() { 
    	// $someValueをさらに別の観点で整形を行う処理
        // この処理を一部変更したためテストを書きたい
    		$this->_result = $result;
    	}
    
    	public function getter() {
    	// 取り出し
    		retrn $this->_result;
    	}
    }
    ?>
    
    <?
    class Sample_Model_Service_B
    {
        // DBからOptionを取り出す
    	public static function getOption($data) {
    		$dao = new Model_Dao_Option();
    		$option = $dao->getOption($data['id']);
    		return $option;
    	}
    }
    ?>

現状書いているテストコード)

    public function testsetResult()
    {
    // とりあえず途中まで動くように定義
    $params = $ids = $data = [];
    // ReflectionClassが使えると読んで使ってみたが、Sample_Model_Service_Aを生成しきれない（DB周りのテストの仕方が分かっていない）
    $sample = new Sample_Model_Service_A($params, $ids, $data);
    $reflection = new ReflectionClass($sample);
    
    $method = $reflection->getMethod('setresult');
    $method->setAccessible(true);
    
    $result = $method->invoke($sample);
    
    // アサート
    $this->assertSame($result);
    }

Answer 1

Mockeryを用いて依存関係のオブジェクトをmockに置き換える事でコードを変えずにテストを書くことができました。
下記参考記事になります。
https://qiita.com/itatemi/items/9e5e37b3b50791f00a73

Answer 2

// ReflectionClassが使えると読んで使ってみたが、Sample_Model_Service_Aを生成しきれない（DB周りのテストの仕方が分かっていない）

「DB周りのテストの仕方が分かっていない」ので困っているという理解で合ってますか？

PHPは詳しくありませんが、DBに限らず外部への依存があると、ユニットテストは書きにくいです。

一案ですが、ユニットテスト用のoptionsをコンストラクタで渡した場合にDBを参照しないようにすれば、DBへの依存を切り離せます。

PHPをあまり書いたことがないので、構文に間違いがあるかもしれませんが、次のようなイメージです。

public function __construct($params, $ids, $data, $options == null) {
    $this->_options = $options == null ? Sample_Model_Service_B::getOption($data) : $options;
}

testsetResultの中で$optionsをセットします。

public function testsetResult()
{
    ...
    $sample = new Sample_Model_Service_A($params, $ids, $data, $options);
    ...
}

Comments

[0matsuki]

コメントありがとうございます！
下記記事にあるような、コンストラクター・インジェクションの考え方でのテストですね！
https://qiita.com/1000k/items/aef6aed46b0fc34cc15e

今回自分がテストを書こうとしているコードに対してご提示していただいた方法でDB接続を回避する場合、7個ほど引数の追加をしなければならなかったため、断念しました。

折角コメントを付けていただいて申し訳ないのですが、今回はMockeryを使う方法で自己解決しましたので、自己解決の方に解決方法を記載させていただきます。
コメントいただきありがとうございました！

[jhashimoto]

解決したようでよかったです。