Do not use SchemaAwareBuilders in CreateStreamUtilTest 03/101503/1
authorRobert Varga <robert.varga@pantheon.tech>
Wed, 8 Jun 2022 14:21:14 +0000 (16:21 +0200)
committerRobert Varga <robert.varga@pantheon.tech>
Wed, 8 Jun 2022 14:21:14 +0000 (16:21 +0200)
The use of builders is clunky at best. Just use plain builders.

Change-Id: Id198e4a52ff54acf3296537d18e6768c5b6f54c9
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>
restconf/restconf-nb-rfc8040/src/test/java/org/opendaylight/restconf/nb/rfc8040/rests/services/impl/CreateStreamUtilTest.java

index dc04c100b6f251792a41264416a1ec427d269488..b5bc8c7ae89d0c7dd4cb1ed92bdf5eabdbd641e1 100644 (file)
@@ -28,11 +28,10 @@ import org.opendaylight.yangtools.yang.common.ErrorTag;
 import org.opendaylight.yangtools.yang.common.ErrorType;
 import org.opendaylight.yangtools.yang.common.QName;
 import org.opendaylight.yangtools.yang.data.api.YangInstanceIdentifier;
-import org.opendaylight.yangtools.yang.data.api.schema.ContainerNode;
-import org.opendaylight.yangtools.yang.data.api.schema.LeafNode;
+import org.opendaylight.yangtools.yang.data.api.YangInstanceIdentifier.NodeIdentifier;
 import org.opendaylight.yangtools.yang.data.api.schema.NormalizedNode;
-import org.opendaylight.yangtools.yang.data.api.schema.builder.DataContainerNodeBuilder;
-import org.opendaylight.yangtools.yang.data.impl.schema.SchemaAwareBuilders;
+import org.opendaylight.yangtools.yang.data.impl.schema.Builders;
+import org.opendaylight.yangtools.yang.data.impl.schema.ImmutableNodes;
 import org.opendaylight.yangtools.yang.model.api.ContainerLike;
 import org.opendaylight.yangtools.yang.model.api.DataSchemaNode;
 import org.opendaylight.yangtools.yang.model.api.EffectiveModelContext;
@@ -97,36 +96,33 @@ public class CreateStreamUtilTest {
             final String inputOutputName) {
         final Module rpcModule = SCHEMA_CTX.findModules("sal-remote").iterator().next();
         final QName rpcQName = QName.create(rpcModule.getQNameModule(), rpcName);
-        ContainerLike rpcInputSchemaNode = null;
+
+        ContainerLike containerSchemaNode = null;
         for (final RpcDefinition rpc : rpcModule.getRpcs()) {
             if (rpcQName.isEqualWithoutRevision(rpc.getQName())) {
-                rpcInputSchemaNode = rpcToContainer.apply(rpc);
+                containerSchemaNode = rpcToContainer.apply(rpc);
                 break;
             }
         }
-        assertNotNull(rpcInputSchemaNode);
-
-        final DataContainerNodeBuilder<YangInstanceIdentifier.NodeIdentifier, ContainerNode> container =
-            SchemaAwareBuilders.containerBuilder(rpcInputSchemaNode);
+        assertNotNull(containerSchemaNode);
 
         final QName lfQName = QName.create(rpcModule.getQNameModule(), inputOutputName);
-        final DataSchemaNode lfSchemaNode = rpcInputSchemaNode.findDataChildByName(lfQName).orElseThrow();
-
+        final DataSchemaNode lfSchemaNode = containerSchemaNode.getDataChildByName(lfQName);
         assertThat(lfSchemaNode, instanceOf(LeafSchemaNode.class));
 
         final Object o;
         if ("toaster".equals(toasterValue)) {
             final QName rpcQname = QName.create("http://netconfcentral.org/ns/toaster", "2009-11-20", toasterValue);
-            o = YangInstanceIdentifier.builder().node(rpcQname).build();
+            o = YangInstanceIdentifier.of(rpcQname);
         } else {
             o = toasterValue;
         }
-        final LeafNode<Object> lfNode = SchemaAwareBuilders.leafBuilder((LeafSchemaNode) lfSchemaNode)
-                .withValue(o).build();
-        container.withChild(lfNode);
 
         return NormalizedNodePayload.of(InstanceIdentifierContext.ofStack(
-            SchemaInferenceStack.of(SCHEMA_CTX, Absolute.of(rpcQName, rpcInputSchemaNode.getQName()))),
-            container.build());
+            SchemaInferenceStack.of(SCHEMA_CTX, Absolute.of(rpcQName, containerSchemaNode.getQName()))),
+            Builders.containerBuilder()
+                .withNodeIdentifier(new NodeIdentifier(containerSchemaNode.getQName()))
+                .withChild(ImmutableNodes.leafNode(lfQName, o))
+                .build());
     }
 }