remove prints

Signed-off-by: Thomas Bouquet <thomas.bouquet@rte-france.com>

Author: bqth29

ra-optimisation/search-tree-rao/src/main/java/com/powsybl/openrao/searchtreerao/commons/costevaluatorresult/SumMaxPerTimestampCostEvaluatorResult.java

@@ -42,29 +42,23 @@ public double getCost(Set<String> contingenciesToExclude, Set<String> cnecsToExc
         Map<FlowCnec, Double> filteredCnecs = marginPerCnec.entrySet().stream()
             .filter(entry -> !cnecsToExclude.contains(entry.getKey().getId()))
             .collect(Collectors.toMap(Map.Entry::getKey, Map.Entry::getValue));
-        System.out.println("filtered cnecs: " + filteredCnecs.keySet());
         if (filteredCnecs.isEmpty()) {
             return -Double.MAX_VALUE;
         }
 
         // Compute state wise cost
         Map<State, Set<FlowCnec>> flowCnecsPerState = groupFlowCnecsPerState(filteredCnecs.keySet());
         Map<State, Double> costPerState = flowCnecsPerState.entrySet().stream().collect(Collectors.toMap(Map.Entry::getKey, entry -> computeCostForState(entry.getValue())));
-        costPerState.forEach((state, cost) -> System.out.println(state.getId() + " " + cost));
 
         Map<OffsetDateTime, Set<State>> statesToEvaluatePerTimestamp = new HashMap<>();
         Set<State> statesToEvaluateWithoutTimestamp = new HashSet<>();
 
         costPerState.keySet().stream().forEach(state -> {
-            System.out.println("checking state: " + state.getId());
             if (statesContingencyMustBeKept(state, contingenciesToExclude)) {
-                System.out.println("state is to be kept: " + state.getId());
                 Optional<OffsetDateTime> timestamp = state.getTimestamp();
                 if (timestamp.isPresent()) {
-                    System.out.println("timestamp present: " + state.getId());
                     statesToEvaluatePerTimestamp.computeIfAbsent(timestamp.get(), s -> new HashSet<>()).add(state);
                 } else {
-                    System.out.println("timestamp missing: " + state.getId());
                     statesToEvaluateWithoutTimestamp.add(state);
                 }
             }

ra-optimisation/search-tree-rao/src/test/java/com/powsybl/openrao/searchtreerao/commons/costevaluatorresult/SumMaxPerTimestampCostEvaluatorResultTest.java

@@ -8,6 +8,7 @@
 package com.powsybl.openrao.searchtreerao.commons.costevaluatorresult;
 
 import com.powsybl.contingency.Contingency;
+import com.powsybl.openrao.commons.OpenRaoException;
 import com.powsybl.openrao.commons.Unit;
 import com.powsybl.openrao.data.crac.api.State;
 import com.powsybl.openrao.data.crac.api.cnec.FlowCnec;
@@ -161,10 +162,15 @@ void testWithoutAnyTimestampWithExclusion() {
         addNullTimestamp();
         Map<FlowCnec, Double> marginPerCnec = Map.of(flowCnecPreventiveT1, -10.0, flowCnecCurative1T1, -50.0, flowCnecCurative12T1, -40.0, flowCnecCurativeT2, 20.0, flowCnecCurativeT3, -17.0);
         SumMaxPerTimestampCostEvaluatorResult evaluatorResult = new SumMaxPerTimestampCostEvaluatorResult(marginPerCnec, List.of(), Unit.MEGAWATT);
-        assertEquals(50.0, evaluatorResult.getCost(Set.of(), Set.of()));
-        assertEquals(17.0, evaluatorResult.getCost(Set.of("contingency-1", "contingency-2"), Set.of()));
-        assertEquals(40.0, evaluatorResult.getCost(Set.of(), Set.of("cnec-curative1")));
-        assertEquals(17.0, evaluatorResult.getCost(Set.of("contingency-1"), Set.of("cnec-curative1")));
+        try {
+            assertEquals(50.0, evaluatorResult.getCost(Set.of(), Set.of()));
+            assertEquals(17.0, evaluatorResult.getCost(Set.of("contingency-1", "contingency-2"), Set.of()));
+            assertEquals(40.0, evaluatorResult.getCost(Set.of(), Set.of("cnec-curative1")));
+            assertEquals(17.0, evaluatorResult.getCost(Set.of("contingency-1"), Set.of("cnec-curative1")));
+        } catch (NullPointerException e) {
+            e.printStackTrace();
+            throw new OpenRaoException(e);
+        }
     }
 
     @Test