fixed tests + stats bacs pros

betagouv · Oct 23, 2024 · 6081dbc · 6081dbc
1 parent 8bd00f0
commit 6081dbc
Show file tree

Hide file tree

Showing 6 changed files with 14 additions and 12 deletions.
diff --git a/app/back/src/main/resources/db/migration/V1_27__reset_table_formation_voeux.sql b/app/back/src/main/resources/db/migration/V1_27__reset_table_formation_voeux.sql
@@ -0,0 +1,2 @@
+
+TRUNCATE TABLE ref_formation CASCADE;
diff --git a/app/etl/etl-common/src/main/java/fr/gouv/monprojetsup/data/model/psup/PsupData.java b/app/etl/etl-common/src/main/java/fr/gouv/monprojetsup/data/model/psup/PsupData.java
@@ -112,7 +112,8 @@ public Collection<Integer> getLasFlCodes() {
 
     public AdmissionStats buildStats() {
 
-        val bacsKeys = getBacs().stream().map(Bac::key).collect(Collectors.toSet());
+        val bacsKeys = new HashSet<>(getBacs().stream().map(Bac::key).toList());
+        bacsKeys.add(TOUS_BACS_CODE_MPS);
         val groups = getGtaToMpsIdMapping();
 
         StatistiquesAdmisParGroupe statsAdmisParGroupe
@@ -218,7 +219,6 @@ public void cleanupAfterUpdate() {
         filActives.retainAll(formations().filieres.keySet());
         //do not restrict to fil actives because we want to keep apprentissage
         formations().cleanup();
-
         Set<String> bacsActifs = new HashSet<>(stats.getBacsWithAtLeastNdAdmis(MIN_NB_ADMIS_FOR_BAC_ACTIF));
         bacsActifs.add(TOUS_BACS_CODE_MPS);
         this.stats.restrictToBacs(bacsActifs);

diff --git a/app/etl/etl-common/src/main/java/fr/gouv/monprojetsup/data/model/stats/PsupStatistiques.java b/app/etl/etl-common/src/main/java/fr/gouv/monprojetsup/data/model/stats/PsupStatistiques.java
@@ -19,7 +19,6 @@ public class PsupStatistiques implements Serializable {
 
     public static final String TOUS_GROUPES_CODE = "";
 
-    public static final String TOUS_BACS_CODE_LEGACY = "";
     public static final String TOUS_BACS_CODE_MPS = "NC";
 
     public static final int PRECISION_PERCENTILES = 40;

diff --git a/...l-common/src/main/java/fr/gouv/monprojetsup/data/model/stats/StatistiquesAdmisParBac.java b/...l-common/src/main/java/fr/gouv/monprojetsup/data/model/stats/StatistiquesAdmisParBac.java
@@ -25,19 +25,20 @@ public void set(Map<String, Map<String, int[]>> stringMapMap) {
 
     public void minimize() {
         parBac.values().forEach(StatistiquesAdmisParMatiere::minimize);
-        StatistiquesAdmisParMatiere s1 = parBac.get(PsupStatistiques.TOUS_BACS_CODE_LEGACY);
-        StatistiquesAdmisParMatiere s2 = parBac.get(PsupStatistiques.TOUS_BACS_CODE_MPS);
-        StatistiquesAdmisParMatiere s = s2 != null ? s2 : s1;
+        StatistiquesAdmisParMatiere s = parBac.get(PsupStatistiques.TOUS_BACS_CODE_MPS);
         if(s == null) {
             parBac.clear();
         } else {
-            s.parMatiere().keySet().removeIf( m -> !m.equals(PsupStatistiques.MATIERE_MOYENNE_GENERALE_CODE) && !m.equals(PsupStatistiques.MATIERE_MOYENNE_BAC_CODE));
+            s.parMatiere().keySet().removeIf(
+                    m -> !m.equals(PsupStatistiques.MATIERE_MOYENNE_GENERALE_CODE)
+                            && !m.equals(PsupStatistiques.MATIERE_MOYENNE_BAC_CODE)
+            );
             Statistique ss = s.parMatiere().get(PsupStatistiques.MATIERE_MOYENNE_GENERALE_CODE);
             if (ss == null) {
                 parBac.clear();
             } else {
                 if (ss.nb() <= 3) {
-                    parBac.keySet().removeIf( b -> !b.equals(PsupStatistiques.TOUS_BACS_CODE_LEGACY) && !b.equals(PsupStatistiques.TOUS_BACS_CODE_MPS));
+                    parBac.keySet().removeIf( b -> !b.equals(PsupStatistiques.TOUS_BACS_CODE_MPS));
                 }
             }
         }

diff --git a/...etl/etl-updatedb/src/test/kotlin/fr/gouv/monprojetsup/data/etl/port/FormationsMpsTests.kt b/...etl/etl-updatedb/src/test/kotlin/fr/gouv/monprojetsup/data/etl/port/FormationsMpsTests.kt
@@ -3,6 +3,7 @@ package fr.gouv.monprojetsup.data.etl.port
 import fr.gouv.monprojetsup.data.Constants.gFlCodToMpsId
 import fr.gouv.monprojetsup.data.Constants.gFrCodToMpsId
 import fr.gouv.monprojetsup.data.TestData
+import fr.gouv.monprojetsup.data.model.stats.PsupStatistiques
 import org.assertj.core.api.Assertions.assertThat
 import org.junit.jupiter.api.Nested
 import org.junit.jupiter.api.Test
@@ -145,7 +146,8 @@ class FormationsMpsTests : DataPortTest() {
 
         @Test
         fun `tous les bacs de toutes les stats sont connus`() {
-            val keys = mpsDataPort.getBacs().map { b -> b.key }
+            val keys = HashSet(mpsDataPort.getBacs().map { b -> b.key })
+            keys.add(PsupStatistiques.TOUS_BACS_CODE_MPS)
             val statsKeys = mpsDataPort.getStatsFormation().flatMap { it.value.nbAdmisParBac.keys }
             assertThat(keys).containsAll(statsKeys)
         }

diff --git a/...ggestions-algo/src/main/java/fr/gouv/monprojetsup/suggestions/algo/AffinityEvaluator.java b/...ggestions-algo/src/main/java/fr/gouv/monprojetsup/suggestions/algo/AffinityEvaluator.java
@@ -32,7 +32,6 @@
 
 import static fr.gouv.monprojetsup.data.Constants.isFiliere;
 import static fr.gouv.monprojetsup.data.Constants.isMetier;
-import static fr.gouv.monprojetsup.data.model.stats.PsupStatistiques.TOUS_BACS_CODE_LEGACY;
 import static fr.gouv.monprojetsup.data.model.stats.PsupStatistiques.TOUS_BACS_CODE_MPS;
 import static fr.gouv.monprojetsup.suggestions.algo.Config.ADMISSIBILITY_10;
 import static fr.gouv.monprojetsup.suggestions.algo.Config.ADMISSIBILITY_25;
@@ -101,8 +100,7 @@ public AffinityEvaluator(ProfileDTO pf, Config cfg, AlgoSuggestions algo, boolea
         this.algo = algo;
 
         String pfBac = pf.bac();
-        if(pfBac == null) pfBac = TOUS_BACS_CODE_MPS;
-        if(pfBac.equals(TOUS_BACS_CODE_LEGACY)) pfBac = TOUS_BACS_CODE_MPS;
+        if(pfBac == null || pfBac.isBlank()) pfBac = TOUS_BACS_CODE_MPS;
         this.bac = pfBac;
 
         try {