Codeception / Codeception/module-laravel
Disable Laravel exception handling
- Langage dominant
- PHP
- Étoiles
- 6
- Forks
- 13
- Métriques de merge des PR
- Aucune PR mergée en 30 j
Description
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!
Guide de contribution
Ouvrir le guide de contribution
Piste de recherche
Ouvrez src/Codeception/Lib/Connector/Laravel5/ExceptionHandlerDecorator.php à la condition référencée autour de la ligne 74 et suivez comment disableExceptionHandling influe sur la relance des exceptions. Vérifiez le comportement pour les exceptions Laravel et Symfony, puis ajoutez ou mettez à jour le test pertinent si le dépôt fournit une couverture ; le travail est terminé lorsque la gestion désactivée relance l'exception quel que soit son type de framework.
Rédigé par le modèle d'indexation à partir du texte de l'issue.
Évaluation
- Stack technique
- laravel, php
- Domaine
- testing
- Type d'issue
- Bug
- Difficulté
- 2/5
- Temps estimé
- 1-3 heures
- Activité
- À l'abandon
- Clarté
- Clairement spécifiée
- Accessibilité débutants
- 35/100