Fix NPE in AutomatonPerimeterResultImpl

ra-optimisation/search-tree-rao/src/main/java/com/powsybl/openrao/searchtreerao/castor/algorithm/AutomatonSimulator.java

@@ -95,16 +95,16 @@ public AutomatonSimulator(Crac crac, RaoParameters raoParameters, ToolProvider t
      */
     AutomatonPerimeterResultImpl simulateAutomatonState(State automatonState, Set<State> curativeStates, Network network, StateTree stateTree, TreeParameters automatonTreeParameters) {
         TECHNICAL_LOGS.info("Optimizing automaton state {}.", automatonState.getId());
-        TECHNICAL_LOGS.info("Initial situation:");
-        RaoLogger.logMostLimitingElementsResults(TECHNICAL_LOGS, prePerimeterSensitivityOutput, Set.of(automatonState), raoParameters.getObjectiveFunctionParameters().getType(), numberLoggedElementsDuringRao);
 
         PrePerimeterSensitivityAnalysis preAutoPstOptimizationSensitivityAnalysis = getPreAutoPerimeterSensitivityAnalysis(automatonState, curativeStates);
-
         // Sensitivity analysis failed :
         if (prePerimeterSensitivityOutput.getSensitivityStatus(automatonState) == ComputationStatus.FAILURE) {
             return createFailedAutomatonPerimeterResult(automatonState, prePerimeterSensitivityOutput, new HashSet<>(), "before");
         }
 
+        TECHNICAL_LOGS.info("Initial situation:");
+        RaoLogger.logMostLimitingElementsResults(TECHNICAL_LOGS, prePerimeterSensitivityOutput, Set.of(automatonState), raoParameters.getObjectiveFunctionParameters().getType(), numberLoggedElementsDuringRao);
+
         // I) Simulate FORCED topological automatons
         TopoAutomatonSimulationResult topoSimulationResult = simulateTopologicalAutomatons(automatonState, network, preAutoPstOptimizationSensitivityAnalysis);
 

ra-optimisation/search-tree-rao/src/main/java/com/powsybl/openrao/searchtreerao/castor/algorithm/CastorSecondPreventive.java

@@ -461,7 +461,7 @@ private static Map<State, Map<RangeAction<?>, Double>> buildSetPointResultsMap(C
         correspondanceMap.forEach((pra, associatedCras) -> {
             setPointResults.get(preventiveState).put(pra, firstPreventiveResult.getOptimizedSetpoint(pra, preventiveState));
             associatedCras.forEach(cra -> contingencyResults.forEach((state, result) -> {
-                if (crac.isRangeActionAvailableInState(cra, state)) {
+                if (crac.isRangeActionAvailableInState(cra, state) && result.getComputationStatus() != FAILURE) {
                     setPointResults.putIfAbsent(state, new HashMap<>());
                     setPointResults.get(state).put(pra, result.getOptimizedSetpoint(cra, state));
                 }
@@ -470,7 +470,7 @@ private static Map<State, Map<RangeAction<?>, Double>> buildSetPointResultsMap(C
         multipleInstantRangeActions.forEach(ra -> {
             setPointResults.get(preventiveState).put(ra, firstPreventiveResult.getOptimizedSetpoint(ra, preventiveState));
             contingencyResults.forEach((state, result) -> {
-                if (crac.isRangeActionAvailableInState(ra, state)) {
+                if (crac.isRangeActionAvailableInState(ra, state) && result.getComputationStatus() != FAILURE) {
                     setPointResults.putIfAbsent(state, new HashMap<>());
                     setPointResults.get(state).put(ra, result.getOptimizedSetpoint(ra, state));
                 }

ra-optimisation/search-tree-rao/src/main/java/com/powsybl/openrao/searchtreerao/result/impl/AutomatonPerimeterResultImpl.java

@@ -183,7 +183,7 @@ public Map<PstRangeAction, Integer> getOptimizedTapsOnState(State state) {
 
     @Override
     public ComputationStatus getSensitivityStatus() {
-        return postAutomatonSensitivityAnalysisOutput.getSensitivityStatus();
+        return postAutomatonSensitivityAnalysisOutput.getSensitivityStatus(optimizedState);
     }
 
     @Override

ra-optimisation/search-tree-rao/src/main/java/com/powsybl/openrao/searchtreerao/result/impl/CurativeWithSecondPraoResult.java

@@ -223,7 +223,7 @@ public Map<PstRangeAction, Integer> getOptimizedTapsOnState(State state) {
 
     @Override
     public ComputationStatus getSensitivityStatus() {
-        return postCraSensitivitySensitivityResult.getSensitivityStatus();
+        return postCraSensitivitySensitivityResult.getSensitivityStatus(state);
     }
 
     @Override

ra-optimisation/search-tree-rao/src/test/java/com/powsybl/openrao/searchtreerao/result/impl/AutomatonPerimeterResultImplTest.java

@@ -188,7 +188,7 @@ void testWrongState() {
 
     @Test
     void testGetSensitivityStatus() {
-        when(postAutoSensitivity.getSensitivityStatus()).thenReturn(ComputationStatus.DEFAULT);
+        when(postAutoSensitivity.getSensitivityStatus(state1)).thenReturn(ComputationStatus.DEFAULT);
         assertEquals(ComputationStatus.DEFAULT, result.getSensitivityStatus());
     }
 

sensitivity-analysis/src/main/java/com/powsybl/openrao/sensitivityanalysis/SystematicSensitivityResult.java

@@ -217,6 +217,9 @@ public SensitivityComputationStatus getStatus() {
     }
 
     public SensitivityComputationStatus getStatus(State state) {
+        if (status == SensitivityComputationStatus.FAILURE) {
+            return status;
+        }
         Optional<Contingency> optionalContingency = state.getContingency();
         if (optionalContingency.isPresent()) {
             List<Integer> possibleInstants = postContingencyResults.keySet().stream()