Se usa una forma lambda para comparar por padron.

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

diff --git a/src/sc_test b/src/sc_test
index 479f75ed6dfd2c17959e73df73e691aef9079acd..5643e611714f379401467a8caedfca8ab90063d0 100755 (executable)
--- a/src/sc_test
+++ b/src/sc_test
@@ -53,6 +53,7 @@ def sigterm(signum, frame):
     global continuar, log
     continuar = False
     log.debug('Señal %d recibida', signum)
+    log.info('Finalizado')
 
 def compilar(intento, mail):
     global log
@@ -61,13 +62,14 @@ def compilar(intento, mail):
     # Compilo
     log.debug('Ejecutando: make -f %s', makefile)
     intento.inicioCompila = datetime.datetime.now()
-    make = subprocess.Popen(('make', '-f', makefile), stderr=subprocess.PIPE,
+    stderr = file(os.path.join(intento.path, 'make.out'), 'w')
+    make = subprocess.Popen(('make', '-f', makefile), stderr=stderr,
         cwd=intento.path)
     make.wait()
     intento.finCompila = datetime.datetime.now()
     log.debug('Fin del comando: make -f %s', makefile)
     # Verifico compilación
-    stderr = make.stderr.read()
+    stderr = file(os.path.join(intento.path, 'make.out')).read()
     intento.compila = not make.returncode
     msg = 'Compilación: '
     if intento.compila and not stderr:
@@ -144,13 +146,18 @@ def probar(intento, caso_de_prueba, mail):
         'cwd': intento.chrootPath(caso_de_prueba),
         'close_fds': True,
         'preexec_fn': secure_process(intento.chrootPath(caso_de_prueba), uid, gid, tiempo_cpu),
+        'stdin': None,
+        'stdout': None,
+        'stderr': None,
     }
     if usa_stdin():
         options['stdin'] = file(os.path.join(caso_de_prueba.path, 'stdin'), 'r')
     if usa_stdout():
-        options['stdout'] = subprocess.PIPE
+        stdout_fn = intento.chrootPath(caso_de_prueba) + '.stdout'
+        options['stdout'] = file(stdout_fn,  'w')
     if usa_stderr():
-        options['stderr'] = subprocess.PIPE
+        stderr_fn = intento.chrootPath(caso_de_prueba) + '.stderr'
+        options['stderr'] = file(stderr_fn,  'w')
     # Ejecuto programa
     params = ['/tp']
     if caso_de_prueba.parametros:
@@ -222,11 +229,11 @@ def probar(intento, caso_de_prueba, mail):
     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, file(stdout_fn).readlines(),
             file(os.path.join(caso_de_prueba.path, 'stdout')).readlines(),
             caso_de_prueba.nombre + '.stdout', longname='La salida estándar')
     if usa_stderr():
-        obs += diff(prueba, mail, proc.stderr.readlines(),
+        obs += diff(prueba, mail, file(stderr_fn).readlines(),
             file(os.path.join(caso_de_prueba.path, 'stderr')).readlines(),
         caso_de_prueba.nombre + '.stderr', longname='La salida de error')
     for f in caso_de_prueba.archivosSalida:
@@ -401,15 +408,16 @@ while continuar:
         log.debug('No hay intento para probar')
         time.sleep(intervalo)
         continue
-    log.info('Nuevo intento a probar (%d %d.%d.%d %d%d%d)',
+    log.info('Nuevo intento a probar (%d %d.%d.%d %d%d%d [%d %d %d])',
         intento.inscripto.padron, intento.entrega.nroEjercicio,
         intento.entrega.entrega, intento.numero, intento.entrega.curso.anio,
-        intento.entrega.curso.cuatrimestre, intento.entrega.curso.curso)
+        intento.entrega.curso.cuatrimestre, intento.entrega.curso.curso,
+        intento.inscripto.id, intento.entrega.id, intento.id)
     mail = MailIntento(intento)
     # Compila
     compilar(intento, mail)
     if not intento.compila:
-        log.info('Intento no compila')
+        log.info('Intento[%d] no compila', intento.id)
         mail.send('NO COMPILA')
         continue
     # Ejecución de casos de prueba
@@ -421,7 +429,8 @@ while continuar:
         # 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)
+            log.info('Prueba %s[%d %d] no pasada', prueba.casoDePrueba.nombre,
+                prueba.casoDePrueba.id, prueba.id)
             resultado = False
         # Limpio chroot
         limpiar(intento, caso_de_prueba)
@@ -429,11 +438,14 @@ while continuar:
     # Envío mail con resultado al alumno
     if resultado:
         mail.send('ACEPTADO')
+        log.info('Pruebas finalizadas (intento[%d] aceptado)', intento.id)
     else:
         mail.send('FALLA PRUEBAS')
+        log.info('Pruebas finalizadas (intento[%d] no aceptado)', intento.id)
     # Limpio directorio
     log.debug('Borrando ejecutable y código objeto (*.o)')
     os.remove(os.path.join(intento.path, 'tp'))
-    [os.remove(obj) for obj in glob.glob(os.path.join(intento.path, '*.o'))]
+    for obj in glob.glob(os.path.join(intento.path, '*.o')):
+        os.remove(obj)
     # time.sleep(intervalo) #XXX Puede servir para enlentecer el server