Bugfix en Intento.getPruebas*Pasadas() que hacía que devuelva true si no compilaba.

[software/sercom-old.git] / src / sc_test

diff --git a/src/sc_test b/src/sc_test
index 59b4eff22c23dcd0d5b07b9d290ac95d50c76f56..8bd2cf07c95904ccfd0b146fdd016089fd5e73f6 100755 (executable)
--- a/src/sc_test
+++ b/src/sc_test
@@ -209,10 +209,10 @@ def probar(intento, caso_de_prueba, mail):
     prueba.pasada = True # Asumo que está bien, ya habrá tiempo para cambiarlo
     obs = diff(prueba, mail, proc.stdout.readlines(),
         file(os.path.join(caso_de_prueba.path, 'stdout')).readlines(),
     prueba.pasada = True # Asumo que está bien, ya habrá tiempo para cambiarlo
     obs = diff(prueba, mail, proc.stdout.readlines(),
         file(os.path.join(caso_de_prueba.path, 'stdout')).readlines(),

-        'stdout', longname='La salida estándar')
+        caso_de_prueba.nombre + '.stdout', longname='La salida estándar')

     obs += diff(prueba, mail, proc.stderr.readlines(),
         file(os.path.join(caso_de_prueba.path, 'stderr')).readlines(),
     obs += diff(prueba, mail, proc.stderr.readlines(),
         file(os.path.join(caso_de_prueba.path, 'stderr')).readlines(),

-        'stderr', longname='La salida de error')
+        caso_de_prueba.nombre + '.stderr', longname='La salida de error')

     for f in caso_de_prueba.archivosSalida:
         if f not in prueba.archivosSalida:
             #TODO agregar error FALTA ARCHIVO
     for f in caso_de_prueba.archivosSalida:
         if f not in prueba.archivosSalida:
             #TODO agregar error FALTA ARCHIVO


@@ -222,7 +222,8 @@ def probar(intento, caso_de_prueba, mail):
             continue
         obs += diff(prueba, mail,
             file(os.path.join(intento.chrootPath, f)).readlines(),
             continue
         obs += diff(prueba, mail,
             file(os.path.join(intento.chrootPath, f)).readlines(),

-            file(os.path.join(caso_de_prueba.pathSalidas, f)).readlines(), f)
+            file(os.path.join(caso_de_prueba.pathSalidas, f)).readlines(),
+            caso_de_prueba.nombre + '.' + f)

     for f in prueba.archivosSalida - (caso_de_prueba.archivosSalida | caso_de_prueba.archivosEntrada):
         #TODO agregar error SOBRA ARCHIVO
         log.debug('El programa debía generar el archivo %s y no lo hizo.', f)
     for f in prueba.archivosSalida - (caso_de_prueba.archivosSalida | caso_de_prueba.archivosEntrada):
         #TODO agregar error SOBRA ARCHIVO
         log.debug('El programa debía generar el archivo %s y no lo hizo.', f)


@@ -315,7 +316,7 @@ class MailIntento(email.MIMEMultipart.MIMEMultipart, object):
         self['To'] = intento.mailRespuesta
         self['Reply-To'] = conf.get('mail', 'admin')
         self['Return-Path'] = conf.get('mail', 'admin')
         self['To'] = intento.mailRespuesta
         self['Reply-To'] = conf.get('mail', 'admin')
         self['Return-Path'] = conf.get('mail', 'admin')

-        self['X-Mailer'] = 'sercom 0.3'
+        self['X-Mailer'] = 'sercom ' + sercom.VERSION

         self['X-Priority'] = '5'
         self.epilogue = 'Para ver correctamente este e-mail su cliente debe ' \
             'soportar MIME.\n\n'
         self['X-Priority'] = '5'
         self.epilogue = 'Para ver correctamente este e-mail su cliente debe ' \
             'soportar MIME.\n\n'


@@ -401,7 +402,6 @@ while continuar:
         preparar(intento)
         # Pruebo y agrego prueba a la lista
         prueba = probar(intento, caso_de_prueba, mail)
         preparar(intento)
         # Pruebo y agrego prueba a la lista
         prueba = probar(intento, caso_de_prueba, mail)

-        resultado = resultado and prueba.pasada

         if not prueba.casoDePrueba.privado and not prueba.pasada:
             resultado = False
         # Limpio chroot
         if not prueba.casoDePrueba.privado and not prueba.pasada:
             resultado = False
         # Limpio chroot