From 8ca59fe226f2dfb936af7c0c85c24e76b0d144bd Mon Sep 17 00:00:00 2001
From: Emmanuel Viennet <emmanuel.viennet@gmail.com>
Date: Wed, 11 Jan 2023 15:09:03 -0300
Subject: [PATCH] Log enregistrement jurys BUT

---
 app/but/jury_but.py                  | 7 ++++---
 app/models/validations.py            | 3 +++
 tests/unit/cursus_but_geii_lyon.yaml | 2 +-
 tests/unit/yaml_setup.py             | 6 +++---
 4 files changed, 11 insertions(+), 7 deletions(-)

diff --git a/app/but/jury_but.py b/app/but/jury_but.py
index e75a6fb0aa..a95bb3df68 100644
--- a/app/but/jury_but.py
+++ b/app/but/jury_but.py
@@ -718,6 +718,7 @@ class DecisionsProposeesAnnee(DecisionsProposees):
             )
             db.session.add(self.validation)
             db.session.commit()
+            log(f"Recording {self}: {code}")
             Scolog.logdb(
                 method="jury_but",
                 etudid=self.etud.id,
@@ -1005,9 +1006,6 @@ class DecisionsProposeesRCUE(DecisionsProposees):
         if code is None:
             self.validation = None
         else:
-            # log(
-            #    f"RCUE.record(etudid={self.etud.id}, ue1_id={self.rcue.ue_1.id}, ue2_id={self.rcue.ue_2.id}, code={code} )"
-            # )
             self.validation = ApcValidationRCUE(
                 etudid=self.etud.id,
                 formsemestre_id=self.rcue.formsemestre_2.id,
@@ -1024,6 +1022,8 @@ class DecisionsProposeesRCUE(DecisionsProposees):
                 msg=f"Validation {self.rcue}: {code}",
                 commit=True,
             )
+            log(f"rcue.record {self}: {code}")
+
             # Modifie au besoin les codes d'UE
             if code == "ADJ":
                 deca = self.deca
@@ -1049,6 +1049,7 @@ class DecisionsProposeesRCUE(DecisionsProposees):
         # par prudence, on requete toutes les validations, en cas de doublons
         validations = self.rcue.query_validations()
         for validation in validations:
+            log(f"DecisionsProposeesRCUE: deleting {validation}")
             db.session.delete(validation)
         db.session.flush()
 
diff --git a/app/models/validations.py b/app/models/validations.py
index aa53251709..f0ec9749ef 100644
--- a/app/models/validations.py
+++ b/app/models/validations.py
@@ -4,6 +4,7 @@
 """
 
 from app import db
+from app import log
 from app.models import SHORT_STR_LEN
 from app.models import CODE_STR_LEN
 from app.models.events import Scolog
@@ -123,6 +124,7 @@ class ScolarAutorisationInscription(db.Model):
         Scolog.logdb(
             "autorise_etud", etudid=etudid, msg=f"Passage vers S{semestre_id}: autorisé"
         )
+        log(f"ScolarAutorisationInscription: recording {autorisation}")
 
     @classmethod
     def delete_autorisation_etud(
@@ -136,6 +138,7 @@ class ScolarAutorisationInscription(db.Model):
         )
         for autorisation in autorisations:
             db.session.delete(autorisation)
+            log(f"ScolarAutorisationInscription: deleting {autorisation}")
             Scolog.logdb(
                 "autorise_etud",
                 etudid=etudid,
diff --git a/tests/unit/cursus_but_geii_lyon.yaml b/tests/unit/cursus_but_geii_lyon.yaml
index 4dd216cc6d..8f4b0e5776 100644
--- a/tests/unit/cursus_but_geii_lyon.yaml
+++ b/tests/unit/cursus_but_geii_lyon.yaml
@@ -248,7 +248,7 @@ Etudiants:
             decisions_rcues: # on repère ici les RCUE par l'acronyme de leur 1ere UE (donc du S1)
               "UE11":
                 code_valide: AJ
-                decision_jury: AJ
+                decision_jury: AJ # inutile de la préciser mais on peut
                 rcue:
                   moy_rcue: 9.00
                   est_compensable: False
diff --git a/tests/unit/yaml_setup.py b/tests/unit/yaml_setup.py
index de495dccce..1c20fe313c 100644
--- a/tests/unit/yaml_setup.py
+++ b/tests/unit/yaml_setup.py
@@ -5,15 +5,15 @@
 ##############################################################################
 
 """
-Met en place une base pour les tests unitaires, à partir d'une description 
+Met en place une base pour les tests unitaires, à partir d'une description
 YAML qui peut donner la formation, son ref. compétences, les formsemestres,
 les étudiants et leurs notes et décisions de jury.
 
-Le traitement est effectué dans l'ordre suivant: 
+Le traitement est effectué dans l'ordre suivant:
 
 setup_from_yaml()
 
-    - setup_but_formation(): 
+    - setup_but_formation():
         - import de la formation (le test utilise une seule formation)
     - associe_ues_et_parcours():
         - crée les associations formation <-> référentiel de compétence