Más modularización.

diff --git a/src/sc_test b/src/sc_test
index a7fe80745bcae7f7fe43566491815c8a823b68f6..34402ff441c583f618a24a4c73ecb78317a2ff93 100755 (executable)
--- a/src/sc_test
+++ b/src/sc_test
@@ -49,6 +49,118 @@ def sigterm(signum, frame):
     continuar = False
     log.debug('Señal %d recibida', signum)
 
     continuar = False
     log.debug('Señal %d recibida', signum)
 

+def compilar(intento, intento_dir, entrega_dir, log):
+    # Busco makefile
+    makefile = os.path.join(entrega_dir, 'Makefile')
+    if not os.path.exists(makefile):
+        makefile = os.path.join(data_dir, 'Makefile')
+    shutil.copy(makefile, intento_dir)
+    # Compilo
+    log.debug('Ejecutando: make -f %s', makefile)
+    intento.inicioCompila = datetime.datetime.now()
+    make = subprocess.Popen(('make', '-f', makefile), stdout=subprocess.PIPE,
+        stderr=subprocess.PIPE, cwd=intento_dir)
+    make.wait()
+    intento.finCompila = datetime.datetime.now()
+    log.debug('Fin del comando: make -f %s', makefile)
+    # Verifico compilación
+    if make.returncode:
+        log.debug('Error al compilar, código de retorno: %d, salida estándar: '
+            '%s, salida de error: %s)', make.returncode, make.stdout.read(),
+            make.stderr.read())
+        intento.compila = False
+        #TODO enviar_respuesta(R_ERR, $mail, "ERROR AL COMPILAR!\n\n$err\n\nCódigo de retorno: $ret\n", $intento);
+        return False
+    log.debug('Compilado OK')
+    intento.compila = True
+    #TODO mail acumulativo
+    return True
+
+def preparar(intento_dir, chroot_dir):
+    # Creo chroot - TODO copiarlo de algún lado donde ande el valgrind?
+    ejecutable = os.path.join(chroot_dir, 'tp')
+    os.mkdir(chroot_dir)
+    shutil.move(os.path.join(intento_dir, 'tp'), ejecutable)
+
+def probar(intento, caso_de_prueba, uid, gid, chroot_dir, conf, conn, log):
+    # Obtengo datos útiles del caso de prueba
+    tiempo_cpu = caso_de_prueba.tiempoCpu
+    if tiempo_cpu is None:
+        tiempo_cpu = conf.get('general', 'tiempo_cpu')
+    tiempo_cpu = int(tiempo_cpu)
+    # Creo prueba nueva
+    prueba = Prueba(intento=intento, casoDePrueba=caso_de_prueba,
+        inicio=datetime.datetime.now(), connection = conn)
+    log.debug('Prueba: %s', prueba)
+    # Abro archivos para fds básicos
+    #XXX sacar nombres de archivos de prueba????
+    #stdin = file(os.path.join(intento_dir, 'stdin'), 'r')
+    stdout = file(os.path.join(intento_dir, 'stdout'), 'w')
+    stderr = file(os.path.join(intento_dir, 'stderr'), 'w')
+    # Ejecuto programa
+    log.debug('Ejecutando /tp %s', caso_de_prueba.parametros)
+    os.seteuid(0) # Dios! (para chroot)
+    # FIXME caso_de_prueba.parametros
+    try:
+        proc = subprocess.Popen('/tp', stdout=stdout, stderr=stderr, #stdin=stdin,
+            preexec_fn=secure_process(chroot_dir, uid, gid, tiempo_cpu))
+    except Exception, e: # FIXME poner en el manejo de exceptiones estandar
+        try:
+            print e.child_traceback
+            raise
+        except:
+            raise
+    os.seteuid(uid) # Mortal de nuevo
+    time.sleep(tiempo_cpu)
+    # Si el proceso sigue andando lo tenemos que matar
+    if proc.poll() is None:
+        log.debug('La prueba tardó más del tiempo permitido (%d segundos)',
+            tiempo_cpu)
+        os.seteuid(0) # Dios! (corre como Dios, hay que matarlo como Dios)
+        os.kill(proc.pid, signal.SIGKILL)
+        os.seteuid(uid) # Mortal de nuevo
+        log.debug('Prueba cancelada (kill)')
+        proc.wait()
+        prueba.observaciones = 'Excedió el límite de tiempo de ejecución ' \
+            '(%d seg)' % tiempo_cpu
+        prueba.fin = datetime.datetime.now()
+        prueba.pasada = False
+        return prueba
+    proc.wait() # Para que no queden zombies
+    prueba.fin = str(datetime.datetime.now())
+    # Salió con una señal?
+    if proc.returncode < 0:
+        sig = -proc.returncode
+        log.debug('El programa salió con la señal %d', sig)
+        prueba.pasada = False
+        #TODO otras señales conocidas
+        if sig == signal.SIGXCPU:
+            prueba.observaciones = 'Excedió el límite de tiempo de CPU ' \
+                '(%d seg)' % tiempo_cpu
+        else:
+            prueba.observaciones = 'Salió con la señal %d' % sig
+        return prueba
+    # Si tenemos que verificar el código de retorno
+    if caso_de_prueba.codigoRetorno is not None:
+        #FIXME trucho lo de 256
+        # Si el código de error esperado es 256 => el código de error debe ser != 0
+        # Si no el código de error esperado debe ser igual al obtenido
+        if caso_de_prueba.codigoRetorno != proc.returncode \
+                or caso_de_prueba.codigoRetorno == 256 \
+                    and proc.returncode != 0:
+            log.debug('Código de retorno incorrecto (debía ser %d y se obtuvo %d)',
+                caso_de_prueba.codigoRetorno, proc.returncode)
+            prueba.pasada = False
+            prueba.observaciones = 'Código de retorno incorrecto (debía ' \
+                'ser %d y se obtuvo %d)' % (caso_de_prueba.codigoRetorno,
+                proc.returncode)
+    #TODO verificar salidas, hacer diff
+    log.debug('Fin de ejecución de caso de prueba (hijo: %d, ret: %d)',
+        proc.pid, proc.returncode)
+    prueba.pasada = True
+    log.debug('Prueba OK: %s', prueba)
+    return prueba
+

 # Conecto señales
 signal.signal(signal.SIGCHLD, sigchld)
 signal.signal(signal.SIGTERM, sigterm)
 # Conecto señales
 signal.signal(signal.SIGCHLD, sigchld)
 signal.signal(signal.SIGTERM, sigterm)


@@ -81,124 +193,21 @@ while continuar:
     # Obtengo paths
     intento_dir = os.path.join(data_dir, intento.path('intentos'))
     entrega_dir = os.path.join(data_dir, 'ejercicios', str(intento.entrega.ejercicioID))
     # Obtengo paths
     intento_dir = os.path.join(data_dir, intento.path('intentos'))
     entrega_dir = os.path.join(data_dir, 'ejercicios', str(intento.entrega.ejercicioID))

-    print entrega_dir
-    # Busco makefile
-    makefile = os.path.join(entrega_dir, 'Makefile')
-    if not os.path.exists(makefile):
-        makefile = os.path.join(data_dir, 'Makefile')
-    shutil.copy(makefile, intento_dir)
-    # Compilo
-    log.debug('Ejecutando: make -f %s', makefile)
-    intento.inicioCompila = datetime.datetime.now()
-    make = subprocess.Popen(('make', '-f', makefile), stdout=subprocess.PIPE,
-        stderr=subprocess.PIPE, cwd=intento_dir)
-    make.wait()
-    intento.finCompila = datetime.datetime.now()
-    log.debug('Fin del comando: make -f %s', makefile)
-    # Verifico compilación
-    if make.returncode:
-        log.debug('Error al compilar, código de retorno: %d, salida estándar: '
-            '%s, salida de error: %s)', make.returncode, make.stdout.read(),
-            make.stderr.read())
-        intento.compila = False
-        #TODO enviar_respuesta(R_ERR, $mail, "ERROR AL COMPILAR!\n\n$err\n\nCódigo de retorno: $ret\n", $intento);
-        continue;
-    else:
-        log.debug('Compilado OK')
-        intento.compila = True
-        #TODO mail acumulativo
-    # Creo chroot - TODO copiarlo de algún lado donde ande el valgrind?

     chroot_dir = os.path.join(intento_dir, 'chroot')
     chroot_dir = os.path.join(intento_dir, 'chroot')

-    ejecutable = os.path.join(chroot_dir, 'tp')
-    os.mkdir(chroot_dir)
-    shutil.move(os.path.join(intento_dir, 'tp'), ejecutable)
-    # Cambio permisos - XXX al pedo? Hice seteuid()
-    os.chmod(chroot_dir, 02770)
-    os.chown(chroot_dir, uid, gid)
-    os.chmod(ejecutable, 0550)
-    os.chown(ejecutable, uid, gid)
+    # Compila
+    if not compilar(intento, intento_dir, entrega_dir, log):
+        continue
+    # Prepara archivos
+    preparar(intento_dir, chroot_dir)

     # Ejecución de casos de prueba
     intento.inicioPruebas = datetime.datetime.now()
     # Ejecución de casos de prueba
     intento.inicioPruebas = datetime.datetime.now()

+    pruebas = []

     for caso_de_prueba in intento.entrega.ejercicio.casosDePrueba:
     for caso_de_prueba in intento.entrega.ejercicio.casosDePrueba:

-        # Obtengo datos útiles del caso de prueba
-        tiempo_cpu = caso_de_prueba.tiempoCpu
-        if tiempo_cpu is None:
-            tiempo_cpu = conf.get('general', 'tiempo_cpu')
-        tiempo_cpu = int(tiempo_cpu)
-        # Creo prueba nueva
-        prueba = Prueba(intento=intento, casoDePrueba=caso_de_prueba,
-            inicio=datetime.datetime.now(), connection = conn)
-        log.debug('Prueba: %s', prueba)
-        # Abro archivos para fds básicos
-        #XXX sacar nombres de archivos de prueba????
-        #stdin = file(os.path.join(intento_dir, 'stdin'), 'r')
-        stdout = file(os.path.join(intento_dir, 'stdout'), 'w')
-        stderr = file(os.path.join(intento_dir, 'stderr'), 'w')
-        # Ejecuto programa
-        log.debug('Ejecutando /tp %s', caso_de_prueba.parametros)
-        os.seteuid(0) # Dios! (para chroot)
-        # FIXME caso_de_prueba.parametros
-        try:
-            proc = subprocess.Popen('/tp', stdout=stdout, stderr=stderr, #stdin=stdin,
-                preexec_fn=secure_process(chroot_dir, uid, gid, tiempo_cpu))
-        except Exception, e: # FIXME poner en el manejo de exceptiones estandar
-            try:
-                print e, e.child_traceback
-            except:
-                print e
-        os.seteuid(uid) # Mortal de nuevo
-        time.sleep(tiempo_cpu)
-        # Si el proceso sigue andando lo tenemos que matar
-        if proc.poll() is None:
-            log.debug('La prueba tardó más del tiempo permitido (%d segundos)',
-                tiempo_cpu)
-            os.seteuid(0) # Dios! (corre como Dios, hay que matarlo como Dios)
-            os.kill(proc.pid, signal.SIGKILL)
-            os.seteuid(uid) # Mortal de nuevo
-            log.debug('Prueba cancelada (kill)')
-            proc.wait()
-            prueba.observaciones = 'Excedió el límite de tiempo de ejecución ' \
-                '(%d seg)' % tiempo_cpu
-            prueba.fin = datetime.datetime.now()
-            prueba.pasada = False
-            continue
-        proc.wait() # Para que no queden zombies
-        prueba.fin = str(datetime.datetime.now())
-        # Salió con una señal?
-        if proc.returncode < 0:
-            sig = -proc.returncode
-            log.debug('El programa salió con la señal %d', sig)
-            prueba.pasada = False
-            #TODO otras señales conocidas
-            if sig == signal.SIGXCPU:
-                prueba.observaciones = 'Excedió el límite de tiempo de CPU ' \
-                    '(%d seg)' % tiempo_cpu
-            else:
-                prueba.observaciones = 'Salió con la señal %d' % sig
-            continue
-        # Si tenemos que verificar el código de retorno
-        if caso_de_prueba.codigoRetorno is not None:
-            #FIXME trucho lo de 256
-            # Si el código de error esperado es 256 => el código de error debe ser != 0
-            # Si no el código de error esperado debe ser igual al obtenido
-            if caso_de_prueba.codigoRetorno != proc.returncode \
-                    or caso_de_prueba.codigoRetorno == 256 \
-                        and proc.returncode != 0:
-                log.debug('Código de retorno incorrecto (debía ser %d y se obtuvo %d)',
-                    caso_de_prueba.codigoRetorno, proc.returncode)
-                prueba.pasada = False
-                prueba.observaciones = 'Código de retorno incorrecto (debía ' \
-                    'ser %d y se obtuvo %d)' % (caso_de_prueba.codigoRetorno,
-                    proc.returncode)
-        #TODO verificar salidas, hacer diff
-        log.debug('Fin de ejecución de caso de prueba (hijo: %d, ret: %d)',
-            proc.pid, proc.returncode)
-        prueba.pasada = True
-        log.debug('Prueba OK: %s', prueba)
+        pruebas.append(probar(intento, caso_de_prueba, uid, gid, chroot_dir, conf, conn, log))

     intento.finPruebas = datetime.datetime.now()
     #TODO make clean
     #TODO Armar mail de respuesta al alumno
     intento.finPruebas = datetime.datetime.now()
     #TODO make clean
     #TODO Armar mail de respuesta al alumno

-    for prueba in Prueba.selectBy(intentoID=intento.id, connection=conn):
+    for prueba in pruebas:

         #TODO Si es publica, veo si se hizo ok o no y voy creando mail
         pass
     time.sleep(intervalo)
         #TODO Si es publica, veo si se hizo ok o no y voy creando mail
         pass
     time.sleep(intervalo)