Improve XmlCodecFactory a bit 10/104910/3
authorRobert Varga <robert.varga@pantheon.tech>
Tue, 14 Mar 2023 12:37:00 +0000 (13:37 +0100)
committerRobert Varga <robert.varga@pantheon.tech>
Wed, 15 Mar 2023 22:07:23 +0000 (23:07 +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>
(cherry picked from commit 90a498521a7ec3fd62ed91c2c0016ab334a93da2)

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 d68424bc51808e616ebe1ec31b3338ef9351c3d6..ff2b317b6742bc50ce7a63f2b80358d8ff242afe 100644 (file)
@@ -63,8 +63,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