Heredar a Prueba de ComandoEjecutado.

De la misma forma que CasoDePrueba hereda de Comando por razones puramente
funcionales, ahora Prueba hereda de ComandoEjecutado.

diff --git a/sercom/model.py b/sercom/model.py
index 0a7bb21801f75b8342d918fe0330bdbdd5095882..a9ef27b56c76809bdecba1836082f9b3847aac9e 100644 (file)
--- a/sercom/model.py
+++ b/sercom/model.py
@@ -779,8 +779,10 @@ class ComandoEjecutado(InheritableSQLObject): #{{{
     exito           = IntCol(default=None)
     observaciones   = UnicodeCol(notNone=True, default=u'')
 
     exito           = IntCol(default=None)
     observaciones   = UnicodeCol(notNone=True, default=u'')
 

-    def __repr__(self):
-        raise NotImplementedError('ComandoEjecutado es una clase abstracta')
+    def __repr__(self, clave='', mas=''):
+        return ('%s(%s inicio=%s, fin=%s, exito=%s, observaciones=%s%s)'
+            % (self.__class__.__name__, clave, self.inicio, self.fin,
+            self.exito, self.observaciones))

 #}}}
 
 class ComandoFuenteEjecutado(ComandoEjecutado): #{{{
 #}}}
 
 class ComandoFuenteEjecutado(ComandoEjecutado): #{{{


@@ -790,10 +792,9 @@ class ComandoFuenteEjecutado(ComandoEjecutado): #{{{
     pk      = DatabaseIndex(comando, entrega, unique=True)
 
     def __repr__(self):
     pk      = DatabaseIndex(comando, entrega, unique=True)
 
     def __repr__(self):

-        return 'ComandoFuenteEjecutado(comando=%s, entrega=%s, inicio=%s, ' \
-            'fin=%s, exito=%s, observaciones=%s)' \
-                % (self.comando.shortrepr(), self.entrega.shortrepr(),
-                    self.inicio, self.fin, self.exito, self.observaciones)
+        return super(ComandoFuenteEjecutado, self).__repr__(
+            'comando=%s, entrega=%s' % (self.comando.shortrepr(),
+                self.entrega.shortrepr()))

 
     def shortrepr(self):
         return '%s-%s' % (self.tarea.shortrepr(), self.entrega.shortrepr())
 
     def shortrepr(self):
         return '%s-%s' % (self.tarea.shortrepr(), self.entrega.shortrepr())


@@ -806,26 +807,20 @@ class ComandoPruebaEjecutado(ComandoEjecutado): #{{{
     pk      = DatabaseIndex(comando, prueba, unique=True)
 
     def __repr__(self):
     pk      = DatabaseIndex(comando, prueba, unique=True)
 
     def __repr__(self):

-        return 'ComandoPruebaEjecutado(comando=%s, prueba=%s, inicio=%s, ' \
-            'fin=%s, exito=%s, observaciones=%s)' \
-                % (self.comando.shortrepr(), self.prueba.shortrepr(),
-                    self.inicio, self.fin, self.exito, self.observaciones)
+        return super(ComandoPruebaEjecutado, self).__repr__(
+            'comando=%s, entrega=%s' % (self.comando.shortrepr(),
+                self.entrega.shortrepr()))

 
     def shortrepr(self):
         return '%s:%s:%s' % (self.tarea.shortrepr(), self.entrega.shortrepr(),
             self.caso_de_prueba.shortrepr())
 #}}}
 
 
     def shortrepr(self):
         return '%s:%s:%s' % (self.tarea.shortrepr(), self.entrega.shortrepr(),
             self.caso_de_prueba.shortrepr())
 #}}}
 

-class Prueba(SQLObject): #{{{
+class Prueba(ComandoEjecutado): #{{{

     # Clave
     entrega             = ForeignKey('Entrega', notNone=True, cascade=False)
     caso_de_prueba      = ForeignKey('CasoDePrueba', notNone=True, cascade=False)
     pk                  = DatabaseIndex(entrega, caso_de_prueba, unique=True)
     # Clave
     entrega             = ForeignKey('Entrega', notNone=True, cascade=False)
     caso_de_prueba      = ForeignKey('CasoDePrueba', notNone=True, cascade=False)
     pk                  = DatabaseIndex(entrega, caso_de_prueba, unique=True)

-    # Campos
-    inicio              = DateTimeCol(notNone=True, default=DateTimeCol.now)
-    fin                 = DateTimeCol(default=None)
-    pasada              = IntCol(default=None)
-    observaciones       = UnicodeCol(notNone=True, default=u'')

     # Joins
     comandos_ejecutados = MultipleJoin('ComandoPruebaEjecutado')
 
     # Joins
     comandos_ejecutados = MultipleJoin('ComandoPruebaEjecutado')
 


@@ -841,10 +836,8 @@ class Prueba(SQLObject): #{{{
         ComandoPruebaEjecutado.pk.get(self.id, comando).destroySelf()
 
     def __repr__(self):
         ComandoPruebaEjecutado.pk.get(self.id, comando).destroySelf()
 
     def __repr__(self):

-        return 'Prueba(entrega=%s, caso_de_prueba=%s, inicio=%s, fin=%s, ' \
-            'pasada=%s, observaciones=%s)' \
-                % (self.entrega.shortrepr(), self.caso_de_prueba.shortrepr(),
-                self.inicio, self.fin, self.pasada, self.observaciones)
+        return super(Prueba, self).__repr__('entrega=%s, caso_de_prueba=%s'
+            % (self.entrega.shortrepr(), self.caso_de_prueba.shortrepr()))

 
     def shortrepr(self):
         return '%s:%s' % (self.entrega.shortrepr(),
 
     def shortrepr(self):
         return '%s:%s' % (self.entrega.shortrepr(),

diff --git a/sercom/tester.py b/sercom/tester.py
index 4d6cb3f0dcfaef18f20bbb66ab803f530ef2fe82..f1229b4ba1640dc3c93dcef4c40e5231f2784e7e 100644 (file)
--- a/sercom/tester.py
+++ b/sercom/tester.py
@@ -252,13 +252,13 @@ def ejecutar_caso_de_prueba(self, path, entrega): #{{{
             for tarea in tareas:
                 tarea.ejecutar(path, prueba)
         except ExecutionFailure, e:
             for tarea in tareas:
                 tarea.ejecutar(path, prueba)
         except ExecutionFailure, e:

-            prueba.pasada = False
+            prueba.exito = False

             if self.rechazar_si_falla:
                 entrega.exito = False
             if self.terminar_si_falla:
                 raise ExecutionError(e.comando, e.tarea, prueba)
         else:
             if self.rechazar_si_falla:
                 entrega.exito = False
             if self.terminar_si_falla:
                 raise ExecutionError(e.comando, e.tarea, prueba)
         else:

-            prueba.pasada = True
+            prueba.exito = True

     finally:
         prueba.fin = datetime.now()
 CasoDePrueba.ejecutar = ejecutar_caso_de_prueba
     finally:
         prueba.fin = datetime.now()
 CasoDePrueba.ejecutar = ejecutar_caso_de_prueba