PHPUnit сообщения об Exception в консоли — нормально ли?

Question

[Anton B.]

Добрый день.

Подскажите, пожалуйста.
Вопрос возник на конкретном примере для Symfony, но вроде бы к фреймворку не привязан.

В тесте для контроллера хочу протестировать ответ для ситуации, когда один из внутренних сервисов, используемых контроллером выдает Exception. Т.е., например, сервис делает запрос к внешнему API, но возникла ситуация, когда данный API недоступен.

В тесте делаю так:

$serviceStub = $this->createMock(ServiceInterface::class);
// имитируем, что someMethod не имеет доступа к внешнему API
$serviceStub->method('someMethod')->willThrowException(
            new MyException(JsonResponse::HTTP_SERVICE_UNAVAILABLE, 'External API is not available.')
        );
// подменяем сервис в контейнере
$this->client->getContainer()->set(self::SERVICE_ID, $serviceStub);

// далее запрос к контроллеру
$this->client->request(Request::METHOD_GET, self::CONTROLLER_ENDPOINT_URL);
// и далее asserts для response

Такой тест проходится, меня смущает то, что вывод в консоли "не чистый".
Результат тестов зеленый, но из строки с willThrowException сообщения попадают в консоль.
Нормально ли это? Или не правильно делаю?

Пробовал на простом примере (без вложенных сервисов) - с отдельным классом/методом.
Там можно убрать Exception сообщения из консоли
с помощью добавление в тест
$this->expectException(MyException::class);

Comments

[Сергей Романенко]

что именно вам не нравится ? ну я например пишу тесты на PEST, да они немного короче и красивее выглядят

test('test __construct() throws exception InvalidArgumentException with object without __toString', function (): void {
    $strings = new Strings((object) []);
})->throws(InvalidArgumentException::class);

[Anton B.]

Сергей Романенко,
У меня получается так - тест пройден. Но, судя по консоли, Uncaught Exception в строке с new MyException.
Выглядит странно. Получается, что метод отрабатывает корректно при вызове из контроллера, но при этом еще кидается исключение прямо в код теста, которое не перехватывается стандартным для этого способом
$this->expectException();
У меня есть предположение, что эти сообщения в консоли можно просто проигнорировать (все-таки это всего лишь тесты и возможно это нормальное поведение для PHPUnit и данной ситуации).
И есть подозрение, что так лучше не делать (возможно закралась какая-то ошибка в логике обработки исключений в рабочем коде - хотя там вроде все прозрачно и просто, или, например, я неправильно тестирую данную ситуация - хотя тут тоже выглядит логично).

Answer 1

[Антон Шаманов]

expectException это не "убрать из консоли", при тесте на исключения ты обязан указать его тип.

class ExceptionTest extends TestCase
{
    public function testException(): void
    {
        $this->expectException(InvalidArgumentException::class);
    }
}

Comments

[Anton B.]

Могли бы показать какой-нибудь рабочий пример с этой конструкцией?

Я пробую так:

namespace App\Tests\Controller;

use Symfony\Bundle\FrameworkBundle\Test\WebTestCase;
use Exception;

class CustomException extends Exception
{

}

class Demo
{
    public function some()
    {
        echo '123';
    }
}

class SomeControllerTest extends WebTestCase
{
    public function testDemo()
    {
       $this->expectException(CustomException::class);
       $demoStub = $this->createMock(Demo::class);
       $demoStub->method('some')->willThrowException(new CustomException('Exception from some method.'));
       $demoIntance = new $demoStub();
       $demoIntance->some();
    }
}

Но выдает:

Failed asserting that exception of type "App\Tests\Controller\CustomException" is thrown.

Т. е. как бы получается, что метод some должен выбрасывать исключение, но либо этого не происходит, либо оно не перехватывается с помощью $this->expectException.

[Антон Шаманов]

/**
     * @expectedException CustomException
     */
    public function testDemo()
    {
        $stub = $this->createMock(Demo::class);
        $stub->method('doSomething')->will($this->throwException(new CustomException('Exception from some method.'));
        $stub->doSomething();
     }

[Anton B.]

Антон Шаманов, точно, спасибо. У меня ошибка была в том, что я создавал экземпляр объекта-заглушки перед вызовом метода с исключением. А createMock уже возвращает экземпляр.