Bump MRI upstreams 21/100121/5
authorRobert Varga <robert.varga@pantheon.tech>
Mon, 14 Mar 2022 07:54:01 +0000 (08:54 +0100)
committerRobert Varga <robert.varga@pantheon.tech>
Wed, 30 Mar 2022 19:48:07 +0000 (21:48 +0200)
Adopt the following versions:
- odlparent-10.0.0
- infrautils-3.0.0
- yangtools-8.0.2
- mdsal-9.0.1
- controller-5.0.1
- aaa-0.15.1
- netconf-3.0.0

Change-Id: Ia4cd90ea1a8aa4f56a3e1853cb3363d40752e2b6
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>
45 files changed:
artifacts/pom.xml
commons/parent/pom.xml
features/features-lispflowmapping/pom.xml
features/odl-lispflowmapping-inmemorydb/pom.xml
features/odl-lispflowmapping-inmemorydb/src/main/feature/feature.xml
features/odl-lispflowmapping-mappingservice-shell/pom.xml
features/odl-lispflowmapping-mappingservice/pom.xml
features/odl-lispflowmapping-models/pom.xml
features/odl-lispflowmapping-models/src/main/feature/feature.xml
features/odl-lispflowmapping-msmr/pom.xml
features/odl-lispflowmapping-southbound/pom.xml
features/odl-lispflowmapping-southbound/src/main/feature/feature.xml
features/pom.xml
integrationtest/pom.xml
integrationtest/src/test/java/org/opendaylight/lispflowmapping/integrationtest/MappingServiceIntegrationTest.java
lispflowmapping-karaf/pom.xml
mappingservice/api/pom.xml
mappingservice/config/pom.xml
mappingservice/dsbackend/pom.xml
mappingservice/dsbackend/src/main/java/org/opendaylight/lispflowmapping/dsbackend/DataStoreBackEnd.java
mappingservice/implementation/pom.xml
mappingservice/implementation/src/main/java/org/opendaylight/lispflowmapping/implementation/MappingService.java
mappingservice/implementation/src/main/java/org/opendaylight/lispflowmapping/implementation/lisp/MapServer.java
mappingservice/implementation/src/main/java/org/opendaylight/lispflowmapping/implementation/util/DSBEInputUtil.java
mappingservice/implementation/src/main/java/org/opendaylight/lispflowmapping/implementation/util/LispNotificationHelper.java
mappingservice/implementation/src/test/java/org/opendaylight/lispflowmapping/implementation/MappingSystemTest.java
mappingservice/implementation/src/test/java/org/opendaylight/lispflowmapping/implementation/lisp/MappingServiceTest.java
mappingservice/implementation/src/test/java/org/opendaylight/lispflowmapping/implementation/util/LispNotificationHelperTest.java
mappingservice/lisp-proto/pom.xml
mappingservice/lisp-proto/src/main/java/org/opendaylight/lispflowmapping/lisp/serializer/address/AfiListSerializer.java
mappingservice/lisp-proto/src/main/java/org/opendaylight/lispflowmapping/lisp/serializer/address/NoAddressSerializer.java
mappingservice/lisp-proto/src/main/java/org/opendaylight/lispflowmapping/lisp/util/LispAddressStringifier.java
mappingservice/lisp-proto/src/main/java/org/opendaylight/lispflowmapping/lisp/util/LispAddressUtil.java
mappingservice/lisp-proto/src/test/java/org/opendaylight/lispflowmapping/lisp/util/LispAddressStringifierTest.java
mappingservice/lisp-proto/src/test/java/org/opendaylight/lispflowmapping/lisp/util/LispAddressUtilTest.java
mappingservice/lisp-proto/src/test/java/org/opendaylight/lispflowmapping/serializer/address/AfiListSerializerTest.java
mappingservice/lisp-proto/src/test/java/org/opendaylight/lispflowmapping/serializer/address/DistinguishedNameSerializerTest.java
mappingservice/mapcache/src/test/java/org/opendaylight/lispflowmapping/mapcache/MultiTableMapCacheTest.java
mappingservice/pom.xml
mappingservice/shell/pom.xml
mappingservice/southbound/pom.xml
mappingservice/southbound/src/main/java/org/opendaylight/lispflowmapping/southbound/LispSouthboundRPC.java
mappingservice/southbound/src/test/java/org/opendaylight/lispflowmapping/southbound/LispSouthboundPluginTest.java
mappingservice/southbound/src/test/java/org/opendaylight/lispflowmapping/southbound/LispSouthboundRpcTest.java
mappingservice/southbound/src/test/java/org/opendaylight/lispflowmapping/southbound/lisp/AuthenticationKeyDataListenerTest.java

index f6fd7d7f782ceeac9e6712190ebe34da19f36d46..d513cb31e3aa84e1c3805a0a1009afada7b65b79 100644 (file)
@@ -12,7 +12,7 @@ and is available at http://www.eclipse.org/legal/epl-v10.html
   <parent>
     <groupId>org.opendaylight.odlparent</groupId>
     <artifactId>odlparent-lite</artifactId>
-    <version>9.0.13</version>
+    <version>10.0.0</version>
     <relativePath/>
   </parent>
 
index f1e4269dd7c0f8d329fdaf538d1d2be37c3cb2e3..79eb742e350682e8750dc92731947f00e43005b1 100644 (file)
@@ -15,7 +15,7 @@
          for plugin configuration etc. -->
     <groupId>org.opendaylight.odlparent</groupId>
     <artifactId>odlparent</artifactId>
-    <version>9.0.13</version>
+    <version>10.0.0</version>
     <relativePath/>
   </parent>
 
@@ -46,9 +46,9 @@
       <dependency>
         <groupId>org.opendaylight.mdsal</groupId>
         <artifactId>mdsal-artifacts</artifactId>
-        <version>8.0.11</version>
+        <version>9.0.1</version>
         <type>pom</type>
-       <scope>import</scope>
+        <scope>import</scope>
       </dependency>
       <dependency>
         <groupId>junit-addons</groupId>
@@ -68,7 +68,7 @@
       <dependency>
         <groupId>org.opendaylight.controller</groupId>
         <artifactId>controller-artifacts</artifactId>
-        <version>4.0.10</version>
+        <version>5.0.1</version>
         <type>pom</type>
         <scope>import</scope>
       </dependency>
index ee018fbc96cc1e8cf8eea085945121ab2a4d96ad..5a75051afe9df2a471ecabacd7424efa90bf8695 100644 (file)
@@ -12,7 +12,7 @@ and is available at http://www.eclipse.org/legal/epl-v10.html
   <parent>
     <groupId>org.opendaylight.odlparent</groupId>
     <artifactId>feature-repo-parent</artifactId>
-    <version>9.0.13</version>
+    <version>10.0.0</version>
     <relativePath/>
   </parent>
 
index ad9b27ff54ab7d38391dca98bab5d010b96b50cd..f0129e35050cb30e39e5ea433212bc11180bf838 100644 (file)
@@ -12,7 +12,7 @@ and is available at http://www.eclipse.org/legal/epl-v10.html
   <parent>
     <groupId>org.opendaylight.odlparent</groupId>
     <artifactId>single-feature-parent</artifactId>
-    <version>9.0.13</version>
+    <version>10.0.0</version>
     <relativePath/>
   </parent>
 
@@ -24,11 +24,28 @@ and is available at http://www.eclipse.org/legal/epl-v10.html
        build failure. Please do not modify this unless you have a good reason. -->
   <name>ODL :: lispflowmapping :: ${project.artifactId}</name>
 
+  <dependencyManagement>
+    <dependencies>
+      <dependency>
+        <groupId>org.opendaylight.controller</groupId>
+        <artifactId>controller-artifacts</artifactId>
+        <version>5.0.1</version>
+        <type>pom</type>
+        <scope>import</scope>
+      </dependency>
+    </dependencies>
+  </dependencyManagement>
+
   <dependencies>
+    <dependency>
+      <groupId>org.opendaylight.controller</groupId>
+      <artifactId>odl-controller-blueprint</artifactId>
+      <type>xml</type>
+      <classifier>features</classifier>
+    </dependency>
     <dependency>
       <groupId>org.opendaylight.controller</groupId>
       <artifactId>odl-mdsal-broker</artifactId>
-      <version>4.0.10</version>
       <type>xml</type>
       <classifier>features</classifier>
     </dependency>
index 297723a5b6c0ebb1f914c986834c26f855f830ad..c1466b1735bac1041faf7c727497ae5e6fd78984 100644 (file)
@@ -8,6 +8,7 @@ and is available at http://www.eclipse.org/legal/epl-v10.html INTERNAL
 -->
 <features xmlns="http://karaf.apache.org/xmlns/features/v1.4.0" name="odl-lispflowmapping-inmemorydb">
     <feature name="odl-lispflowmapping-inmemorydb">
-        <feature version="[4,5)">odl-mdsal-broker</feature>
+        <feature version="[5,6)">odl-mdsal-broker</feature>
+        <feature version="[5,6)">odl-controller-blueprint</feature>
     </feature>
 </features>
index f498ff985e7d77e87e223637d93ed5ad7c595778..84f64df5967ab30b98e537512c04af7e5e61e2df 100644 (file)
@@ -12,7 +12,7 @@ and is available at http://www.eclipse.org/legal/epl-v10.html
   <parent>
     <groupId>org.opendaylight.odlparent</groupId>
     <artifactId>single-feature-parent</artifactId>
-    <version>9.0.13</version>
+    <version>10.0.0</version>
     <relativePath/>
   </parent>
 
index 9adf2f4be1105ea9a420cc600a81ed113eeee39c..7007eb5a93d6299fae3de0bc01793f1733e336c3 100644 (file)
@@ -12,7 +12,7 @@ and is available at http://www.eclipse.org/legal/epl-v10.html
   <parent>
     <groupId>org.opendaylight.odlparent</groupId>
     <artifactId>single-feature-parent</artifactId>
-    <version>9.0.13</version>
+    <version>10.0.0</version>
     <relativePath/>
   </parent>
 
@@ -28,7 +28,7 @@ and is available at http://www.eclipse.org/legal/epl-v10.html
     <dependency>
       <groupId>org.opendaylight.netconf</groupId>
       <artifactId>odl-restconf</artifactId>
-      <version>2.0.14</version>
+      <version>3.0.0</version>
       <type>xml</type>
       <classifier>features</classifier>
     </dependency>
index fbdf4a98443eeed7451f8ec0f137804a3f488bf7..f114562a0ce26e6e1e9f95a0844ff6450753c669 100644 (file)
@@ -12,7 +12,7 @@ and is available at http://www.eclipse.org/legal/epl-v10.html
   <parent>
     <groupId>org.opendaylight.odlparent</groupId>
     <artifactId>single-feature-parent</artifactId>
-    <version>9.0.13</version>
+    <version>10.0.0</version>
     <relativePath/>
   </parent>
 
@@ -29,7 +29,7 @@ and is available at http://www.eclipse.org/legal/epl-v10.html
       <dependency>
         <groupId>org.opendaylight.mdsal</groupId>
         <artifactId>mdsal-artifacts</artifactId>
-        <version>8.0.11</version>
+        <version>9.0.1</version>
         <type>pom</type>
         <scope>import</scope>
       </dependency>
index 788751c84abb7e1e7c93621b01ec91f282c15b3d..0a5c3c9f01301d02ea0439adf9adac3b30a4835c 100644 (file)
@@ -8,7 +8,7 @@ and is available at http://www.eclipse.org/legal/epl-v10.html INTERNAL
 -->
 <features xmlns="http://karaf.apache.org/xmlns/features/v1.4.0" name="odl-lispflowmapping-models">
     <feature name="odl-lispflowmapping-models">
-        <feature version="[9,10)">odl-apache-commons-lang3</feature>
-        <feature version="[8,9)">odl-mdsal-model-rfc6991</feature>
+        <feature version="[10,11)">odl-apache-commons-lang3</feature>
+        <feature version="[9,10)">odl-mdsal-model-rfc6991</feature>
     </feature>
 </features>
index 06d601ca412812dbc9a100942a28dd44fa80a8e0..256e0fe42495bc16603c42bea54092d1b1ec4a4e 100644 (file)
@@ -12,7 +12,7 @@ and is available at http://www.eclipse.org/legal/epl-v10.html
   <parent>
     <groupId>org.opendaylight.odlparent</groupId>
     <artifactId>single-feature-parent</artifactId>
-    <version>9.0.13</version>
+    <version>10.0.0</version>
     <relativePath/>
   </parent>
 
index 0f8a88fb8beec18a9a29e7dfcf7e9e4ffea20dce..824bd80d261e918326a59d9cdd5a5744f6a8eff4 100644 (file)
@@ -12,7 +12,7 @@ and is available at http://www.eclipse.org/legal/epl-v10.html
   <parent>
     <groupId>org.opendaylight.odlparent</groupId>
     <artifactId>single-feature-parent</artifactId>
-    <version>9.0.13</version>
+    <version>10.0.0</version>
     <relativePath/>
   </parent>
 
@@ -28,14 +28,14 @@ and is available at http://www.eclipse.org/legal/epl-v10.html
     <dependency>
       <groupId>org.opendaylight.netconf</groupId>
       <artifactId>odl-restconf</artifactId>
-      <version>2.0.14</version>
+      <version>3.0.0</version>
       <type>xml</type>
       <classifier>features</classifier>
     </dependency>
     <dependency>
       <groupId>org.opendaylight.controller</groupId>
       <artifactId>odl-mdsal-broker</artifactId>
-      <version>4.0.10</version>
+      <version>5.0.1</version>
       <type>xml</type>
       <classifier>features</classifier>
     </dependency>
index 657eaf0f61778580bac6ff1d9fd6d5fc1b822c7b..729aae01d1e42b0389221d185077039dfc1e491f 100644 (file)
@@ -8,7 +8,7 @@ and is available at http://www.eclipse.org/legal/epl-v10.html INTERNAL
 -->
 <features xmlns="http://karaf.apache.org/xmlns/features/v1.4.0" name="odl-lispflowmapping-southbound">
     <feature name="odl-lispflowmapping-southbound">
-        <feature version="[9,10)">odl-netty-4</feature>
-        <feature version="[4,5)">odl-mdsal-broker</feature>
+        <feature version="[10,11)">odl-netty-4</feature>
+        <feature version="[5,6)">odl-mdsal-broker</feature>
     </feature>
 </features>
index 3d29369de8c95f6efe06573b07f490726968f978..1f0db8a1837f01275ac99e9e8b1a789115ef14d8 100644 (file)
@@ -12,7 +12,7 @@ and is available at http://www.eclipse.org/legal/epl-v10.html
   <parent>
     <groupId>org.opendaylight.odlparent</groupId>
     <artifactId>odlparent-lite</artifactId>
-    <version>9.0.13</version>
+    <version>10.0.0</version>
     <relativePath/>
   </parent>
 
index 8c5f5b363598c6520642681c15018d04e724509f..bbe23f17f42c37ccf9d99aec8d3247266a2f3bf5 100644 (file)
@@ -11,7 +11,7 @@ and is available at http://www.eclipse.org/legal/epl-v10.html
   <parent>
     <groupId>org.opendaylight.controller</groupId>
     <artifactId>mdsal-it-parent</artifactId>
-    <version>4.0.10</version>
+    <version>5.0.1</version>
     <relativePath/>
   </parent>
 
index 8e84a68590a579677d2ba2ed9804616b634f8d7b..f78241564f3d52e93e8621f983dc52fc4b073489 100644 (file)
@@ -33,6 +33,7 @@ import static org.ops4j.pax.exam.karaf.options.KarafDistributionOption.configure
 import static org.ops4j.pax.exam.karaf.options.KarafDistributionOption.editConfigurationFilePut;
 import static org.ops4j.pax.exam.karaf.options.KarafDistributionOption.keepRuntimeFolder;
 
+import com.google.common.collect.ImmutableSet;
 import com.google.common.collect.Lists;
 import com.google.common.collect.Sets;
 import java.io.File;
@@ -47,6 +48,7 @@ import java.nio.ByteBuffer;
 import java.util.ArrayList;
 import java.util.Arrays;
 import java.util.HashSet;
+import java.util.Iterator;
 import java.util.List;
 import java.util.Set;
 import javax.inject.Inject;
@@ -505,9 +507,9 @@ public class MappingServiceIntegrationTest extends AbstractMdsalTestBase {
         final MapReply mapReply1 = lms.handleMapRequest(
                 new MapRequestBuilder(requests1.get(0))
                         .setSourceEid(new SourceEidBuilder().setEid(subscriberEid).build())
-                        .setItrRloc(Lists.newArrayList(new ItrRlocBuilder()
+                        .setItrRloc(List.of(new ItrRlocBuilder()
                                 .setRloc(LispAddressUtil.asIpv4Rloc(subscriberSrcRloc1)).build()))
-                        .setEidItem(Lists.newArrayList(new EidItemBuilder().setEid(eid1).build()))
+                        .setEidItem(List.of(new EidItemBuilder().setEid(eid1).build()))
                         .setSmrInvoked(true)
                         .setSmr(false).build());
 
@@ -517,9 +519,9 @@ public class MappingServiceIntegrationTest extends AbstractMdsalTestBase {
         final MapReply mapReply2 = lms.handleMapRequest(
                 new MapRequestBuilder(requests2.get(0))
                         .setSourceEid(new SourceEidBuilder().setEid(subscriberEid).build())
-                        .setItrRloc(Lists.newArrayList(new ItrRlocBuilder()
+                        .setItrRloc(List.of(new ItrRlocBuilder()
                                 .setRloc(LispAddressUtil.asIpv4Rloc(subscriberSrcRloc2)).build()))
-                        .setEidItem(Lists.newArrayList(new EidItemBuilder().setEid(eid1).build()))
+                        .setEidItem(List.of(new EidItemBuilder().setEid(eid1).build()))
                         .setSmrInvoked(true)
                         .setSmr(false).build());
 
@@ -554,7 +556,7 @@ public class MappingServiceIntegrationTest extends AbstractMdsalTestBase {
             LOG.error("Unknown address {}.", address, e);
         }
 
-        final List<MapRequest> requests = Lists.newArrayList();
+        final List<MapRequest> requests = new ArrayList<>();
         byte[][] buffers = reader.getBuffers(expectedSmrs);
         for (byte[] buf : buffers) {
             ByteBuffer packet = ByteBuffer.wrap(buf);
@@ -2595,11 +2597,10 @@ public class MappingServiceIntegrationTest extends AbstractMdsalTestBase {
         cleanUP();
         String macString = "01:02:03:04:05:06";
         String ipString = "10.20.255.30";
-        List<SimpleAddress> addresses = new ArrayList<SimpleAddress>();
-        addresses.add(new SimpleAddress(new IpAddress(new Ipv4Address(ipString))));
-        addresses.add(new SimpleAddress(new MacAddress(macString)));
         AfiListBuilder listbuilder = new AfiListBuilder();
-        listbuilder.setAddressList(addresses);
+        listbuilder.setAddressList(ImmutableSet.of(
+                new SimpleAddress(new IpAddress(new Ipv4Address(ipString))),
+                new SimpleAddress(new MacAddress(macString))));
 
         EidBuilder eb = new EidBuilder();
         eb.setAddressType(AfiListLcaf.class);
@@ -2614,8 +2615,9 @@ public class MappingServiceIntegrationTest extends AbstractMdsalTestBase {
         assertEquals(AfiListLcaf.class, receivedAddress.getAddressType());
 
         AfiList listAddrFromNetwork = (AfiList) receivedAddress.getAddress();
-        SimpleAddress receivedAddr1 = (SimpleAddress) listAddrFromNetwork.getAfiList().getAddressList().get(0);
-        SimpleAddress receivedAddr2 = (SimpleAddress) listAddrFromNetwork.getAfiList().getAddressList().get(1);
+        Iterator<SimpleAddress> addrList = listAddrFromNetwork.getAfiList().getAddressList().iterator();
+        SimpleAddress receivedAddr1 = addrList.next();
+        SimpleAddress receivedAddr2 = addrList.next();
 
         assertNotNull(receivedAddr1.getIpAddress().getIpv4Address());
         assertNotNull(receivedAddr2.getMacAddress());
@@ -2675,8 +2677,8 @@ public class MappingServiceIntegrationTest extends AbstractMdsalTestBase {
         ExplicitLocatorPath receivedAddress = (ExplicitLocatorPath) reply.getMappingRecordItem().get(0)
                 .getMappingRecord().getEid().getAddress();
 
-        Hop receivedHop1 = (Hop) receivedAddress.getExplicitLocatorPath().getHop().get(0);
-        Hop receivedHop2 = (Hop) receivedAddress.getExplicitLocatorPath().getHop().get(1);
+        Hop receivedHop1 = receivedAddress.getExplicitLocatorPath().getHop().get(0);
+        Hop receivedHop2 = receivedAddress.getExplicitLocatorPath().getHop().get(1);
 
         assertEquals(true, receivedHop1.getLrsBits().getLookup());
         assertEquals(false, receivedHop1.getLrsBits().getRlocProbe());
index 102f549ea3077e2714e260938512f31c4cf1f955..670ec1c1d83992df013387ea7e2a2cd3385b68ae 100644 (file)
@@ -12,7 +12,7 @@ and is available at http://www.eclipse.org/legal/epl-v10.html
   <parent>
     <groupId>org.opendaylight.odlparent</groupId>
     <artifactId>karaf4-parent</artifactId>
-    <version>9.0.13</version>
+    <version>10.0.0</version>
     <relativePath/>
   </parent>
 
index dea3de3795f062335cde9c6d71305845e878fca7..58f1832b0e9bade168cb668b69ce89baa696542c 100644 (file)
@@ -5,7 +5,7 @@
   <parent>
     <groupId>org.opendaylight.mdsal</groupId>
     <artifactId>binding-parent</artifactId>
-    <version>8.0.11</version>
+    <version>9.0.1</version>
     <relativePath/>
   </parent>
 
index a9c247ecc36f9f13d8131f1d3533454d1fc685a8..9514fc67f8e91b90ab03146943b7d4d9b061077c 100644 (file)
@@ -17,7 +17,7 @@
   <dependencies>
     <dependency>
       <groupId>org.osgi</groupId>
-      <artifactId>osgi.core</artifactId>
+      <artifactId>org.osgi.framework</artifactId>
     </dependency>
     <dependency>
       <groupId>org.opendaylight.lispflowmapping</groupId>
index a99c77d7cff58a2d41e998c17980bc6ac1cce5b4..932b5005c84ee52ce7abbf5b2130f767eefd2545 100644 (file)
       <groupId>junit-addons</groupId>
       <artifactId>junit-addons</artifactId>
     </dependency>
+    <dependency>
+      <groupId>org.mockito</groupId>
+      <artifactId>mockito-core</artifactId>
+      <version>3.12.4</version>
+      <scope>test</scope>
+    </dependency>
     <dependency>
       <groupId>org.powermock</groupId>
       <artifactId>powermock-api-mockito2</artifactId>
+      <version>2.0.9</version>
+      <scope>test</scope>
     </dependency>
     <dependency>
       <groupId>org.powermock</groupId>
       <artifactId>powermock-module-junit4</artifactId>
+      <version>2.0.9</version>
+      <scope>test</scope>
     </dependency>
   </dependencies>
 </project>
index 91c92d6e60d82a8296ac756463d40228cfbae308..f682977315168066c2146ab652d1c91c740e845f 100644 (file)
@@ -12,6 +12,7 @@ import static java.util.Objects.requireNonNull;
 import com.google.common.util.concurrent.FutureCallback;
 import com.google.common.util.concurrent.ListenableFuture;
 import com.google.common.util.concurrent.MoreExecutors;
+import edu.umd.cs.findbugs.annotations.SuppressFBWarnings;
 import java.util.ArrayList;
 import java.util.Date;
 import java.util.List;
@@ -54,6 +55,7 @@ public class DataStoreBackEnd implements TransactionChainListener {
     private final DataBroker broker;
     private TransactionChain txChain;
 
+    @SuppressFBWarnings(value = "MC_OVERRIDABLE_METHOD_CALL_IN_CONSTRUCTOR", justification = "Non-final for mocking")
     public DataStoreBackEnd(DataBroker broker) {
         this.broker = broker;
         createTransactionChain();
index ae3f9c444c03bfa9223ba104e8a0bc410efbb5ee..1078cff5e5ef40d1bb12164ecb368a73042acc64 100644 (file)
@@ -21,7 +21,7 @@
   <dependencies>
     <dependency>
       <groupId>org.osgi</groupId>
-      <artifactId>osgi.core</artifactId>
+      <artifactId>org.osgi.framework</artifactId>
     </dependency>
     <dependency>
       <groupId>${project.groupId}</groupId>
       <groupId>junit-addons</groupId>
       <artifactId>junit-addons</artifactId>
     </dependency>
+    <dependency>
+      <groupId>org.mockito</groupId>
+      <artifactId>mockito-core</artifactId>
+      <version>3.12.4</version>
+      <scope>test</scope>
+    </dependency>
     <dependency>
       <groupId>org.powermock</groupId>
       <artifactId>powermock-api-mockito2</artifactId>
+      <version>2.0.9</version>
+      <scope>test</scope>
     </dependency>
     <dependency>
       <groupId>org.powermock</groupId>
       <artifactId>powermock-module-junit4</artifactId>
+      <version>2.0.9</version>
+      <scope>test</scope>
     </dependency>
     <dependency>
       <groupId>org.opendaylight.lispflowmapping</groupId>
index 86040aa35fa09f0c48c3930746a1036b4588290a..c4cf63f579aa16b41668c93cc0cb28d4abdb597b 100644 (file)
@@ -9,7 +9,6 @@ package org.opendaylight.lispflowmapping.implementation;
 
 import static java.util.Objects.requireNonNull;
 
-import com.google.common.util.concurrent.Futures;
 import com.google.common.util.concurrent.ListenableFuture;
 import java.util.ArrayList;
 import java.util.List;
@@ -91,7 +90,8 @@ import org.opendaylight.yang.gen.v1.urn.opendaylight.lfm.mappingservice.rev15090
 import org.opendaylight.yang.gen.v1.urn.opendaylight.lfm.mappingservice.rev150906.UpdateMappingOutputBuilder;
 import org.opendaylight.yang.gen.v1.urn.opendaylight.lfm.mappingservice.rev150906.UpdateMappingsInput;
 import org.opendaylight.yang.gen.v1.urn.opendaylight.lfm.mappingservice.rev150906.UpdateMappingsOutput;
-import org.opendaylight.yangtools.yang.common.RpcError;
+import org.opendaylight.yangtools.yang.common.ErrorTag;
+import org.opendaylight.yangtools.yang.common.ErrorType;
 import org.opendaylight.yangtools.yang.common.RpcResult;
 import org.opendaylight.yangtools.yang.common.RpcResultBuilder;
 import org.slf4j.Logger;
@@ -110,8 +110,6 @@ import org.slf4j.LoggerFactory;
  */
 public class MappingService implements OdlMappingserviceService, IMappingService, AutoCloseable {
     private static final Logger LOG = LoggerFactory.getLogger(MappingService.class);
-    private static final String NOT_FOUND_TAG = "data-missing";
-    private static final String DATA_EXISTS_TAG = "data-exists";
 
     private MappingSystem mappingSystem;
     private DataStoreBackEnd dsbe;
@@ -168,7 +166,7 @@ public class MappingService implements OdlMappingserviceService, IMappingService
     @Override
     public ListenableFuture<RpcResult<AddKeyOutput>> addKey(AddKeyInput input) {
         requireNonNull(input, "add-key RPC input must be not null!");
-        LOG.trace("RPC received to add the following key: " + input.toString());
+        LOG.trace("RPC received to add the following key: {}", input);
 
         RpcResultBuilder<AddKeyOutput> rpcResultBuilder;
 
@@ -177,20 +175,20 @@ public class MappingService implements OdlMappingserviceService, IMappingService
         if (key != null) {
             String message = "Key already exists! Please use update-key if you want to change it.";
             rpcResultBuilder = RpcResultBuilder.<AddKeyOutput>failed()
-                    .withError(RpcError.ErrorType.PROTOCOL, DATA_EXISTS_TAG, message);
-            return Futures.immediateFuture(rpcResultBuilder.build());
+                    .withError(ErrorType.PROTOCOL, ErrorTag.DATA_EXISTS, message);
+            return rpcResultBuilder.buildFuture();
         }
 
         dsbe.addAuthenticationKey(RPCInputConvertorUtil.toAuthenticationKey(input));
         rpcResultBuilder = RpcResultBuilder.success(new AddKeyOutputBuilder().build());
 
-        return Futures.immediateFuture(rpcResultBuilder.build());
+        return rpcResultBuilder.buildFuture();
     }
 
     @Override
     public ListenableFuture<RpcResult<AddMappingOutput>> addMapping(AddMappingInput input) {
         requireNonNull(input, "add-mapping RPC input must be not null!");
-        LOG.trace("RPC received to add the following mapping: " + input.toString());
+        LOG.trace("RPC received to add the following mapping: {}", input);
 
         dsbe.addMapping(RPCInputConvertorUtil.toMapping(input));
 
@@ -198,7 +196,7 @@ public class MappingService implements OdlMappingserviceService, IMappingService
 
         rpcResultBuilder = RpcResultBuilder.success(new AddMappingOutputBuilder().build());
 
-        return Futures.immediateFuture(rpcResultBuilder.build());
+        return rpcResultBuilder.buildFuture();
     }
 
     @Override
@@ -230,7 +228,7 @@ public class MappingService implements OdlMappingserviceService, IMappingService
     @Override
     public ListenableFuture<RpcResult<GetKeyOutput>> getKey(GetKeyInput input) {
         requireNonNull(input, "get-key RPC input must be not null!");
-        LOG.trace("RPC received to get the following key: " + input.toString());
+        LOG.trace("RPC received to get the following key: {}", input);
 
         RpcResultBuilder<GetKeyOutput> rpcResultBuilder;
 
@@ -239,18 +237,18 @@ public class MappingService implements OdlMappingserviceService, IMappingService
         if (key == null) {
             String message = "Key was not found in the mapping database";
             rpcResultBuilder = RpcResultBuilder.<GetKeyOutput>failed()
-                    .withError(RpcError.ErrorType.APPLICATION, NOT_FOUND_TAG, message);
+                    .withError(ErrorType.APPLICATION, ErrorTag.DATA_MISSING, message);
         } else {
-            rpcResultBuilder = RpcResultBuilder.success(new GetKeyOutputBuilder().setMappingAuthkey(key));
+            rpcResultBuilder = RpcResultBuilder.success(new GetKeyOutputBuilder().setMappingAuthkey(key).build());
         }
 
-        return Futures.immediateFuture(rpcResultBuilder.build());
+        return rpcResultBuilder.buildFuture();
     }
 
     @Override
     public ListenableFuture<RpcResult<GetMappingOutput>> getMapping(GetMappingInput input) {
         requireNonNull(input, "get-mapping RPC input must be not null!");
-        LOG.trace("RPC received to get the following mapping: " + input.toString());
+        LOG.trace("RPC received to get the following mapping: {}", input);
 
         RpcResultBuilder<GetMappingOutput> rpcResultBuilder;
 
@@ -259,13 +257,14 @@ public class MappingService implements OdlMappingserviceService, IMappingService
         if (reply == null) {
             String message = "No mapping was found in the mapping database";
             rpcResultBuilder = RpcResultBuilder.<GetMappingOutput>failed()
-                    .withError(RpcError.ErrorType.APPLICATION, NOT_FOUND_TAG, message);
+                    .withError(ErrorType.APPLICATION, ErrorTag.DATA_MISSING, message);
         } else {
             final MappingRecord convertedReply = convertFromBinaryIfNecessary(reply.getRecord());
-            rpcResultBuilder = RpcResultBuilder.success(new GetMappingOutputBuilder().setMappingRecord(convertedReply));
+            rpcResultBuilder = RpcResultBuilder.success(
+                new GetMappingOutputBuilder().setMappingRecord(convertedReply).build());
         }
 
-        return Futures.immediateFuture(rpcResultBuilder.build());
+        return rpcResultBuilder.buildFuture();
     }
 
     @Override
@@ -296,7 +295,7 @@ public class MappingService implements OdlMappingserviceService, IMappingService
     @Override
     public ListenableFuture<RpcResult<GetMappingWithXtrIdOutput>> getMappingWithXtrId(GetMappingWithXtrIdInput input) {
         requireNonNull(input, "get-mapping RPC input must be not null!");
-        LOG.trace("RPC received to get the following mapping: " + input.toString());
+        LOG.trace("RPC received to get the following mapping: {}", input);
 
         RpcResultBuilder<GetMappingWithXtrIdOutput> rpcResultBuilder;
 
@@ -306,20 +305,20 @@ public class MappingService implements OdlMappingserviceService, IMappingService
         if (reply == null) {
             String message = "No mapping was found in the mapping database";
             rpcResultBuilder = RpcResultBuilder.<GetMappingWithXtrIdOutput>failed()
-                    .withError(RpcError.ErrorType.APPLICATION, NOT_FOUND_TAG, message);
+                    .withError(ErrorType.APPLICATION, ErrorTag.DATA_MISSING, message);
         } else {
             final MappingRecord convertedReply = convertFromBinaryIfNecessary(reply.getRecord());
             rpcResultBuilder = RpcResultBuilder.success(new GetMappingWithXtrIdOutputBuilder()
-                    .setMappingRecord(convertedReply));
+                    .setMappingRecord(convertedReply).build());
         }
 
-        return Futures.immediateFuture(rpcResultBuilder.build());
+        return rpcResultBuilder.buildFuture();
     }
 
     @Override
     public ListenableFuture<RpcResult<RemoveKeyOutput>> removeKey(RemoveKeyInput input) {
         requireNonNull(input, "remove-key RPC input must be not null!");
-        LOG.trace("RPC received to remove the following key: " + input.toString());
+        LOG.trace("RPC received to remove the following key: {}", input);
 
         RpcResultBuilder<RemoveKeyOutput> rpcResultBuilder;
 
@@ -327,13 +326,13 @@ public class MappingService implements OdlMappingserviceService, IMappingService
 
         rpcResultBuilder = RpcResultBuilder.success(new RemoveKeyOutputBuilder().build());
 
-        return Futures.immediateFuture(rpcResultBuilder.build());
+        return rpcResultBuilder.buildFuture();
     }
 
     @Override
     public ListenableFuture<RpcResult<RemoveMappingOutput>> removeMapping(RemoveMappingInput input) {
         requireNonNull(input, "remove-mapping RPC input must be not null!");
-        LOG.trace("RPC received to remove the following mapping: " + input.toString());
+        LOG.trace("RPC received to remove the following mapping: {}", input);
 
         RpcResultBuilder<RemoveMappingOutput> rpcResultBuilder;
 
@@ -341,7 +340,7 @@ public class MappingService implements OdlMappingserviceService, IMappingService
 
         rpcResultBuilder = RpcResultBuilder.success(new RemoveMappingOutputBuilder().build());
 
-        return Futures.immediateFuture(rpcResultBuilder.build());
+        return rpcResultBuilder.buildFuture();
     }
 
     @Override
@@ -355,7 +354,7 @@ public class MappingService implements OdlMappingserviceService, IMappingService
     @Override
     public ListenableFuture<RpcResult<UpdateKeyOutput>> updateKey(UpdateKeyInput input) {
         requireNonNull(input, "update-key RPC input must be not null!");
-        LOG.trace("RPC received to update the following key: " + input.toString());
+        LOG.trace("RPC received to update the following key: {}", input);
 
         RpcResultBuilder<UpdateKeyOutput> rpcResultBuilder;
 
@@ -364,19 +363,19 @@ public class MappingService implements OdlMappingserviceService, IMappingService
         if (key == null) {
             String message = "Key doesn't exist! Please use add-key if you want to create a new authentication key.";
             rpcResultBuilder = RpcResultBuilder.<UpdateKeyOutput>failed()
-                    .withError(RpcError.ErrorType.PROTOCOL, NOT_FOUND_TAG, message);
-            return Futures.immediateFuture(rpcResultBuilder.build());
+                    .withError(ErrorType.PROTOCOL, ErrorTag.DATA_MISSING, message);
+            return rpcResultBuilder.buildFuture();
         }
 
         dsbe.updateAuthenticationKey(RPCInputConvertorUtil.toAuthenticationKey(input));
         rpcResultBuilder = RpcResultBuilder.success(new UpdateKeyOutputBuilder().build());
 
-        return Futures.immediateFuture(rpcResultBuilder.build());
+        return rpcResultBuilder.buildFuture();
     }
 
     @Override
     public ListenableFuture<RpcResult<UpdateMappingOutput>> updateMapping(UpdateMappingInput input) {
-        LOG.trace("RPC received to update the following mapping: " + input.toString());
+        LOG.trace("RPC received to update the following mapping: {}", input);
         requireNonNull(input, "update-mapping RPC input must be not null!");
 
         RpcResultBuilder<UpdateMappingOutput> rpcResultBuilder;
@@ -385,7 +384,7 @@ public class MappingService implements OdlMappingserviceService, IMappingService
 
         rpcResultBuilder = RpcResultBuilder.success(new UpdateMappingOutputBuilder().build());
 
-        return Futures.immediateFuture(rpcResultBuilder.build());
+        return rpcResultBuilder.buildFuture();
     }
 
     @Override
@@ -477,7 +476,7 @@ public class MappingService implements OdlMappingserviceService, IMappingService
 
         rpcResultBuilder = RpcResultBuilder.success(new RemoveAllOperationalContentOutputBuilder().build());
 
-        return Futures.immediateFuture(rpcResultBuilder.build());
+        return rpcResultBuilder.buildFuture();
     }
 
     @Override
index 9d70f1c650b3da916c37498c471c7c771c281fab..f17ad7ec7f25519f5c789fd47e247f8a4f921373 100644 (file)
@@ -241,7 +241,7 @@ public class MapServer implements IMapServerAsync, OdlMappingserviceListener, IS
             return;
         }
         final MapRequestBuilder mrb = MapRequestUtil.prepareSMR(eid, LispAddressUtil.toRloc(getLocalAddress()));
-        LOG.trace("Built SMR packet template (EID field will be set later): " + mrb.build().toString());
+        LOG.trace("Built SMR packet template (EID field will be set later): {}", mrb.build());
 
         scheduler.scheduleSmrs(mrb, subscribers.iterator());
     }
@@ -251,7 +251,7 @@ public class MapServer implements IMapServerAsync, OdlMappingserviceListener, IS
             Enumeration<NetworkInterface> interfaces = NetworkInterface.getNetworkInterfaces();
             while (interfaces.hasMoreElements()) {
                 NetworkInterface current = interfaces.nextElement();
-                LOG.trace("Interface " + current.toString());
+                LOG.trace("Interface {}", current);
                 if (!current.isUp() || current.isLoopback() || current.isVirtual()) {
                     continue;
                 }
@@ -300,7 +300,7 @@ public class MapServer implements IMapServerAsync, OdlMappingserviceListener, IS
             while (subscribers.hasNext()) {
                 Subscriber subscriber = subscribers.next();
                 if (subscriber.timedOut()) {
-                    LOG.debug("Lazy removing expired subscriber entry " + subscriber.getString());
+                    LOG.debug("Lazy removing expired subscriber entry {}", subscriber.getString());
                     subscribers.remove();
                 } else {
                     final ScheduledFuture<?> future = executor.scheduleAtFixedRate(new CancellableRunnable(
index 22aa8914855d381312d0172d7cdc3ffb64b1c10a..41495ec7f180b093e1dc7fd2eea3264cf9432651 100644 (file)
@@ -10,8 +10,7 @@ package org.opendaylight.lispflowmapping.implementation.util;
 import static java.util.Objects.requireNonNull;
 
 import edu.umd.cs.findbugs.annotations.Nullable;
-import java.util.Arrays;
-import java.util.List;
+import java.util.Set;
 import org.opendaylight.lispflowmapping.lisp.type.MappingData;
 import org.opendaylight.lispflowmapping.lisp.util.LispAddressStringifier;
 import org.opendaylight.yang.gen.v1.urn.opendaylight.lfm.lisp.proto.rev151105.SiteId;
@@ -41,7 +40,7 @@ public final class DSBEInputUtil {
 
     public static Mapping toMapping(MappingOrigin origin, Eid key, SiteId siteId, MappingData mappingData) {
         MappingRecord record = mappingData != null ? mappingData.getRecord() : null;
-        List<SiteId> siteIds = siteId != null ? Arrays.asList(siteId) : null;
+        Set<SiteId> siteIds = siteId != null ? Set.of(siteId) : null;
         return new MappingBuilder()
                     .setEidUri(new EidUri(LispAddressStringifier.getURIString(key)))
                     .setOrigin(origin)
@@ -53,7 +52,7 @@ public final class DSBEInputUtil {
         MappingRecord record = mappingData != null ? mappingData.getRecord() :
                                                         new MappingRecordBuilder().setEid(key).build();
         SiteId siteId = record != null ? record.getSiteId() : null;
-        List<SiteId> siteIds = siteId != null ? Arrays.asList(siteId) : null;
+        Set<SiteId> siteIds = siteId != null ? Set.of(siteId) : null;
         return new MappingBuilder()
                 .setEidUri(new EidUri(LispAddressStringifier.getURIString(key)))
                 .setOrigin(origin)
index c05660701899b5886ed41a5b4537a98a34c8dce8..3f5155e2b0f3212f9709097506f646732cb2fbd0 100644 (file)
@@ -14,6 +14,7 @@ import java.net.UnknownHostException;
 import java.util.ArrayList;
 import java.util.Iterator;
 import java.util.List;
+import java.util.Set;
 import org.opendaylight.lispflowmapping.lisp.type.LispMessage;
 import org.opendaylight.lispflowmapping.lisp.util.LispAddressStringifier;
 import org.opendaylight.yang.gen.v1.urn.ietf.params.xml.ns.yang.ietf.inet.types.rev130715.IpAddress;
@@ -127,12 +128,9 @@ public final class LispNotificationHelper {
         return mappings;
     }
 
-    public static List<SiteId> getSiteId(MapRegister mapRegister) {
+    public static Set<SiteId> getSiteId(MapRegister mapRegister) {
         if (mapRegister.getXtrSiteIdPresent()) {
-            List<SiteId> siteIds = new ArrayList<>();
-            SiteId siteId = new SiteId(mapRegister.getSiteId());
-            siteIds.add(siteId);
-            return siteIds;
+            return Set.of(new SiteId(mapRegister.getSiteId()));
         } else {
             return null;
         }
index cebcb93d5718386bf21d68f867cc21ad23afe2d1..867d8a874d0d18bf4c6f2f463fdc2e7502d64e3a 100644 (file)
@@ -19,6 +19,7 @@ import java.util.Arrays;
 import java.util.Date;
 import java.util.EnumMap;
 import java.util.List;
+import java.util.Set;
 import org.junit.Before;
 import org.junit.Test;
 import org.junit.runner.RunWith;
@@ -162,7 +163,7 @@ public class MappingSystemTest {
     /**
      * This method changes the lookup policy.
      */
-    private static void setLookupPolicy(IMappingService.LookupPolicy policy) throws NoSuchFieldException,
+    private static void setLookupPolicy(final IMappingService.LookupPolicy policy) throws NoSuchFieldException,
             IllegalAccessException {
         final Field lookupPolicy = ConfigIni.class.getDeclaredField("lookupPolicy");
         lookupPolicy.setAccessible(true);
@@ -182,7 +183,7 @@ public class MappingSystemTest {
         final Mapping mapping = new MappingBuilder()
                 .setEidUri(new EidUri("ipv4:" + IPV4_DST))
                 .setOrigin(MappingOrigin.Southbound)
-                .setSiteId(Lists.newArrayList(SITE_ID))
+                .setSiteId(Set.of(SITE_ID))
                 .setMappingRecord(mappingData.getRecord()).build();
 
         assertNull(mappingSystem.getMapping(EID_IPV4_SRC, EID_IPV4_DST));
@@ -354,7 +355,7 @@ public class MappingSystemTest {
         final Mapping mapping = new MappingBuilder()
                 .setEidUri(new EidUri("ipv4:" + IPV4_DST))
                 .setOrigin(MappingOrigin.Southbound)
-                .setSiteId(Lists.newArrayList(SITE_ID))
+                .setSiteId(Set.of(SITE_ID))
                 .setMappingRecord(mappingRecord).build();
 
         assertEquals(nbMappingData, mappingSystem.getMapping(EID_IPV4_SRC, EID_IPV4_DST));
@@ -594,7 +595,7 @@ public class MappingSystemTest {
                 .setEid(EID_IPV4_DST);
     }
 
-    private static ServicePath getDefaultServicePath(short index) {
+    private static ServicePath getDefaultServicePath(final short index) {
         org.opendaylight.yang.gen.v1.urn.ietf.params.xml.ns.yang.ietf.lisp.address.types.rev151105.lisp.address.address
                 .service.path.ServicePathBuilder servicePathBuilder = new org.opendaylight.yang.gen.v1.urn.ietf.params
                 .xml.ns.yang.ietf.lisp.address.types.rev151105.lisp.address.address.service.path.ServicePathBuilder();
index 66622b28d8afb6737617a353c00366e05de2d5ee..af644bd2a98593912a90aaf0a34eaccd7e52ab35 100644 (file)
@@ -83,6 +83,8 @@ import org.opendaylight.yang.gen.v1.urn.opendaylight.lfm.mappingservice.rev15090
 import org.opendaylight.yang.gen.v1.urn.opendaylight.lfm.mappingservice.rev150906.UpdateMappingOutput;
 import org.opendaylight.yang.gen.v1.urn.opendaylight.lfm.mappingservice.rev150906.UpdateMappingOutputBuilder;
 import org.opendaylight.yang.gen.v1.urn.opendaylight.lfm.mappingservice.rev150906.UpdateMappingsInput;
+import org.opendaylight.yangtools.yang.common.ErrorTag;
+import org.opendaylight.yangtools.yang.common.ErrorType;
 import org.opendaylight.yangtools.yang.common.RpcError;
 import org.opendaylight.yangtools.yang.common.RpcResult;
 import org.opendaylight.yangtools.yang.common.RpcResultBuilder;
@@ -130,7 +132,7 @@ public class MappingServiceTest {
         Mockito.when(mappingSystem.getAuthenticationKey(IPV4_EID)).thenReturn(MAPPING_AUTHKEY);
 
         // input
-        final RpcResult<Object> rpc = RpcResultBuilder.failed().withError(RpcError.ErrorType.PROTOCOL, "data-exists",
+        final RpcResult<Object> rpc = RpcResultBuilder.failed().withError(ErrorType.PROTOCOL, ErrorTag.DATA_EXISTS,
                 "Key already exists! Please use update-key if you want to change it.").build();
         // equals() not implemented int RpcError
         final RpcError error = rpc.getErrors().iterator().next();
@@ -211,7 +213,7 @@ public class MappingServiceTest {
         Mockito.when(mappingSystem.getAuthenticationKey(getKeyInput.getEid())).thenReturn(null);
 
         final RpcResult<Object> rpc = RpcResultBuilder.failed()
-                .withError(RpcError.ErrorType.APPLICATION, "data-missing", "Key was not found in the mapping database")
+                .withError(ErrorType.APPLICATION, ErrorTag.DATA_MISSING, "Key was not found in the mapping database")
                 .build();
         final RpcError error = rpc.getErrors().iterator().next();
 
@@ -245,7 +247,7 @@ public class MappingServiceTest {
         Mockito.when(mappingSystem.getMapping(getMappingInput.getEid())).thenReturn(mappingData);
 
         final RpcResult<GetMappingOutput> rpc = RpcResultBuilder
-                .success(new GetMappingOutputBuilder().setMappingRecord(nonBinaryMappingRecord)).build();
+                .success(new GetMappingOutputBuilder().setMappingRecord(nonBinaryMappingRecord).build()).build();
 
         //result
         final Future<RpcResult<GetMappingOutput>> result = mappingService.getMapping(getMappingInput);
@@ -265,8 +267,8 @@ public class MappingServiceTest {
         final GetMappingInput getMappingInput = new GetMappingInputBuilder().setEid(IPV4_EID).build();
         Mockito.when(mappingSystem.getMapping(getMappingInput.getEid())).thenReturn(null);
 
-        final RpcResult<Object> rpc = RpcResultBuilder.failed().withError(RpcError.ErrorType.APPLICATION,
-                "data-missing", "No mapping was found in the mapping database").build();
+        final RpcResult<Object> rpc = RpcResultBuilder.failed().withError(ErrorType.APPLICATION, ErrorTag.DATA_MISSING,
+                "No mapping was found in the mapping database").build();
         final RpcError error = rpc.getErrors().iterator().next();
 
         //result
@@ -340,7 +342,7 @@ public class MappingServiceTest {
         Mockito.when(mappingSystem.getAuthenticationKey(IPV4_EID)).thenReturn(null);
 
         // input
-        final RpcResult<Object> rpc = RpcResultBuilder.failed().withError(RpcError.ErrorType.PROTOCOL, "data-missing",
+        final RpcResult<Object> rpc = RpcResultBuilder.failed().withError(ErrorType.PROTOCOL, ErrorTag.DATA_MISSING,
                 "Key doesn't exist! Please use add-key if you want to create a new authentication key.").build();
         final RpcError error = rpc.getErrors().iterator().next();
 
index 8d036a809a20557447a3b46b7d01ff84a041a997..c3f3a275bb798431d17ef4076cd335132a1c5b59 100644 (file)
@@ -14,6 +14,7 @@ import com.google.common.collect.Lists;
 import com.google.common.net.InetAddresses;
 import java.net.InetAddress;
 import java.net.UnknownHostException;
+import java.util.Set;
 import org.junit.Test;
 import org.opendaylight.lispflowmapping.lisp.type.LispMessage;
 import org.opendaylight.lispflowmapping.lisp.util.LispAddressUtil;
@@ -231,13 +232,13 @@ public class LispNotificationHelperTest {
                 .setMappingRecord(new MappingRecordBuilder().setEid(EID_IPV4).build())
                 .setEidUri(new EidUri("ipv4:" + IPV4_STRING))
                 .setOrigin(MappingOrigin.Southbound)
-                .setSiteId(Lists.newArrayList(SITE_ID));
+                .setSiteId(Set.of(SITE_ID));
 
         final MappingBuilder mappingBuilder_2 = new MappingBuilder()
                 .setMappingRecord(new MappingRecordBuilder().setEid(EID_IPV6).build())
                 .setEidUri(new EidUri("ipv6:" + IPV6_STRING))
                 .setOrigin(MappingOrigin.Southbound)
-                .setSiteId(Lists.newArrayList(SITE_ID));
+                .setSiteId(Set.of(SITE_ID));
 
         assertEquals(Lists.newArrayList(mappingBuilder_1.build(), mappingBuilder_2.build()),
                 LispNotificationHelper.getMapping(getDefaultAddMapping(true)));
@@ -298,7 +299,7 @@ public class LispNotificationHelperTest {
                 .build();
     }
 
-    private static AddMapping getDefaultAddMapping(boolean isXtrSiteIdPresent) {
+    private static AddMapping getDefaultAddMapping(final boolean isXtrSiteIdPresent) {
         final MappingRecordItem mappingRecordItem_1 = new MappingRecordItemBuilder()
                 .withKey(new MappingRecordItemKey(IPV4_STRING))
                 .setMappingRecord(new MappingRecordBuilder().setEid(EID_IPV4).build()).build();
index 476631469c8eb7eb9b16a21537d57eae2a055681..5430999ca7be55d63678be0b1e5058943e2e65c5 100644 (file)
@@ -12,7 +12,7 @@ and is available at http://www.eclipse.org/legal/epl-v10.html
   <parent>
     <groupId>org.opendaylight.mdsal</groupId>
     <artifactId>binding-parent</artifactId>
-    <version>8.0.11</version>
+    <version>9.0.1</version>
     <relativePath/>
   </parent>
 
index c47a6edbdbc685c67947ffa766082c8436664b65..6bf628812f23f17cabdb6314a1bdfa66ce15ee75 100644 (file)
@@ -7,9 +7,9 @@
  */
 package org.opendaylight.lispflowmapping.lisp.serializer.address;
 
+import com.google.common.collect.ImmutableSet;
 import java.nio.ByteBuffer;
-import java.util.ArrayList;
-import java.util.List;
+import java.util.Set;
 import org.opendaylight.lispflowmapping.lisp.type.LispCanonicalAddressFormatEnum;
 import org.opendaylight.yang.gen.v1.urn.ietf.params.xml.ns.yang.iana.afn.safi.rev130704.AddressFamily;
 import org.opendaylight.yang.gen.v1.urn.ietf.params.xml.ns.yang.ietf.lisp.address.types.rev151105.AfiListLcaf;
@@ -42,8 +42,8 @@ public final class AfiListSerializer extends LcafSerializer {
     @Override
     protected short getLcafLength(LispAddress lispAddress) {
         short totalSize = 0;
-        AfiList afiList = (((org.opendaylight.yang.gen.v1.urn.ietf.params.xml.ns.yang.ietf.lisp.address.types.rev151105
-                .lisp.address.address.AfiList) lispAddress.getAddress()).getAfiList());
+        AfiList afiList = ((org.opendaylight.yang.gen.v1.urn.ietf.params.xml.ns.yang.ietf.lisp.address.types.rev151105
+                .lisp.address.address.AfiList) lispAddress.getAddress()).getAfiList();
         for (SimpleAddress address : afiList.getAddressList()) {
             totalSize += SimpleAddressSerializer.getInstance().getAddressSize(address);
         }
@@ -82,14 +82,21 @@ public final class AfiListSerializer extends LcafSerializer {
         return rb.build();
     }
 
-    private Address deserializeData(ByteBuffer buffer, short lcafLength, LispAddressSerializerContext ctx) {
-        List<SimpleAddress> addresses = new ArrayList<SimpleAddress>();
-        short length = lcafLength;
-        while (length > 0) {
-            SimpleAddress address = SimpleAddressSerializer.getInstance().deserialize(buffer, ctx);
-            length -= SimpleAddressSerializer.getInstance().getAddressSize(address);
-            addresses.add(address);
+    private static Address deserializeData(ByteBuffer buffer, short lcafLength, LispAddressSerializerContext ctx) {
+        final Set<SimpleAddress> addresses;
+        if (lcafLength > 0) {
+            ImmutableSet.Builder<SimpleAddress> builder = ImmutableSet.builderWithExpectedSize(lcafLength);
+            short length = lcafLength;
+            while (length > 0) {
+                SimpleAddress address = SimpleAddressSerializer.getInstance().deserialize(buffer, ctx);
+                length -= SimpleAddressSerializer.getInstance().getAddressSize(address);
+                builder.add(address);
+            }
+            addresses = builder.build();
+        } else {
+            addresses = Set.of();
         }
+
         return new org.opendaylight.yang.gen.v1.urn.ietf.params.xml.ns.yang.ietf.lisp.address.types.rev151105.lisp
                 .address.address.AfiListBuilder().setAfiList(new AfiListBuilder().setAddressList(addresses).build())
                 .build();
index 082652dfc0fea9b063e63f0628b07c0a1b06bc93..6ee4fd91196a35b7b71f46b2719ae14a050b5b30 100644 (file)
@@ -47,7 +47,7 @@ public final class NoAddressSerializer extends LispAddressSerializer {
         EidBuilder eb = new EidBuilder();
         eb.setAddressType(NoAddressAfi.class);
         eb.setVirtualNetworkId(getVni(ctx));
-        eb.setAddress(new NoAddressBuilder().setNoAddress(Empty.getInstance()).build());
+        eb.setAddress(new NoAddressBuilder().setNoAddress(Empty.value()).build());
         return eb.build();
     }
 
@@ -56,7 +56,7 @@ public final class NoAddressSerializer extends LispAddressSerializer {
         RlocBuilder rb = new RlocBuilder();
         rb.setAddressType(NoAddressAfi.class);
         rb.setVirtualNetworkId(null);
-        rb.setAddress(new NoAddressBuilder().setNoAddress(Empty.getInstance()).build());
+        rb.setAddress(new NoAddressBuilder().setNoAddress(Empty.value()).build());
         return rb.build();
     }
 
index 390f5d1bcc0bd5db3a84a6b97998e7bdee35c569..eb110a093d862cccf2e20ac941935762d8b3304e 100644 (file)
@@ -324,10 +324,9 @@ public final class LispAddressStringifier {
         // AFI 16387, LCAF Type 1; Address List
         // Example rendering:
         //    {192.0.2.1,192.0.2.2,2001:db8::1}
-        List<SimpleAddress> addresses = addr.getAfiList().getAddressList();
         StringBuilder sb = new StringBuilder("{");
         boolean needComma = false;
-        for (SimpleAddress a : addresses) {
+        for (SimpleAddress a : addr.getAfiList().getAddressList()) {
             if (needComma) {
                 sb.append(",");
             }
index f7fac320f0b0815dd9eea349e4fccae2fd2d6f31..295d4b85fb85d5d09bd59a26467e6fcea3ae166d 100644 (file)
@@ -770,7 +770,7 @@ public final class LispAddressUtil {
         EidBuilder builder = new EidBuilder();
         builder.setAddressType(NoAddressAfi.class);
         builder.setVirtualNetworkId(null);
-        builder.setAddress(new NoAddressBuilder().setNoAddress(Empty.getInstance()).build());
+        builder.setAddress(new NoAddressBuilder().setNoAddress(Empty.value()).build());
         return builder.build();
     }
 
index f35546ed0b37ee782373ff269d7afcb5f2d60963..90700b6bfd8c8a3f1e67ac1d8b5562eafb37e52f 100644 (file)
@@ -12,7 +12,9 @@ import static org.junit.Assert.assertNull;
 
 import com.google.common.io.BaseEncoding;
 import java.util.ArrayList;
+import java.util.LinkedHashSet;
 import java.util.List;
+import java.util.Set;
 import org.junit.Test;
 import org.opendaylight.yang.gen.v1.urn.ietf.params.xml.ns.yang.ietf.inet.types.rev130715.AsNumber;
 import org.opendaylight.yang.gen.v1.urn.ietf.params.xml.ns.yang.ietf.inet.types.rev130715.IpAddress;
@@ -154,13 +156,14 @@ public class LispAddressStringifierTest {
     private static final LispAddress LISP_AS_NUMBER = new EidBuilder().setAddress(AS_NUMBER_LISP_TYPE).build();
 
     // NoAddress
-    private static final NoAddress NO_ADDRESS = new NoAddressBuilder().setNoAddress(Empty.getInstance()).build();
+    private static final NoAddress NO_ADDRESS = new NoAddressBuilder().setNoAddress(Empty.value()).build();
     private static final LispAddress LISP_NO_ADDRESS = new EidBuilder().setAddress(NO_ADDRESS).build();
 
     // AfiList
     private static final SimpleAddress SIMPLE_ADDRESS_1 = new SimpleAddress(new IpAddress(IPV4_ADDRESS));
     private static final SimpleAddress SIMPLE_ADDRESS_2 = new SimpleAddress(new IpAddress(IPV6_ADDRESS));
-    private static final List<SimpleAddress> ADDRESS_LIST = new ArrayList<>();
+    // FIXME: this is ugly as hell, this Set should be immutable
+    private static final Set<SimpleAddress> ADDRESS_LIST = new LinkedHashSet<>();
     private static final AfiList AFI_LIST = new AfiListBuilder().setAfiList(new org.opendaylight.yang.gen.v1.urn.ietf
             .params.xml.ns.yang.ietf.lisp.address.types.rev151105.lisp.address.address.afi.list.AfiListBuilder()
             .setAddressList(ADDRESS_LIST).build()).build();
index 0793d7c4cbb2fabef3e0fe8886252f3c0a17fb9c..33c0e2af54a0a1cc31592a127acd5325b1f05ffb 100644 (file)
@@ -1275,7 +1275,7 @@ public class LispAddressUtilTest {
         final Eid noAddressEid = LispAddressUtil.getNoAddressEid();
         assertEquals(NoAddressAfi.class, noAddressEid.getAddressType());
         assertNull(noAddressEid.getVirtualNetworkId());
-        assertEquals(((NoAddress) noAddressEid.getAddress()).getNoAddress(), Empty.getInstance());
+        assertEquals(((NoAddress) noAddressEid.getAddress()).getNoAddress(), Empty.value());
     }
 
     /**
index 83d90d6c5b70f29cf918ad6633d7ff0c1bc796ba..864ec1fecdec02d344fdeb247f73bb4015493242 100644 (file)
@@ -9,9 +9,10 @@ package org.opendaylight.lispflowmapping.serializer.address;
 
 import static org.junit.Assert.assertEquals;
 
+import com.google.common.collect.ImmutableSet;
 import java.nio.ByteBuffer;
-import java.util.ArrayList;
-import java.util.List;
+import java.util.Iterator;
+import java.util.Set;
 import junitx.framework.ArrayAssert;
 import junitx.framework.Assert;
 import org.junit.Test;
@@ -38,11 +39,12 @@ public class AfiListSerializerTest extends BaseTestCase {
         assertEquals(AfiListLcaf.class, address.getAddressType());
         AfiList afiList = (AfiList) address.getAddress();
 
-        List<SimpleAddress> addressList = afiList.getAfiList().getAddressList();
+        Set<SimpleAddress> addressList = afiList.getAfiList().getAddressList();
         assertEquals(2, addressList.size());
 
-        assertEquals("170.187.204.221", addressList.get(0).stringValue());
-        assertEquals("1122:3344:1122:3344:1122:3344:1122:3344", addressList.get(1).stringValue());
+        Iterator<SimpleAddress> it = addressList.iterator();
+        assertEquals("170.187.204.221", it.next().stringValue());
+        assertEquals("1122:3344:1122:3344:1122:3344:1122:3344", it.next().stringValue());
     }
 
     @Test
@@ -53,8 +55,7 @@ public class AfiListSerializerTest extends BaseTestCase {
         assertEquals(AfiListLcaf.class, address.getAddressType());
         AfiList afiList = (AfiList) address.getAddress();
 
-        List<SimpleAddress> addressList = afiList.getAfiList().getAddressList();
-        assertEquals(0, addressList.size());
+        assertEquals(Set.of(), afiList.getAfiList().getAddressList());
     }
 
     @Test(expected = LispSerializationException.class)
@@ -74,10 +75,9 @@ public class AfiListSerializerTest extends BaseTestCase {
     @Test
     public void serialize__Simple() throws Exception {
         AfiListBuilder listBuilder = new AfiListBuilder();
-        List<SimpleAddress> addressList = new ArrayList<>();
-        addressList.add(SimpleAddressBuilder.getDefaultInstance("170.187.204.221"));
-        addressList.add(SimpleAddressBuilder.getDefaultInstance("1122:3344:1122:3344:1122:3344:1122:3344"));
-        listBuilder.setAddressList(addressList);
+        listBuilder.setAddressList(ImmutableSet.of(
+            SimpleAddressBuilder.getDefaultInstance("170.187.204.221"),
+            SimpleAddressBuilder.getDefaultInstance("1122:3344:1122:3344:1122:3344:1122:3344")));
 
         RlocBuilder rb = new RlocBuilder();
         rb.setAddressType(AfiListLcaf.class);
@@ -98,8 +98,7 @@ public class AfiListSerializerTest extends BaseTestCase {
     @Test
     public void serialize__NoAddresses() throws Exception {
         AfiListBuilder listBuilder = new AfiListBuilder();
-        List<SimpleAddress> addressList = new ArrayList<>();
-        listBuilder.setAddressList(addressList);
+        listBuilder.setAddressList(Set.of());
 
         RlocBuilder rb = new RlocBuilder();
         rb.setAddressType(AfiListLcaf.class);
@@ -120,17 +119,14 @@ public class AfiListSerializerTest extends BaseTestCase {
         final SimpleAddress ip1 = SimpleAddressBuilder.getDefaultInstance("0:0:0:0:0:0:0:1");
         final SimpleAddress ip2 = SimpleAddressBuilder.getDefaultInstance("0:0:0:0:0:0:0:2");
 
-        AfiListBuilder listBuilder = new AfiListBuilder().setAddressList(new ArrayList<SimpleAddress>());
+        AfiListBuilder listBuilder = new AfiListBuilder().setAddressList(Set.of(ip1));
 
-        listBuilder.getAddressList().add(ip1);
         final org.opendaylight.yang.gen.v1.urn.ietf.params.xml.ns.yang.ietf.lisp.address.types.rev151105
                 .lisp.address.address.afi.list.AfiList address1 = listBuilder.build();
-        listBuilder.setAddressList(new ArrayList<SimpleAddress>());
-        listBuilder.getAddressList().add(ip1);
+        listBuilder.setAddressList(Set.of(ip1));
         final org.opendaylight.yang.gen.v1.urn.ietf.params.xml.ns.yang.ietf.lisp.address.types.rev151105
                 .lisp.address.address.afi.list.AfiList address2 = listBuilder.build();
-        listBuilder.setAddressList(new ArrayList<SimpleAddress>());
-        listBuilder.getAddressList().add(ip2);
+        listBuilder.setAddressList(Set.of(ip2));
         final org.opendaylight.yang.gen.v1.urn.ietf.params.xml.ns.yang.ietf.lisp.address.types.rev151105
                 .lisp.address.address.afi.list.AfiList address3 = listBuilder.build();
 
index dede1175fa9c100dba9bab64f02194563caad791..e09b4e47f8e438af3cfd8b50c5ed589b4830a669 100644 (file)
@@ -56,7 +56,7 @@ public class DistinguishedNameSerializerTest extends BaseTestCase {
                 + "00 11 64 61 76 69 64 00"), null);
 
         assertEquals(AfiListLcaf.class, address.getAddressType());
-        assertEquals("david", ((AfiList) address.getAddress()).getAfiList().getAddressList().get(0)
+        assertEquals("david", ((AfiList) address.getAddress()).getAfiList().getAddressList().iterator().next()
                 .getDistinguishedNameType().getValue());
 
     }
index de57122416c572786e09b1da38030030f2e31d99..fee9bc83af695710004cd8c06a891abf48637e32 100644 (file)
@@ -11,14 +11,13 @@ import static org.junit.Assert.assertEquals;
 import static org.junit.Assert.assertNull;
 import static org.mockito.Mockito.mock;
 import static org.mockito.Mockito.verify;
-import static org.mockito.Mockito.verifyZeroInteractions;
+import static org.mockito.Mockito.verifyNoInteractions;
 import static org.mockito.Mockito.when;
 
 import java.util.Map;
 import java.util.concurrent.ConcurrentHashMap;
 import org.junit.Before;
 import org.junit.Test;
-import org.mockito.Mockito;
 import org.opendaylight.lispflowmapping.interfaces.dao.ILispDAO;
 import org.opendaylight.lispflowmapping.interfaces.dao.MappingEntry;
 import org.opendaylight.lispflowmapping.interfaces.dao.SubKeys;
@@ -205,8 +204,8 @@ public class MultiTableMapCacheTest {
         when(daoMock.getSpecific(VNI, SubKeys.VNI)).thenReturn(null);
 
         multiTableMapCache.removeMapping(EID_SOURCE_DEST_KEY_TYPE);
-        verifyZeroInteractions(tableDaoMock);
-        verifyZeroInteractions(dbMock);
+        verifyNoInteractions(tableDaoMock);
+        verifyNoInteractions(dbMock);
     }
 
     /**
@@ -300,7 +299,7 @@ public class MultiTableMapCacheTest {
         when(daoMock.getSpecific(VNI, SubKeys.VNI)).thenReturn(null);
         multiTableMapCache.removeData(EID_TEST, SubKeys.RECORD);
 
-        Mockito.verifyZeroInteractions(tableDaoMock);
+        verifyNoInteractions(tableDaoMock);
     }
 
     /**
index e2b77dd1487437e3ab992762f7adad57ba73242b..60dc52dc52c2dc4040bdd3fd9899a70f14e9dbe6 100644 (file)
@@ -5,7 +5,7 @@
   <parent>
     <groupId>org.opendaylight.odlparent</groupId>
     <artifactId>odlparent</artifactId>
-    <version>9.0.13</version>
+    <version>10.0.0</version>
     <relativePath/>
   </parent>
 
@@ -38,7 +38,7 @@
       <dependency>
         <groupId>org.opendaylight.mdsal</groupId>
         <artifactId>mdsal-artifacts</artifactId>
-        <version>8.0.11</version>
+        <version>9.0.1</version>
         <type>pom</type>
         <scope>import</scope>
       </dependency>
index f679dd9d3faddbabeaa120c3999b1cf21ca6fdf0..ae88209c8fced6b74d1978ef8529a84892e59f21 100644 (file)
@@ -17,7 +17,7 @@
   <dependencies>
     <dependency>
       <groupId>org.osgi</groupId>
-      <artifactId>osgi.core</artifactId>
+      <artifactId>org.osgi.framework</artifactId>
     </dependency>
     <dependency>
       <groupId>org.apache.karaf.shell</groupId>
index 487e062f3494c13200eaf71a8e3dedf5f10e93f0..8e3fb4cb245291d8a3963d72bcc5719a3aa7bb31 100644 (file)
@@ -21,7 +21,7 @@
   <dependencies>
     <dependency>
       <groupId>org.osgi</groupId>
-      <artifactId>osgi.core</artifactId>
+      <artifactId>org.osgi.framework</artifactId>
     </dependency>
     <dependency>
       <groupId>${project.groupId}</groupId>
index a6da0553625b6fb0765adb2f456e2b306c3ec7e2..b71906978a5ae05de0bb125c035b0bd32ebee70d 100644 (file)
@@ -40,7 +40,8 @@ import org.opendaylight.yang.gen.v1.urn.opendaylight.lfm.lisp.sb.rev150904.ctrl.
 import org.opendaylight.yang.gen.v1.urn.opendaylight.lfm.lisp.sb.rev150904.ctrl.msg.stats.ControlMessageBuilder;
 import org.opendaylight.yang.gen.v1.urn.opendaylight.lfm.lisp.sb.rev150904.get.stats.output.ControlMessageStatsBuilder;
 import org.opendaylight.yang.gen.v1.urn.opendaylight.lfm.lisp.sb.rev150904.get.stats.output.MapRegisterCacheStatsBuilder;
-import org.opendaylight.yangtools.yang.common.RpcError;
+import org.opendaylight.yangtools.yang.common.ErrorTag;
+import org.opendaylight.yangtools.yang.common.ErrorType;
 import org.opendaylight.yangtools.yang.common.RpcResult;
 import org.opendaylight.yangtools.yang.common.RpcResultBuilder;
 import org.slf4j.Logger;
@@ -134,7 +135,7 @@ public class LispSouthboundRPC implements OdlLispSbService {
 
         if (stats == null) {
             rpcResultBuilder = RpcResultBuilder.<GetStatsOutput>failed()
-                    .withError(RpcError.ErrorType.APPLICATION, "data-missing", "No stats found");
+                    .withError(ErrorType.APPLICATION, ErrorTag.DATA_MISSING, "No stats found");
         } else {
             rpcResultBuilder = RpcResultBuilder.success(createGetStatsOutput(stats));
         }
@@ -149,7 +150,7 @@ public class LispSouthboundRPC implements OdlLispSbService {
 
         if (stats == null) {
             return Futures.immediateFuture(RpcResultBuilder.<ResetStatsOutput>failed()
-                    .withError(RpcError.ErrorType.APPLICATION, "data-missing", "No stats found")
+                    .withError(ErrorType.APPLICATION, ErrorTag.DATA_MISSING, "No stats found")
                     .build());
         } else {
             stats.resetStats();
index 00a29d55fc1071d42d52a36c3f8f9264c296aef0..c175ec99a91775430689c105b5fa92791245e004 100644 (file)
@@ -144,8 +144,8 @@ public class LispSouthboundPluginTest {
         injectField("bootstrap", BOOTSTRAP_MOCK);
         lispSouthboundPlugin.setLispAddress(ADDRESS_1);
 
-        Mockito.verifyZeroInteractions(BOOTSTRAP_MOCK);
-        Mockito.verifyZeroInteractions(channel);
+        Mockito.verifyNoInteractions(BOOTSTRAP_MOCK);
+        Mockito.verifyNoInteractions(channel);
     }
 
     /**
@@ -165,7 +165,7 @@ public class LispSouthboundPluginTest {
     public void shouldListenOnXtrPortTest_false() throws NoSuchFieldException, IllegalAccessException {
         lispSouthboundPlugin.shouldListenOnXtrPort(false);
 
-        Mockito.verifyZeroInteractions(xtrChannel);
+        Mockito.verifyNoInteractions(xtrChannel);
     }
 
     /**
index 06592a9f50cbc116c7960edfbaec6a55dcae15ad..130c75599d3c6fc323bcfac2224b1d4274141c9d 100644 (file)
@@ -46,6 +46,8 @@ import org.opendaylight.yang.gen.v1.urn.opendaylight.lfm.lisp.sb.rev150904.SendM
 import org.opendaylight.yang.gen.v1.urn.opendaylight.lfm.lisp.sb.rev150904.SendMapReplyInput;
 import org.opendaylight.yang.gen.v1.urn.opendaylight.lfm.lisp.sb.rev150904.SendMapRequestInput;
 import org.opendaylight.yang.gen.v1.urn.opendaylight.lfm.lisp.sb.rev150904.get.stats.output.ControlMessageStats;
+import org.opendaylight.yangtools.yang.common.ErrorTag;
+import org.opendaylight.yangtools.yang.common.ErrorType;
 import org.opendaylight.yangtools.yang.common.RpcError;
 import org.opendaylight.yangtools.yang.common.RpcResult;
 import org.opendaylight.yangtools.yang.common.RpcResultBuilder;
@@ -59,7 +61,7 @@ public class LispSouthboundRpcTest {
     private static final RpcResult<Void> RPC_RESULT_FAILURE = RpcResultBuilder.<Void>failed().build();
     private static final RpcResult<Void> RPC_RESULT_SUCCESS = RpcResultBuilder.<Void>success().build();
     private static final RpcResult<GetStatsOutput> RPC_RESULT_ERROR = RpcResultBuilder.<GetStatsOutput>failed()
-            .withError(RpcError.ErrorType.APPLICATION, "data-missing", "No stats found").build();
+            .withError(ErrorType.APPLICATION, ErrorTag.DATA_MISSING, "No stats found").build();
 
     /**
      * Tests {@link LispSouthboundRPC#sendMapNotify} method.
@@ -88,7 +90,7 @@ public class LispSouthboundRpcTest {
         assertEquals(RPC_RESULT_FAILURE.isSuccessful(),
                 lispSouthboundRPC.sendMapNotify(null).get().isSuccessful());
 
-        Mockito.verifyZeroInteractions(lispSouthboundPlugin);
+        Mockito.verifyNoInteractions(lispSouthboundPlugin);
     }
 
     /**
@@ -118,7 +120,7 @@ public class LispSouthboundRpcTest {
         assertEquals(RPC_RESULT_FAILURE.isSuccessful(),
                 lispSouthboundRPC.sendMapReply(null).get().isSuccessful());
 
-        Mockito.verifyZeroInteractions(lispSouthboundPlugin);
+        Mockito.verifyNoInteractions(lispSouthboundPlugin);
     }
 
     /**
@@ -148,7 +150,7 @@ public class LispSouthboundRpcTest {
         assertEquals(RPC_RESULT_FAILURE.isSuccessful(),
                 lispSouthboundRPC.sendMapRequest(null).get().isSuccessful());
 
-        Mockito.verifyZeroInteractions(lispSouthboundPlugin);
+        Mockito.verifyNoInteractions(lispSouthboundPlugin);
     }
 
     /**
@@ -178,7 +180,7 @@ public class LispSouthboundRpcTest {
         assertEquals(RPC_RESULT_FAILURE.isSuccessful(),
                 lispSouthboundRPC.sendMapRegister(null).get().isSuccessful());
 
-        Mockito.verifyZeroInteractions(lispSouthboundPlugin);
+        Mockito.verifyNoInteractions(lispSouthboundPlugin);
     }
 
     /**
index f48adcfaa245f392ea1172fc93f3358d5378ead0..18bd462eea7bc9dcc024b173f5ab5ee997a8849d 100644 (file)
@@ -154,7 +154,7 @@ public class AuthenticationKeyDataListenerTest {
         Mockito.when(mod_nullModType.getModificationType()).thenReturn(null);
 
         authenticationKeyDataListener.onDataTreeChanged(Lists.newArrayList(change_nullModType));
-        Mockito.verifyZeroInteractions(akdbMock);
+        Mockito.verifyNoInteractions(akdbMock);
     }
 
     /**