Checkstyle

Signed-off-by: Sylvestre Prabakaran <sylvestre.prabakaran@rte-france.com>

Author: SylvestreSakti

src/test/java/com/powsybl/openloadflow/OperationalLimitsTest.java

@@ -77,7 +77,7 @@ void testLineCurrentLimits() {
     }
 
     private double getLimitValueFromAcceptableDuration(LfBranch branch, int acceptableDuration, TwoSides side, LimitType type) {
-        List<LfBranch.LfLimitsGroup> limitsGroups = (side == TwoSides.ONE ? branch.getLimits1(type, null) : branch.getLimits2(type, null));
+        List<LfBranch.LfLimitsGroup> limitsGroups = side == TwoSides.ONE ? branch.getLimits1(type, null) : branch.getLimits2(type, null);
         if (limitsGroups.isEmpty()) {
             return Double.NaN;
         }

src/test/java/com/powsybl/openloadflow/sa/OpenSecurityAnalysisTest.java

@@ -79,8 +79,8 @@
  */
 class OpenSecurityAnalysisTest extends AbstractOpenSecurityAnalysisTest {
 
-    // TO DO : use powsybl-core LimitViolations.comparator() instead when it'll compare also OperationalLimtisGroupId
-    Comparator<LimitViolation> COMPARATOR = LimitViolations.comparator()
+    // TO DO : use powsybl-core LimitViolations.comparator() instead when it'll compare also OperationalLimitsGroupId
+    Comparator<LimitViolation> limitViolationComparator = LimitViolations.comparator()
             .thenComparing(LimitViolation::getOperationalLimitsGroupId);
 
     @Test
@@ -1355,7 +1355,7 @@ void testViolationOnThreeWindingsTransformersLeg() {
         assertEquals(1, result.getPreContingencyResult().getLimitViolationsResult().getLimitViolations().size());
         LimitViolation expected = new LimitViolation("3wt", null, "DEFAULT", LimitViolationType.CURRENT, "permanent",
                 60, 400., 1.0F, 435.0831773201809, TwoSides.TWO.toThreeSides(), null);
-        int compare = COMPARATOR.compare(expected,
+        int compare = limitViolationComparator.compare(expected,
                 result.getPreContingencyResult().getLimitViolationsResult().getLimitViolations().get(0));
         assertEquals(0, compare);
     }
@@ -2739,21 +2739,21 @@ void testAcceptableDurations(SecurityAnalysisParameters securityAnalysisParamete
 
         LimitViolation violation0 = new LimitViolation("NHV1_NHV2_2", null, "DEFAULT", LimitViolationType.CURRENT, "20'",
                 60, 1000.0, 1.0F, 1047.8598237521767, TwoSides.TWO.toThreeSides(), null);
-        int compare0 = COMPARATOR.compare(violation0, result.getPostContingencyResults().get(0)
+        int compare0 = limitViolationComparator.compare(violation0, result.getPostContingencyResults().get(0)
                 .getLimitViolationsResult().getLimitViolations().get(0));
         assertEquals(0, compare0);
 
         LimitViolation violation1 = new LimitViolation("NHV1_NHV2_1", null, "DEFAULT", LimitViolationType.CURRENT, "10'",
                 60, 1000.0, 1.0F, 1047.8598237521767, TwoSides.TWO.toThreeSides(), null);
-        int compare1 = COMPARATOR.compare(violation1, result.getPostContingencyResults().get(1)
+        int compare1 = limitViolationComparator.compare(violation1, result.getPostContingencyResults().get(1)
                 .getLimitViolationsResult().getLimitViolations().get(0));
         assertEquals(0, compare1);
 
-        int compare2 = COMPARATOR.compare(violation0, result.getPostContingencyResults().get(2)
+        int compare2 = limitViolationComparator.compare(violation0, result.getPostContingencyResults().get(2)
                 .getLimitViolationsResult().getLimitViolations().get(0));
         assertEquals(0, compare2); // FIXME line open at one side
 
-        int compare3 = COMPARATOR.compare(violation1, result.getPostContingencyResults().get(3)
+        int compare3 = limitViolationComparator.compare(violation1, result.getPostContingencyResults().get(3)
                 .getLimitViolationsResult().getLimitViolations().get(0));
         assertEquals(0, compare3); // FIXME line open at one side
 
@@ -2764,12 +2764,12 @@ void testAcceptableDurations(SecurityAnalysisParameters securityAnalysisParamete
 
         LimitViolation violation4 = new LimitViolation("NHV1_NHV2_2", null, "DEFAULT", LimitViolationType.CURRENT, "permanent",
                 2147483647, 899.9999999999999, 1.0F, 911.6056881941461, TwoSides.ONE.toThreeSides(), null);
-        int compare4 = COMPARATOR.compare(violation4, result2.getPostContingencyResults().get(0)
+        int compare4 = limitViolationComparator.compare(violation4, result2.getPostContingencyResults().get(0)
                 .getLimitViolationsResult().getLimitViolations().get(0));
         assertEquals(0, compare4);
         LimitViolation violation5 = new LimitViolation("NHV1_NHV2_2", null, "DEFAULT", LimitViolationType.CURRENT, "permanent",
                 1200, 899.9999999999999, 1.0F, 911.6056881941461, TwoSides.TWO.toThreeSides(), null);
-        int compare5 = COMPARATOR.compare(violation5, result2.getPostContingencyResults().get(0)
+        int compare5 = limitViolationComparator.compare(violation5, result2.getPostContingencyResults().get(0)
                 .getLimitViolationsResult().getLimitViolations().get(1));
         assertEquals(0, compare5);
 
@@ -2779,12 +2779,12 @@ void testAcceptableDurations(SecurityAnalysisParameters securityAnalysisParamete
         SecurityAnalysisResult result3 = runSecurityAnalysis(network, contingencies.getContingencies(network), Collections.emptyList(), securityAnalysisParameters);
         LimitViolation violation6 = new LimitViolation("NHV1_NHV2_2", null, "DEFAULT", LimitViolationType.CURRENT, "permanent",
                 2147483647, 899.9999999999999, 1.0F, 911.6056881941463, TwoSides.ONE.toThreeSides(), null);
-        int compare6 = COMPARATOR.compare(violation6, result3.getPostContingencyResults().get(0)
+        int compare6 = limitViolationComparator.compare(violation6, result3.getPostContingencyResults().get(0)
                 .getLimitViolationsResult().getLimitViolations().get(0));
         assertEquals(0, compare6);
         LimitViolation violation7 = new LimitViolation("NHV1_NHV2_2", null, "DEFAULT", LimitViolationType.CURRENT, "permanent",
                 1200, 899.9999999999999, 1.0F, 911.6056881941463, TwoSides.TWO.toThreeSides(), null);
-        int compare7 = COMPARATOR.compare(violation7, result3.getPostContingencyResults().get(0)
+        int compare7 = limitViolationComparator.compare(violation7, result3.getPostContingencyResults().get(0)
                 .getLimitViolationsResult().getLimitViolations().get(1));
         assertEquals(0, compare7);
     }