Как покрыть подобный метод юнит тестом?

Question

[Twitt]

Есть метод, который нужно покрыть юнит тестом. Суть его такова: на вход ему приходит $clientId, сам метод делает

try {
            $data = $this->read($clientId);
        }
        catch (\Exception $exception) {
            // тут обработка эксепшена
        }

т.е. он идет в БД и ищет этот clientId, в ином случае кидается эксепшн. Вопрос, как ему подкинуть корректный clientId в юнит тесте в дата провайдере? Ведь если я укажу существующие сейчас clientId СТАТИЧЕСКИ, и с ними пройдут юнит тесты, и когда то удалят эти clientId из БД, то юнит тесты зафейлятся. Как быть в таких ситуациях? Мне нужно в юнит тесте сходить в бд, и выбрать те clientId, которые существуют, и их прокинуть в датапровайдер? Как сделать это по умному?

Answer 1

[green_goo]

Ведь если я укажу существующие сейчас clientId, и с ними пройдут юнит тесты, и когда то удалят эти clientId из БД, то юнит тесты зафейлятся

Юнит тесты ходят в реальную бд? Это неправильно.
Нужно заменять в тесте слой взаимодействия с бд заглушкой, которая будет возвращать нужные данные, либо для тестов своя бд, где перед тестом она заполняется нужными данными

Comments

[Twitt]

не совсем понимаю, как это сделать
допустим код который я писал выше находится в классе UserInfo в методе getInfoByClientId, я в юнит тесте делаю метод testGetInfoByClientId, в котором я делаю
$info = new UserInfo();
$info->getInfoByClientId(тут айди из дата провайдера);

Он же полюбому будет обращаться к БД, потому что мы обращаемся к UserInfo, который уже приучен ходить в базу данных. Разве нет?

[green_goo]

Twitt,

Он же полюбому будет обращаться к БД, потому что мы обращаемся к UserInfo, который уже приучен ходить в базу данных. Разве нет?

Может так у тебя и реализовано, но это неправильно.
Тестовое окружение должно быть отдельно и не зависеть от внешних условий
А если использовать в своем коде DI, то всегда можно в тестах использовать заглушки.

[Twitt]

green_goo, ну я же дергаю метод класса UserInfo, который даже не подозревает что он необходим для юнит тестов. Он автоматом сходит в БД.

"Нужно заменять в тесте слой взаимодействия с бд заглушкой, которая будет возвращать нужные данные, либо для тестов своя бд, где перед тестом она заполняется нужными данными"

1. Как заменить слой?
2. Для PhpUnit возможно где-то указать свою БД?

[green_goo]

Twitt, если использовать DI, то подбросить зависимости в тестах не проблема. Если нет, то могут возникнуть сложности.

Answer 2

[Николай Конюхов]

Сделайте мок тестируемого сервиса, переопределите методы, которые находятся в тестируемом методе, но нет необходимости их тестировать, верните ожидаемое для тестового сценария значение:

$serivceMock = $this->getMockBuilder(Service::class)
            ->setMethods(['read',])
            ->getMock();
$serviceMock->expects($this->once())
            ->method('read')
            ->willReturn($preparedResult);
$serviceMock->execute($clientId);
// there is checks about work with preparedResult

Или, например, при проверке ситуации с ненайденным пользователем

$serivceMock = $this->getMockBuilder(Service::class)
            ->setMethods(['read',])
            ->getMock();
$serviceMock->expects($this->once())
            ->method('read')
            ->willThrowException(new NotFoundException());
$serviceMock->execute($clientId);
// there is checks about work with not found exception