From a9059a70a875089f5801cbf8e9202bda58b6bcc4 Mon Sep 17 00:00:00 2001 From: "Claudio D. Gasparini" Date: Fri, 9 Nov 2018 13:27:47 +0100 Subject: [PATCH] Do not create empty route list container to keep consistency with mdsal changes Change-Id: Ibd58e7538fbefda96f96d7b3166553c3f8f27181 Signed-off-by: Claudio D. Gasparini --- .../protocol/bgp/evpn/impl/EvpnRibSupportTest.java | 4 +--- .../protocol/bgp/flowspec/FlowspecIpv4RIBSupportTest.java | 4 +--- .../protocol/bgp/flowspec/FlowspecIpv6RIBSupportTest.java | 4 +--- .../bgp/flowspec/FlowspecL3vpnIpv4RIBSupportTest.java | 4 +--- .../bgp/flowspec/FlowspecL3vpnIpv6RIBSupportTest.java | 4 +--- .../opendaylight/protocol/bgp/inet/IPv4RIBSupportTest.java | 5 +---- .../opendaylight/protocol/bgp/inet/IPv6RIBSupportTest.java | 6 ++---- .../bgp/l3vpn/mcast/L3vpnMcastIpv4RIBSupportTest.java | 5 +---- .../bgp/l3vpn/mcast/L3vpnMcastIpv6RIBSupportTest.java | 5 +---- .../bgp/l3vpn/unicast/ipv4/VpnIpv4RIBSupportTest.java | 4 +--- .../bgp/l3vpn/unicast/ipv6/VpnIpv6RIBSupportTest.java | 4 +--- .../labeled/unicast/LabeledUnicastIpv4RIBSupportTest.java | 4 +--- .../labeled/unicast/LabeledUnicastIpv6RIBSupportTest.java | 4 +--- .../protocol/bgp/linkstate/LinkstateRIBSupportTest.java | 4 +--- .../protocol/bgp/mvpn/impl/MvpnIpv4RIBSupportTest.java | 4 +--- .../protocol/bgp/mvpn/impl/MvpnIpv6RIBSupportTest.java | 4 +--- .../impl/RouteTargetConstrainRIBSupportTest.java | 5 +---- .../protocol/bgp/rib/impl/EffectiveRibInWriter.java | 1 - .../protocol/bgp/rib/spi/AbstractRIBSupport.java | 1 - .../org/opendaylight/protocol/bgp/rib/spi/RIBSupport.java | 1 + .../protocol/bgp/rib/spi/AbstractRIBSupportTest.java | 7 +++---- .../opendaylight/protocol/bmp/impl/app/TableContext.java | 6 +----- .../protocol/bmp/impl/app/BmpMonitorImplTest.java | 3 +-- .../protocol/bmp/impl/session/BmpDispatcherImplTest.java | 2 +- 24 files changed, 25 insertions(+), 70 deletions(-) diff --git a/bgp/extensions/evpn/src/test/java/org/opendaylight/protocol/bgp/evpn/impl/EvpnRibSupportTest.java b/bgp/extensions/evpn/src/test/java/org/opendaylight/protocol/bgp/evpn/impl/EvpnRibSupportTest.java index 2c8e524eb2..3667d5b6ec 100644 --- a/bgp/extensions/evpn/src/test/java/org/opendaylight/protocol/bgp/evpn/impl/EvpnRibSupportTest.java +++ b/bgp/extensions/evpn/src/test/java/org/opendaylight/protocol/bgp/evpn/impl/EvpnRibSupportTest.java @@ -114,9 +114,7 @@ public final class EvpnRibSupportTest extends AbstractRIBSupportTest e : tableKey.getKeyValues().entrySet()) { tb.withChild(ImmutableNodes.leafNode(e.getKey(), e.getValue())); } - final ChoiceNode routes - = (ChoiceNode) this.tableSupport.emptyTable().getChild(ROUTES_NODE_ID).get(); tx.put(LogicalDatastoreType.OPERATIONAL, this.tableId, - tb.withChild(ImmutableChoiceNodeBuilder.create(routes).withNodeIdentifier( + tb.withChild(ImmutableChoiceNodeBuilder.create().withNodeIdentifier( new NodeIdentifier(TablesUtil.BMP_ROUTES_QNAME)).build()).build()); } diff --git a/bmp/bmp-impl/src/test/java/org/opendaylight/protocol/bmp/impl/app/BmpMonitorImplTest.java b/bmp/bmp-impl/src/test/java/org/opendaylight/protocol/bmp/impl/app/BmpMonitorImplTest.java index 506d2fb91e..b191fee836 100644 --- a/bmp/bmp-impl/src/test/java/org/opendaylight/protocol/bmp/impl/app/BmpMonitorImplTest.java +++ b/bmp/bmp-impl/src/test/java/org/opendaylight/protocol/bmp/impl/app/BmpMonitorImplTest.java @@ -321,6 +321,7 @@ public class BmpMonitorImplTest extends AbstractConcurrentDataBrokerTest { readDataOperational(getDataBroker(), routerIId, router -> { final List peers = router.getPeer(); + assertNotNull(peers.size()); assertEquals(1, peers.size()); final Peer peer = peers.get(0); assertEquals(PeerType.Global, peer.getType()); @@ -337,7 +338,6 @@ public class BmpMonitorImplTest extends AbstractConcurrentDataBrokerTest { assertEquals(Ipv4AddressFamily.class, prePolicyTable.getAfi()); assertEquals(UnicastSubsequentAddressFamily.class, prePolicyTable.getSafi()); assertFalse(prePolicyTable.getAttributes().isUptodate()); - assertNotNull(prePolicyTable.getRoutes()); assertNotNull(peer.getPostPolicyRib()); assertEquals(1, peer.getPostPolicyRib().getTables().size()); @@ -345,7 +345,6 @@ public class BmpMonitorImplTest extends AbstractConcurrentDataBrokerTest { assertEquals(Ipv4AddressFamily.class, postPolicyTable.getAfi()); assertEquals(UnicastSubsequentAddressFamily.class, postPolicyTable.getSafi()); assertFalse(postPolicyTable.getAttributes().isUptodate()); - assertNotNull(postPolicyTable.getRoutes()); assertNotNull(peer.getPeerSession()); final PeerSession peerSession = peer.getPeerSession(); diff --git a/bmp/bmp-impl/src/test/java/org/opendaylight/protocol/bmp/impl/session/BmpDispatcherImplTest.java b/bmp/bmp-impl/src/test/java/org/opendaylight/protocol/bmp/impl/session/BmpDispatcherImplTest.java index 727d30067c..cb2d6a3cf5 100644 --- a/bmp/bmp-impl/src/test/java/org/opendaylight/protocol/bmp/impl/session/BmpDispatcherImplTest.java +++ b/bmp/bmp-impl/src/test/java/org/opendaylight/protocol/bmp/impl/session/BmpDispatcherImplTest.java @@ -9,7 +9,7 @@ package org.opendaylight.protocol.bmp.impl.session; import static org.junit.Assert.assertTrue; -import static org.mockito.Matchers.any; +import static org.mockito.ArgumentMatchers.any; import static org.mockito.Mockito.doNothing; import static org.mockito.Mockito.timeout; import static org.mockito.Mockito.verify; -- 2.36.6