Fix get{Min,Max}Elements() usage 97/93897/1
authorRobert Varga <robert.varga@pantheon.tech>
Wed, 18 Nov 2020 12:33:10 +0000 (13:33 +0100)
committerRobert Varga <robert.varga@pantheon.tech>
Fri, 20 Nov 2020 16:56:35 +0000 (17:56 +0100)
These return a nullable, UTs should be using
assertEquals(Object, Object) to compare them.

Change-Id: I6d6aefc94695c5118b4ec4552f64bb45b45dcdc8
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>
(cherry picked from commit de729828fea23328647b2cc169fad665e9934bee)

yang/yang-parser-rfc7950/src/test/java/org/opendaylight/yangtools/yang/stmt/Bug9244Test.java
yang/yang-parser-rfc7950/src/test/java/org/opendaylight/yangtools/yang/stmt/DeviationResolutionTest.java
yang/yang-parser-rfc7950/src/test/java/org/opendaylight/yangtools/yang/stmt/EffectiveUsesRefineAndConstraintsTest.java
yang/yang-parser-rfc7950/src/test/java/org/opendaylight/yangtools/yang/stmt/GroupingTest.java
yang/yang-parser-rfc7950/src/test/java/org/opendaylight/yangtools/yang/stmt/ListTest.java
yang/yang-parser-rfc7950/src/test/java/org/opendaylight/yangtools/yang/stmt/YangParserTest.java
yang/yang-parser-rfc7950/src/test/java/org/opendaylight/yangtools/yang/stmt/YangParserWithContextTest.java

index 768183cd217e7a64547255727f60e690cdc4bd44..2f959bb5b62e133a8741dbc8beb981c7bb30394b 100644 (file)
@@ -5,7 +5,6 @@
  * terms of the Eclipse Public License v1.0 which accompanies this distribution,
  * and is available at http://www.eclipse.org/legal/epl-v10.html
  */
-
 package org.opendaylight.yangtools.yang.stmt;
 
 import static org.junit.Assert.assertEquals;
@@ -40,8 +39,8 @@ public class Bug9244Test {
                 QName.create(barModule.getQNameModule(), "bar-leaf-list"));
         assertNotNull(barLeafList);
         final ElementCountConstraint constraint = barLeafList.getElementCountConstraint().get();
-        assertEquals(5, constraint.getMinElements().intValue());
-        assertEquals(10, constraint.getMaxElements().intValue());
+        assertEquals((Object) 5, constraint.getMinElements());
+        assertEquals((Object) 10, constraint.getMaxElements());
 
         final LeafSchemaNode barLeaf = (LeafSchemaNode) barModule.getDataChildByName(
                 QName.create(barModule.getQNameModule(), "bar-leaf"));
index c3292d8f0fd74adac369f577dccbbaa99027139e..77170fe94b9463d4e6453940b14ea8b43de9608e 100644 (file)
@@ -90,8 +90,8 @@ public class DeviationResolutionTest {
         assertEquals(3, myLeafList.getDefaults().size());
 
         final ElementCountConstraint constraint = myLeafList.getElementCountConstraint().get();
-        assertEquals(10, constraint.getMaxElements().intValue());
-        assertEquals(5, constraint.getMinElements().intValue());
+        assertEquals((Object) 10, constraint.getMaxElements());
+        assertEquals((Object) 5, constraint.getMinElements());
         assertNotNull(myLeafList.getType().getUnits());
 
         final ListSchemaNode myList = (ListSchemaNode) barModule.getDataChildByName(
@@ -149,8 +149,8 @@ public class DeviationResolutionTest {
         assertNotNull(myLeafList);
 
         final ElementCountConstraint constraint = myLeafList.getElementCountConstraint().get();
-        assertEquals(6, constraint.getMaxElements().intValue());
-        assertEquals(3, constraint.getMinElements().intValue());
+        assertEquals((Object) 6, constraint.getMaxElements());
+        assertEquals((Object) 3, constraint.getMinElements());
         assertTrue(myLeafList.isConfiguration());
 
         final ChoiceSchemaNode myChoice = (ChoiceSchemaNode) barModule.getDataChildByName(
index 01d5d77122c36b36cc9eaf6ab49224dc6a7ddbdd..ef6dcd8758647e3b7452aac3a034beae254f24b9 100644 (file)
@@ -106,8 +106,8 @@ public class EffectiveUsesRefineAndConstraintsTest {
         assertFalse(listSchemaNode.isConfiguration());
 
         ElementCountConstraint listConstraints = listSchemaNode.getElementCountConstraint().get();
-        assertEquals(10, listConstraints.getMinElements().intValue());
-        assertEquals(20, listConstraints.getMaxElements().intValue());
+        assertEquals((Object) 10, listConstraints.getMinElements());
+        assertEquals((Object) 20, listConstraints.getMaxElements());
         assertEquals(1, listSchemaNode.getMustConstraints().size());
     }
 
@@ -141,8 +141,8 @@ public class EffectiveUsesRefineAndConstraintsTest {
         assertTrue(listSchemaNode.isConfiguration());
 
         ElementCountConstraint listConstraints = listSchemaNode.getElementCountConstraint().get();
-        assertEquals(5, listConstraints.getMinElements().intValue());
-        assertEquals(7, listConstraints.getMaxElements().intValue());
+        assertEquals((Object) 5, listConstraints.getMinElements());
+        assertEquals((Object) 7, listConstraints.getMaxElements());
         assertEquals(2, listSchemaNode.getMustConstraints().size());
     }
 }
index dec90ae129ae807f532260ece873683718abff87..75e53e5a5e63d19579d899faf339c7bb353a1054 100644 (file)
@@ -124,7 +124,7 @@ public class GroupingTest {
         assertFalse(refineList.isConfiguration());
 
         final ElementCountConstraint constraint = refineList.getElementCountConstraint().get();
-        assertEquals(2, constraint.getMinElements().intValue());
+        assertEquals((Object) 2, constraint.getMinElements());
         assertNull(constraint.getMaxElements());
 
         // leaf id
index c8479e0b77d315d1bf78ee866244775d63fcb02f..2a4c44ca761e2028adb587176f9b09f4ff0680ff 100644 (file)
@@ -5,7 +5,6 @@
  * terms of the Eclipse Public License v1.0 which accompanies this distribution,
  * and is available at http://www.eclipse.org/legal/epl-v10.html
  */
-
 package org.opendaylight.yangtools.yang.stmt;
 
 import static org.junit.Assert.assertEquals;
@@ -52,8 +51,8 @@ public class ListTest {
         assertEquals("key2", keys.get(1).getLocalName());
 
         ElementCountConstraint constraint = list.getElementCountConstraint().get();
-        assertEquals(1, constraint.getMinElements().intValue());
-        assertEquals(10, constraint.getMaxElements().intValue());
+        assertEquals((Object) 1, constraint.getMinElements());
+        assertEquals((Object) 10, constraint.getMaxElements());
 
         assertEquals(5, list.getChildNodes().size());
 
index cb5ad927d0ad9f5c8e53e33d437b6d90950c3f32..517e58fb733d464279379d124b81879a8743fa39 100644 (file)
@@ -139,8 +139,8 @@ public class YangParserTest {
         // assertNull(constraints.getWhenCondition());
         assertEquals(0, ifEntry.getMustConstraints().size());
         ElementCountConstraint constraints = ifEntry.getElementCountConstraint().get();
-        assertEquals(1, constraints.getMinElements().intValue());
-        assertEquals(11, constraints.getMaxElements().intValue());
+        assertEquals((Object) 1, constraints.getMinElements());
+        assertEquals((Object) 11, constraints.getMaxElements());
         // test AugmentationTarget args
         final Collection<? extends AugmentationSchemaNode> availableAugmentations = ifEntry.getAvailableAugmentations();
         assertEquals(2, availableAugmentations.size());
index 48dbcb6505f64569e1967ce01fb96fa2372309da..d10119b1e946c03a551d97da59644cabb1fe699b 100644 (file)
@@ -268,8 +268,8 @@ public class YangParserWithContextTest {
         assertEquals(Optional.of("addresses reference added by refine"), refineList.getReference());
         assertFalse(refineList.isConfiguration());
         final ElementCountConstraint constraint = refineList.getElementCountConstraint().get();
-        assertEquals(2, constraint.getMinElements().intValue());
-        assertEquals(12, constraint.getMaxElements().intValue());
+        assertEquals((Object) 2, constraint.getMinElements());
+        assertEquals((Object) 12, constraint.getMaxElements());
     }
 
     @Test