marcellinodour / MIDO-SVG

Other
0 stars 0 forks source link

DEV BDD IT3 : Add class ROFDATABASE + JU #28

Closed marcellinodour closed 4 years ago

marcellinodour commented 4 years ago

La récupération du département s’est faite sur deux étapes:

Ajout des classes getOrgUnit et getOrgUnits dans Querier à partir de la clé correspondant au département MIDO dans refRof/root/OrgUnit on récupère le département La récupération des formations ne pouvait se faire en cascade à partir du département. En effet, comme renseigné dans le diagramme de classe ROFDataBase, il n’existe pas de lien entre OrgUnit et Programs ou Mentions. Par conséquent, nous avons cherché les clés correspondant aux formations MIDO dans Rof et que nous avons utilisé ensuite pour récupérer les Mentions et convertir en Licence ou Master selon le niveau d’étude. Il existe d’ailleurs des doublons qu’il faut gérer.

La récupération des cours s’est faite à partir des programmes. En effet, chaque program pointe vers une liste de program (dans refProgram) et chaque Program dans cette liste pointe vers une autre liste de Program et ainsi de suite. Quand on arrive aux niveaux les plus fins refProgram est vide et refCourse contient tous les cours correspondant aux Program(s) père(s).

WARNING : WORKFLOW non fonctionnel car refROF indisponible, l'action sera relancé vers 14h

TajouriSarra commented 4 years ago

Oui ceci remplace la PR #27 Voici un lien vers une documentation (toujours en cours de rédaction pour l'it 5 UML) https://github.com/marcellinodour/MIDO-SVG/blob/master/Doc/ROFDauphineDoc.adoc Pour les tests je crains qu'ils passeront pas, étant donné le problème des Mentions qui ont disparu dans testapi (comme discuté il y a quelques jours). Je tiens à préciser que tous les tests passaient avant cet incident.

marcellinodour commented 4 years ago

Monsieur, Pour le GH WF, la récupération WSDL a échoué de votre côté car vous il a été run à une heure de "fermeture REFROF" (de souvenir la base n'est plus accessible après 22h et entre 12h-14h).

Pour la suite, après échange avec @TajouriSarra nous nous sommes rendu compte d'une autre problème sous-jacent. Récemment, il semblerait que la base REFROF a fait l'objet d'une maintenance rendant ainsi nos test obsolètes. Il s'avère que les tests initialement programmé concernait un cas bien précis. Cas malheureusement inéxsistant à ce jour. A la vue du temps et du retard déjà entreprit pour cette tâche, nous nous sommes permis de désactiver ces tests. Une évolution est donc à prévoir dans l'it 4 Java. Qu'en pensez-vous ?

TajouriSarra commented 4 years ago

Monsieur, Pour compléter le commentaire de Marcellino, j'ai essayé de modifier les tests pour utiliser des mentions présentes dans testapi à ce jour. Cependant je me rends compte d'un autre problème qui n'existait pas auparavant. La récupération des noms de Program et Course n'est plus fonctionnelle, bien que celle de ects, programId, courseId... se fait normalement. Ce problème est peut être dû à la présence de deux champs (fr et en) dans courseName et programName bien que cela ne posait aucun problème il y a quelques jours. Il faut bien creuser pour régler ce problème et donc je me dis qu'il est peut être plus judicieux de le faire à l'it4 java. Je rejoins donc Marcellino sur la demande de désactiver les tests.

TajouriSarra commented 4 years ago

Oui, Maven échoue à cause de 9 erreurs de compilation de ce type ==> "incompatible types: CourseName cannot be converted to java.lang.String " (à l'origine de l'échec de la récupération des courseName, programName...) Il semblerait que la méthode getValue() qui permettait de récupérer la valeur de la donnée des champs JAXBElement ne remplisse plus sa fonction. Investigation en cours ...

oliviercailloux commented 4 years ago

Notez qu’un problème de compilation ne peut pas être dû à un problème à l’exécution (donc le problème rencontré par Maven ne peut pas être dû à un problème lors de l’exécution de getValue).

oliviercailloux commented 4 years ago

Plusieurs problèmes relevés ici semblent en fait liés à la PR #25.

brulej commented 4 years ago

l'it3 est cloturée