global continuar, log
continuar = False
log.debug('Señal %d recibida', signum)
+ log.info('Finalizado')
def compilar(intento, mail):
global log
# 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:
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)
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),
+ '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:
# 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
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:
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:
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
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)
+ log.info('Prueba %s[%d %d] no pasada', prueba.casoDePrueba.nombre,
+ prueba.casoDePrueba.id, prueba.id)
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:
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