Trabajando en issue #11, arreglando modAsociaciones

Author: juanAFernandez

SMS-Back-End/microservicio1/APIDB/DBCreator_v0_1.sql

@@ -79,15 +79,15 @@ CREATE TABLE Clase(
 #Un grupo es la asociación de una asignatura y un curso, por ejemplo: 1ºESO-Francés que identifica perfectamente un grupo de alumnos.
 CREATE TABLE Asocia(
 
+ id_clase INT ,
  id_asignatura INT,
- id_clase INT,
 
  #Especificamos que se trata de claves foráneas (claves primarias de otras tablas)
- FOREIGN KEY (id_asignatura) REFERENCES Asignatura(id),
  FOREIGN KEY (id_clase) REFERENCES Clase(id),
+ FOREIGN KEY (id_asignatura) REFERENCES Asignatura(id),
 
  #Especificamos la formación de la clave primaria en esta tabla.
- PRIMARY KEY (id_asignatura, id_clase)
+ PRIMARY KEY (id_clase, id_asignatura)
 
 );
 

SMS-Back-End/microservicio1/APIDB/GestorAsociacionesSQL.py

@@ -13,7 +13,7 @@
 #Uso de variables generales par la conexión a la BD.
 import dbParams
 #Variable global de para act/desactivar el modo verbose para imprimir mensajes en terminal.
-v=0
+v=1
 
 '''Clase controladora de Asociacions. Que usando la clase que define el modelo de Asociacion (la info en BD que de el se guarda)
 ofrece una interface de gestión que simplifica y abstrae el uso.
@@ -24,15 +24,16 @@ class GestorAsociaciones:
     """
 
     @classmethod
-    def nuevaAsociacion(self, id_asignatura, id_curso):
+    def nuevaAsociacion(self, id_clase, id_asignatura):
+        '''Introduce entidades en la tabla Asocia'''
 
         db = MySQLdb.connect(dbParams.host, dbParams.user, dbParams.password, dbParams.db)
 
         #Añadimos al principio y al final una comilla simple a todos los elementos.
         id_asignatura='\''+id_asignatura+'\''
-        id_curso='\''+id_curso+'\''
+        id_clase='\''+id_clase+'\''
 
-        query="INSERT INTO Asocia VALUES("+id_asignatura+","+id_curso+");"
+        query="INSERT INTO Asocia VALUES("+id_clase+","+id_asignatura+");"
         if v:
             print '\n'+query
         cursor = db.cursor()
@@ -62,12 +63,15 @@ def nuevaAsociacion(self, id_asignatura, id_curso):
             return 'OK'
         if salida==1062:
             return 'Elemento duplicado'
-        if salida=='1452':
+        if salida==1452:
             return 'Alguno de los elementos no existe'
 
 
     @classmethod
     def getAsociaciones(self):
+        '''
+        Devuelve una lista simplificada de todas las asociaciones dadas de alta en el sistema.
+        '''
         db = MySQLdb.connect(dbParams.host, dbParams.user, dbParams.password, dbParams.db)
         cursor = db.cursor()
 
@@ -89,7 +93,7 @@ def getAsociaciones(self):
             asociacion.id_asignatura=row[0]
             asociacion.id_curso=row[1]
 
-            lista.append(curso)
+            lista.append(asociacion)
             #print row[0], row[1]
             row = cursor.fetchone()
 
@@ -101,17 +105,17 @@ def getAsociaciones(self):
         #Una de las opciones es convertirlo en un objeto y devolverlo
 
     @classmethod
-    def getAsociacion(self, id_asignatura, id_curso):
+    def getAsociacion(self, id_clase, id_asignatura):
         """
         Recupera TODA la información de un Asociacion en concreto a través de la clave primaria, su id.
         """
         db = MySQLdb.connect(dbParams.host, dbParams.user, dbParams.password, dbParams.db)
         cursor = db.cursor()
 
+        id_clase='\''+id_clase+'\''
         id_asignatura='\''+id_asignatura+'\''
-        id_curso='\''+id_curso+'\''
 
-        query="select * from Asocia where id_asignatura="+id_asignatura+" and id_curso="+id_curso+";"
+        query="select * from Asocia where id_clase="+id_clase+" and id_asignatura="+id_asignatura+";"
         if v:
             print '\n'+query
         try:
@@ -127,6 +131,9 @@ def getAsociacion(self, id_asignatura, id_curso):
             except IndexError:
                 print "MySQL Error: %s" % str(e)
 
+        if v:
+            print "Salida MySQL: "+str(salida)
+
         cursor.close()
         db.close()
 
@@ -141,34 +148,30 @@ def getAsociacion(self, id_asignatura, id_curso):
         if salida==0:
             return 'Elemento no encontrado'
 
-    '''
+
     @classmethod
-    def modAsociacion(self, id_asignatura, id_curso, campoACambiar, nuevoValor):
+    def modAsociacion(self, id_asignatura, id_clase, campoACambiar, nuevoValor):
         """
         Esta función permite cambiar cualquier atributo de una Asociacion.
         Parámetros:
         campoACambiar: nombre del atributo que se quiere cambiar
         nuevoValor: nuevo valor que se quiere guardar en ese campo.
-
         Este caso puede ser delicado al tener sólo dos atributos y ambos ser claves foráneas. Por eso no permitiremos que
         se haga, para modificar la relación antes tendremos que destruirla y volverla a crear.
-
         """
-        db = MySQLdb.connect(host="localhost", user="root", passwd="root", db="smm"); #La conexión está clara.
+        db = MySQLdb.connect(host="localhost", user="root", passwd="root", db="sms"); #La conexión está clara.
         nuevoValor='\''+nuevoValor+'\''
         id_asignatura='\''+id_asignatura+'\''
-        id_curso='\''+id_curso+'\''
-
-        query="UPDATE Asocia SET "+campoACambiar+"="+nuevoValor+" WHERE id_asignatura="+id_asignatura+" and id_curso="+id_curso+";"
+        id_clase='\''+id_clase+'\''
+        query="UPDATE Asocia SET "+campoACambiar+"="+nuevoValor+" WHERE id_asignatura="+id_asignatura+" and id_clase="+id_clase+";"
         if v:
             print '\n'+query
-
         cursor = db.cursor()
         salida =''
         '''
         #Como la ejecución de esta consulta (query) puede producir excepciones como por ejemplo que el Asociacion con clave
         #que estamos pasando ya exista tendremos que tratar esas excepciones y conformar una respuesta entendible.
-    '''
+        '''
         try:
             salida = cursor.execute(query);
         except MySQLdb.Error, e:
@@ -179,19 +182,17 @@ def modAsociacion(self, id_asignatura, id_curso, campoACambiar, nuevoValor):
                 salida=e.args[0]
             except IndexError:
                 print "MySQL Error: %s" % str(e)
-
         #Efectuamos los cambios
         db.commit()
         cursor.close()
         db.close()
-
         if salida==1:
             return 'OK'
         elif salida==1062:
             return 'Elemento duplicado'
         elif salida==0:
             return 'Elemento no encontrado'
-    '''
+
     @classmethod
     def delAsociacion(self, id_asignatura, id_curso):
         db = MySQLdb.connect(host="localhost", user="root", passwd="root", db="smm"); #La conexión está clara.

SMS-Back-End/microservicio1/APIDB/test/testUnitario.py

@@ -449,43 +449,118 @@ def test_36_NumeroClases(self):
 
 ### RELACIONES, (entre entidades u otras relaciones), como Matricula, Asocia... ###
 
+class TestRelacionAsocia(unittest.TestCase):
+    '''
+    Comprobación de las operaciones CRUD sobre la tabla Asocia que relaciona las entidades Curso y Asignatura de la BD.
+    '''
+    #Los métodos, por convención empiezan por la palabra test, representando que métodos componen el test.
+    def test_41_InserccionRelacionAsocia(self):
+        '''
+        Comprueba el método nuevaClase del gestor GestorAsociacionesSQL.py
+        '''
+        #Nos aseguramos de que la base de datos se encuentra en estado CERO creándola en el momento.
+        os.system('mysql -u root -p\'root\' < ../DBCreator_v0_1.sql')
+        #Preparamos las variables
+        testA=testB=testC=False;
 
-if False:
+        #1. Insertamos una asociacion en la base de datos y comprobamos que la salida es 'OK'
+        #Primero creamos una clase.
+        GestorClases.nuevaClase('1','A','ESO')
+        #Después creamos una asignatura:
+        GestorAsignaturas.nuevaAsignatura('frances')
+        #Usamos ambos ides (1 y 1 porque se acaban de introducir y son autoincrementables) para relacionarlos.
+        if GestorAsociaciones.nuevaAsociacion('1','1') == 'OK':
+            testA=True
+        else:
+            testA=False
 
-    ### RELACIONES ###
+        #2. Comprobamos que no podemos introducir una asociación entre una Clase y una Asignatura si alguna de las dos
+        # no existe previamente (por las foreing keys)
+        print "GestorAsociaciones"+str(GestorAsociaciones.nuevaAsociacion('1','2'))
+        if GestorAsociaciones.nuevaAsociacion('1','2') == 'Alguno de los elementos no existe':
+            testB=True
+        else:
+            testB=False
+
+        #3. Comprobamos que insertar una asociació que ya existe nos da un error.
+        print GestorAsociaciones.nuevaAsociacion('1','1')
+        if GestorAsociaciones.nuevaAsociacion('1','1') == 'Elemento duplicado':
+            testC=True
+        else:
+            testC=False
+
+        self.assertEqual(testA and testB and testC, True)
+
+
+    def test_42_LecturaRelacionesAsocia(self):
+        '''Recupera la lista de todas las asociaciones (Clase-Asignatura) almacenadas en la base de datos.'''
+        os.system('mysql -u root -p\'root\' < ../DBCreator_v0_1.sql')
+        #Creamos una clase
+        GestorClases.nuevaClase('1','A','ESO')
+        #Después creamos dos asignaturas:
+        GestorAsignaturas.nuevaAsignatura('frances')
+        GestorAsignaturas.nuevaAsignatura('lengua')
+        GestorAsociaciones.nuevaAsociacion('1','1')
+        GestorAsociaciones.nuevaAsociacion('1','2')
+
+        self.assertEqual(len(GestorAsociaciones.getAsociaciones()),2)
+
+
+    def test_43_LecturaAsociacion(self):
+        ''' Comprobación de la lectura correcta de una asociacion concreta'''
+        os.system('mysql -u root -p\'root\' < ../DBCreator_v0_1.sql')
+        #Damos de alta una clase
+        GestorClases.nuevaClase('1','A','ESO')
+        #Después creamos dos asignaturas:
+        GestorAsignaturas.nuevaAsignatura('frances')
+        GestorAsociaciones.nuevaAsociacion('1','1')
+
+        if GestorClases.getClase('1') != 'Elemento no encontrado' and GestorClases.getClase('2') == 'Elemento no encontrado':
+            test=True
+        else:
+            test=False
+
+        self.assertEqual(test, True)
+
+    def test_44_ModificacionAsociacion(self):
+        '''Comprobación de como cualquier atributo de una asociación puede modificarse, método modAsociacion de GestorAsociacionesSQL.py'''
+        #Nos aseguramos de que la base de datos se encuentra en estado CERO creándola en el momento.
+        os.system('mysql -u root -p\'root\' < ../DBCreator_v0_1.sql')
+        #Preparamos las variables
+        testA=testB=testC=False;
+
+        #Creamos una asociación:
+        GestorClases.nuevaClase('1','A','ESO')
+        GestorAsignaturas.nuevaAsignatura('frances')
+        GestorAsignaturas.nuevaAsignatura('ingles')
+        #Relacionamos la clase con la primera asignatura.
+        GestorAsociaciones.nuevaAsociacion('1','1')
+
+        #1. Modificamos la asignatura de la asociación por la asignatura 2
+        if GestorAsociaciones.modAsociacion('1','1','id_asignatura', '2') == 'OK':
+            testA=True
 
-    class TestRelacionAsocia(unittest.TestCase):
+
+        #Intentamos asociar una clase con una asignatura que no existe (la asignatura con id 3, inexistente):
+        #if GestorAsociaciones.modAsociacion('1','1','id_asignatura', '3') == 'Elemento no encontrado':
+        #    testB=True
+
+
+        #Comprobamos que si realizamos una modificación que da lugar a una asociacioón que ya existe lo detecta.
         '''
-        Comprobación de las operaciones CRUD sobre la tabla Asocia que relaciona las entidades Curso y Asignatura de la BD.
+        #Creamos una nueva clase
+        GestorClases.nuevaClase('1','A','ESO') #LA anterior quedó como 1ABACH
+        #Intentamos cambiar un parámetro quedando como tra existente, debe de dar Elemento duplicado como error.
+        if GestorClases.modClase('2', 'nivel', 'BACH') == 'Elemento duplicado':
+            testC=True #El error se da.
         '''
 
+        #Comprobamos que el nombre ha sido cambiado.
+        self.assertEqual(testA, True)
 
-        #Los métodos, por convención empiezan por la palabra test, representando que métodos componen el test.
-        def test_41_InserccionRelacionAsocia(self):
-            #Creamos una asignatura
-            salidaA=GestorAsignaturas.nuevaAsignatura('fr','Frances')
-            if salidaA=='OK':
-                salidaA=True
-            else:
-                salidaA=False
 
-            #Creamos un curso
-            salidaB=GestorCursos.nuevoCurso('curso','1','A','ESO')
-            if salidaB=='OK':
-                salidaB=True
-            else:
-                salidaB=False
-            #Testeamos la creación de la asociación
-            salidaC=GestorAsociaciones.nuevaAsociacion('fr','curso')
-            if salidaC=='OK':
-                salidaC=True
-            else:
-                salidaC=False
-
-            self.assertEqual(salidaA and salidaB and salidaC, True)
 
-        def test_42_LecturaRelacionAsocia(self):
-            self.assertNotEqual(GestorAsociaciones.getAsociacion('fr','curso'),'Elemento no encontrado')
+    if 0:
         '''
         def test_43_ModificacionRelacionAsocia(self):
             #Intentamos modificar el cambio de la asociación de un curso a una asignatura
@@ -506,27 +581,36 @@ def test_44_EliminacionRelacionAsocia(self):
             self.assertEqual(salida,'OK')
 
 
-        def test_45_AlumnosMAtriculadosEnUnaAsignaturaYCurso(self):
-            aprovisionadorDBconInterfaz.aprovisiona()
-            sizeA=len(GestorAsociaciones.getAlumnos('mt','1AESO'))
-            sizeB=len(GestorAsociaciones.getAlumnos('fr','1AESO'))
-            sizeC=len(GestorAsociaciones.getAlumnos('mt','1BESO'))
-            os.system('mysql -u root -p\'root\' < ../DBCreator_v0.sql')
-            salida=False
-            if sizeA==1 and sizeB==3 and sizeC==3:
-                salida=True
-            self.assertEqual(salida,True)
+        if 0:
+
+            def test_45_AlumnosMAtriculadosEnUnaAsignaturaYCurso(self):
+                aprovisionadorDBconInterfaz.aprovisiona()
+                sizeA=len(GestorAsociaciones.getAlumnos('mt','1AESO'))
+                sizeB=len(GestorAsociaciones.getAlumnos('fr','1AESO'))
+                sizeC=len(GestorAsociaciones.getAlumnos('mt','1BESO'))
+                os.system('mysql -u root -p\'root\' < ../DBCreator_v0.sql')
+                salida=False
+                if sizeA==1 and sizeB==3 and sizeC==3:
+                    salida=True
+                self.assertEqual(salida,True)
+
+            def test_46_ProfesoresQueImpartenEnUnCursoYAsignatura(self):
+                aprovisionadorDBconInterfaz.aprovisiona()
+                sizeA=len(GestorAsociaciones.getProfesores('mt','1AESO'))
+                sizeB=len(GestorAsociaciones.getProfesores('fr','1AESO'))
+                sizeC=len(GestorAsociaciones.getProfesores('mt','1BESO'))
+                os.system('mysql -u root -p\'root\' < ../DBCreator_v0.sql')
+                salida=False
+                if sizeA==0 and sizeB==2 and sizeC==1:
+                    salida=True
+                self.assertEqual(salida,True)
+
+
+
+if False:
+
+    ### RELACIONES ###
 
-        def test_46_ProfesoresQueImpartenEnUnCursoYAsignatura(self):
-            aprovisionadorDBconInterfaz.aprovisiona()
-            sizeA=len(GestorAsociaciones.getProfesores('mt','1AESO'))
-            sizeB=len(GestorAsociaciones.getProfesores('fr','1AESO'))
-            sizeC=len(GestorAsociaciones.getProfesores('mt','1BESO'))
-            os.system('mysql -u root -p\'root\' < ../DBCreator_v0.sql')
-            salida=False
-            if sizeA==0 and sizeB==2 and sizeC==1:
-                salida=True
-            self.assertEqual(salida,True)