Codeception / Codeception/module-laravel

Disable Laravel exception handling

Abierto
#2 1 comentario 0 reacciones 0 asignados Ver en GitHub
Lenguaje dominante
PHP
Estrellas
6
Forks
13
Métricas de merge de PR
Sin PR fusionados en 30 d

Descripción

Laravel v7.10.3

I was trying to `$i->disableExceptionHandling()` and noticed that it wasn't actually getting taken care of. I would gladly PR this as it seems like a simple change:

https://github.com/Codeception/module-laravel5/blob/bd2c604e8aa02d2b24737de01c6716473e8db96c/src/Codeception/Lib/Connector/Laravel5/ExceptionHandlerDecorator.php#L74

Any objections to removing the 2nd condition from the if statement? From a "disable exception handling" standpoint I don't care if it's a laravel or symfony exception, I just want it to get re-thrown.

Any objections to this change?

It looks like @janhenkgerritsen is familiar with these issues in some capacity, so I am tagging for visibility!

Guía de contribución

Abrir la guía de contribución

Línea de trabajo

Abre src/Codeception/Lib/Connector/Laravel5/ExceptionHandlerDecorator.php en la condición referenciada alrededor de la línea 74 y sigue cómo disableExceptionHandling afecta al relanzamiento de excepciones. Verifica el comportamiento para las excepciones de Laravel y Symfony, y después añade o actualiza la prueba relevante si el repositorio proporciona cobertura; se considera terminado cuando el manejo deshabilitado vuelve a lanzar la excepción independientemente de su tipo de framework.

Escrito por el modelo de indexación a partir del texto del issue.

Evaluación

Stack tecnológico
laravel, php
Área
testing
Tipo de issue
Error
Dificultad
2/5
Tiempo estimado
1-3 horas
Estado de actividad
Estancado
Claridad
Bien especificado
Aptitud para principiantes
35/100

Recibe los nuevos issues en tu correo

Un resumen breve de issues de GitHub para principiantes.