Phpunit: Supprimez l'attribut $maxDepth, le paramètre, etc.
Déplacé ici depuis https://github.com/sebastianbergmann/phpunit/pull/3335#issuecomment -428570649.
sebastianbergmann
Tous les 4 commentaires
Remplace également #3180.
localheinz
le 10 oct. 2018
La suppression du paramètre $maxDepth de assertEquals() etc. cassera tous les tests qui utilisent assertEquals() avec des paramètres optionnels qui vont après $maxDepth .
La seule façon de nettoyer assertEquals() qui {espérons-le|probablement} cause moins de problèmes serait de nettoyer plus radicalement et non seulement de supprimer $maxDepth mais aussi tous les paramètres optionnels qui viennent après. Cela aussi n'est pas vraiment attrayant.
Tous ces paramètres optionnels n'auraient jamais dû être ajoutés, ils font une API terrible. Je ne suis pas sûr, cependant, si l'avantage de nettoyer assertEquals() etc. en supprimant ces paramètres facultatifs et en ajoutant des méthodes d'assertion plus spécifiques à leur place l'emporte sur la douleur de briser la compatibilité.
sebastianbergmann
le 10 oct. 2018
Le seul paramètre que j'ai utilisé parfois est celui d'ignorer la casse. Jamais utilisé d'autre.
Migrer vers une assertion dédiée pour cela me semble un chemin de mise à niveau raisonnable.
Jean85
le 10 oct. 2018
Remplacé par #3341.
sebastianbergmann
le 11 oct. 2018
Questions connexes
ezzatron
·
3Commentaires
gellis
·
4Commentaires
edyan
·
4Commentaires
sebastianbergmann
·
4Commentaires
kunjalpopat
·
4Commentaires
Commentaire le plus utile
La suppression du paramètre
$maxDepthdeassertEquals()etc. cassera tous les tests qui utilisentassertEquals()avec des paramètres optionnels qui vont après$maxDepth.La seule façon de nettoyer
assertEquals()qui {espérons-le|probablement} cause moins de problèmes serait de nettoyer plus radicalement et non seulement de supprimer$maxDepthmais aussi tous les paramètres optionnels qui viennent après. Cela aussi n'est pas vraiment attrayant.Tous ces paramètres optionnels n'auraient jamais dû être ajoutés, ils font une API terrible. Je ne suis pas sûr, cependant, si l'avantage de nettoyer
assertEquals()etc. en supprimant ces paramètres facultatifs et en ajoutant des méthodes d'assertion plus spécifiques à leur place l'emporte sur la douleur de briser la compatibilité.