Как замокать внутренние вызовы методов?

Question

[v3shin]

Всем привет. Есть класс с последовательными вызовами:

class MyClass
{
    public function getExternalData1($request): ExternalResponse1 { ... }
    public function getExternalData2($request): ExternalResponse2 { ... }
    public function static functionToTest($someData)
    {
        $data1 = getExternalData1($someData);
        $data2 = getExternalData2($data1->value);
        return $data2;
    }
}

Пытаюсь его замокать:

public function testFunctionToTest()
{
    $getExternalData1Value = new ExternalResponse1(['value' => 'someValue']);
    $getExternalData2Value = new ExternalResponse2();

    $myClassMock = $this->getMockBuilder(MyClass::class)
        ->disableProxyingToOriginalMethods()
        ->onlyMethods(['getExternalData1', 'getExternalData2'])
        ->getMock();

    $myClassMock->method('getExternalData1')->willReturn($getExternalData1Value);
    $myClassMock->method('getExternalData2')->willReturn($getExternalData2Value);

    $myClassMock->functionToTest([ ... ]);
}

Получаю ошибку:

[TypeError] Argument 1 passed to Mock_MyClass::getExternalData2() must be of the type string, null given, called in ...

При этом $myClassMock->getExternalData1('...')->value возвращает правильное значение 'someValue'. Как это правильно протестировать?

Comments

[Дмитрий]

Респонс можно чистый отдавать и без моков. Методы для ответов нужно сделать приватными, они протестируются через один, все три

[v3shin]

Дмитрий, getExternalData1() и getExternalData2() получают данные со стороннего сервиса. Как мне их протестировать, если не моками?

[Дмитрий]

v3shin, подумал это обычные ответы. А у вас версия юнита какая?
Может аргумент в моке не указали

[v3shin]

Дмитрий, 8.5.21.
Кажись, я догадался. Прошу прощения, не указал важную, как оказалось особенность: functionToTest() я объявил статической. Переписал все на обычные методы - заработало.

Answer 1

[v3shin]

Всем спасибо за участие. Мой косяк: я указал functionToTest() статической. Переписал все на обычные методы - тесты заработали.

Answer 2

[AgentSmith72]

Вы не передаёте аргумент в ExternalResponse2()
$getExternalData2Value = new ExternalResponse2();
И если, метод используется, только внутри класса, то нужно объявлять его как private.

private function getExternalData2($request): ExternalResponse2 { ... }

Comments

[v3shin]

1. На самом деле передаю, но в данном случае это не важно.
2. Получаю ошибку

[PHPUnit\Framework\Warning] Trying to configure method "getExternalData1" which cannot be configured because it does not exist, has not been specified, is final, or is static