Movido aquí desde https://github.com/sebastianbergmann/phpunit/pull/3335#issuecomment -428570649.
También reemplaza al #3180.
Eliminar el parámetro $maxDepth
de assertEquals()
etc. romperá todas las pruebas que usan assertEquals()
con parámetros opcionales que van después $maxDepth
.
La única forma de limpiar assertEquals()
que {con suerte|posiblemente} cause menos problemas sería limpiar más drásticamente y no solo eliminar $maxDepth
sino también todos los parámetros opcionales que vienen después. Esto tampoco es realmente atractivo.
Todos estos parámetros opcionales nunca deberían haberse agregado, son una API terrible. No estoy seguro, sin embargo, si el beneficio de limpiar assertEquals()
etc. eliminando estos parámetros opcionales y agregando métodos de aserción más específicos en su lugar supera el dolor de romper la compatibilidad.
El único parámetro que utilicé a veces es el caso de ignorar uno. Nunca usé ningún otro.
Migrar a una aserción dedicada para eso me parece una ruta de actualización razonable.
Reemplazado por #3341.
Comentario más útil
Eliminar el parámetro
$maxDepth
deassertEquals()
etc. romperá todas las pruebas que usanassertEquals()
con parámetros opcionales que van después$maxDepth
.La única forma de limpiar
assertEquals()
que {con suerte|posiblemente} cause menos problemas sería limpiar más drásticamente y no solo eliminar$maxDepth
sino también todos los parámetros opcionales que vienen después. Esto tampoco es realmente atractivo.Todos estos parámetros opcionales nunca deberían haberse agregado, son una API terrible. No estoy seguro, sin embargo, si el beneficio de limpiar
assertEquals()
etc. eliminando estos parámetros opcionales y agregando métodos de aserción más específicos en su lugar supera el dolor de romper la compatibilidad.