Codeception / Codeception/module-laravel
Disable Laravel exception handling
- 主要言語
- PHP
- スター
- 6
- フォーク
- 13
- PR マージ指標
- 30日以内にマージされた PR はありません
説明
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!
コントリビューションガイド
調査の方向性
src/Codeception/Lib/Connector/Laravel5/ExceptionHandlerDecorator.php を開き、74行目付近にある参照された条件を確認して、disableExceptionHandling が例外の再スローにどのように影響するかを追跡します。Laravel と Symfony の例外について動作を検証し、リポジトリにカバレッジがある場合は関連するテストを追加または更新します。無効化されたハンドリングが、フレームワークの種類に関係なく例外を再スローすれば完了です。
索引モデルが issue の本文から書いたものです。
評価
- 技術スタック
- laravel, php
- 領域
- testing
- issue の種類
- バグ
- 難易度
- 2/5
- 見積もり時間
- 1〜3時間
- 活発さ
- 停滞
- 明瞭さ
- 明確に書かれている
- 初心者へのやさしさ
- 35/100