Почему expectException не работает в функциональных тестах?

Question

[Денис]

Почему expectException не работает в функциональных тестах? То есть элементарно это утверждение никогда ни при каких условиях не срабатывает в функциональных тестах. В юнит вроде всё хорошо. Причём приходится использовать альтернативу:

$response = $this->post(route('items.use', ['id' => -1, 'count' => 0]));

$this->assertTrue($response->exception->getCode() == 0);

В контроллере:

$item = Item::findOrFail($id);

А вот такие утверждения:

$this->expectException(InvalidArgumentException::class);

Никогда не срабатывают.

А свойство "исключение" вообще никак не связано с утверждением expectException, expectExceptionCode, expectExceptionMessage и так далее. Они все не работают. Да, в принципе можно работать и со свойством exception, но
у него нет метода возвращающего тип исключения. Не удобно идентифицировать исключения по коду или ошибке. Ну и в принципе хочется разобраться когда и что использовать и в чём разница.

Comments

[FanatPHP]

а если написать \InvalidArgumentException::class

[Денис]

FanatPHP, я пробовал все варианты, и так, и выносить в экспорт, и в кавычках как строку, все-все способы

[FanatPHP]

стоп. а там точно InvalidArgumentException кидается?

[Денис]

FanatPHP,
Я думаю что это связано как-то с http. Потому что выбрасываются они а не обычные. Точно не помню но кажись в unit тестах выбрасываются обычные и они норм ловятся

[FanatPHP]

я извиняюсь, но тут написано совсем не InvalidArgumentException
ну то есть я не понял логику.
зачем писать $this->expectException(InvalidArgumentException::class); и ждать что исключение будет поймано, если кидается совсем другое исключение?

[Денис]

FanatPHP, но суть даже не совсем в этом. Вот я отладил и увидел какой класс исключения, норм ок. но тест говорит об обратном: "Failed asserting that exception of type "Symfony\Component\HttpKernel\Exception\NotFoundHttpException" is thrown." Абсолютно одинаково что на скрине что тут. Может генерируется совсем другое исключение которое редиректит на страницу 404.blade и там переменная excеption отображает другую информацию. хз

[FanatPHP]

Ну здесь не могу сказать
подписался на комментарии

[Денис]

FanatPHP, Вот, я написал выше. Суть не в имени исключения. Это у меня там разные выкидывались и поэтому приводил другое. Даже если и совпадают то не работает. Вот еще информация:

dd(get_class($response->exception));

"Illuminate\Database\Eloquent\ModelNotFoundException"

Было бы прекрасно если бы это было решением проблемы, но всё также:

Failed asserting that exception of type "Illuminate\Database\Eloquent\ModelNotFoundException" is thrown.

[Денис]

FanatPHP,

$this->assertTrue(get_class($response->exception) == ModelNotFoundException::class);

Что думаете о таком варианте? Не костыльно? работает так.

[neol]

Вангую, что у вас в тесте

$response = …;
$this->expectException(InvalidArgumentException::class);

а должно быть

$this->expectException(InvalidArgumentException::class);
$response = …;

т. е. сначала говорите phpunit, что ждёте исключение, потом вызываете код, бросающий его, а не наоборот.

[Денис]

neol, эх если бы было так просто :( Я знаю о том что это должно быть в начале, хотя пробовал и так и так. Ну в общем я использую как описал выше с get_class - в принципе работает. Все исключения библиотеки Assertion (у меня большинство их) имеют один и тот же класс что удобно. Остальные вроде как в основном это модели, с которыми вроде как разобрался тоже.

Answer 1

[jazzus]

В http тестах findorfail проверяется максимум assertNotFound. Это логично. Http запрос ничего про php код/Ларавел не знает и знать не должен. Если нужно тестировать код, тогда нужно писать юнит тесты.

Comments

[Денис]

спасибо за ответ) но я немного не понял первое ваше предложение. В итоге, если я проверяю исключения как описал в комментариях под вопросом

$this->assertTrue(get_class($response->exception) == ModelNotFoundException::class);

Это логично?

[jazzus]

Денис, нет. http запрос получает код ответа, а не эксепшен. Findorfail вернет 404 в случае fail. Который и проверяет assertNotFound. И в http когда нужно протестировать ответ тестируют $response, а не $this. $this к запросу вообще не относится.

$this->post($route)
->assertNotFound();

[jazzus]

Денис, https://laravel.com/docs/8.x/http-tests#assert-not...

[Денис]

jazzus, хм, интересно, спасибо, не знал) действительно работает. Получается все запросы в бд от модели возвращают такое? Еще наверное несуществующие страницы. А что еще можно протестировать этим assertNotFound?

[jazzus]

Денис, да нету там никаких моделей. assertNotFound проверяет что СТАТУС ответа 404. ВСЕ. Ты делаешь http запрос и получаешь ответ. findorfail для удобства возвращает not found, чтобы ты это руками не делал и проверки не писал. И модели тут непричем. Ларавел просто формирует ответ за тебя. Я ссылку дал. Ничего больше проверять кроме как в документации не нужно. Ни одного велосипеда. Как и в целом по Ларавел. Есть по ссылке твой пример с ModelNotFound? Нет. Ну и забудь) Значит решение не верное.