Funciones de pruebas unitarias con efectos secundarios?

Question

Digamos que está escribiendo una función para verificar si una página fue alcanzada por la URL apropiada. La página tiene un código "canónico"; por ejemplo, mientras se puede acceder a una página en stackoverflow.com/questions/123, preferiríamos (por motivos de SEO) redirigirla a stackoverflow.com/questions/123/how-do -i-move-the-turtle-in-logo - y la redirección real está contenida de forma segura en su propio método (por ejemplo, redirectPage ($ url)), pero ¿cómo se prueba adecuadamente la función que lo llama?

Por ejemplo, tomemos la siguiente función:

function checkStub($questionId, $baseUrl, $stub) { 
    canonicalStub = model->getStub($questionId); 
    if ($stub != $canonicalStub) { 
    redirectPage($baseUrl . $canonicalStub); 
    } 
} 

Si se va a prueba unidad de la función checkStub(), ¿no la redirección en el camino?

Esto es parte de un problema mayor en el que ciertas funciones parecen ser demasiado grandes y dejan el reino de las pruebas unitarias y en el mundo de las pruebas de integración. Inmediatamente, mi mente piensa que los enrutadores y controladores tienen este tipo de problemas, ya que probarlos necesariamente conduce a la generación de páginas en lugar de limitarse a su propia función.

¿Fallo en las pruebas unitarias?

Answer 1

Parece que tiene otro caso de prueba. Debe verificar que el talón se identifica correctamente como un talón con pruebas positivas y negativas, y debe comprobar que la página a la que se le redirige es correcta.

¿O entiendo totalmente la pregunta?

Answer 2

Usted dice ...

Esto es parte de un problema mayor en ciertas funciones parecen ser demasiado grande y dejar el reino de las pruebas unitarias y en el mundo de las pruebas de integración

Creo que esta es la razón por la cual las pruebas unitarias son (1) duras y (2) conducen a un código que no se desmorona por su propio peso. Tienes que ser meticuloso para romper todas tus dependencias o terminas con pruebas de unidad == pruebas de integración.

En su ejemplo, se inyectaría un redirector como dependencia. Utiliza un simulacro, doble o espía. Luego haces las pruebas cuando @atk establece. A veces no vale la pena. Más a menudo te obliga a escribir un mejor código. Y es difícil hacerlo sin un contenedor de IOC.

Answer 3

Esta es una vieja pregunta, pero creo que esta respuesta es relevante. @Rob indica que se inyectaría un redirector como dependencia y, por supuesto, esto funciona. Sin embargo, su problema es que no tiene una buena separación de preocupaciones.

Necesita hacer que sus funciones sean lo más atómicas posible, y luego componer una funcionalidad más grande usando las funciones granulares que ha creado. Usted escribió esto:

function checkStub($questionId, $baseUrl, $stub) { 
    canonicalStub = model->getStub($questionId); 
    if ($stub != $canonicalStub) { 
    redirectPage($baseUrl . $canonicalStub); 
    } 
} 

me gustaría escribir esto:

function checkStubEquality($stub1, $stub2) { 
    return $stub1 == $stub2; 
    } 

    canonicalStub = model->getStub($questionId); 
    if (!checkStubEquality(canonicalStub, $stub)) redirectPage($baseUrl . $canonicalStub);