Use local variable type inference 61/101061/2
authorRobert Varga <robert.varga@pantheon.tech>
Wed, 11 May 2022 11:07:22 +0000 (13:07 +0200)
committerRobert Varga <robert.varga@pantheon.tech>
Wed, 11 May 2022 11:22:12 +0000 (13:22 +0200)
Eclipse has a number of warnings which boil down to capturing the
nullability contract. Use a simple 'var' to reduce verbosity and fix a
number of warnings.

Change-Id: I607a77e0c90d14cb51ecbd0bf5b11ab190d2d1f2
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>
17 files changed:
parser/yang-parser-rfc7950/src/test/java/org/opendaylight/yangtools/yang/parser/stmt/rfc7950/IdentityrefStatementTest.java
parser/yang-parser-rfc7950/src/test/java/org/opendaylight/yangtools/yang/stmt/AugmentTest.java
parser/yang-parser-rfc7950/src/test/java/org/opendaylight/yangtools/yang/stmt/Bug1412Test.java
parser/yang-parser-rfc7950/src/test/java/org/opendaylight/yangtools/yang/stmt/Bug3799Test.java
parser/yang-parser-rfc7950/src/test/java/org/opendaylight/yangtools/yang/stmt/Bug5942Test.java
parser/yang-parser-rfc7950/src/test/java/org/opendaylight/yangtools/yang/stmt/Bug5946Test.java
parser/yang-parser-rfc7950/src/test/java/org/opendaylight/yangtools/yang/stmt/DeclaredStatementsTest.java
parser/yang-parser-rfc7950/src/test/java/org/opendaylight/yangtools/yang/stmt/EffectiveIdentityTest.java
parser/yang-parser-rfc7950/src/test/java/org/opendaylight/yangtools/yang/stmt/ExtensionStmtTest.java
parser/yang-parser-rfc7950/src/test/java/org/opendaylight/yangtools/yang/stmt/GroupingAndUsesStmtTest.java
parser/yang-parser-rfc7950/src/test/java/org/opendaylight/yangtools/yang/stmt/GroupingTest.java
parser/yang-parser-rfc7950/src/test/java/org/opendaylight/yangtools/yang/stmt/IdentityStmtTest.java
parser/yang-parser-rfc7950/src/test/java/org/opendaylight/yangtools/yang/stmt/OrderingTest.java
parser/yang-parser-rfc7950/src/test/java/org/opendaylight/yangtools/yang/stmt/YangParserTest.java
parser/yang-parser-rfc7950/src/test/java/org/opendaylight/yangtools/yang/stmt/YangParserWithContextTest.java
parser/yang-parser-rfc7950/src/test/java/org/opendaylight/yangtools/yang/stmt/yin/YinFileChoiceStmtTest.java
parser/yang-parser-rfc7950/src/test/java/org/opendaylight/yangtools/yang/stmt/yin/YinFileExtensionStmtTest.java

index 5b2ea8e6e26965eaa32989f09ee1e409ab2f3d7c..5d54e9532e3c4402c2a726cbf072e33b16d64046 100644 (file)
@@ -12,12 +12,9 @@ import static org.hamcrest.MatcherAssert.assertThat;
 import static org.hamcrest.collection.IsIterableContainingInAnyOrder.containsInAnyOrder;
 import static org.junit.Assert.assertEquals;
 
-import java.util.Collection;
-import java.util.Set;
 import org.junit.Test;
 import org.opendaylight.yangtools.yang.common.QName;
 import org.opendaylight.yangtools.yang.common.Revision;
-import org.opendaylight.yangtools.yang.model.api.IdentitySchemaNode;
 import org.opendaylight.yangtools.yang.model.api.LeafSchemaNode;
 import org.opendaylight.yangtools.yang.model.api.Module;
 import org.opendaylight.yangtools.yang.model.api.type.IdentityrefTypeDefinition;
@@ -30,13 +27,13 @@ public class IdentityrefStatementTest extends AbstractYangTest {
         final var context = assertEffectiveModel("/rfc7950/identityref-stmt/foo.yang");
 
         final Module foo = context.findModule("foo", Revision.of("2017-01-11")).get();
-        final Collection<? extends IdentitySchemaNode> identities = foo.getIdentities();
+        final var identities = foo.getIdentities();
         assertEquals(3, identities.size());
 
         final LeafSchemaNode idrefLeaf = (LeafSchemaNode) foo.getDataChildByName(QName.create(foo.getQNameModule(),
                 "idref-leaf"));
         final IdentityrefTypeDefinition idrefType = (IdentityrefTypeDefinition) idrefLeaf.getType();
-        final Set<? extends IdentitySchemaNode> referencedIdentities = idrefType.getIdentities();
+        final var referencedIdentities = idrefType.getIdentities();
         assertEquals(3, referencedIdentities.size());
         assertThat(referencedIdentities, containsInAnyOrder(identities.toArray()));
         assertEquals("id-a", idrefType.getIdentities().iterator().next().getQName().getLocalName());
index b2a54ba4877ec26d4ad4b24963326484dc30f77b..18c6c283e88c5ad5ae2fdf0685ba5cf1eac65675 100644 (file)
@@ -12,7 +12,6 @@ import static org.junit.Assert.assertFalse;
 import static org.junit.Assert.assertNotNull;
 import static org.junit.Assert.assertTrue;
 
-import java.util.Collection;
 import org.junit.BeforeClass;
 import org.junit.Test;
 import org.opendaylight.yangtools.yang.common.QName;
@@ -57,14 +56,14 @@ public class AugmentTest extends AbstractYangTest {
     public void testAugmentParsing() {
         // foo.yang
         final Module module1 = AUGMENT_IN_AUGMENT.findModules("foo").iterator().next();
-        Collection<? extends AugmentationSchemaNode> augmentations = module1.getAugmentations();
+        var augmentations = module1.getAugmentations();
         assertEquals(1, augmentations.size());
         final AugmentationSchemaNode augment = augmentations.iterator().next();
         assertNotNull(augment);
 
         assertEquals(Absolute.of(Q0, Q1, Q2), augment.getTargetPath());
 
-        final Collection<? extends DataSchemaNode> augmentChildren = augment.getChildNodes();
+        final var augmentChildren = augment.getChildNodes();
         assertEquals(4, augmentChildren.size());
         for (final DataSchemaNode dsn : augmentChildren) {
             checkIsAugmenting(dsn, false);
@@ -196,7 +195,7 @@ public class AugmentTest extends AbstractYangTest {
         // augment "/br:interfaces/br:ifEntry/bz:augment-holder"
         final ChoiceSchemaNode odl = (ChoiceSchemaNode) augmentedHolder.getDataChildByName(QName.create(FOO, "odl"));
         assertNotNull(odl);
-        final Collection<? extends CaseSchemaNode> cases = odl.getCases();
+        final var cases = odl.getCases();
         assertEquals(4, cases.size());
 
         CaseSchemaNode id = null;
@@ -223,22 +222,22 @@ public class AugmentTest extends AbstractYangTest {
 
         // case id
         assertEquals(QName.create(FOO, "id"), id.getQName());
-        final Collection<? extends DataSchemaNode> idChildren = id.getChildNodes();
+        final var idChildren = id.getChildNodes();
         assertEquals(1, idChildren.size());
 
         // case node1
         assertEquals(QName.create(FOO, "node1"), node1.getQName());
-        final Collection<? extends DataSchemaNode> node1Children = node1.getChildNodes();
+        final var node1Children = node1.getChildNodes();
         assertTrue(node1Children.isEmpty());
 
         // case node2
         assertEquals(QName.create(FOO, "node2"), node2.getQName());
-        final Collection<? extends DataSchemaNode> node2Children = node2.getChildNodes();
+        final var node2Children = node2.getChildNodes();
         assertTrue(node2Children.isEmpty());
 
         // case node3
         assertEquals(QName.create(FOO, "node3"), node3.getQName());
-        final Collection<? extends DataSchemaNode> node3Children = node3.getChildNodes();
+        final var node3Children = node3.getChildNodes();
         assertEquals(1, node3Children.size());
 
         // test cases
@@ -260,7 +259,7 @@ public class AugmentTest extends AbstractYangTest {
         final XMLNamespace NS_FOO = XMLNamespace.of("urn:opendaylight:foo");
         final Revision revision = Revision.of("2013-10-11");
         final Module bar = context.findModules("bar").iterator().next();
-        final Collection<? extends RpcDefinition> rpcs = bar.getRpcs();
+        final var rpcs = bar.getRpcs();
         assertEquals(2, rpcs.size());
 
         RpcDefinition submit = null;
@@ -282,7 +281,7 @@ public class AugmentTest extends AbstractYangTest {
         final QName argumentsQName = QName.create(NS_BAR, revision, "arguments");
         assertEquals(argumentsQName, arguments.getQName());
         assertFalse(arguments.isAugmenting());
-        final Collection<? extends CaseSchemaNode> cases = arguments.getCases();
+        final var cases = arguments.getCases();
         assertEquals(3, cases.size());
 
         CaseSchemaNode attach = null;
@@ -307,17 +306,17 @@ public class AugmentTest extends AbstractYangTest {
 
         // case attach
         assertEquals(QName.create(NS_FOO, revision, "attach"), attach.getQName());
-        final Collection<? extends DataSchemaNode> attachChildren = attach.getChildNodes();
+        final var attachChildren = attach.getChildNodes();
         assertEquals(1, attachChildren.size());
 
         // case create
         assertEquals(QName.create(NS_FOO, revision, "create"), create.getQName());
-        final Collection<? extends DataSchemaNode> createChildren = create.getChildNodes();
+        final var createChildren = create.getChildNodes();
         assertEquals(1, createChildren.size());
 
         // case attach
         assertEquals(QName.create(NS_FOO, revision, "destroy"), destroy.getQName());
-        final Collection<? extends DataSchemaNode> destroyChildren = destroy.getChildNodes();
+        final var destroyChildren = destroy.getChildNodes();
         assertEquals(1, destroyChildren.size());
     }
 
@@ -335,7 +334,7 @@ public class AugmentTest extends AbstractYangTest {
                 "nodes"));
         final ContainerSchemaNode node = (ContainerSchemaNode) nodes.getDataChildByName(QName.create(
                 test.getQNameModule(), "node"));
-        final Collection<? extends AugmentationSchemaNode> augments = node.getAvailableAugmentations();
+        final var augments = node.getAvailableAugmentations();
         assertEquals(1, augments.size());
         assertEquals(1, node.getChildNodes().size());
         final LeafSchemaNode id = (LeafSchemaNode) node.getDataChildByName(QName.create(test.getQNameModule(), "id"));
index 668e93dfa17eb85d128ed61ecaf81bd986208b18..048714d85fc58f43a22431e6e9474b88ffd94268 100644 (file)
@@ -10,13 +10,9 @@ package org.opendaylight.yangtools.yang.stmt;
 import static org.junit.Assert.assertEquals;
 import static org.junit.Assert.assertNotNull;
 
-import java.util.Collection;
 import org.junit.Test;
 import org.opendaylight.yangtools.yang.common.Empty;
 import org.opendaylight.yangtools.yang.common.QName;
-import org.opendaylight.yangtools.yang.common.QNameModule;
-import org.opendaylight.yangtools.yang.common.Revision;
-import org.opendaylight.yangtools.yang.common.XMLNamespace;
 import org.opendaylight.yangtools.yang.model.api.ContainerSchemaNode;
 import org.opendaylight.yangtools.yang.model.api.Module;
 import org.opendaylight.yangtools.yang.model.api.stmt.UnrecognizedStatement;
@@ -34,12 +30,10 @@ public class Bug1412Test extends AbstractYangTest {
 
         final ContainerSchemaNode node = (ContainerSchemaNode) bug1412.getDataChildByName(QName.create(
                 bug1412.getQNameModule(), "node"));
-        Collection<? extends UnrecognizedStatement> unknownNodes = node.asEffectiveStatement().getDeclared()
-            .declaredSubstatements(UnrecognizedStatement.class);
+        var unknownNodes = node.asEffectiveStatement().getDeclared().declaredSubstatements(UnrecognizedStatement.class);
         assertEquals(1, unknownNodes.size());
         final UnrecognizedStatement action = unknownNodes.iterator().next();
 
-        final QNameModule qm = QNameModule.create(XMLNamespace.of("urn:test:bug1412"), Revision.of("2014-07-25"));
         assertEquals(QName.create("urn:test:bug1412:ext:definitions", "2014-07-25", "action"),
             action.statementDefinition().getStatementName());
         assertEquals("hello", action.argument());
index cbd0cf4c18b4b89bf831c37f209076ae1e1d5748..ac09eb5b1a33517bc1d383803fa128102a55bcd3 100644 (file)
@@ -11,7 +11,6 @@ import static org.junit.Assert.assertEquals;
 import static org.junit.Assert.assertNotNull;
 import static org.junit.Assert.assertTrue;
 
-import java.util.Collection;
 import org.junit.Test;
 import org.opendaylight.yangtools.yang.model.api.DataSchemaNode;
 import org.opendaylight.yangtools.yang.model.api.LeafSchemaNode;
@@ -24,23 +23,23 @@ public class Bug3799Test extends AbstractYangTest {
     public void test() {
         final var schema = assertEffectiveModelDir("/bugs/bug3799");
 
-        Collection<? extends Module> modules = schema.getModules();
+        var modules = schema.getModules();
         assertNotNull(modules);
         assertEquals(1, modules.size());
 
         Module testModule = modules.iterator().next();
-        Collection<? extends Submodule> subModules = testModule.getSubmodules();
+        var subModules = testModule.getSubmodules();
         assertNotNull(subModules);
         assertEquals(1, subModules.size());
 
         Submodule testSubmodule = subModules.iterator().next();
 
-        Collection<? extends NotificationDefinition> notifications = testSubmodule.getNotifications();
+        var notifications = testSubmodule.getNotifications();
         assertNotNull(notifications);
         assertEquals(1, notifications.size());
 
         NotificationDefinition bazNotification = notifications.iterator().next();
-        Collection<? extends DataSchemaNode> childNodes = bazNotification.getChildNodes();
+        var childNodes = bazNotification.getChildNodes();
         assertNotNull(childNodes);
         assertEquals(1, childNodes.size());
 
index 95af9388c87b3758f950ba56d8e84a8ad7302224..150f9dbbae9b30ef47a89fbf7a838d70a9d87f68 100644 (file)
@@ -10,7 +10,6 @@ package org.opendaylight.yangtools.yang.stmt;
 import static org.junit.Assert.assertEquals;
 import static org.junit.Assert.assertTrue;
 
-import java.util.Collection;
 import java.util.Optional;
 import org.junit.Test;
 import org.opendaylight.yangtools.yang.common.QName;
@@ -28,7 +27,7 @@ public class Bug5942Test extends AbstractYangTest {
         final DataSchemaNode root = schemaContext.getDataChildByName(QName.create("foo", "2016-06-02", "root"));
         assertTrue(root instanceof ContainerSchemaNode);
 
-        final Collection<? extends UsesNode> uses = ((ContainerSchemaNode) root).getUses();
+        final var uses = ((ContainerSchemaNode) root).getUses();
         assertEquals(1, uses.size());
         final UsesNode usesNode = uses.iterator().next();
 
index 2af5328248a1c1ae47cf609a0b22dacc622413af..21a27611424700741dba38e531a1cb38a44a0929 100644 (file)
@@ -7,10 +7,12 @@
  */
 package org.opendaylight.yangtools.yang.stmt;
 
+import static org.hamcrest.CoreMatchers.startsWith;
+import static org.hamcrest.MatcherAssert.assertThat;
 import static org.junit.Assert.assertEquals;
 import static org.junit.Assert.assertNotNull;
+import static org.junit.Assert.assertThrows;
 import static org.junit.Assert.assertTrue;
-import static org.junit.Assert.fail;
 
 import java.io.File;
 import java.util.Collection;
@@ -21,7 +23,6 @@ import org.opendaylight.yangtools.yang.model.api.DataSchemaNode;
 import org.opendaylight.yangtools.yang.model.api.ListSchemaNode;
 import org.opendaylight.yangtools.yang.model.api.SchemaContext;
 import org.opendaylight.yangtools.yang.model.api.stmt.SchemaNodeIdentifier;
-import org.opendaylight.yangtools.yang.model.api.stmt.SchemaNodeIdentifier.Descendant;
 import org.opendaylight.yangtools.yang.model.api.stmt.UniqueEffectiveStatement;
 import org.opendaylight.yangtools.yang.parser.spi.meta.ReactorException;
 
@@ -45,27 +46,25 @@ public class Bug5946Test {
                 .getResource("/bugs/bug5946/foo.yang").toURI()));
         assertNotNull(context);
 
-        Collection<? extends UniqueEffectiveStatement> uniqueConstraints = getListConstraints(context, WITHOUT_UNIQUE);
+        var uniqueConstraints = getListConstraints(context, WITHOUT_UNIQUE);
         assertNotNull(uniqueConstraints);
         assertTrue(uniqueConstraints.isEmpty());
 
-        Collection<? extends UniqueEffectiveStatement> simpleUniqueConstraints =
-            getListConstraints(context, SIMPLE_UNIQUE);
+        var simpleUniqueConstraints = getListConstraints(context, SIMPLE_UNIQUE);
         assertNotNull(simpleUniqueConstraints);
         assertEquals(1, simpleUniqueConstraints.size());
-        Collection<Descendant> simpleUniqueConstraintTag = simpleUniqueConstraints.iterator().next().argument();
+        var simpleUniqueConstraintTag = simpleUniqueConstraints.iterator().next().argument();
         assertTrue(simpleUniqueConstraintTag.contains(L1_ID));
         assertTrue(simpleUniqueConstraintTag.contains(C_L3_ID));
 
-        Collection<? extends UniqueEffectiveStatement> multipleUniqueConstraints =
-            getListConstraints(context, MULTIPLE_UNIQUE);
+        var multipleUniqueConstraints = getListConstraints(context, MULTIPLE_UNIQUE);
         assertNotNull(multipleUniqueConstraints);
         assertEquals(3, multipleUniqueConstraints.size());
         boolean l1l2 = false;
         boolean l1cl3 = false;
         boolean cl3l2 = false;
         for (UniqueEffectiveStatement uniqueConstraint : multipleUniqueConstraints) {
-            Collection<Descendant> uniqueConstraintTag = uniqueConstraint.argument();
+            var uniqueConstraintTag = uniqueConstraint.argument();
             if (uniqueConstraintTag.contains(L1_ID) && uniqueConstraintTag.contains(L2_ID)) {
                 l1l2 = true;
             } else if (uniqueConstraintTag.contains(L1_ID) && uniqueConstraintTag.contains(C_L3_ID)) {
@@ -79,17 +78,14 @@ public class Bug5946Test {
 
     @Test
     public void testInvalid() throws Exception {
-        try {
-            StmtTestUtils.parseYangSources(new File(getClass().getResource("/bugs/bug5946/foo-invalid.yang").toURI()));
-            fail("Should fail due to invalid argument of unique constraint");
-        } catch (ReactorException e) {
-            assertTrue(e.getCause().getMessage().startsWith(
-                    "Unique statement argument '/simple-unique/l1' contains schema node identifier '/simple-unique/l1'"
-                            + " which is not in the descendant node identifier form."));
-        }
+        final var cause = assertThrows(ReactorException.class, () -> StmtTestUtils.parseYangSources(
+            new File(getClass().getResource("/bugs/bug5946/foo-invalid.yang").toURI())))
+            .getCause();
+        assertThat(cause.getMessage(), startsWith("Unique statement argument '/simple-unique/l1' contains schema node "
+            + "identifier '/simple-unique/l1' which is not in the descendant node identifier form."));
     }
 
-    private static @NonNull Collection<? extends UniqueEffectiveStatement> getListConstraints(
+    private static @NonNull Collection<? extends @NonNull UniqueEffectiveStatement> getListConstraints(
             final SchemaContext context, final QName listQName) {
         DataSchemaNode dataChildByName = context.getDataChildByName(listQName);
         assertTrue(dataChildByName instanceof ListSchemaNode);
index 0c6e99304b4aab8b933e2091d77e06504464252c..77919c8c4ba715840d331b0a16c2144045a0c134 100644 (file)
@@ -90,18 +90,18 @@ public class DeclaredStatementsTest {
         final QName name = anyxmlStatement.argument();
         assertNotNull(name);
 
-        final WhenStatement whenStatement = anyxmlStatement.getWhenStatement().get();
+        final WhenStatement whenStatement = anyxmlStatement.getWhenStatement().orElseThrow();
         assertNotNull(whenStatement.argument());
-        final DescriptionStatement whenStatementDescription = whenStatement.getDescription().get();
+        final DescriptionStatement whenStatementDescription = whenStatement.getDescription().orElseThrow();
         assertTrue(whenStatement.getReference().isPresent());
 
-        final Collection<? extends IfFeatureStatement> ifFeatureStatements = anyxmlStatement.getIfFeatures();
+        final var ifFeatureStatements = anyxmlStatement.getIfFeatures();
         assertNotNull(ifFeatureStatements);
         assertEquals(1, ifFeatureStatements.size());
         final Predicate<Set<QName>> ifFeaturePredicate = ifFeatureStatements.iterator().next().getIfFeaturePredicate();
         assertNotNull(ifFeaturePredicate);
 
-        final Collection<? extends MustStatement> mustStatements = anyxmlStatement.getMustStatements();
+        final var mustStatements = anyxmlStatement.getMustStatements();
         assertNotNull(mustStatements);
         assertEquals(1, mustStatements.size());
         final MustStatement mustStatement = mustStatements.iterator().next();
@@ -111,17 +111,17 @@ public class DeclaredStatementsTest {
         assertTrue(mustStatement.getDescription().isPresent());
         assertTrue(mustStatement.getReference().isPresent());
 
-        final ConfigStatement configStatement = anyxmlStatement.getConfig().get();
+        final ConfigStatement configStatement = anyxmlStatement.getConfig().orElseThrow();
         assertFalse(configStatement.getValue());
 
-        final StatusStatement statusStatement = anyxmlStatement.getStatus().get();
+        final StatusStatement statusStatement = anyxmlStatement.getStatus().orElseThrow();
         final Status status = statusStatement.argument();
         assertNotNull(status);
 
-        final DescriptionStatement descriptionStatement = anyxmlStatement.getDescription().get();
+        final DescriptionStatement descriptionStatement = anyxmlStatement.getDescription().orElseThrow();
         assertEquals("anyxml description", descriptionStatement.argument());
 
-        final ReferenceStatement referenceStatement = anyxmlStatement.getReference().get();
+        final ReferenceStatement referenceStatement = anyxmlStatement.getReference().orElseThrow();
         assertEquals("anyxml reference", referenceStatement.argument());
 
         assertTrue(anyxmlStatement.getMandatory().isPresent());
@@ -146,33 +146,32 @@ public class DeclaredStatementsTest {
         final QName name = choiceStatement.argument();
         assertNotNull(name);
 
-        final DefaultStatement defaultStatement = choiceStatement.getDefault().get();
+        final DefaultStatement defaultStatement = choiceStatement.getDefault().orElseThrow();
         assertEquals("case-two", defaultStatement.argument());
 
         assertTrue(choiceStatement.getConfig().isPresent());
         assertTrue(choiceStatement.getMandatory().isPresent());
 
-        final Collection<? extends CaseStatement> caseStatements = choiceStatement.getCases();
+        final var caseStatements = choiceStatement.getCases();
         assertNotNull(caseStatements);
         assertEquals(3, caseStatements.size());
         final CaseStatement caseStatement = caseStatements.iterator().next();
         final QName caseStatementName = caseStatement.argument();
         assertNotNull(caseStatementName);
-        final WhenStatement caseStatementWhen = caseStatement.getWhenStatement().get();
-        final Collection<? extends IfFeatureStatement> caseStatementIfFeatures = caseStatement.getIfFeatures();
+        final WhenStatement caseStatementWhen = caseStatement.getWhenStatement().orElseThrow();
+        final var caseStatementIfFeatures = caseStatement.getIfFeatures();
         assertNotNull(caseStatementIfFeatures);
         assertEquals(1, caseStatementIfFeatures.size());
-        final Collection<? extends DataDefinitionStatement> caseStatementDataDefinitions =
-                caseStatement.getDataDefinitions();
+        final var caseStatementDataDefinitions = caseStatement.getDataDefinitions();
         assertNotNull(caseStatementDataDefinitions);
         assertEquals(1, caseStatementDataDefinitions.size());
         assertTrue(caseStatement.getStatus().isPresent());
         assertTrue(caseStatement.getDescription().isPresent());
         assertTrue(caseStatement.getReference().isPresent());
 
-        final WhenStatement whenStatement = choiceStatement.getWhenStatement().get();
+        final WhenStatement whenStatement = choiceStatement.getWhenStatement().orElseThrow();
 
-        final Collection<? extends IfFeatureStatement> ifFeatureStatements = choiceStatement.getIfFeatures();
+        final var ifFeatureStatements = choiceStatement.getIfFeatures();
         assertNotNull(ifFeatureStatements);
         assertEquals(1, ifFeatureStatements.size());
 
@@ -271,7 +270,7 @@ public class DeclaredStatementsTest {
         assertNotNull(schemaContext);
 
         final Revision revision = Revision.of("2016-09-28");
-        final Module testModule = schemaContext.findModule("root-module-declared-test", revision).get();
+        final Module testModule = schemaContext.findModule("root-module-declared-test", revision).orElseThrow();
         assertNotNull(testModule);
 
         final ModuleStatement moduleStatement = ((ModuleEffectiveStatement) testModule).getDeclared();
@@ -282,31 +281,31 @@ public class DeclaredStatementsTest {
         assertEquals("imdt", importStatement.getPrefix().argument());
         assertEquals(revision, importStatement.getRevisionDate().argument());
 
-        assertEquals("test description", moduleStatement.getDescription().get().argument());
-        assertEquals("test reference", moduleStatement.getReference().get().argument());
-        assertEquals("test organization", moduleStatement.getOrganization().get().argument());
-        assertEquals("test contact", moduleStatement.getContact().get().argument());
+        assertEquals("test description", moduleStatement.getDescription().orElseThrow().argument());
+        assertEquals("test reference", moduleStatement.getReference().orElseThrow().argument());
+        assertEquals("test organization", moduleStatement.getOrganization().orElseThrow().argument());
+        assertEquals("test contact", moduleStatement.getContact().orElseThrow().argument());
 
         assertEquals(1, moduleStatement.getRevisions().size());
         final RevisionStatement revisionStatement = moduleStatement.getRevisions().iterator().next();
         assertEquals(revision, revisionStatement.argument());
-        assertEquals("test description", revisionStatement.getDescription().get().argument());
-        assertEquals("test reference", revisionStatement.getReference().get().argument());
+        assertEquals("test description", revisionStatement.getDescription().orElseThrow().argument());
+        assertEquals("test reference", revisionStatement.getReference().orElseThrow().argument());
 
         assertEquals(1, moduleStatement.getExtensions().size());
         final ExtensionStatement extensionStatement = moduleStatement.getExtensions().iterator().next();
-        assertEquals(Status.CURRENT, extensionStatement.getStatus().get().argument());
-        assertEquals("test description", extensionStatement.getDescription().get().argument());
-        assertEquals("test reference", extensionStatement.getReference().get().argument());
+        assertEquals(Status.CURRENT, extensionStatement.getStatus().orElseThrow().argument());
+        assertEquals("test description", extensionStatement.getDescription().orElseThrow().argument());
+        assertEquals("test reference", extensionStatement.getReference().orElseThrow().argument());
         final ArgumentStatement argumentStatement = extensionStatement.getArgument();
         assertEquals("ext-argument", argumentStatement.argument().getLocalName());
         assertTrue(argumentStatement.getYinElement().argument());
 
         assertEquals(2, moduleStatement.getFeatures().size());
         final FeatureStatement featureStatement = moduleStatement.getFeatures().iterator().next();
-        assertEquals(Status.CURRENT, featureStatement.getStatus().get().argument());
-        assertEquals("test description", featureStatement.getDescription().get().argument());
-        assertEquals("test reference", featureStatement.getReference().get().argument());
+        assertEquals(Status.CURRENT, featureStatement.getStatus().orElseThrow().argument());
+        assertEquals("test description", featureStatement.getDescription().orElseThrow().argument());
+        assertEquals("test reference", featureStatement.getReference().orElseThrow().argument());
         assertEquals("test-feature", featureStatement.argument().getLocalName());
         assertEquals(1, featureStatement.getIfFeatures().size());
 
@@ -319,19 +318,19 @@ public class DeclaredStatementsTest {
         }
 
         assertEquals("test-base-id", identityStatement.getBases().iterator().next().argument().getLocalName());
-        assertEquals(Status.CURRENT, identityStatement.getStatus().get().argument());
-        assertEquals("test description", identityStatement.getDescription().get().argument());
-        assertEquals("test reference", identityStatement.getReference().get().argument());
+        assertEquals(Status.CURRENT, identityStatement.getStatus().orElseThrow().argument());
+        assertEquals("test description", identityStatement.getDescription().orElseThrow().argument());
+        assertEquals("test reference", identityStatement.getReference().orElseThrow().argument());
         assertEquals("test-id", identityStatement.argument().getLocalName());
 
         assertEquals(1, moduleStatement.getTypedefs().size());
         final TypedefStatement typedefStatement = moduleStatement.getTypedefs().iterator().next();
-        assertEquals(Status.CURRENT, typedefStatement.getStatus().get().argument());
-        assertEquals("test description", typedefStatement.getDescription().get().argument());
-        assertEquals("test reference", typedefStatement.getReference().get().argument());
+        assertEquals(Status.CURRENT, typedefStatement.getStatus().orElseThrow().argument());
+        assertEquals("test description", typedefStatement.getDescription().orElseThrow().argument());
+        assertEquals("test reference", typedefStatement.getReference().orElseThrow().argument());
         assertEquals("test-typedef", typedefStatement.argument().getLocalName());
         assertEquals("int32", typedefStatement.getType().rawArgument());
-        assertEquals("meter", typedefStatement.getUnits().get().argument());
+        assertEquals("meter", typedefStatement.getUnits().orElseThrow().argument());
     }
 
     @Test
@@ -354,7 +353,7 @@ public class DeclaredStatementsTest {
         final QName name = containerStatement.argument();
         assertNotNull(name);
 
-        final WhenStatement containerStatementWhen = containerStatement.getWhenStatement().get();
+        final WhenStatement containerStatementWhen = containerStatement.getWhenStatement().orElseThrow();
 
         final Collection<? extends IfFeatureStatement> containerStatementIfFeatures =
                 containerStatement.getIfFeatures();
index 648d4542f04c759ea76de58f6467b94b7d8a7725..e3ba4bcca0fea8048fe33b005ba2c4bae0cc7f14 100644 (file)
@@ -7,25 +7,28 @@
  */
 package org.opendaylight.yangtools.yang.stmt;
 
+import static org.hamcrest.CoreMatchers.instanceOf;
+import static org.hamcrest.CoreMatchers.startsWith;
+import static org.hamcrest.MatcherAssert.assertThat;
 import static org.junit.Assert.assertEquals;
 import static org.junit.Assert.assertFalse;
 import static org.junit.Assert.assertNotNull;
+import static org.junit.Assert.assertThrows;
 import static org.junit.Assert.assertTrue;
 import static org.opendaylight.yangtools.yang.stmt.StmtTestUtils.sourceForResource;
 
 import com.google.common.collect.Iterables;
 import java.net.URISyntaxException;
-import java.util.Collection;
 import org.junit.Test;
 import org.opendaylight.yangtools.yang.model.api.IdentitySchemaNode;
 import org.opendaylight.yangtools.yang.model.api.Module;
 import org.opendaylight.yangtools.yang.model.api.SchemaContext;
 import org.opendaylight.yangtools.yang.parser.rfc7950.reactor.RFC7950Reactors;
+import org.opendaylight.yangtools.yang.parser.spi.meta.InferenceException;
 import org.opendaylight.yangtools.yang.parser.spi.meta.ReactorException;
 import org.opendaylight.yangtools.yang.parser.spi.meta.SomeModifiersUnresolvedException;
 import org.opendaylight.yangtools.yang.parser.spi.source.SourceException;
 import org.opendaylight.yangtools.yang.parser.spi.source.StatementStreamSource;
-import org.opendaylight.yangtools.yang.parser.stmt.reactor.CrossSourceStatementReactor;
 
 public class EffectiveIdentityTest {
 
@@ -35,17 +38,17 @@ public class EffectiveIdentityTest {
     private static final StatementStreamSource CYCLIC_IDENTITY_TEST = sourceForResource(
             "/stmt-test/identity/cyclic-identity-test.yang");
 
-    @Test(expected = SomeModifiersUnresolvedException.class)
+    @Test
     public void cyclicefineTest() throws SourceException, ReactorException, URISyntaxException {
-
-        CrossSourceStatementReactor.BuildAction reactor = RFC7950Reactors.defaultReactor().newBuild()
-                .addSources(CYCLIC_IDENTITY_TEST);
-        try {
-            reactor.buildEffective();
-        } catch (SomeModifiersUnresolvedException e) {
-            StmtTestUtils.log(e, "      ");
-            throw e;
-        }
+        final var reactor = RFC7950Reactors.defaultReactor().newBuild().addSources(CYCLIC_IDENTITY_TEST);
+        final var cause = assertThrows(SomeModifiersUnresolvedException.class, reactor::buildEffective).getCause();
+        assertThat(cause, instanceOf(InferenceException.class));
+        assertThat(cause.getMessage(), startsWith("Yang model processing phase STATEMENT_DEFINITION failed [at "));
+
+        final var cause1 = cause.getCause();
+        assertThat(cause1, instanceOf(InferenceException.class));
+        assertThat(cause1.getMessage(), startsWith("Unable to resolve identity (cyclic.identity.test)child-identity-1 "
+            + "and base identity (cyclic.identity.test)child-identity-2 [at "));
     }
 
     @Test
@@ -54,8 +57,8 @@ public class EffectiveIdentityTest {
         SchemaContext result = RFC7950Reactors.defaultReactor().newBuild().addSources(IDENTITY_TEST).buildEffective();
         assertNotNull(result);
 
-        Module module = result.findModule("identity-test").get();
-        Collection<? extends IdentitySchemaNode> identities = module.getIdentities();
+        Module module = result.findModule("identity-test").orElseThrow();
+        var identities = module.getIdentities();
 
         assertNotNull(identities);
         assertEquals(4, identities.size());
@@ -90,7 +93,7 @@ public class EffectiveIdentityTest {
 
         assertTrue(root.getBaseIdentities().isEmpty());
 
-        Collection<? extends IdentitySchemaNode> rootDerivedIdentities = result.getDerivedIdentities(root);
+        var rootDerivedIdentities = result.getDerivedIdentities(root);
         assertEquals(2, rootDerivedIdentities.size());
 
         assertTrue(rootDerivedIdentities.contains(child1));
@@ -103,7 +106,7 @@ public class EffectiveIdentityTest {
 
         assertEquals(0, result.getDerivedIdentities(child2).size());
 
-        Collection<? extends IdentitySchemaNode> child1DerivedIdentities = result.getDerivedIdentities(child1);
+        var child1DerivedIdentities = result.getDerivedIdentities(child1);
         assertEquals(1, child1DerivedIdentities.size());
         assertTrue(child1DerivedIdentities.contains(child12));
         assertFalse(child1DerivedIdentities.contains(child1));
index 55c888b51a1a636357c29976ca6642a3aa2b826a..1b1361ea0a178d1c22effe495da1ba276f7eba37 100644 (file)
@@ -12,7 +12,6 @@ import static org.junit.Assert.assertNotNull;
 import static org.junit.Assert.assertTrue;
 import static org.opendaylight.yangtools.yang.stmt.StmtTestUtils.sourceForResource;
 
-import java.util.Collection;
 import org.junit.Test;
 import org.opendaylight.yangtools.yang.common.QName;
 import org.opendaylight.yangtools.yang.model.api.ExtensionDefinition;
@@ -36,7 +35,7 @@ public class ExtensionStmtTest {
 
         assertEquals(1, testModule.getExtensionSchemaNodes().size());
 
-        final Collection<? extends ExtensionDefinition> extensions = testModule.getExtensionSchemaNodes();
+        final var extensions = testModule.getExtensionSchemaNodes();
         final ExtensionDefinition extension = extensions.iterator().next();
         assertEquals("opendaylight", extension.getQName().getLocalName());
         assertEquals("name", extension.getArgument());
@@ -56,7 +55,7 @@ public class ExtensionStmtTest {
 
         assertEquals(1, testModule1.getExtensionSchemaNodes().size());
 
-        final Collection<? extends ExtensionDefinition> extensions = testModule1.getExtensionSchemaNodes();
+        final var extensions = testModule1.getExtensionSchemaNodes();
         final ExtensionDefinition extensionDefinition = extensions.iterator().next();
 
         final Module testModule2 = result.findModules("ext-use").iterator().next();
@@ -66,7 +65,7 @@ public class ExtensionStmtTest {
             QName.create(testModule2.getQNameModule(), "value"));
         assertNotNull(leaf);
 
-        final Collection<? extends UnrecognizedStatement> unknownNodes = leaf.asEffectiveStatement().getDeclared()
+        final var unknownNodes = leaf.asEffectiveStatement().getDeclared()
             .declaredSubstatements(UnrecognizedStatement.class);
         assertEquals(1, unknownNodes.size());
         final UnrecognizedStatement extensionUse = unknownNodes.iterator().next();
index 114783985faaf3cc642036a45992cd3bfa1655c2..18d02790a830f45075d78b385b6dcbd200c4fed2 100644 (file)
@@ -14,7 +14,6 @@ import static org.opendaylight.yangtools.yang.stmt.StmtTestUtils.sourceForResour
 import com.google.common.collect.ImmutableList;
 import java.util.ArrayList;
 import java.util.Collection;
-import java.util.Iterator;
 import java.util.Map;
 import org.junit.Test;
 import org.opendaylight.yangtools.yang.common.QName;
@@ -51,11 +50,10 @@ public class GroupingAndUsesStmtTest {
         final Module testModule = result.findModules("baz").iterator().next();
         assertNotNull(testModule);
 
-        final Collection<? extends GroupingDefinition> groupings = testModule.getGroupings();
+        final var groupings = testModule.getGroupings();
         assertEquals(1, groupings.size());
 
-        final Iterator<? extends GroupingDefinition> groupingsIterator = groupings.iterator();
-        final GroupingDefinition grouping = groupingsIterator.next();
+        final GroupingDefinition grouping = groupings.iterator().next();
         assertEquals("target", grouping.getQName().getLocalName());
         assertEquals(5, grouping.getChildNodes().size());
 
@@ -97,7 +95,7 @@ public class GroupingAndUsesStmtTest {
 
         final Module testModule = result.findModules("foo").iterator().next();
 
-        final Collection<? extends UsesNode> usesNodes = testModule.getUses();
+        final var usesNodes = testModule.getUses();
         assertEquals(1, usesNodes.size());
 
         UsesNode usesNode = usesNodes.iterator().next();
index b78807d75faad1d6da944e21e17f514d62f298dc..44a8c3e61ffe4d77ab2cc6e9d18fac4a024819bd 100644 (file)
@@ -15,7 +15,6 @@ import static org.junit.Assert.assertNull;
 import static org.junit.Assert.assertTrue;
 
 import com.google.common.collect.Iterables;
-import java.util.Collection;
 import java.util.Map;
 import java.util.Optional;
 import org.junit.Test;
@@ -50,7 +49,7 @@ public class GroupingTest extends AbstractModelTest {
         final ContainerSchemaNode peer = (ContainerSchemaNode) FOO.getDataChildByName(fooQName("peer"));
         final ContainerSchemaNode destination = (ContainerSchemaNode) peer.getDataChildByName(fooQName("destination"));
 
-        final Collection<? extends UsesNode> usesNodes = destination.getUses();
+        final var usesNodes = destination.getUses();
         assertEquals(1, usesNodes.size());
         final UsesNode usesNode = usesNodes.iterator().next();
         final Map<Descendant, SchemaNode> refines = usesNode.getRefines();
@@ -85,7 +84,7 @@ public class GroupingTest extends AbstractModelTest {
         assertEquals(Optional.of("address reference added by refine"), refineLeaf.getReference());
         assertEquals(Optional.of(Boolean.FALSE), refineLeaf.effectiveConfig());
         assertFalse(refineLeaf.isMandatory());
-        final Collection<? extends MustDefinition> leafMustConstraints = refineLeaf.getMustConstraints();
+        final var leafMustConstraints = refineLeaf.getMustConstraints();
         assertEquals(1, leafMustConstraints.size());
         final MustDefinition leafMust = leafMustConstraints.iterator().next();
         assertEquals("ifType != 'ethernet' or (ifType = 'ethernet' and ifMTU = 1500)", leafMust.getXpath().toString());
@@ -96,7 +95,7 @@ public class GroupingTest extends AbstractModelTest {
 
         // container port
         assertNotNull(refineContainer);
-        final Collection<? extends MustDefinition> mustConstraints = refineContainer.getMustConstraints();
+        final var mustConstraints = refineContainer.getMustConstraints();
         assertTrue(mustConstraints.isEmpty());
         assertEquals(Optional.of("description of port defined by refine"), refineContainer.getDescription());
         assertEquals(Optional.of("port reference added by refine"), refineContainer.getReference());
@@ -119,10 +118,10 @@ public class GroupingTest extends AbstractModelTest {
 
     @Test
     public void testGrouping() {
-        final Collection<? extends GroupingDefinition> groupings = BAZ.getGroupings();
+        final var groupings = BAZ.getGroupings();
         assertEquals(1, groupings.size());
         final GroupingDefinition grouping = groupings.iterator().next();
-        final Collection<? extends DataSchemaNode> children = grouping.getChildNodes();
+        final var children = grouping.getChildNodes();
         assertEquals(5, children.size());
     }
 
@@ -133,7 +132,7 @@ public class GroupingTest extends AbstractModelTest {
 
 
         // get grouping
-        final Collection<? extends GroupingDefinition> groupings = BAZ.getGroupings();
+        final var groupings = BAZ.getGroupings();
         assertEquals(1, groupings.size());
         final GroupingDefinition grouping = groupings.iterator().next();
 
@@ -142,7 +141,7 @@ public class GroupingTest extends AbstractModelTest {
         final ContainerSchemaNode destination = (ContainerSchemaNode) peer.getDataChildByName(fooQName("destination"));
 
         // check uses
-        final Collection<? extends UsesNode> uses = destination.getUses();
+        final var uses = destination.getUses();
         assertEquals(1, uses.size());
 
         // check uses process
@@ -194,11 +193,11 @@ public class GroupingTest extends AbstractModelTest {
         assertNotEquals(addresses_u, addresses_g);
 
         // grouping defined by 'uses'
-        final Collection<? extends GroupingDefinition> groupings_u = destination.getGroupings();
+        final var groupings_u = destination.getGroupings();
         assertEquals(0, groupings_u.size());
 
         // grouping defined in 'grouping' node
-        final Collection<? extends GroupingDefinition> groupings_g = grouping.getGroupings();
+        final var groupings_g = grouping.getGroupings();
         assertEquals(1, groupings_g.size());
         final GroupingDefinition grouping_g = groupings_g.iterator().next();
         assertIsAddedByUses(grouping_g, false);
@@ -214,12 +213,12 @@ public class GroupingTest extends AbstractModelTest {
         // suffix _g = defined in grouping
 
         // get grouping
-        final Collection<? extends GroupingDefinition> groupings = BAZ.getGroupings();
+        final var groupings = BAZ.getGroupings();
         assertEquals(1, groupings.size());
         final GroupingDefinition grouping = groupings.iterator().next();
 
         // check uses
-        final Collection<? extends UsesNode> uses = FOO.getUses();
+        final var uses = FOO.getUses();
         assertEquals(1, uses.size());
 
         // check uses process
@@ -233,7 +232,7 @@ public class GroupingTest extends AbstractModelTest {
         final ChoiceSchemaNode how_u = (ChoiceSchemaNode) FOO.getDataChildByName(fooQName("how"));
         assertIsAddedByUses(how_u, true);
         assertFalse(how_u.isAugmenting());
-        final Collection<? extends CaseSchemaNode> cases_u = how_u.getCases();
+        final var cases_u = how_u.getCases();
         assertEquals(2, cases_u.size());
         final CaseSchemaNode interval = how_u.findCaseNodes("interval").iterator().next();
         assertFalse(interval.isAugmenting());
@@ -276,11 +275,11 @@ public class GroupingTest extends AbstractModelTest {
         assertNotEquals(addresses_u, addresses_g);
 
         // grouping defined by 'uses'
-        final Collection<? extends GroupingDefinition> groupings_u = FOO.getGroupings();
+        final var groupings_u = FOO.getGroupings();
         assertEquals(0, groupings_u.size());
 
         // grouping defined in 'grouping' node
-        final Collection<? extends GroupingDefinition> groupings_g = grouping.getGroupings();
+        final var groupings_g = grouping.getGroupings();
         assertEquals(1, groupings_g.size());
         final GroupingDefinition grouping_g = groupings_g.iterator().next();
         assertIsAddedByUses(grouping_g, false);
@@ -290,11 +289,11 @@ public class GroupingTest extends AbstractModelTest {
             .size());
 
         final UsesNode un = uses.iterator().next();
-        final Collection<? extends AugmentationSchemaNode> usesAugments = un.getAugmentations();
+        final var usesAugments = un.getAugmentations();
         assertEquals(1, usesAugments.size());
         final AugmentationSchemaNode augment = usesAugments.iterator().next();
         assertEquals(Optional.of("inner augment"), augment.getDescription());
-        final Collection<? extends DataSchemaNode> children = augment.getChildNodes();
+        final var children = augment.getChildNodes();
         assertEquals(1, children.size());
         final DataSchemaNode leaf = children.iterator().next();
         assertTrue(leaf instanceof LeafSchemaNode);
@@ -308,7 +307,7 @@ public class GroupingTest extends AbstractModelTest {
 
         final Module testModule =  Iterables.getOnlyElement(loadModules.findModules("cascade-uses"));
         final QNameModule namespace = testModule.getQNameModule();
-        final Collection<? extends GroupingDefinition> groupings = testModule.getGroupings();
+        final var groupings = testModule.getGroupings();
 
         GroupingDefinition gu = null;
         GroupingDefinition gv = null;
@@ -352,7 +351,7 @@ public class GroupingTest extends AbstractModelTest {
             Revision.of("2013-07-18"));
 
         // grouping-U
-        Collection<? extends DataSchemaNode> childNodes = gu.getChildNodes();
+        var childNodes = gu.getChildNodes();
         assertEquals(7, childNodes.size());
 
         final LeafSchemaNode leafGroupingU = (LeafSchemaNode) gu.getDataChildByName(
index b8c52f1e19683003e7827a7b2799b17c2b035ece..9d1f1226aee8a3f2acd129de7ceff42b2c024800 100644 (file)
@@ -8,17 +8,15 @@
 package org.opendaylight.yangtools.yang.stmt;
 
 import static org.hamcrest.CoreMatchers.anyOf;
+import static org.hamcrest.CoreMatchers.instanceOf;
 import static org.hamcrest.CoreMatchers.is;
 import static org.hamcrest.CoreMatchers.startsWith;
 import static org.hamcrest.MatcherAssert.assertThat;
 import static org.junit.Assert.assertEquals;
 import static org.junit.Assert.assertNotNull;
-import static org.junit.Assert.fail;
+import static org.junit.Assert.assertThrows;
 import static org.opendaylight.yangtools.yang.stmt.StmtTestUtils.sourceForResource;
 
-import com.google.common.base.Throwables;
-import java.util.Collection;
-import java.util.Iterator;
 import org.junit.Test;
 import org.opendaylight.yangtools.yang.model.api.IdentitySchemaNode;
 import org.opendaylight.yangtools.yang.model.api.Module;
@@ -82,10 +80,10 @@ public class IdentityStmtTest {
         Module testModule = result.findModules("legal-chained-identity-test").iterator().next();
         assertNotNull(testModule);
 
-        Collection<? extends IdentitySchemaNode> identities = testModule.getIdentities();
+        var identities = testModule.getIdentities();
         assertEquals(4, identities.size());
 
-        Iterator<? extends IdentitySchemaNode> identitiesIterator = identities.iterator();
+        var identitiesIterator = identities.iterator();
         IdentitySchemaNode identity = identitiesIterator.next();
         assertThat(identity.getQName().getLocalName(), anyOf(is("first-identity"), is("second-identity"),
             is("third-identity"), is("fourth-identity")));
@@ -105,12 +103,9 @@ public class IdentityStmtTest {
 
     @Test
     public void duplicateIdentityTest() throws ReactorException {
-        try {
-            RFC7950Reactors.defaultReactor().newBuild().addSource(DUPLICATE_IDENTITY_MODULE).buildEffective();
-            fail("Duplicate identities should have been detected");
-        } catch (SomeModifiersUnresolvedException e) {
-            final SourceException cause = Throwables.getCauseAs(e, SourceException.class);
-            assertThat(cause.getMessage(), startsWith("Duplicate identity definition "));
-        }
+        final var reactor = RFC7950Reactors.defaultReactor().newBuild().addSource(DUPLICATE_IDENTITY_MODULE);
+        final var cause = assertThrows(SomeModifiersUnresolvedException.class, reactor::buildEffective).getCause();
+        assertThat(cause, instanceOf(SourceException.class));
+        assertThat(cause.getMessage(), startsWith("Duplicate identity definition "));
     }
 }
index f1b44c5cabea2704006430fbc6c078f5c9dda14f..f351ab7ba334e908cb25a583e766071ca7ad3aa2 100644 (file)
@@ -11,7 +11,6 @@ import static org.junit.Assert.assertArrayEquals;
 import static org.junit.Assert.assertEquals;
 import static org.junit.Assert.assertNotNull;
 
-import java.util.Collection;
 import org.junit.Test;
 import org.opendaylight.yangtools.yang.common.QName;
 import org.opendaylight.yangtools.yang.model.api.AugmentationSchemaNode;
@@ -24,7 +23,7 @@ import org.opendaylight.yangtools.yang.model.api.TypeDefinition;
 public class OrderingTest extends AbstractModelTest {
     @Test
     public void testOrderingTypedef() {
-        final Collection<? extends TypeDefinition<?>> typedefs = BAR.getTypeDefinitions();
+        final var typedefs = BAR.getTypeDefinitions();
         final String[] expectedOrder = { "int32-ext1", "int32-ext2", "string-ext1", "string-ext2", "string-ext3",
             "string-ext4", "multiple-pattern-string", "my-decimal-type", "my-union", "my-union-ext", "nested-union2"
         };
@@ -63,7 +62,7 @@ public class OrderingTest extends AbstractModelTest {
 
     @Test
     public void testOrderingNestedChildNodes1() {
-        final Collection<? extends DataSchemaNode> childNodes = FOO.getChildNodes();
+        final var childNodes = FOO.getChildNodes();
         final String[] expectedOrder = { "int32-leaf", "string-leaf", "multiple-pattern-string-leaf",
             "multiple-pattern-direct-string-def-leaf", "length-leaf", "decimal-leaf", "decimal-leaf2", "ext",
             "union-leaf", "custom-union-leaf", "transfer", "datas", "mycont", "data", "how", "address", "port",
@@ -81,11 +80,11 @@ public class OrderingTest extends AbstractModelTest {
 
     @Test
     public void testOrderingNestedChildNodes2() {
-        final Collection<? extends GroupingDefinition> groupings = BAZ.getGroupings();
+        final var groupings = BAZ.getGroupings();
         assertEquals(1, groupings.size());
         final GroupingDefinition target = groupings.iterator().next();
 
-        final Collection<? extends DataSchemaNode> childNodes = target.getChildNodes();
+        final var childNodes = target.getChildNodes();
         final String[] expectedOrder = { "data", "how", "address", "port", "addresses" };
         final String[] actualOrder = new String[childNodes.size()];
 
@@ -102,7 +101,7 @@ public class OrderingTest extends AbstractModelTest {
         final Module justFoo = assertEffectiveModel("/ordering/foo.yang").getModules().iterator().next();
         final ContainerSchemaNode x = (ContainerSchemaNode) justFoo
                 .getDataChildByName(QName.create(justFoo.getQNameModule(), "x"));
-        final Collection<? extends DataSchemaNode> childNodes = x.getChildNodes();
+        final var childNodes = x.getChildNodes();
 
         final String[] expectedOrder = { "x15", "x10", "x5", "x1", "a5", "a1", "x2", "b5", "b1", "x3", "ax15", "ax5" };
         final String[] actualOrder = new String[childNodes.size()];
index 86c91d030c3a4d506d7906c090435f4487161aeb..3cc6f0a7d623ac894b7f424c22ae4274986e2226 100644 (file)
@@ -22,10 +22,8 @@ import com.google.common.collect.RangeSet;
 import java.text.ParseException;
 import java.util.ArrayList;
 import java.util.Collection;
-import java.util.Iterator;
 import java.util.List;
 import java.util.Optional;
-import java.util.Set;
 import org.junit.Test;
 import org.opendaylight.yangtools.yang.common.QName;
 import org.opendaylight.yangtools.yang.common.Revision;
@@ -41,7 +39,6 @@ import org.opendaylight.yangtools.yang.model.api.DeviateKind;
 import org.opendaylight.yangtools.yang.model.api.Deviation;
 import org.opendaylight.yangtools.yang.model.api.ElementCountConstraint;
 import org.opendaylight.yangtools.yang.model.api.ExtensionDefinition;
-import org.opendaylight.yangtools.yang.model.api.FeatureDefinition;
 import org.opendaylight.yangtools.yang.model.api.LeafSchemaNode;
 import org.opendaylight.yangtools.yang.model.api.ListSchemaNode;
 import org.opendaylight.yangtools.yang.model.api.ModuleImport;
@@ -55,7 +52,6 @@ import org.opendaylight.yangtools.yang.model.api.type.DecimalTypeDefinition;
 import org.opendaylight.yangtools.yang.model.api.type.Int16TypeDefinition;
 import org.opendaylight.yangtools.yang.model.api.type.Int32TypeDefinition;
 import org.opendaylight.yangtools.yang.model.api.type.PatternConstraint;
-import org.opendaylight.yangtools.yang.model.api.type.RangeConstraint;
 import org.opendaylight.yangtools.yang.model.api.type.StringTypeDefinition;
 import org.opendaylight.yangtools.yang.model.api.type.TypeDefinitions;
 import org.opendaylight.yangtools.yang.model.api.type.Uint32TypeDefinition;
@@ -107,7 +103,7 @@ public class YangParserTest extends AbstractModelTest {
         // ifEntry should be a context node ?
         // assertNull(constraints.getWhenCondition());
         assertEquals(0, ifEntry.getMustConstraints().size());
-        ElementCountConstraint constraints = ifEntry.getElementCountConstraint().get();
+        ElementCountConstraint constraints = ifEntry.getElementCountConstraint().orElseThrow();
         assertEquals((Object) 1, constraints.getMinElements());
         assertEquals((Object) 11, constraints.getMaxElements());
         // test AugmentationTarget args
@@ -141,11 +137,10 @@ public class YangParserTest extends AbstractModelTest {
         assertEquals(Optional.of("mile"), leafType.getUnits());
         assertEquals(Optional.of("11"), leafType.getDefaultValue());
 
-        final RangeSet<? extends Number> rangeset = leafType.getRangeConstraint().get().getAllowedRanges();
-        final Set<? extends Range<? extends Number>> ranges = rangeset.asRanges();
+        final var ranges = leafType.getRangeConstraint().orElseThrow().getAllowedRanges().asRanges();
         assertEquals(1, ranges.size());
 
-        final Range<? extends Number> range = ranges.iterator().next();
+        final var range = ranges.iterator().next();
         assertEquals(12, range.lowerEndpoint().intValue());
         assertEquals(20, range.upperEndpoint().intValue());
 
@@ -154,15 +149,14 @@ public class YangParserTest extends AbstractModelTest {
         assertEquals(Optional.of("mile"), firstBaseType.getUnits());
         assertEquals(Optional.of("11"), firstBaseType.getDefaultValue());
 
-        final RangeSet<? extends Number> firstRangeset = firstBaseType.getRangeConstraint().get().getAllowedRanges();
-        final Set<? extends Range<? extends Number>> baseRanges = firstRangeset.asRanges();
+        final var baseRanges = firstBaseType.getRangeConstraint().orElseThrow().getAllowedRanges().asRanges();
         assertEquals(2, baseRanges.size());
 
-        final Iterator<? extends Range<? extends Number>> it = baseRanges.iterator();
-        final Range<? extends Number> baseTypeRange1 = it.next();
+        final var it = baseRanges.iterator();
+        final var baseTypeRange1 = it.next();
         assertEquals(3, baseTypeRange1.lowerEndpoint().intValue());
         assertEquals(9, baseTypeRange1.upperEndpoint().intValue());
-        final Range<? extends Number> baseTypeRange2 = it.next();
+        final var baseTypeRange2 = it.next();
         assertEquals(11, baseTypeRange2.lowerEndpoint().intValue());
         assertEquals(20, baseTypeRange2.upperEndpoint().intValue());
 
@@ -171,10 +165,9 @@ public class YangParserTest extends AbstractModelTest {
         assertEquals(Optional.empty(), secondBaseType.getUnits());
         assertEquals(Optional.empty(), secondBaseType.getDefaultValue());
 
-        final Set<? extends Range<? extends Number>> secondRanges = secondBaseType.getRangeConstraint().get()
-                .getAllowedRanges().asRanges();
+        final var secondRanges = secondBaseType.getRangeConstraint().orElseThrow().getAllowedRanges().asRanges();
         assertEquals(1, secondRanges.size());
-        final Range<? extends Number> secondRange = secondRanges.iterator().next();
+        final var secondRange = secondRanges.iterator().next();
         assertEquals(2, secondRange.lowerEndpoint().intValue());
         assertEquals(20, secondRange.upperEndpoint().intValue());
 
@@ -194,7 +187,7 @@ public class YangParserTest extends AbstractModelTest {
         assertEquals(1, patterns.size());
         PatternConstraint pattern = patterns.iterator().next();
         assertEquals("^(?:[e-z]*)$", pattern.getJavaPatternString());
-        assertEquals(1, type.getLengthConstraint().get().getAllowedRanges().asRanges().size());
+        assertEquals(1, type.getLengthConstraint().orElseThrow().getAllowedRanges().asRanges().size());
 
         final StringTypeDefinition baseType1 = type.getBaseType();
         assertEquals(barQName("string-ext3"), baseType1.getQName());
@@ -204,14 +197,14 @@ public class YangParserTest extends AbstractModelTest {
         assertEquals(1, patterns.size());
         pattern = patterns.iterator().next();
         assertEquals("^(?:[b-u]*)$", pattern.getJavaPatternString());
-        assertEquals(1, baseType1.getLengthConstraint().get().getAllowedRanges().asRanges().size());
+        assertEquals(1, baseType1.getLengthConstraint().orElseThrow().getAllowedRanges().asRanges().size());
 
         final StringTypeDefinition baseType2 = baseType1.getBaseType();
         assertEquals(barQName("string-ext2"), baseType2.getQName());
         assertEquals(Optional.empty(), baseType2.getUnits());
         assertEquals(Optional.empty(), baseType2.getDefaultValue());
         assertTrue(baseType2.getPatternConstraints().isEmpty());
-        final RangeSet<Integer> baseType2Lengths = baseType2.getLengthConstraint().get().getAllowedRanges();
+        final RangeSet<Integer> baseType2Lengths = baseType2.getLengthConstraint().orElseThrow().getAllowedRanges();
         assertEquals(1, baseType2Lengths.asRanges().size());
         Range<Integer> length = baseType2Lengths.span();
         assertEquals(6, length.lowerEndpoint().intValue());
@@ -225,7 +218,7 @@ public class YangParserTest extends AbstractModelTest {
         assertEquals(1, patterns.size());
         pattern = patterns.iterator().next();
         assertEquals("^(?:[a-k]*)$", pattern.getJavaPatternString());
-        final RangeSet<Integer> baseType3Lengths = baseType3.getLengthConstraint().get().getAllowedRanges();
+        final RangeSet<Integer> baseType3Lengths = baseType3.getLengthConstraint().orElseThrow().getAllowedRanges();
         assertEquals(1, baseType3Lengths.asRanges().size());
         length = baseType3Lengths.span();
         assertEquals(5, length.lowerEndpoint().intValue());
@@ -246,7 +239,7 @@ public class YangParserTest extends AbstractModelTest {
         assertEquals(2, patterns.size());
         assertEquals("^(?:[A-Z]*-%22!\\^\\^)$", patterns.get(0).getJavaPatternString());
         assertEquals("^(?:[e-z]*)$", patterns.get(1).getJavaPatternString());
-        assertEquals(1, type.getLengthConstraint().get().getAllowedRanges().asRanges().size());
+        assertEquals(1, type.getLengthConstraint().orElseThrow().getAllowedRanges().asRanges().size());
 
         final LeafSchemaNode multiplePatternDirectStringDefLeaf = (LeafSchemaNode) FOO.getDataChildByName(
             fooQName("multiple-pattern-direct-string-def-leaf"));
@@ -271,7 +264,7 @@ public class YangParserTest extends AbstractModelTest {
         assertEquals(Optional.empty(), type.getUnits());
         assertEquals(Optional.empty(), type.getDefaultValue());
         assertTrue(type.getPatternConstraints().isEmpty());
-        final RangeSet<Integer> typeLengths = type.getLengthConstraint().get().getAllowedRanges();
+        final RangeSet<Integer> typeLengths = type.getLengthConstraint().orElseThrow().getAllowedRanges();
         assertEquals(1, typeLengths.asRanges().size());
         Range<Integer> length = typeLengths.span();
         assertEquals(7, length.lowerEndpoint().intValue());
@@ -282,7 +275,7 @@ public class YangParserTest extends AbstractModelTest {
         assertEquals(Optional.empty(), baseType1.getUnits());
         assertEquals(Optional.empty(), baseType1.getDefaultValue());
         assertTrue(baseType1.getPatternConstraints().isEmpty());
-        final RangeSet<Integer> baseType2Lengths = baseType1.getLengthConstraint().get().getAllowedRanges();
+        final RangeSet<Integer> baseType2Lengths = baseType1.getLengthConstraint().orElseThrow().getAllowedRanges();
         assertEquals(1, baseType2Lengths.asRanges().size());
         length = baseType2Lengths.span();
         assertEquals(6, length.lowerEndpoint().intValue());
@@ -296,7 +289,7 @@ public class YangParserTest extends AbstractModelTest {
         assertEquals(1, patterns.size());
         final PatternConstraint pattern = patterns.iterator().next();
         assertEquals("^(?:[a-k]*)$", pattern.getJavaPatternString());
-        final RangeSet<Integer> baseType3Lengths = baseType2.getLengthConstraint().get().getAllowedRanges();
+        final RangeSet<Integer> baseType3Lengths = baseType2.getLengthConstraint().orElseThrow().getAllowedRanges();
         assertEquals(1, baseType3Lengths.asRanges().size());
         length = baseType3Lengths.span();
         assertEquals(5, length.lowerEndpoint().intValue());
@@ -315,14 +308,14 @@ public class YangParserTest extends AbstractModelTest {
         assertEquals(Optional.empty(), type.getUnits());
         assertEquals(Optional.empty(), type.getDefaultValue());
         assertEquals(6, type.getFractionDigits());
-        assertEquals(1, type.getRangeConstraint().get().getAllowedRanges().asRanges().size());
+        assertEquals(1, type.getRangeConstraint().orElseThrow().getAllowedRanges().asRanges().size());
 
         final DecimalTypeDefinition typeBase = type.getBaseType();
         assertEquals(barQName("decimal64"), typeBase.getQName());
         assertEquals(Optional.empty(), typeBase.getUnits());
         assertEquals(Optional.empty(), typeBase.getDefaultValue());
         assertEquals(6, typeBase.getFractionDigits());
-        assertEquals(1, typeBase.getRangeConstraint().get().getAllowedRanges().asRanges().size());
+        assertEquals(1, typeBase.getRangeConstraint().orElseThrow().getAllowedRanges().asRanges().size());
 
         assertNull(typeBase.getBaseType());
     }
@@ -337,7 +330,7 @@ public class YangParserTest extends AbstractModelTest {
         assertEquals(Optional.empty(), type.getUnits());
         assertEquals(Optional.empty(), type.getDefaultValue());
         assertEquals(6, type.getFractionDigits());
-        assertEquals(1, type.getRangeConstraint().get().getAllowedRanges().asRanges().size());
+        assertEquals(1, type.getRangeConstraint().orElseThrow().getAllowedRanges().asRanges().size());
 
         final DecimalTypeDefinition baseTypeDecimal = type.getBaseType();
         assertEquals(6, baseTypeDecimal.getFractionDigits());
@@ -367,11 +360,11 @@ public class YangParserTest extends AbstractModelTest {
         assertEquals(Optional.empty(), unionType1.getUnits());
         assertEquals(Optional.empty(), unionType1.getDefaultValue());
 
-        final RangeConstraint<?> ranges = unionType1.getRangeConstraint().get();
+        final var ranges = unionType1.getRangeConstraint().orElseThrow();
         assertEquals(1, ranges.getAllowedRanges().asRanges().size());
-        final Range<?> range = ranges.getAllowedRanges().span();
-        assertEquals((short)1, range.lowerEndpoint());
-        assertEquals((short)100, range.upperEndpoint());
+        final var range = ranges.getAllowedRanges().span();
+        assertEquals(Short.valueOf("1"), range.lowerEndpoint());
+        assertEquals(Short.valueOf("100"), range.upperEndpoint());
         assertEquals(BaseTypes.int16Type(), unionType1.getBaseType());
 
         assertEquals(BaseTypes.int32Type(), unionTypes.get(1));
@@ -430,10 +423,9 @@ public class YangParserTest extends AbstractModelTest {
         assertEquals(TypeDefinitions.INT16, int16Ext.getQName());
         assertEquals(Optional.empty(), int16Ext.getUnits());
         assertEquals(Optional.empty(), int16Ext.getDefaultValue());
-        final Set<? extends Range<? extends Number>> ranges = int16Ext.getRangeConstraint().get().getAllowedRanges()
-                .asRanges();
+        final var ranges = int16Ext.getRangeConstraint().orElseThrow().getAllowedRanges().asRanges();
         assertEquals(1, ranges.size());
-        final Range<? extends Number> range = ranges.iterator().next();
+        final var range = ranges.iterator().next();
         assertEquals(1, range.lowerEndpoint().intValue());
         assertEquals(100, range.upperEndpoint().intValue());
 
@@ -444,7 +436,7 @@ public class YangParserTest extends AbstractModelTest {
     public void testChoice() {
         final ContainerSchemaNode transfer = (ContainerSchemaNode) FOO.getDataChildByName(fooQName("transfer"));
         final ChoiceSchemaNode how = (ChoiceSchemaNode) transfer.getDataChildByName(fooQName("how"));
-        final Collection<? extends CaseSchemaNode> cases = how.getCases();
+        final var cases = how.getCases();
         assertEquals(5, cases.size());
         CaseSchemaNode input = null;
         CaseSchemaNode output = null;
@@ -461,7 +453,7 @@ public class YangParserTest extends AbstractModelTest {
 
     @Test
     public void testDeviation() {
-        final Collection<? extends Deviation> deviations = FOO.getDeviations();
+        final var deviations = FOO.getDeviations();
         assertEquals(1, deviations.size());
         final Deviation dev = deviations.iterator().next();
         assertEquals(Optional.of("system/user ref"), dev.getReference());
@@ -473,7 +465,7 @@ public class YangParserTest extends AbstractModelTest {
     @Test
     public void testUnknownNode() {
         final ContainerSchemaNode network = (ContainerSchemaNode) BAZ.getDataChildByName(bazQName("network"));
-        final Collection<? extends UnrecognizedStatement> unknownNodes = network.asEffectiveStatement().getDeclared()
+        final var unknownNodes = network.asEffectiveStatement().getDeclared()
             .declaredSubstatements(UnrecognizedStatement.class);
         assertEquals(1, unknownNodes.size());
         assertEquals("point", unknownNodes.iterator().next().argument());
@@ -481,13 +473,13 @@ public class YangParserTest extends AbstractModelTest {
 
     @Test
     public void testFeature() {
-        final Collection<? extends FeatureDefinition> features = BAZ.getFeatures();
+        final var features = BAZ.getFeatures();
         assertEquals(3, features.size());
     }
 
     @Test
     public void testExtension() {
-        final Collection<? extends ExtensionDefinition> extensions = BAZ.getExtensionSchemaNodes();
+        final var extensions = BAZ.getExtensionSchemaNodes();
         assertEquals(1, extensions.size());
         final ExtensionDefinition extension = extensions.iterator().next();
         assertEquals("name", extension.getArgument());
@@ -499,7 +491,7 @@ public class YangParserTest extends AbstractModelTest {
 
     @Test
     public void testNotification() {
-        final Collection<? extends NotificationDefinition> notifications = BAZ.getNotifications();
+        final var notifications = BAZ.getNotifications();
         assertEquals(1, notifications.size());
 
         final NotificationDefinition notification = notifications.iterator().next();
@@ -523,7 +515,7 @@ public class YangParserTest extends AbstractModelTest {
 
     @Test
     public void testRpc() {
-        final Collection<? extends RpcDefinition> rpcs = BAZ.getRpcs();
+        final var rpcs = BAZ.getRpcs();
         assertEquals(1, rpcs.size());
 
         final RpcDefinition rpc = rpcs.iterator().next();
@@ -545,7 +537,7 @@ public class YangParserTest extends AbstractModelTest {
 
     @Test
     public void testTypePath2() throws ParseException {
-        final Collection<? extends TypeDefinition<?>> types = BAR.getTypeDefinitions();
+        final var types = BAR.getTypeDefinitions();
 
         // my-decimal-type
         final DecimalTypeDefinition myDecType = (DecimalTypeDefinition) TestUtils.findTypedef(types, "my-decimal-type");
index 74f25d65a2bf5d18f8363cb72e1ce1b616649af1..eeed9778f959a206e95820d8bc3d95920241c0b8 100644 (file)
@@ -15,7 +15,6 @@ import static org.opendaylight.yangtools.yang.stmt.StmtTestUtils.sourceForResour
 
 import com.google.common.collect.Iterables;
 import com.google.common.collect.Range;
-import java.util.Collection;
 import java.util.Map;
 import java.util.Optional;
 import java.util.Set;
@@ -113,7 +112,7 @@ public class YangParserWithContextTest {
         final Module testModule = context.findModule("test2", Revision.of("2013-06-18")).get();
         final Module contextModule = context.findModules(XMLNamespace.of("urn:opendaylight.baz")).iterator().next();
         assertNotNull(contextModule);
-        final Collection<? extends GroupingDefinition> groupings = contextModule.getGroupings();
+        final var groupings = contextModule.getGroupings();
         assertEquals(1, groupings.size());
         final GroupingDefinition grouping = groupings.iterator().next();
 
@@ -124,7 +123,7 @@ public class YangParserWithContextTest {
                 testModule.getQNameModule(), "destination"));
 
         // check uses
-        final Collection<? extends UsesNode> uses = destination.getUses();
+        final var uses = destination.getUses();
         assertEquals(1, uses.size());
 
         // check uses process
@@ -184,11 +183,11 @@ public class YangParserWithContextTest {
         assertFalse(addresses_u.equals(addresses_g));
 
         // grouping defined by 'uses'
-        final Collection<? extends GroupingDefinition> groupings_u = destination.getGroupings();
+        final var groupings_u = destination.getGroupings();
         assertEquals(0, groupings_u.size());
 
         // grouping defined in 'grouping' node
-        final Collection<? extends GroupingDefinition> groupings_g = grouping.getGroupings();
+        final var groupings_g = grouping.getGroupings();
         assertEquals(1, groupings_g.size());
         final GroupingDefinition grouping_g = groupings_g.iterator().next();
         assertFalse(grouping_g.isAddedByUses());
@@ -205,7 +204,7 @@ public class YangParserWithContextTest {
                 module.getQNameModule(), "peer"));
         final ContainerSchemaNode destination = (ContainerSchemaNode) peer.getDataChildByName(QName.create(
                 module.getQNameModule(), "destination"));
-        final Collection<? extends UsesNode> usesNodes = destination.getUses();
+        final var usesNodes = destination.getUses();
         assertEquals(1, usesNodes.size());
         final UsesNode usesNode = usesNodes.iterator().next();
 
@@ -238,14 +237,14 @@ public class YangParserWithContextTest {
         assertEquals(Optional.of("address reference added by refine"), refineLeaf.getReference());
         assertEquals(Optional.of(Boolean.FALSE), refineLeaf.effectiveConfig());
         assertTrue(refineLeaf.isMandatory());
-        final Collection<? extends MustDefinition> leafMustConstraints = refineLeaf.getMustConstraints();
+        final var leafMustConstraints = refineLeaf.getMustConstraints();
         assertEquals(1, leafMustConstraints.size());
         final MustDefinition leafMust = leafMustConstraints.iterator().next();
         assertEquals("ifType != 'ethernet' or (ifType = 'ethernet' and ifMTU = 1500)", leafMust.getXpath().toString());
 
         // container port
         assertNotNull(refineContainer);
-        final Collection<? extends MustDefinition> mustConstraints = refineContainer.getMustConstraints();
+        final var mustConstraints = refineContainer.getMustConstraints();
         assertTrue(mustConstraints.isEmpty());
         assertEquals(Optional.of("description of port defined by refine"), refineContainer.getDescription());
         assertEquals(Optional.of("port reference added by refine"), refineContainer.getReference());
@@ -271,7 +270,7 @@ public class YangParserWithContextTest {
                 .buildEffective();
 
         final Module module = context.findModule("test3", Revision.of("2013-06-18")).get();
-        final Collection<? extends IdentitySchemaNode> identities = module.getIdentities();
+        final var identities = module.getIdentities();
         assertEquals(1, identities.size());
 
         final IdentitySchemaNode identity = identities.iterator().next();
@@ -298,8 +297,7 @@ public class YangParserWithContextTest {
         final Module module = context.findModule("test3", Revision.of("2013-06-18")).get();
         final ContainerStatement network = ((ContainerSchemaNode) module.getDataChildByName(
             QName.create(module.getQNameModule(), "network"))).asEffectiveStatement().getDeclared();
-        final Collection<? extends UnrecognizedStatement> unknownNodes =
-            network.declaredSubstatements(UnrecognizedStatement.class);
+        final var unknownNodes = network.declaredSubstatements(UnrecognizedStatement.class);
         assertEquals(1, unknownNodes.size());
 
         final UnrecognizedStatement un = unknownNodes.iterator().next();
@@ -351,7 +349,7 @@ public class YangParserWithContextTest {
                 .buildEffective();
 
         final Module testModule = context.findModule("deviation-test", Revision.of("2013-02-27")).get();
-        final Collection<? extends Deviation> deviations = testModule.getDeviations();
+        final var deviations = testModule.getDeviations();
         assertEquals(1, deviations.size());
         final Deviation dev = deviations.iterator().next();
 
index bdd86e51ad814708321f7b6b9fc366af38279705..f423e15b00d5b81c2ff7ccc1edb20bf2013d1661 100644 (file)
@@ -12,7 +12,6 @@ import static org.junit.Assert.assertFalse;
 import static org.junit.Assert.assertNotNull;
 import static org.junit.Assert.assertTrue;
 
-import java.util.Iterator;
 import java.util.Optional;
 import org.junit.Test;
 import org.opendaylight.yangtools.yang.common.QName;
@@ -40,7 +39,7 @@ public class YinFileChoiceStmtTest extends AbstractYinModulesTest {
         assertEquals(1, choice.getCases().size());
 
         // this choice is augmented (see main-impl.yang.xml)
-        final Iterator<? extends CaseSchemaNode> casesIterator = choice.getCases().iterator();
+        final var casesIterator = choice.getCases().iterator();
         final CaseSchemaNode caseNode = casesIterator.next();
         assertEquals("main-impl", caseNode.getQName().getLocalName());
         assertEquals(13, caseNode.getChildNodes().size());
index 5f6f06cf867ec42c9963ef79e5702048f479483b..57a5570e8e0a4694d7da8ed868ac27e634e71dd8 100644 (file)
@@ -11,7 +11,6 @@ import static org.junit.Assert.assertEquals;
 import static org.junit.Assert.assertNotNull;
 import static org.junit.Assert.assertNull;
 
-import java.util.Collection;
 import java.util.Iterator;
 import java.util.Optional;
 import org.junit.Test;
@@ -25,7 +24,7 @@ public class YinFileExtensionStmtTest extends AbstractYinModulesTest {
         Module testModule = context.findModules("config").iterator().next();
         assertNotNull(testModule);
 
-        Collection<? extends ExtensionDefinition> extensions = testModule.getExtensionSchemaNodes();
+        var extensions = testModule.getExtensionSchemaNodes();
         assertEquals(5, extensions.size());
 
         Iterator<? extends ExtensionDefinition> extIterator = extensions.iterator();