Как замокать результаты метода, чтобы он не делал API call?

Question

[driverx18]

есть ситуация, есть метод, который я хочу протестировать, есть метод:

public function getPaidShops() : array
    {
        $categories = $this->getCategories();

        $data = $this->getLicense($categories);

        return $this->format($data);
    }

всё бы хорошо, но вот тут вызывается метод getLicense который делает API call. Внутри него происходит такое:

private function getLicense(array $categories)
    {
        return app(APIService::class)->getLicenses($categories);
    }

Я хотел бы, чтобы этот метод не лез в апишку (тем более оно вызвать ее не может из тестов, потому что выдается ошибка, что session не установлена), а хотел бы конкретно замокать этот метод. Как это можно сделать?
Как вариант, можно переписать код, чтобы в тестируемый класс в конструктор передавался APIService (а не создавался напрямую в методе), и в юнит тестах в конструктор передавать мок этого класса с данными которые подкинутся в getLicenses из дата провайдера
Но переписывать код не хотелось бы и заменять во всех местах проекта, может есть какие-то еще варианты решения?

Comments

[Максим Федоров]

переписать на DI

Answer 1

[Илья]

Метод getLicense сделайте protected и для теста наследуйтесь от основного класса и подменяйте метод getLicense.
Тогда вообще ничего переписывать не придется.

public function testCase()
{
    $mockObject = $this->createMock(NeedClass::class);
    // чем это принципиально отличается от мока с точки зрения теста - непонятно
    $mockObject = new class extends NeedClass {
        protected function getLicense(array $categories)
        {
            return 'need license';
        }
    };

    $this->assertEqual($mockObject->getPaidShops(), 'actual value');
}

Как вариант, можно переписать код, чтобы в тестируемый класс в конструктор передавался APIService (а не создавался напрямую в методе), и в юнит тестах в конструктор передавать мок этого класса с данными которые подкинутся в getLicenses из сервис провайдера

Ну вообще так и надо сделать, DI все дела :)

Comments

[driverx18]

не совсем понял
имеете ввиду мне нужно создать какой то класс, который будет наследовать основной, и в нем переопределять метод getLicenses?
Но в таком случае это только плодит ненужный класс, который будет нужен только для тестирования, но будет создан не в папке tests

[Максим Федоров]

ого вы извращенец в первом предложении

[Илья]

Максим Федоров, пару причин почему это извращение? Тот же самый мок, только на "живом" объекте, где подменяется только 1 метод.

[Илья]

driverx18,

Но в таком случае это только плодит ненужный класс, который будет нужен только для тестирования, но будет создан не в папке tests

Для начала: почему не в папке тест? Для продолжения можно создать анонимный класс внутри теста. Моки создаются на лету, а этот класс у вас будет под конкретный тест (может хоть в файле теста его создать), принциальной разницы с моком (который повторю на лету строится) не будет.

Для продолжения: "нужен только для тестирования", а простите моки для чего нужны?