1 files changed, 29 insertions, 1 deletions
diff --git a/routing/routing-impl/src/test/java/io/fd/hc2vpp/routing/write/RoutingProtocolCustomizerTest.java b/routing/routing-impl/src/test/java/io/fd/hc2vpp/routing/write/RoutingProtocolCustomizerTest.java
index d734aab66..21a2bdc14 100644
--- a/routing/routing-impl/src/test/java/io/fd/hc2vpp/routing/write/RoutingProtocolCustomizerTest.java
+++ b/routing/routing-impl/src/test/java/io/fd/hc2vpp/routing/write/RoutingProtocolCustomizerTest.java
@@ -17,6 +17,7 @@
 package io.fd.hc2vpp.routing.write;
 
 import static io.fd.hc2vpp.routing.helpers.RoutingRequestTestHelper.ROUTE_PROTOCOL_NAME;
+import static io.fd.hc2vpp.routing.helpers.RoutingRequestTestHelper.ROUTE_PROTOCOL_NAME_2;
 import static org.junit.Assert.assertTrue;
 import static org.junit.Assert.fail;
 
@@ -39,6 +40,7 @@ public class RoutingProtocolCustomizerTest extends WriterCustomizerTest {
 
     private InstanceIdentifier<RoutingProtocol> validId;
     private RoutingProtocol validData;
+    private RoutingProtocol validData2;
     private RoutingProtocol invalidData;
     private RoutingProtocolCustomizer customizer;
     private NamingContext routingProtocolContext;
@@ -56,6 +58,16 @@ public class RoutingProtocolCustomizerTest extends WriterCustomizerTest {
                         .build())
                 .build();
 
+        validData2= new RoutingProtocolBuilder()
+                .setName(ROUTE_PROTOCOL_NAME_2)
+                .setType(Static.class)
+                .addAugmentation(RoutingProtocolVppAttr.class, new RoutingProtocolVppAttrBuilder()
+                        .setVppProtocolAttributes(new VppProtocolAttributesBuilder()
+                                .setPrimaryVrf(new VniReference(1L))
+                                .build())
+                        .build())
+                .build();
+
         invalidData = new RoutingProtocolBuilder()
                 .setType(Direct.class)
                 .build();
@@ -74,12 +86,28 @@ public class RoutingProtocolCustomizerTest extends WriterCustomizerTest {
         }
     }
 
+    /**
+     * Should not fail, just ignore re-mapping same name
+     * */
     @Test
-    public void testWriteIsStaticAllreadyExist() throws WriteFailedException {
+    public void testWriteIsStaticSameAllreadyExist() throws WriteFailedException {
         defineMapping(mappingContext, ROUTE_PROTOCOL_NAME, 1, "routing-protocol-context");
         try {
             customizer.writeCurrentAttributes(validId, validData, writeContext);
         } catch (Exception e) {
+            fail("Test should have passed without throwing exception");
+        }
+    }
+
+    /**
+     * Should fail, because of attempt to map different name to same index
+     * */
+    @Test
+    public void testWriteIsStaticOtherAllreadyExist() throws WriteFailedException {
+        defineMapping(mappingContext, ROUTE_PROTOCOL_NAME, 1, "routing-protocol-context");
+        try {
+            customizer.writeCurrentAttributes(validId, validData2, writeContext);
+        } catch (Exception e) {
             assertTrue(e instanceof IllegalStateException);
             return;
         }