Improve XmlCodecFactory a bit 75/104875/1
authorRobert Varga <robert.varga@pantheon.tech>
Tue, 14 Mar 2023 12:37:00 +0000 (13:37 +0100)
committerRobert Varga <robert.varga@pantheon.tech>
Tue, 14 Mar 2023 12:38:00 +0000 (13:38 +0100)
Remove an unneeded cast in test code by exposing the fact that
instanceIdentifierCodec() operates on YangInstanceIdentifier.

JIRA: YANGTOOLS-1473
Change-Id: Ib915a0b4c93674e1e266359070bb6b06da20094c
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>
codec/yang-data-codec-xml/src/main/java/org/opendaylight/yangtools/yang/data/codec/xml/XmlCodecFactory.java
codec/yang-data-codec-xml/src/test/java/org/opendaylight/yangtools/yang/data/codec/xml/YT1473Test.java

index 3a9f81d4155241f92fb8e1ba8e91b96d4e267dd9..95a9262623b5ddfc024d400923c4966edb237a6b 100644 (file)
@@ -15,6 +15,7 @@ import java.util.List;
 import org.opendaylight.yangtools.rfc8528.data.api.MountPointContext;
 import org.opendaylight.yangtools.rfc8528.data.util.EmptyMountPointContext;
 import org.opendaylight.yangtools.yang.common.QNameModule;
+import org.opendaylight.yangtools.yang.data.api.YangInstanceIdentifier;
 import org.opendaylight.yangtools.yang.data.impl.codec.AbstractIntegerStringCodec;
 import org.opendaylight.yangtools.yang.data.impl.codec.BinaryStringCodec;
 import org.opendaylight.yangtools.yang.data.impl.codec.BitsStringCodec;
@@ -112,7 +113,7 @@ public final class XmlCodecFactory extends AbstractCodecFactory<XmlCodec<?>> {
     }
 
     @Override
-    protected XmlCodec<?> instanceIdentifierCodec(final InstanceIdentifierTypeDefinition type) {
+    protected XmlCodec<YangInstanceIdentifier> instanceIdentifierCodec(final InstanceIdentifierTypeDefinition type) {
         return new XmlStringInstanceIdentifierCodec(getEffectiveModelContext(), this);
     }
 
index 163f9c0324563476c4c34708909eff9b77be57b4..639fbcc9a62173f8c0a2957afaf898d80372193a 100644 (file)
@@ -64,8 +64,7 @@ public class YT1473Test {
         assertTrue(id instanceof LeafSchemaNode);
         final var type = ((LeafSchemaNode) id).getType();
         assertTrue(type instanceof InstanceIdentifierTypeDefinition);
-        CODEC = (XmlStringInstanceIdentifierCodec) XmlCodecFactory.create(modelContext)
-                .instanceIdentifierCodec((InstanceIdentifierTypeDefinition) type);
+        CODEC = XmlCodecFactory.create(modelContext).instanceIdentifierCodec((InstanceIdentifierTypeDefinition) type);
     }
 
     @AfterAll