また、#3180を置き換えます。
assertEquals()
$maxDepth
パラメータを削除すると、$ $maxDepth
の後に続くオプションのパラメータでassertEquals()
を使用するすべてのテストが失敗します。
{うまくいけば|おそらく}問題が少ないassertEquals()
をクリーンアップする唯一の方法は、より大幅にクリーンアップし、 $maxDepth
だけでなく、その後に続くすべてのオプションのパラメーターも削除することです。 これもあまり魅力的ではありません。
これらのオプションのパラメータはすべて追加されるべきではありませんでした。それらはひどいAPIになります。 ただし、これらのオプションのパラメーターを削除し、代わりにさらに具体的なアサーションメソッドを追加することで、 assertEquals()
などをクリーンアップすることの利点が、互換性を壊す苦痛を上回るかどうかはわかりません。
私が時々使用した唯一のパラメーターは、大文字と小文字を区別しないものです。 他のものを使用したことはありません。
そのための専用のアサーションに移行することは、私にとって合理的なアップグレードパスのようです。
#3341に置き換えられました。
最も参考になるコメント
assertEquals()
$maxDepth
パラメータを削除すると、$$maxDepth
の後に続くオプションのパラメータでassertEquals()
を使用するすべてのテストが失敗します。{うまくいけば|おそらく}問題が少ない
assertEquals()
をクリーンアップする唯一の方法は、より大幅にクリーンアップし、$maxDepth
だけでなく、その後に続くすべてのオプションのパラメーターも削除することです。 これもあまり魅力的ではありません。これらのオプションのパラメータはすべて追加されるべきではありませんでした。それらはひどいAPIになります。 ただし、これらのオプションのパラメーターを削除し、代わりにさらに具体的なアサーションメソッドを追加することで、
assertEquals()
などをクリーンアップすることの利点が、互換性を壊す苦痛を上回るかどうかはわかりません。