Movido aqui de https://github.com/sebastianbergmann/phpunit/pull/3335#issuecomment -428570649.
Também substitui #3180.
A remoção do parâmetro $maxDepth
de assertEquals()
etc. interromperá todos os testes que usam assertEquals()
com parâmetros opcionais que seguem $maxDepth
.
A única maneira de limpar assertEquals()
que {espero|possivelmente} causa menos problemas seria limpar mais drasticamente e não apenas remover $maxDepth
mas também todos os parâmetros opcionais que vêm depois. Isso também não é realmente atraente.
Todos esses parâmetros opcionais nunca deveriam ter sido adicionados, eles criam uma API terrível. Não tenho certeza, porém, se o benefício de limpar assertEquals()
etc. removendo esses parâmetros opcionais e adicionando métodos de asserção mais específicos em seu lugar supera a dor de quebrar a compatibilidade.
O único parâmetro que usei às vezes é o caso de ignorar. Nunca usei nenhum outro.
Migrar para uma declaração dedicada para isso parece um caminho de atualização razoável para mim.
Substituído por #3341.
Comentários muito úteis
A remoção do parâmetro
$maxDepth
deassertEquals()
etc. interromperá todos os testes que usamassertEquals()
com parâmetros opcionais que seguem$maxDepth
.A única maneira de limpar
assertEquals()
que {espero|possivelmente} causa menos problemas seria limpar mais drasticamente e não apenas remover$maxDepth
mas também todos os parâmetros opcionais que vêm depois. Isso também não é realmente atraente.Todos esses parâmetros opcionais nunca deveriam ter sido adicionados, eles criam uma API terrível. Não tenho certeza, porém, se o benefício de limpar
assertEquals()
etc. removendo esses parâmetros opcionais e adicionando métodos de asserção mais específicos em seu lugar supera a dor de quebrar a compatibilidade.