X-Git-Url: https://git.opendaylight.org/gerrit/gitweb?a=blobdiff_plain;f=opendaylight%2Fmd-sal%2Fsal-distributed-datastore%2Fsrc%2Ftest%2Fjava%2Forg%2Fopendaylight%2Fcontroller%2Fcluster%2Fdatastore%2FDataChangeListenerTest.java;h=19f0f8c5514663224f1bdb83c083bf202a4a4c93;hb=9c34ce103df5efac991297dc25a64c9b8d6019f3;hp=d5a12c73c57c0deb4fda3be2f11aac435a428bdc;hpb=621ad520d61103e0e5c19cff5a1b00bdd11db86f;p=controller.git diff --git a/opendaylight/md-sal/sal-distributed-datastore/src/test/java/org/opendaylight/controller/cluster/datastore/DataChangeListenerTest.java b/opendaylight/md-sal/sal-distributed-datastore/src/test/java/org/opendaylight/controller/cluster/datastore/DataChangeListenerTest.java index d5a12c73c5..19f0f8c551 100644 --- a/opendaylight/md-sal/sal-distributed-datastore/src/test/java/org/opendaylight/controller/cluster/datastore/DataChangeListenerTest.java +++ b/opendaylight/md-sal/sal-distributed-datastore/src/test/java/org/opendaylight/controller/cluster/datastore/DataChangeListenerTest.java @@ -13,6 +13,7 @@ import org.opendaylight.controller.cluster.datastore.messages.EnableNotification import org.opendaylight.controller.md.cluster.datastore.model.CompositeModel; import org.opendaylight.controller.md.sal.common.api.data.AsyncDataChangeEvent; import org.opendaylight.controller.md.sal.common.api.data.AsyncDataChangeListener; +import org.opendaylight.yangtools.yang.model.api.SchemaContext; public class DataChangeListenerTest extends AbstractActorTest { @@ -28,7 +29,7 @@ public class DataChangeListenerTest extends AbstractActorTest { // Let the DataChangeListener know that notifications should be enabled subject.tell(new EnableNotification(true), getRef()); - subject.tell(new DataChanged(CompositeModel.createTestContext(), mockChangeEvent), + subject.tell(new DataChanged(mockChangeEvent), getRef()); expectMsgClass(DataChangedReply.class); @@ -47,7 +48,7 @@ public class DataChangeListenerTest extends AbstractActorTest { final ActorRef subject = getSystem().actorOf(props, "testDataChangedNotificationsDisabled"); - subject.tell(new DataChanged(CompositeModel.createTestContext(), mockChangeEvent), + subject.tell(new DataChanged(mockChangeEvent), getRef()); new Within(duration("1 seconds")) { @@ -73,8 +74,7 @@ public class DataChangeListenerTest extends AbstractActorTest { getSystem().eventStream().subscribe(getRef(), DeadLetter.class); - subject.tell(new DataChanged(CompositeModel.createTestContext(), mockChangeEvent), - ActorRef.noSender()); + subject.tell(new DataChanged(mockChangeEvent), ActorRef.noSender()); // Make sure no DataChangedReply is sent to DeadLetters. while(true) { @@ -92,4 +92,38 @@ public class DataChangeListenerTest extends AbstractActorTest { } }}; } + + @SuppressWarnings({ "rawtypes", "unchecked" }) + @Test + public void testDataChangedWithListenerRuntimeEx(){ + new JavaTestKit(getSystem()) {{ + AsyncDataChangeEvent mockChangeEvent1 = Mockito.mock(AsyncDataChangeEvent.class); + AsyncDataChangeEvent mockChangeEvent2 = Mockito.mock(AsyncDataChangeEvent.class); + AsyncDataChangeEvent mockChangeEvent3 = Mockito.mock(AsyncDataChangeEvent.class); + + AsyncDataChangeListener mockListener = Mockito.mock(AsyncDataChangeListener.class); + Mockito.doThrow(new RuntimeException("mock")).when(mockListener).onDataChanged(mockChangeEvent2); + + Props props = DataChangeListener.props(mockListener); + ActorRef subject = getSystem().actorOf(props, "testDataChangedWithListenerRuntimeEx"); + + // Let the DataChangeListener know that notifications should be enabled + subject.tell(new EnableNotification(true), getRef()); + + SchemaContext schemaContext = CompositeModel.createTestContext(); + + subject.tell(new DataChanged(mockChangeEvent1),getRef()); + expectMsgClass(DataChangedReply.class); + + subject.tell(new DataChanged(mockChangeEvent2),getRef()); + expectMsgClass(DataChangedReply.class); + + subject.tell(new DataChanged(mockChangeEvent3),getRef()); + expectMsgClass(DataChangedReply.class); + + Mockito.verify(mockListener).onDataChanged(mockChangeEvent1); + Mockito.verify(mockListener).onDataChanged(mockChangeEvent2); + Mockito.verify(mockListener).onDataChanged(mockChangeEvent3); + }}; + } }