Phpunit: Eliminar el atributo $maxDepth, parámetro, etc.

Creado en 10 oct. 2018  ·  4Comentarios  ·  Fuente: sebastianbergmann/phpunit

typbackward-compatibility

Comentario más útil

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.

Todos 4 comentarios

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.

¿Fue útil esta página
0 / 5 - 0 calificaciones