web-dev-qa-db-ja.com

新しいクラスオブジェクトによって呼び出されたメソッドをモックするための単体テスト

私はこのような既存のコードの単体テストを書いています

class someClass {
    public function __construct() { ... }

    public function someFoo($var) {
        ...
        $var = "something";
        ...
        $model = new someClass();
        model->someOtherFoo($var);
    }

    public someOtherFoo($var){
         // some code which has to be mocked
    }
}

ここで、someOtherFooの内部で "some code"が実行されないように、関数 "someOtherFoo"の呼び出しをモックする方法を教えてください。

class someClassTest {
   public function someFoo() {
      $fixture = $this->getMock('someClass ', array('someOtherFoo'));
      $var = "something";
      ....
      // How to mock the call to someOtherFoo() here
   }

}

コンストラクターをモックアウトして、独自に作成した関数または変数を返すことはできますか?

ありがとう

32
Sumitk

テスト中のメソッドにnew XXX(...)がある場所はどこでもあります。同じクラスの新しいメソッド--createSomeClass(...)--にインスタンス化を抽出します。これにより、新しいメソッドからスタブ値またはモック値を返す、テスト中のクラスの部分モックを作成できます。

_class someClass {
    public function someFoo($var) {
        $model = $this->createSomeClass();  // call method instead of using new
        model->someOtherFoo($var);
    }

    public function createSomeClass() {  // now you can mock this method in the test
        return new someClass();
    }

    public function someOtherFoo($var){
         // some code which has to be mocked
    }
}
_

テストでは、createSomeClass()を呼び出すインスタンスでsomeFoo()をモックし、最初のモックされた呼び出しから戻るインスタンスでsomeOtherFoo()をモックします。

_function testSomeFoo() {
    // mock someOtherFoo() to ensure it gets the correct value for $arg
    $created = $this->getMock('someClass', array('someOtherFoo'));
    $created->expects($this->once())
            ->method('someOtherFoo')
            ->with('foo');

    // mock createSomeClass() to return the mock above
    $creator = $this->getMock('someClass', array('createSomeClass'));
    $creator->expects($this->once())
            ->method('createSomeClass')
            ->will($this->returnValue($created));

    // call someFoo() with the correct $arg
    $creator->someFoo('foo');
}
_

インスタンスは同じクラスの別のインスタンスを作成しているため、通常は2つのインスタンスが関係することに注意してください。明確になっている場合は、ここで同じモックインスタンスを使用できます。

_function testSomeFoo() {
    $fixture = $this->getMock('someClass', array('createSomeClass', 'someOtherFoo'));

    // mock createSomeClass() to return the mock
    $fixture->expects($this->once())
            ->method('createSomeClass')
            ->will($this->returnValue($fixture));

    // mock someOtherFoo() to ensure it gets the correct value for $arg
    $fixture->expects($this->once())
            ->method('someOtherFoo')
            ->with('foo');

    // call someFoo() with the correct $arg
    $fixture->someFoo('foo');
}
_
29
David Harkness

モッククラス名の前にoverload:を付けることができます

Mocking Hard Dependencies のドキュメントを確認してください。

あなたの例は次のようになります:

/**
 * @runTestsInSeparateProcesses
 * @preserveGlobalState disabled
 */
class SomeClassTest extends \PHPUnit\Framework\TestCase
{
    public function test_some_foo()
    {
        $someOtherClassMock = \Mockery::mock('overload:SomeOtherClass');
        $someOtherClassMock->shouldReceive('someOtherFoo')
            ->once()
            ->with('something')
            ->andReturn();

        $systemUnderTest = new SomeClass();

        $systemUnderTest->someFoo('something');
    }

}

@runTestsInSeparateProcessesアノテーションを追加しました。通常、モックされたクラスは他のテストでも使用されるためです。アノテーションがないと、class already existsエラーのためにオートローダーがクラッシュします。

これが、モックされたクラスがテストスイートで使用される唯一の場所である場合は、アノテーションを削除する必要があります。

6
Jeff Puckett

ここで、クラス__constructorをホワイトボックステストして、それがクラスメソッドを呼び出し、一部のデータが__constructorに渡されることを確認する方法を見つけました。

同じ理由で他の誰かがここにいる場合、私は私が最終的に使用したメソッドを共有すると思いました(この質問で使用されているファクトリスタイルのcreateSomeClass()メソッドなしで)。

<?php
class someClass {

  public function __constructor($param1) {
    // here is the method in the constructor we want to call
    $this->someOtherFoo($param1);
  }

  public function someOtherFoo($var){  }

}

PHPUnitテスト:

<?php
$paramData = 'someData';

// set up the mock class here
$model = $this->getMock('someClass', 
  array('someOtherFoo'), // override the method we want to check
  array($paramData) // we need to pass in a parameter to the __constructor
);

// test that someOtherFoo() is called once, with out test data
$model->expects($this->once())
      ->with($paramData)
      ->method('someOtherFoo');

// directly call the constructor, instead of doing "new someClass" like normal
$model->__construct($paramData);
2
thaddeusmt