Phpunit: $ maxDepth属性、パラメーターなどを削除します。

作成日 2018年10月10日  ·  4コメント  ·  ソース: sebastianbergmann/phpunit

typbackward-compatibility

最も参考になるコメント

assertEquals() $maxDepthパラメータを削除すると、$ $maxDepthの後に続くオプションのパラメータでassertEquals()を使用するすべてのテストが失敗します。

{うまくいけば|おそらく}問題が少ないassertEquals()をクリーンアップする唯一の方法は、より大幅にクリーンアップし、 $maxDepthだけでなく、その後に続くすべてのオプションのパラメーターも削除することです。 これもあまり魅力的ではありません。

これらのオプションのパラメータはすべて追加されるべきではありませんでした。それらはひどいAPIになります。 ただし、これらのオプションのパラメーターを削除し、代わりにさらに具体的なアサーションメソッドを追加することで、 assertEquals()などをクリーンアップすることの利点が、互換性を壊す苦痛を上回るかどうかはわかりません。

全てのコメント4件

また、#3180を置き換えます。

assertEquals() $maxDepthパラメータを削除すると、$ $maxDepthの後に続くオプションのパラメータでassertEquals()を使用するすべてのテストが失敗します。

{うまくいけば|おそらく}問題が少ないassertEquals()をクリーンアップする唯一の方法は、より大幅にクリーンアップし、 $maxDepthだけでなく、その後に続くすべてのオプションのパラメーターも削除することです。 これもあまり魅力的ではありません。

これらのオプションのパラメータはすべて追加されるべきではありませんでした。それらはひどいAPIになります。 ただし、これらのオプションのパラメーターを削除し、代わりにさらに具体的なアサーションメソッドを追加することで、 assertEquals()などをクリーンアップすることの利点が、互換性を壊す苦痛を上回るかどうかはわかりません。

私が時々使用した唯一のパラメーターは、大文字と小文字を区別しないものです。 他のものを使用したことはありません。

そのための専用のアサーションに移行することは、私にとって合理的なアップグレードパスのようです。

#3341に置き換えられました。

このページは役に立ちましたか?
0 / 5 - 0 評価