X-Git-Url: https://git.opendaylight.org/gerrit/gitweb?a=blobdiff_plain;f=opendaylight%2Fmd-sal%2Fsal-distributed-datastore%2Fsrc%2Ftest%2Fjava%2Forg%2Fopendaylight%2Fcontroller%2Fcluster%2Fdatastore%2Fpersisted%2FShardDataTreeSnapshotTest.java;h=9d172e653a69f6e7adeac56bf1b50a06f1f5cdb2;hb=HEAD;hp=3051194a6f019c0ceb494713ff6c24062570f406;hpb=abaef4a5ae37f27542155457fe7306a4662b1eeb;p=controller.git diff --git a/opendaylight/md-sal/sal-distributed-datastore/src/test/java/org/opendaylight/controller/cluster/datastore/persisted/ShardDataTreeSnapshotTest.java b/opendaylight/md-sal/sal-distributed-datastore/src/test/java/org/opendaylight/controller/cluster/datastore/persisted/ShardDataTreeSnapshotTest.java index 3051194a6f..9d172e653a 100644 --- a/opendaylight/md-sal/sal-distributed-datastore/src/test/java/org/opendaylight/controller/cluster/datastore/persisted/ShardDataTreeSnapshotTest.java +++ b/opendaylight/md-sal/sal-distributed-datastore/src/test/java/org/opendaylight/controller/cluster/datastore/persisted/ShardDataTreeSnapshotTest.java @@ -8,9 +8,7 @@ package org.opendaylight.controller.cluster.datastore.persisted; import static org.junit.Assert.assertEquals; -import static org.junit.Assert.assertTrue; -import com.google.common.collect.ImmutableMap; import java.io.ByteArrayInputStream; import java.io.ByteArrayOutputStream; import java.io.Externalizable; @@ -23,10 +21,9 @@ import java.util.Map; import java.util.Optional; import org.junit.Test; import org.opendaylight.controller.md.cluster.datastore.model.TestModel; -import org.opendaylight.yangtools.yang.data.api.YangInstanceIdentifier; -import org.opendaylight.yangtools.yang.data.api.schema.NormalizedNode; -import org.opendaylight.yangtools.yang.data.impl.schema.ImmutableNodes; -import org.opendaylight.yangtools.yang.data.impl.schema.builder.impl.ImmutableContainerNodeBuilder; +import org.opendaylight.yangtools.yang.data.api.YangInstanceIdentifier.NodeIdentifier; +import org.opendaylight.yangtools.yang.data.api.schema.ContainerNode; +import org.opendaylight.yangtools.yang.data.spi.node.ImmutableNodes; /** * Unit tests for ShardDataTreeSnapshot. @@ -34,11 +31,10 @@ import org.opendaylight.yangtools.yang.data.impl.schema.builder.impl.ImmutableCo * @author Thomas Pantelis */ public class ShardDataTreeSnapshotTest { - @Test public void testShardDataTreeSnapshotWithNoMetadata() throws Exception { - NormalizedNode expectedNode = ImmutableContainerNodeBuilder.create() - .withNodeIdentifier(new YangInstanceIdentifier.NodeIdentifier(TestModel.TEST_QNAME)) + ContainerNode expectedNode = ImmutableNodes.newContainerBuilder() + .withNodeIdentifier(new NodeIdentifier(TestModel.TEST_QNAME)) .withChild(ImmutableNodes.leafNode(TestModel.DESC_QNAME, "foo")).build(); MetadataShardDataTreeSnapshot snapshot = new MetadataShardDataTreeSnapshot(expectedNode); @@ -48,28 +44,26 @@ public class ShardDataTreeSnapshotTest { } final byte[] bytes = bos.toByteArray(); - assertEquals(236, bytes.length); + assertEquals(202, bytes.length); ShardDataTreeSnapshot deserialized; try (ObjectInputStream in = new ObjectInputStream(new ByteArrayInputStream(bytes))) { deserialized = ShardDataTreeSnapshot.deserialize(in).getSnapshot(); } - Optional actualNode = deserialized.getRootNode(); - assertTrue("rootNode present", actualNode.isPresent()); - assertEquals("rootNode", expectedNode, actualNode.get()); + assertEquals("rootNode", Optional.of(expectedNode), deserialized.getRootNode()); assertEquals("Deserialized type", MetadataShardDataTreeSnapshot.class, deserialized.getClass()); assertEquals("Metadata size", 0, ((MetadataShardDataTreeSnapshot)deserialized).getMetadata().size()); } @Test public void testShardDataTreeSnapshotWithMetadata() throws Exception { - NormalizedNode expectedNode = ImmutableContainerNodeBuilder.create() - .withNodeIdentifier(new YangInstanceIdentifier.NodeIdentifier(TestModel.TEST_QNAME)) + ContainerNode expectedNode = ImmutableNodes.newContainerBuilder() + .withNodeIdentifier(new NodeIdentifier(TestModel.TEST_QNAME)) .withChild(ImmutableNodes.leafNode(TestModel.DESC_QNAME, "foo")).build(); Map>, ShardDataTreeSnapshotMetadata> expMetadata = - ImmutableMap.of(TestShardDataTreeSnapshotMetadata.class, new TestShardDataTreeSnapshotMetadata("test")); + Map.of(TestShardDataTreeSnapshotMetadata.class, new TestShardDataTreeSnapshotMetadata("test")); MetadataShardDataTreeSnapshot snapshot = new MetadataShardDataTreeSnapshot(expectedNode, expMetadata); ByteArrayOutputStream bos = new ByteArrayOutputStream(); try (ObjectOutputStream out = new ObjectOutputStream(bos)) { @@ -77,22 +71,21 @@ public class ShardDataTreeSnapshotTest { } final byte[] bytes = bos.toByteArray(); - assertEquals(384, bytes.length); + assertEquals(350, bytes.length); ShardDataTreeSnapshot deserialized; try (ObjectInputStream in = new ObjectInputStream(new ByteArrayInputStream(bytes))) { deserialized = ShardDataTreeSnapshot.deserialize(in).getSnapshot(); } - Optional actualNode = deserialized.getRootNode(); - assertTrue("rootNode present", actualNode.isPresent()); - assertEquals("rootNode", expectedNode, actualNode.get()); + assertEquals("rootNode", Optional.of(expectedNode), deserialized.getRootNode()); assertEquals("Deserialized type", MetadataShardDataTreeSnapshot.class, deserialized.getClass()); assertEquals("Metadata", expMetadata, ((MetadataShardDataTreeSnapshot)deserialized).getMetadata()); } static class TestShardDataTreeSnapshotMetadata extends ShardDataTreeSnapshotMetadata { + @java.io.Serial private static final long serialVersionUID = 1L; private final String data; @@ -118,11 +111,13 @@ public class ShardDataTreeSnapshotTest { @Override public boolean equals(final Object obj) { - return obj instanceof TestShardDataTreeSnapshotMetadata - && data.equals(((TestShardDataTreeSnapshotMetadata)obj).data); + return obj instanceof TestShardDataTreeSnapshotMetadata other && data.equals(other.data); } private static class Proxy implements Externalizable { + @java.io.Serial + private static final long serialVersionUID = 7534948936595056176L; + private String data; @SuppressWarnings("checkstyle:RedundantModifier")