Be more defensive when checking nodes

We have a couple of failures, where we end up throwing a NPE
rather than an assertion. Clean them up.

Change-Id: Ib70a04fb398ba4ce789fcef9b08bb90f4b21c967
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

diff --git a/pcep/topology/topology-provider/src/test/java/org/opendaylight/bgpcep/pcep/topology/provider/PCEPTopologySessionListenerTest.java b/pcep/topology/topology-provider/src/test/java/org/opendaylight/bgpcep/pcep/topology/provider/PCEPTopologySessionListenerTest.java
index c9a616b195cf0927a0412e05aee8808e19ba4463..15282bf0b3214510402963374facac33e44f84b7 100644 (file)
--- a/pcep/topology/topology-provider/src/test/java/org/opendaylight/bgpcep/pcep/topology/provider/PCEPTopologySessionListenerTest.java
+++ b/pcep/topology/topology-provider/src/test/java/org/opendaylight/bgpcep/pcep/topology/provider/PCEPTopologySessionListenerTest.java
@@ -396,7 +396,7 @@ public class PCEPTopologySessionListenerTest
                 Optional.of(MsgBuilderUtil.createSrp(srpId)), MsgBuilderUtil.createPath(req.getEro().getSubobject()));
         this.listener.onMessage(this.session, pcRpt);
         readDataOperational(getDataBroker(), TOPO_IID, topology -> {
-            assertEquals(1, topology.getNode().size());
+            assertEquals(1, topology.nonnullNode().size());
             return topology;
         });
     }
@@ -420,7 +420,7 @@ public class PCEPTopologySessionListenerTest
                 Optional.of(MsgBuilderUtil.createSrp(srpId)), MsgBuilderUtil.createPath(req.getEro().getSubobject()));
         this.listener.onMessage(this.session, pcRpt);
         readDataOperational(getDataBroker(), TOPO_IID, topology -> {
-            assertEquals(1, topology.getNode().size());
+            assertEquals(1, topology.nonnullNode().size());
             return topology;
         });
 
@@ -459,7 +459,7 @@ public class PCEPTopologySessionListenerTest
                 Optional.of(MsgBuilderUtil.createSrp(srpId)), MsgBuilderUtil.createPath(req.getEro().getSubobject()));
         this.listener.onMessage(this.session, pcRpt);
         readDataOperational(getDataBroker(), TOPO_IID, topology -> {
-            assertEquals(1, topology.getNode().size());
+            assertEquals(1, topology.nonnullNode().size());
             return topology;
         });
 
@@ -494,7 +494,7 @@ public class PCEPTopologySessionListenerTest
         this.listener.onSessionUp(this.session);
         this.listener.onMessage(this.session, rptmsg);
         readDataOperational(getDataBroker(), TOPO_IID, node -> {
-            assertFalse(node.getNode().isEmpty());
+            assertFalse(node.nonnullNode().isEmpty());
             return node;
         });
     }