Evita Usar isTesting u Otros Flags Similares

No dejes que el código de prueba llegue a producción

Resumen: Evita agregar flags como isTesting o similares en tu código.

Problemas ??

• Abstracción deficiente
• Contaminación con código no relacionado al negocio
• Código frágil
• Comportamiento inconsistente
• Dependencias ocultas
• Dificultad en la depuración
• Uso de flags booleanas
• Pruebas no confiables
• Código dependiente del entorno de producción

Soluciones ??

1. Eliminar condicionantes innecesarios
2. Utilizar inyección de dependencias
3. Modelar servicios externos sin usar mocks
4. Separar configuraciones
5. Aislar la lógica de prueba
6. Mantener límites de comportamiento bien definidos

Refactorización ??

El uso de patrones adecuados permite eliminar flags innecesarias en la lógica del código.

Contexto ??

Cuando agregas flags como isTesting, mezclas código de prueba y producción, creando caminos ocultos que solo se activan en pruebas. Esto impide cubrir correctamente el código en producción y aumenta el riesgo de errores.

Ejemplo de Código ??

Incorrecto ?

struct PaymentService {
    is_testing: bool,
}

impl PaymentService {
    fn process_payment(&self, amount: f64) {
        if self.is_testing {
            println!('Modo de prueba: Omitiendo pago real');
            return;
        }
        println!('Procesando pago de ${}', amount);
    }
}

Correcto ??

trait PaymentProcessor {
    fn process(&self, amount: f64);
}

struct RealPaymentProcessor;
impl PaymentProcessor for RealPaymentProcessor {
    fn process(&self, amount: f64) {
        println!('Procesando pago de ${}', amount);
    }
}

struct TestingPaymentProcessor;
impl PaymentProcessor for TestingPaymentProcessor {
    fn process(&self, _: f64) {
        println!('No se ejecuta pago: Omitiendo transacción real');
    }
}

struct PaymentService {
    processor: T,
}

impl PaymentService {
    fn process_payment(&self, amount: f64) {
        self.processor.process(amount);
    }
}

Detección ??

• [x] Semi-Automática

Se puede detectar esta mala práctica buscando flags condicionales como isTesting, environment == 'test' o DEBUG_MODE, indicios de que la lógica de prueba se está filtrando a la producción.

Etiquetas ???

• Pruebas

Nivel ??

• [x] Intermedio

Importancia de la separación de entornos ???

Es esencial mantener una separación clara entre código de prueba y producción. Si se mezclan, se rompe la correspondencia entre el comportamiento real y el funcionamiento del programa.

Generación de código por IA ??

Las herramientas de IA suelen introducir este problema al priorizar soluciones rápidas sobre un buen diseño.

Detección de IA ??

Si se configuran correctamente, las herramientas de detección pueden identificar este problema al analizar lógica condicional basada en estados de prueba.

Conclusión ??

Evita utilizar flags como isTesting en tu código. En su lugar, aplica inyección de dependencias y modela los entornos para garantizar una separación clara entre la lógica de prueba y producción.

En Q2BSTUDIO, una empresa especializada en desarrollo y servicios tecnológicos, promovemos las mejores prácticas de desarrollo para garantizar código limpio y mantenible. Nuestro equipo de expertos se asegura de aplicar principios sólidos de arquitectura de software, evitando problemas como la filtración de código de prueba en producción.