Ir al contenido
wannacode

Tipos de revisión

En resumen: Una revisión tiene cinco pasos: planificación, inicio, revisión individual, comunicación y análisis, y corrección e informe. Los roles son autor, revisores, moderador, secretario, líder de la revisión y gerente. Los tipos se diferencian en la formalidad: informal, recorrido, revisión técnica e inspección. La lista de comprobación dice qué buscar, y se revisa el documento, no al autor.

Problema

«Lo hemos revisado»

La especificación de las devoluciones está lista. La analista la lleva a la reunión del lunes. Seis personas la ven por primera vez en la pantalla. Alguien pregunta por la tipografía, dos discuten cómo se guardará en la base de datos, el dueño dice «tiene buena pinta». Una hora después se acaba la reunión. Nadie anotó nada.

Dos semanas después resulta que los «14 días» se cuentan desde el pedido, mientras que la ley cuenta desde la recepción. Todos estuvieron en la reunión. Nadie leyó el texto.

La reunión no fue una revisión. Fue gente reunida alrededor de un documento.

La idea

Los cinco pasos

Una revisión de verdad tiene un proceso. En ISTQB CTFL tiene cinco pasos:

  1. Planificación. Qué se revisa, con qué objetivo, quién, para cuándo, con qué lista de comprobación, y cuándo termina la revisión.
  2. Inicio. Todos reciben el documento y conocen su rol y qué buscar.
  3. Revisión individual. Cada uno lee solo, antes de la reunión, y anota sus hallazgos. Aquí es donde más se encuentra.
  4. Comunicación y análisis. Los hallazgos se discuten juntos. Para cada uno: ¿es un problema?, ¿qué importancia tiene?, ¿quién lo corrige?
  5. Corrección e informe. El autor corrige. Se registran los hallazgos y la decisión: ¿se acepta el documento, o necesita otra revisión?

La reunión del lunes fue solo el paso 4, sin 1, 2 y 3. Por eso se habló de la tipografía.

Más precisamente, para curiosos

CTFL v4.0.1 (sección 3.2) describe el proceso de revisión de un producto de trabajo, con referencia a la norma ISO/IEC 20246. Cuánto del proceso se sigue depende del tipo de revisión: una informal puede saltarse casi todo, una inspección lo sigue por completo.

Pruébalo

Ordena el proceso

Ordena los pasos de la revisión.

  1. Corrección e informe: el autor corrige, y se registran la decisión y los hallazgos
  2. Comunicación y análisis: los hallazgos se discuten juntos y se toman decisiones
  3. Inicio: todos reciben el documento y conocen su rol
  4. Revisión individual: cada uno lee solo y anota sus hallazgos
  5. Planificación: qué se revisa, quién, para cuándo, con qué lista de comprobación

La idea

Quién hace qué

Rol Qué hace En la revisión de las devoluciones
autor creó el documento y lo corrige la analista
gerente decide qué se revisa, aporta personas y tiempo el jefe del equipo
moderador (facilitador) dirige la reunión, vigila el tiempo y el tono alguien neutral, no el autor
secretario (scribe) registra los hallazgos y las decisiones alguien del equipo
revisor lee y encuentra problemas QA, un programador, el almacén, el dueño
líder de la revisión responde de la revisión en conjunto: quién participa, cuándo, dónde a menudo el mismo que el moderador

En un equipo pequeño una persona tiene varios roles. Lo que importa es que los roles existan, no que haya seis personas.

Los revisores son más útiles cuando miran desde sitios distintos. El almacén preguntará «¿quién recibe el producto devuelto?». Contabilidad preguntará «¿cuándo se reembolsa el dinero?». QA preguntará «¿y el día 14?».

Pruébalo

¿De quién es el rol?

1. La analista que escribió la especificación del código promocional responde preguntas y después corrige el texto.
2. Alguien dirige la reunión: da la palabra, devuelve la charla a los hallazgos cuando dos personas empiezan a discutir cómo se va a construir.
3. Alguien anota cada hallazgo con su lugar y lo que se decidió sobre él.
4. El jefe del equipo decide que la revisión se hará y da tiempo a la gente para ella.
5. El tester y la persona del almacén leen la especificación y buscan problemas.

La idea

Los cuatro tipos

Las revisiones se diferencian en lo formales que son. Cuanto más arriesgado es el documento, más compensa la formalidad.

Tipo Quién dirige Objetivo En la tienda
revisión informal nadie; sin proceso encontrar problemas, rápido «échale un vistazo rápido a los criterios»
recorrido (walkthrough) el autor encontrar problemas, que la gente entienda el documento y se ponga de acuerdo la analista guía al almacén por la especificación de las devoluciones
revisión técnica un moderador consenso y una decisión sobre una cuestión técnica cómo se calculan los descuentos
inspección un moderador formado encontrar tantos problemas como sea posible; las métricas mejoran el proceso la especificación del pago

Una revisión informal no es mala. Es la más frecuente y la más barata. Pero cuando un error cuesta dinero a los clientes, como en el pago, compensa el proceso completo.

Más precisamente, para curiosos

En CTFL v4.0.1 la inspección es el único tipo donde el proceso se sigue por completo y se recogen métricas. Allí el autor no puede actuar como líder de la revisión ni como secretario. El tipo se elige según el objetivo, el riesgo, la cultura del equipo y los requisitos del ámbito: en ámbitos regulados (medicina, transporte) las inspecciones suelen ser obligatorias.

Pruébalo

¿Qué tipo es?

1. El programador escribe en el chat: «Échale un vistazo rápido a los criterios antes de que empiece». El compañero responde con dos comentarios.
2. La analista reúne al dueño, al almacén y a QA y recorre la especificación de las devoluciones línea por línea para que todos la entiendan.
3. Tres programadores con experiencia y la arquitecta se reúnen para decidir cómo calculará la tienda los descuentos, después de que cada uno haya leído la propuesta.
4. La especificación del pago pasa por el proceso completo: roles formados, una lista de comprobación, cada hallazgo registrado, su número seguido de proyecto en proyecto.

La idea

La lista de comprobación

Un revisor que lee «a ver si todo está bien» ve poco. Uno que tiene preguntas ve mucho. Una lista de comprobación es una lista de preguntas que se hacen a todos los documentos de un mismo tipo.

Una lista de comprobación para los requisitos de la tienda:

  • ¿Dice quién (cliente, empleado, cualquier visitante)?
  • ¿Hay una medida en lugar de «rápido», «cómodo», «claro»?
  • ¿Está claro qué pasa en el límite (50,00 €, el día 14, la décima unidad)?
  • ¿Dice qué pasa en el caso negativo (cancelación, error, código caducado)?
  • ¿Hay una lista cerrada en lugar de «etc.»?
  • ¿Dice qué, no cómo?
  • ¿Contradice a otro requisito?

La lista de comprobación está viva. Cada vez que un defecto se escapa de una revisión, el equipo pregunta «¿qué pregunta lo habría cazado?» y la añade.

Y una regla sobre el tono: se revisa el documento, no al autor. «Aquí el límite no está claro», no «no pensaste en el límite». Una revisión en la que se juzga a alguien no se repite de buena gana.

Más precisamente, para curiosos

CTFL v4.0.1 enumera también qué hace que las revisiones tengan éxito: objetivos y criterios de salida claros; el tipo adecuado para el objetivo; trozos pequeños, para que los revisores no se cansen; tiempo suficiente para prepararse; apoyo de la dirección; una cultura donde una revisión sirve para aprender, no para castigar. Además de las listas de comprobación hay otras formas de lectura: basada en escenarios (el revisor recorre el documento con un uso concreto) y basada en perspectivas (uno lee como cliente, otro como tester, un tercero como el almacén).

Ejercicio

Una revisión con lista de comprobación

1. ¿Qué pregunta lo caza? Para cada requisito, elige la pregunta de la lista de comprobación que encuentra el problema.

1. El requisito: «El cliente recibe puntos de fidelidad por cada pedido.» ¿Qué pregunta de la lista de comprobación encuentra un problema?
2. El requisito: «El envío es gratis para pedidos de más de 50 €.» ¿Qué pregunta encuentra un problema?
3. El requisito: «El descuento se guarda en el campo discount_pct de la tabla orders.» ¿Qué pregunta encuentra un problema?

Test

Ponte a prueba

1. ¿Qué es cierto de la revisión individual?
2. ¿Qué tipo de revisión es el más formal?
3. ¿Quién dirige un recorrido (walkthrough)?
4. ¿Por qué una revisión discute el documento y no al autor?
5. ¿Por qué es útil una lista de comprobación?

Creado por Ivo Gergov como parte de la campaña de Evo Tech por una educación accesible en programación ·