X-Git-Url: https://git.llucax.com/software/sercom-old.git/blobdiff_plain/979651724ed20e70239d742aa29e0cbeac4bec8b..c6b56e0a25a47f75854a46dbcc9201988d7ee949:/src/sc_test

diff --git a/src/sc_test b/src/sc_test
index aeac6d8..479f75e 100755
--- a/src/sc_test
+++ b/src/sc_test
@@ -91,26 +91,27 @@ Salida:
     intento.observaciones = msg + '\n\n'
     log.debug(msg)
 
-def preparar(intento):
+def preparar(intento, caso_de_prueba):
     # Creo chroot - TODO copiarlo de algún lado donde ande el valgrind?
-    os.mkdir(intento.chrootPath)
+    os.makedirs(intento.chrootPath(caso_de_prueba))
     shutil.copy(os.path.join(intento.path, 'tp'),
-        os.path.join(intento.chrootPath, 'tp'))
+        os.path.join(intento.chrootPath(caso_de_prueba), 'tp'))
 
-def limpiar(intento):
+def limpiar(intento, caso_de_prueba):
     # Borro chroot entero
-    shutil.rmtree(intento.chrootPath)
+    #shutil.rmtree(intento.chrootPath)
+    os.unlink(os.path.join(intento.chrootPath(caso_de_prueba), 'tp'))
 
 def probar(intento, caso_de_prueba, mail):
     def diff(prueba, mail, orig, new, name, origname='alumno', newname='catedra', longname=None):
         from difflib import unified_diff, HtmlDiff
-        #TODO diferenciar pruebas públicas de privadas
         if longname is None: longname = name
         diff = ''.join(list(unified_diff(orig, new, fromfile=name+'.'+origname, tofile=name+'.'+newname)))
         nota = ''
         if diff:
             prueba.pasada = False
-            nota = "- %s no coincide con lo esperado (archivo '%s.diff').\n" % (longname, name)
+            nota += "* %s no coincide con lo esperado (archivo '%s.diff').\n" % (longname, name)
+            nota += ('-' * 72) + '\n' + diff + ('-' * 72) + '\n'
             if not prueba.casoDePrueba.privado:
                 mail.attachText(diff, name + '.diff')
                 diff = HtmlDiff().make_file(orig, new, fromdesc=name+'.'+origname, todesc=name+'.'+newname, context=True, numlines=3)
@@ -140,9 +141,9 @@ def probar(intento, caso_de_prueba, mail):
     log.debug('Caso de prueba: %s', prueba.casoDePrueba)
     # Abro archivos para fds básicos
     options = {
-        'cwd': intento.chrootPath,
+        'cwd': intento.chrootPath(caso_de_prueba),
         'close_fds': True,
-        'preexec_fn': secure_process(intento.chrootPath, uid, gid, tiempo_cpu),
+        'preexec_fn': secure_process(intento.chrootPath(caso_de_prueba), uid, gid, tiempo_cpu),
     }
     if usa_stdin():
         options['stdin'] = file(os.path.join(caso_de_prueba.path, 'stdin'), 'r')
@@ -160,7 +161,7 @@ def probar(intento, caso_de_prueba, mail):
         proc = subprocess.Popen(params, **options)
     except Exception, e: # FIXME poner en el manejo de exceptiones estandar
         try:
-            print e.child_traceback
+            log.debug('ERROR! Trace del hijo: %s', e.child_traceback)
             raise
         except:
             raise
@@ -187,6 +188,7 @@ def probar(intento, caso_de_prueba, mail):
     # Salió con una señal?
     if proc.returncode < 0:
         sig = -proc.returncode
+        # Conversor de nro de señal a string
         sigs = {}
         for s in [s for s in dir(signal) if s.startswith('SIG') and s.isalpha()]:
             sigs[getattr(signal, s)] = s
@@ -218,8 +220,9 @@ def probar(intento, caso_de_prueba, mail):
             return prueba
     # Verifico salidas estándar/de error
     prueba.pasada = True # Asumo que está bien, ya habrá tiempo para cambiarlo
+    obs = ''
     if usa_stdout():
-        obs = diff(prueba, mail, proc.stdout.readlines(),
+        obs += diff(prueba, mail, proc.stdout.readlines(),
             file(os.path.join(caso_de_prueba.path, 'stdout')).readlines(),
             caso_de_prueba.nombre + '.stdout', longname='La salida estándar')
     if usa_stderr():
@@ -228,20 +231,18 @@ def probar(intento, caso_de_prueba, mail):
         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
             log.debug('Falta el archivo %s y no lo hizo.', f)
             prueba.pasada = False
-            obs += "- Falta el archivo de salida '%s'.\n" % f
+            obs += "* Falta el archivo de salida '%s'.\n" % f
             continue
         obs += diff(prueba, mail,
-            file(os.path.join(intento.chrootPath, f)).readlines(),
+            file(os.path.join(intento.chrootPath(caso_de_prueba), f)).readlines(),
             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)
         prueba.pasada = False
-        obs += "- Se generó un archivo ('%s') que no se esperaba.\n" % f
+        obs += "* Se generó un archivo ('%s') que no se esperaba.\n" % f
     log.debug('Fin de ejecución de caso de prueba (hijo: %d, ret: %d)',
         proc.pid, proc.returncode)
     if obs:
@@ -416,14 +417,14 @@ while continuar:
     resultado = True
     for caso_de_prueba in intento.entrega.ejercicio.casosDePrueba:
         # Preparo chroot
-        preparar(intento)
+        preparar(intento, caso_de_prueba)
         # Pruebo y agrego prueba a la lista
         prueba = probar(intento, caso_de_prueba, mail)
         if not prueba.casoDePrueba.privado and not prueba.pasada:
             log.info('Prueba %s no pasada' % prueba.casoDePrueba.nombre)
             resultado = False
         # Limpio chroot
-        limpiar(intento)
+        limpiar(intento, caso_de_prueba)
     intento.finPruebas = datetime.datetime.now()
     # Envío mail con resultado al alumno
     if resultado: