Migrate YangInstanceIdentifier.EMPTY users 75/84175/19
authorRobert Varga <robert.varga@pantheon.tech>
Tue, 3 Sep 2019 10:27:41 +0000 (12:27 +0200)
committerRobert Varga <robert.varga@pantheon.tech>
Tue, 24 Sep 2019 11:02:59 +0000 (13:02 +0200)
This moves over to use the replacement empty() method.

Change-Id: I133d5025ee176e5487dbcd8f71517c990c299682
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>
15 files changed:
netconf/mdsal-netconf-connector/src/main/java/org/opendaylight/netconf/mdsal/connector/ops/CopyConfig.java
netconf/mdsal-netconf-connector/src/main/java/org/opendaylight/netconf/mdsal/connector/ops/get/AbstractGet.java
netconf/netconf-topology-singleton/src/main/java/org/opendaylight/netconf/topology/singleton/impl/ProxyDOMRpcService.java
netconf/netconf-topology-singleton/src/main/java/org/opendaylight/netconf/topology/singleton/impl/actors/NetconfNodeActor.java
netconf/netconf-topology-singleton/src/test/java/org/opendaylight/netconf/topology/singleton/impl/ProxyDOMDataBrokerTest.java
netconf/netconf-topology-singleton/src/test/java/org/opendaylight/netconf/topology/singleton/impl/actors/ReadTransactionActorTestAdapter.java
netconf/netconf-topology-singleton/src/test/java/org/opendaylight/netconf/topology/singleton/impl/tx/ProxyReadWriteTransactionTest.java
netconf/sal-netconf-connector/src/test/java/org/opendaylight/netconf/sal/connect/netconf/util/NetconfBaseOpsTest.java
restconf/restconf-nb-bierman02/src/main/java/org/opendaylight/netconf/sal/restconf/impl/ControllerContext.java
restconf/restconf-nb-rfc8040/src/test/java/org/opendaylight/restconf/nb/rfc8040/rests/services/impl/RestconfDataServiceImplTest.java
restconf/restconf-nb-rfc8040/src/test/java/org/opendaylight/restconf/nb/rfc8040/rests/utils/DeleteDataTransactionUtilTest.java
restconf/restconf-nb-rfc8040/src/test/java/org/opendaylight/restconf/nb/rfc8040/rests/utils/PutDataTransactionUtilTest.java
restconf/restconf-nb-rfc8040/src/test/java/org/opendaylight/restconf/nb/rfc8040/utils/parser/IdentifierCodecTest.java
restconf/restconf-nb-rfc8040/src/test/java/org/opendaylight/restconf/nb/rfc8040/utils/parser/ParserIdentifierTest.java
restconf/restconf-nb-rfc8040/src/test/java/org/opendaylight/restconf/nb/rfc8040/utils/parser/YangInstanceIdentifierSerializerTest.java

index dd3889eec94e3a0fcfd32468f0526c49524a465e..939fcb1d3cdbe417ea017bd98a1fb9e245c52317 100644 (file)
@@ -104,7 +104,7 @@ public final class CopyConfig extends AbstractEdit {
         // <copy-config>, unlike <edit-config>, always replaces entire configuration,
         // so remove old configuration first:
         final DOMDataTreeReadWriteTransaction rwTx = transactionProvider.getOrCreateTransaction();
-        rwTx.put(LogicalDatastoreType.CONFIGURATION, YangInstanceIdentifier.EMPTY, EMPTY_ROOT_NODE);
+        rwTx.put(LogicalDatastoreType.CONFIGURATION, YangInstanceIdentifier.empty(), EMPTY_ROOT_NODE);
 
         // Then create nodes present in the <config> element:
         for (final XmlElement element : configElements) {
@@ -169,7 +169,7 @@ public final class CopyConfig extends AbstractEdit {
     private ContainerNode readData(final XmlElement source) throws DocumentedException {
         final Datastore sourceDatastore = getDatastore(source);
         final DOMDataTreeReadWriteTransaction rwTx = getTransaction(sourceDatastore);
-        final YangInstanceIdentifier dataRoot = YangInstanceIdentifier.EMPTY;
+        final YangInstanceIdentifier dataRoot = YangInstanceIdentifier.empty();
         try {
             final Optional<NormalizedNode<?, ?>> normalizedNodeOptional = rwTx.read(
                 LogicalDatastoreType.CONFIGURATION, dataRoot).get();
index f02bc08adf0a1c5c95d34b225a1e64cc1a966049..af319616dafc2e6891d504f38005a9b2645c06e1 100644 (file)
@@ -43,7 +43,7 @@ import org.w3c.dom.Node;
 
 public abstract class AbstractGet extends AbstractSingletonNetconfOperation {
     private static final XMLOutputFactory XML_OUTPUT_FACTORY;
-    private static final YangInstanceIdentifier ROOT = YangInstanceIdentifier.EMPTY;
+    private static final YangInstanceIdentifier ROOT = YangInstanceIdentifier.empty();
     private static final String FILTER = "filter";
 
     static {
index 8936b110c069452d3b59b1f68b54eb7b0be5284a..5c5fd4a77e56a31d942ae9a36682fe8351a9dfea 100644 (file)
@@ -57,7 +57,7 @@ public class ProxyDOMRpcService implements DOMRpcService {
         LOG.trace("{}: Rpc operation invoked with schema type: {} and node: {}.", id, type, input);
 
         final NormalizedNodeMessage normalizedNodeMessage = input != null
-                ? new NormalizedNodeMessage(YangInstanceIdentifier.EMPTY, input) : null;
+                ? new NormalizedNodeMessage(YangInstanceIdentifier.empty(), input) : null;
         final Future<Object> scalaFuture = Patterns.ask(masterActorRef,
                 new InvokeRpcMessage(new SchemaPathMessage(type), normalizedNodeMessage), actorResponseWaitTime);
 
index e99834d07a74dacdd7bf4c41df0e8e3d6bc33e4a..e6183e17e9aa1e2cb16296a6e334432b5c1645ba 100644 (file)
@@ -254,7 +254,7 @@ public class NetconfNodeActor extends AbstractUntypedActor {
                 }
                 NormalizedNodeMessage nodeMessageReply = null;
                 if (domRpcResult.getResult() != null) {
-                    nodeMessageReply = new NormalizedNodeMessage(YangInstanceIdentifier.EMPTY,
+                    nodeMessageReply = new NormalizedNodeMessage(YangInstanceIdentifier.empty(),
                             domRpcResult.getResult());
                 }
                 recipient.tell(new InvokeRpcMessageReply(nodeMessageReply, domRpcResult.getErrors()), getSelf());
index 5384856bd64415f5733e9c795c911c532fcc5474..337552101d0e0c2adb9223714dde621c2b0b2bd4 100644 (file)
@@ -59,7 +59,7 @@ public class ProxyDOMDataBrokerTest {
 
         assertEquals(DEVICE_ID, tx.getIdentifier());
 
-        tx.read(LogicalDatastoreType.CONFIGURATION, YangInstanceIdentifier.EMPTY);
+        tx.read(LogicalDatastoreType.CONFIGURATION, YangInstanceIdentifier.empty());
         masterActor.expectMsgClass(ReadRequest.class);
     }
 
@@ -71,7 +71,7 @@ public class ProxyDOMDataBrokerTest {
 
         assertEquals(DEVICE_ID, tx.getIdentifier());
 
-        tx.delete(LogicalDatastoreType.CONFIGURATION, YangInstanceIdentifier.EMPTY);
+        tx.delete(LogicalDatastoreType.CONFIGURATION, YangInstanceIdentifier.empty());
         masterActor.expectMsgClass(DeleteRequest.class);
     }
 
@@ -83,7 +83,7 @@ public class ProxyDOMDataBrokerTest {
 
         assertEquals(DEVICE_ID, tx.getIdentifier());
 
-        tx.read(LogicalDatastoreType.CONFIGURATION, YangInstanceIdentifier.EMPTY);
+        tx.read(LogicalDatastoreType.CONFIGURATION, YangInstanceIdentifier.empty());
         masterActor.expectMsgClass(ReadRequest.class);
     }
 
index 9080ed47d8b5bbcca21f15e865e3a6ea7a634043..cb1324b472920f1a9cf11f1822dad9c8b6462a74 100644 (file)
@@ -40,7 +40,7 @@ import org.opendaylight.yangtools.yang.data.impl.schema.Builders;
  * @author Thomas Pantelis
  */
 public abstract class ReadTransactionActorTestAdapter {
-    static final YangInstanceIdentifier PATH = YangInstanceIdentifier.EMPTY;
+    static final YangInstanceIdentifier PATH = YangInstanceIdentifier.empty();
     static final LogicalDatastoreType STORE = LogicalDatastoreType.CONFIGURATION;
     static final Timeout TIMEOUT = Timeout.apply(5, TimeUnit.SECONDS);
     static final NormalizedNode<?, ?> NODE = Builders.containerBuilder()
index 60cf7fab8e399b216fbbbd18d1b3b74985910c6d..591b57067aa7e9724ea0f4807410f0dfa2b23ab2 100644 (file)
@@ -56,7 +56,7 @@ public class ProxyReadWriteTransactionTest {
     private static final FiniteDuration EXP_NO_MESSAGE_TIMEOUT = Duration.apply(300, TimeUnit.MILLISECONDS);
     private static final RemoteDeviceId DEVICE_ID =
             new RemoteDeviceId("dev1", InetSocketAddress.createUnresolved("localhost", 17830));
-    private static final YangInstanceIdentifier PATH = YangInstanceIdentifier.EMPTY;
+    private static final YangInstanceIdentifier PATH = YangInstanceIdentifier.empty();
     private static final LogicalDatastoreType STORE = LogicalDatastoreType.CONFIGURATION;
 
     private static ActorSystem system = ActorSystem.apply();
index 316b2ae651afe3d2832132e707d1d6dd14db9e71..0fce822e6a116b2a4334ddd2c75b0687d1295df9 100644 (file)
@@ -177,7 +177,7 @@ public class NetconfBaseOpsTest extends AbstractTestModelTest {
     @Test
     public void testGetConfigRunningData() throws Exception {
         final Optional<NormalizedNode<?, ?>> dataOpt =
-                baseOps.getConfigRunningData(callback, Optional.of(YangInstanceIdentifier.EMPTY)).get();
+                baseOps.getConfigRunningData(callback, Optional.of(YangInstanceIdentifier.empty())).get();
         Assert.assertTrue(dataOpt.isPresent());
         Assert.assertEquals(NetconfUtil.NETCONF_DATA_QNAME, dataOpt.get().getNodeType());
     }
@@ -185,7 +185,7 @@ public class NetconfBaseOpsTest extends AbstractTestModelTest {
     @Test
     public void testGetData() throws Exception {
         final Optional<NormalizedNode<?, ?>> dataOpt =
-                baseOps.getData(callback, Optional.of(YangInstanceIdentifier.EMPTY)).get();
+                baseOps.getData(callback, Optional.of(YangInstanceIdentifier.empty())).get();
         Assert.assertTrue(dataOpt.isPresent());
         Assert.assertEquals(NetconfUtil.NETCONF_DATA_QNAME, dataOpt.get().getNodeType());
     }
index 5908624515f80a1a0b66695fda555236c6e78c0f..b36b0c5c1bd9af084e84097041f5f992f2328f1b 100644 (file)
@@ -171,7 +171,7 @@ public final class ControllerContext implements SchemaContextListener, Closeable
         checkPreconditions();
 
         if (restconfInstance == null) {
-            return new InstanceIdentifierContext<>(YangInstanceIdentifier.EMPTY, this.globalSchema, null,
+            return new InstanceIdentifierContext<>(YangInstanceIdentifier.empty(), this.globalSchema, null,
                     this.globalSchema);
         }
 
index 101213e5e33e583638be09abd7d9d3995ae0ea4a..7d474846ab1cd8475680ec78ba7d78c3dcec74ee 100644 (file)
@@ -231,10 +231,10 @@ public class RestconfDataServiceImplTest {
         doReturn(new MultivaluedHashMap<String, String>()).when(this.uriInfo).getQueryParameters();
         doReturn(immediateFluentFuture(Optional.of(wrapNodeByDataRootContainer(this.buildBaseContConfig))))
                 .when(this.read)
-                .read(LogicalDatastoreType.CONFIGURATION, YangInstanceIdentifier.EMPTY);
+                .read(LogicalDatastoreType.CONFIGURATION, YangInstanceIdentifier.empty());
         doReturn(immediateFluentFuture(Optional.of(wrapNodeByDataRootContainer(this.buildBaseContOperational))))
                 .when(this.read)
-                .read(LogicalDatastoreType.OPERATIONAL, YangInstanceIdentifier.EMPTY);
+                .read(LogicalDatastoreType.OPERATIONAL, YangInstanceIdentifier.empty());
         final Response response = this.dataService.readData(this.uriInfo);
         assertNotNull(response);
         assertEquals(200, response.getStatus());
index 0f95c8eebda810da352ca915d77402ed73c24e69..696ae89ee59c7cc83a2b7d2d878a15ecf6faef06 100644 (file)
@@ -49,7 +49,7 @@ public class DeleteDataTransactionUtilTest {
         MockitoAnnotations.initMocks(this);
         Mockito.when(this.transactionChain.newReadWriteTransaction()).thenReturn(this.readWrite);
         Mockito.doReturn(CommitInfo.emptyFluentFuture()).when(this.readWrite).commit();
-        Mockito.when(this.context.getInstanceIdentifier()).thenReturn(YangInstanceIdentifier.EMPTY);
+        Mockito.when(this.context.getInstanceIdentifier()).thenReturn(YangInstanceIdentifier.empty());
 
         Mockito.doReturn(transactionChain).when(mockDataBroker).createTransactionChain(Mockito.any());
         transactionChainHandler = new TransactionChainHandler(mockDataBroker);
@@ -62,7 +62,7 @@ public class DeleteDataTransactionUtilTest {
     public void deleteData() throws Exception {
         // assert that data to delete exists
         Mockito.when(this.transactionChain.newReadWriteTransaction().exists(LogicalDatastoreType.CONFIGURATION,
-                YangInstanceIdentifier.EMPTY)).thenReturn(immediateTrueFluentFuture());
+                YangInstanceIdentifier.empty())).thenReturn(immediateTrueFluentFuture());
 
         // test
         final Response response = DeleteDataTransactionUtil.deleteData(
@@ -79,7 +79,7 @@ public class DeleteDataTransactionUtilTest {
     public void deleteDataNegativeTest() throws Exception {
         // assert that data to delete does NOT exist
         Mockito.when(this.transactionChain.newReadWriteTransaction().exists(LogicalDatastoreType.CONFIGURATION,
-                YangInstanceIdentifier.EMPTY)).thenReturn(immediateFalseFluentFuture());
+                YangInstanceIdentifier.empty())).thenReturn(immediateFalseFluentFuture());
 
         // test and assert error
         try {
index 6d026c5d7d8dc89c02cb5458208c8ecba5003652..27a0a0631f044da279b7555a16da20e8d76ba2c5 100644 (file)
@@ -188,7 +188,7 @@ public class PutDataTransactionUtilTest {
         final InstanceIdentifierContext<DataSchemaNode> iidContext =
                 new InstanceIdentifierContext<>(this.iid, this.schemaNode, null, this.schema);
         final NormalizedNodeContext payload = new NormalizedNodeContext(iidContext, this.buildLeaf);
-        PutDataTransactionUtil.validTopLevelNodeName(YangInstanceIdentifier.EMPTY, payload);
+        PutDataTransactionUtil.validTopLevelNodeName(YangInstanceIdentifier.empty(), payload);
     }
 
     @Test(expected = RestconfDocumentedException.class)
index fed9756f6580ddaf19df822f5551413bd74cd599..f4fc263f869c91e7b189fa8d3fa98906230dbb49 100644 (file)
@@ -73,7 +73,7 @@ public class IdentifierCodecTest {
     @Test
     public void codecDeserializeNullTest() {
         final YangInstanceIdentifier dataYangII = IdentifierCodec.deserialize(null, this.schemaContext);
-        assertEquals("Failed codec deserialization test", YangInstanceIdentifier.EMPTY, dataYangII);
+        assertEquals("Failed codec deserialization test", YangInstanceIdentifier.empty(), dataYangII);
     }
 
     /**
@@ -82,7 +82,7 @@ public class IdentifierCodecTest {
      */
     @Test
     public void codecSerializeEmptyTest() {
-        final String serialized = IdentifierCodec.serialize(YangInstanceIdentifier.EMPTY, this.schemaContext);
+        final String serialized = IdentifierCodec.serialize(YangInstanceIdentifier.empty(), this.schemaContext);
         assertTrue("Failed codec serialization test", serialized.isEmpty());
     }
 
@@ -92,8 +92,8 @@ public class IdentifierCodecTest {
      */
     @Test
     public void codecDeserializeAndSerializeEmptyTest() {
-        final String serialized = IdentifierCodec.serialize(YangInstanceIdentifier.EMPTY, this.schemaContext);
+        final String serialized = IdentifierCodec.serialize(YangInstanceIdentifier.empty(), this.schemaContext);
         assertEquals("Failed codec serialization and deserialization test",
-                YangInstanceIdentifier.EMPTY, IdentifierCodec.deserialize(serialized, this.schemaContext));
+                YangInstanceIdentifier.empty(), IdentifierCodec.deserialize(serialized, this.schemaContext));
     }
 }
index 2ad973f7b6cbfb7b068dde3a22c6be2608e5b596..d5f3568b6b1a0baf16307a83c48c52ad4d8175c1 100644 (file)
@@ -128,7 +128,7 @@ public class ParserIdentifierTest {
 
         // register mount point with null schema context
         when(this.mockMountPoint.getSchemaContext()).thenReturn(null);
-        when(this.mockMountPointService.getMountPoint(YangInstanceIdentifier.EMPTY))
+        when(this.mockMountPointService.getMountPoint(YangInstanceIdentifier.empty()))
                 .thenReturn(Optional.of(this.mockMountPoint));
     }
 
@@ -190,7 +190,7 @@ public class ParserIdentifierTest {
         final InstanceIdentifierContext<?> context = ParserIdentifier.toInstanceIdentifier(
                 null, this.schemaContext, Optional.empty());
         assertEquals("Returned not expected identifier",
-                YangInstanceIdentifier.EMPTY, context.getInstanceIdentifier());
+                YangInstanceIdentifier.empty(), context.getInstanceIdentifier());
     }
 
     /**
@@ -211,7 +211,7 @@ public class ParserIdentifierTest {
         final InstanceIdentifierContext<?> context = ParserIdentifier.toInstanceIdentifier(
                 "", this.schemaContext, Optional.empty());
         assertEquals("Returned not expected identifier",
-                YangInstanceIdentifier.EMPTY, context.getInstanceIdentifier());
+                YangInstanceIdentifier.empty(), context.getInstanceIdentifier());
     }
 
     /**
index 8721346af16251099bf58b228dfac007d7693121..36ee08edc3daea01a230bf4cf2c88094c9707d3c 100644 (file)
@@ -191,7 +191,7 @@ public class YangInstanceIdentifierSerializerTest {
     @Test
     public void serializeNullSchemaContextNegativeTest() {
         this.thrown.expect(NullPointerException.class);
-        YangInstanceIdentifierSerializer.create(null, YangInstanceIdentifier.EMPTY);
+        YangInstanceIdentifierSerializer.create(null, YangInstanceIdentifier.empty());
     }
 
     /**
@@ -212,7 +212,8 @@ public class YangInstanceIdentifierSerializerTest {
      */
     @Test
     public void serializeEmptyDataTest() {
-        final String result = YangInstanceIdentifierSerializer.create(this.schemaContext, YangInstanceIdentifier.EMPTY);
+        final String result = YangInstanceIdentifierSerializer.create(this.schemaContext,
+            YangInstanceIdentifier.empty());
         assertTrue("Empty identifier is expected", result.isEmpty());
     }