Phpunit: Remova o atributo $maxDepth, parâmetro, etc.

Criado em 10 out. 2018  ·  4Comentários  ·  Fonte: sebastianbergmann/phpunit

typbackward-compatibility

Comentários muito úteis

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.

Todos 4 comentários

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.

Esta página foi útil?
0 / 5 - 0 avaliações