Un poco de contexto: hoy tuve que actualizar un código SQL que proporcionó otro colega mío, y como es un script bastante grande, se almacena como un archivo separado (que luego se lee y ejecuta en tiempo de ejecución). Mientras hacía esto, volví a introducir accidentalmente dos errores que teníamos hace unos meses, a saber:
- Por cualquier motivo, el archivo ASCII se codificó en UTF-16 (el colega me envió el archivo por correo electrónico, lo que podría haberlo causado).
- En el script faltaban las declaraciones iniciales
SET
(necesarias debido a algunos elementos del controlador en producción, pero no en una instalación limpia localmente).
Después de depurar esto durante aproximadamente una hora (otra vez) decidí escribir algunas pruebas unitarias para garantizar que esto nunca volvería a suceder (e incluir una forma rápida de corregirlo en el mensaje de confirmación para proporcionar una solución fácil para los futuros desarrolladores).
Sin embargo, cuando presioné este código, otro colega (que también es el líder de nuestro equipo) se me acerca y me dice que no debería volver a hacer estas cosas porque:
"Estas cosas no pertenecen a las pruebas unitarias"
"Las pruebas unitarias solo deben usarse para verificar el flujo de su código"
Ahora estoy bastante en conflicto ya que todavía pienso que lo que estoy haciendo no está mal, ya que este error no se volvería a presentar en el futuro, sin embargo, este colega trabaja como senior y al final del día Para decidir en qué gastamos nuestro tiempo. ¿Qué tengo que hacer? ¿Me equivoco al hacerlo de esta manera? ¿Se considera una mala práctica?