Codeception / Codeception/module-laravel
Disable Laravel exception handling
- 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
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