X-Git-Url: https://git.opendaylight.org/gerrit/gitweb?p=controller.git;a=blobdiff_plain;f=opendaylight%2Fmd-sal%2Fsal-distributed-datastore%2Fsrc%2Ftest%2Fjava%2Forg%2Fopendaylight%2Fcontroller%2Fcluster%2Fdatastore%2FDataChangeListenerTest.java;h=f3dcaa2b709810d169ae1ae1cc0b54c716051fe9;hp=c4ec8b45fc2726cfda7911287b0c59b6b82f9edd;hb=013a6679470bf692753f2e04ab4398c97fd9f5d0;hpb=86f71bd3f15ce34a9181bb01f0f71fc6979917c5 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 c4ec8b45fc..f3dcaa2b70 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 @@ -1,154 +1,142 @@ +/* + * Copyright (c) 2014, 2015 Cisco Systems, Inc. and others. All rights reserved. + * + * This program and the accompanying materials are made available under the + * terms of the Eclipse Public License v1.0 which accompanies this distribution, + * and is available at http://www.eclipse.org/legal/epl-v10.html + */ + package org.opendaylight.controller.cluster.datastore; +import static org.opendaylight.controller.md.cluster.datastore.model.TestModel.TEST_PATH; + import akka.actor.ActorRef; +import akka.actor.DeadLetter; import akka.actor.Props; import akka.testkit.JavaTestKit; +import org.junit.Assert; import org.junit.Test; +import org.mockito.Mockito; import org.opendaylight.controller.cluster.datastore.messages.DataChanged; import org.opendaylight.controller.cluster.datastore.messages.DataChangedReply; 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.data.api.YangInstanceIdentifier; -import org.opendaylight.yangtools.yang.data.api.schema.NormalizedNode; - -import java.util.HashMap; -import java.util.HashSet; -import java.util.Map; -import java.util.Set; - -import static org.junit.Assert.assertNotNull; -import static org.junit.Assert.assertTrue; public class DataChangeListenerTest extends AbstractActorTest { - private static class MockDataChangedEvent implements AsyncDataChangeEvent> { - Map> createdData = new HashMap(); - Map> updatedData = new HashMap(); - Map> originalData = new HashMap(); - - - - @Override - public Map> getCreatedData() { - createdData.put(CompositeModel.FAMILY_PATH, CompositeModel.createFamily()); - return createdData; - } - - @Override - public Map> getUpdatedData() { - updatedData.put(CompositeModel.FAMILY_PATH, CompositeModel.createFamily()); - return updatedData; - - } - - @Override - public Set getRemovedPaths() { - Setids = new HashSet(); - ids.add( CompositeModel.TEST_PATH); - return ids; - } - - @Override - public Map> getOriginalData() { - originalData.put(CompositeModel.FAMILY_PATH, CompositeModel.createFamily()); - return originalData; - } - - @Override public NormalizedNode getOriginalSubtree() { + @SuppressWarnings({ "rawtypes", "unchecked" }) + @Test + public void testDataChangedWhenNotificationsAreEnabled() { + new JavaTestKit(getSystem()) { + { + final AsyncDataChangeEvent mockChangeEvent = Mockito.mock(AsyncDataChangeEvent.class); + final AsyncDataChangeListener mockListener = Mockito.mock(AsyncDataChangeListener.class); + final Props props = DataChangeListener.props(mockListener, TEST_PATH); + final ActorRef subject = getSystem().actorOf(props, "testDataChangedNotificationsEnabled"); + // Let the DataChangeListener know that notifications should be + // enabled + subject.tell(new EnableNotification(true), getRef()); - return originalData.put(CompositeModel.FAMILY_PATH, CompositeModel.createFamily()); - } + subject.tell(new DataChanged(mockChangeEvent), getRef()); - @Override public NormalizedNode getUpdatedSubtree() { + expectMsgClass(DataChangedReply.class); - //fixme: need to have some valid data here - return originalData.put(CompositeModel.FAMILY_PATH, CompositeModel.createFamily()); - } + Mockito.verify(mockListener).onDataChanged(mockChangeEvent); + } + }; } - private class MockDataChangeListener implements AsyncDataChangeListener> { - private boolean gotIt = false; - private AsyncDataChangeEvent> change; - - @Override public void onDataChanged( - AsyncDataChangeEvent> change) { - gotIt = true;this.change=change; - } - - public boolean gotIt() { - return gotIt; - } - public AsyncDataChangeEvent> getChange(){ - return change; - } + @SuppressWarnings({ "rawtypes", "unchecked" }) + @Test + public void testDataChangedWhenNotificationsAreDisabled() { + new JavaTestKit(getSystem()) { + { + final AsyncDataChangeEvent mockChangeEvent = Mockito.mock(AsyncDataChangeEvent.class); + final AsyncDataChangeListener mockListener = Mockito.mock(AsyncDataChangeListener.class); + final Props props = DataChangeListener.props(mockListener, TEST_PATH); + final ActorRef subject = getSystem().actorOf(props, "testDataChangedNotificationsDisabled"); + + subject.tell(new DataChanged(mockChangeEvent), getRef()); + + new Within(duration("1 seconds")) { + @Override + protected void run() { + expectNoMsg(); + + Mockito.verify(mockListener, Mockito.never()) + .onDataChanged(Mockito.any(AsyncDataChangeEvent.class)); + } + }; + } + }; } + @SuppressWarnings({ "rawtypes", "unchecked" }) @Test - public void testDataChangedWhenNotificationsAreEnabled(){ - new JavaTestKit(getSystem()) {{ - final MockDataChangeListener listener = new MockDataChangeListener(); - final Props props = DataChangeListener.props(CompositeModel.createTestContext(),listener,CompositeModel.FAMILY_PATH ); - final ActorRef subject = - getSystem().actorOf(props, "testDataChangedNotificationsEnabled"); - - new Within(duration("1 seconds")) { - protected void run() { - - // Let the DataChangeListener know that notifications should - // be enabled - subject.tell(new EnableNotification(true), getRef()); - - subject.tell( - new DataChanged(CompositeModel.createTestContext(),new MockDataChangedEvent()).toSerializable(), - getRef()); - - final Boolean out = new ExpectMsg(duration("800 millis"), "dataChanged") { - // do not put code outside this method, will run afterwards - protected Boolean match(Object in) { - if (in != null && in.getClass().equals(DataChangedReply.SERIALIZABLE_CLASS)) { - - return true; - } else { - throw noMatch(); - } - } - }.get(); // this extracts the received message - - assertTrue(out); - assertTrue(listener.gotIt()); - assertNotNull(listener.getChange().getCreatedData()); - - expectNoMsg(); + public void testDataChangedWithNoSender() { + new JavaTestKit(getSystem()) { + { + final AsyncDataChangeEvent mockChangeEvent = Mockito.mock(AsyncDataChangeEvent.class); + final AsyncDataChangeListener mockListener = Mockito.mock(AsyncDataChangeListener.class); + final Props props = DataChangeListener.props(mockListener, TEST_PATH); + final ActorRef subject = getSystem().actorOf(props, "testDataChangedWithNoSender"); + + getSystem().eventStream().subscribe(getRef(), DeadLetter.class); + + subject.tell(new DataChanged(mockChangeEvent), ActorRef.noSender()); + + // Make sure no DataChangedReply is sent to DeadLetters. + while (true) { + DeadLetter deadLetter; + try { + deadLetter = expectMsgClass(duration("1 seconds"), DeadLetter.class); + } catch (AssertionError e) { + // Timed out - got no DeadLetter - this is good + break; + } + + // We may get DeadLetters for other messages we don't care + // about. + Assert.assertFalse("Unexpected DataChangedReply", deadLetter.message() instanceof DataChangedReply); } - - - }; - }}; + } + }; } + @SuppressWarnings({ "rawtypes", "unchecked" }) @Test - public void testDataChangedWhenNotificationsAreDisabled(){ - new JavaTestKit(getSystem()) {{ - final MockDataChangeListener listener = new MockDataChangeListener(); - final Props props = DataChangeListener.props(CompositeModel.createTestContext(),listener,CompositeModel.FAMILY_PATH ); - final ActorRef subject = - getSystem().actorOf(props, "testDataChangedNotificationsDisabled"); + public void testDataChangedWithListenerRuntimeEx() { + new JavaTestKit(getSystem()) { + { + final AsyncDataChangeEvent mockChangeEvent1 = Mockito.mock(AsyncDataChangeEvent.class); + final AsyncDataChangeEvent mockChangeEvent2 = Mockito.mock(AsyncDataChangeEvent.class); + final AsyncDataChangeEvent mockChangeEvent3 = Mockito.mock(AsyncDataChangeEvent.class); - new Within(duration("1 seconds")) { - protected void run() { + AsyncDataChangeListener mockListener = Mockito.mock(AsyncDataChangeListener.class); + Mockito.doThrow(new RuntimeException("mock")).when(mockListener).onDataChanged(mockChangeEvent2); - subject.tell( - new DataChanged(CompositeModel.createTestContext(),new MockDataChangedEvent()).toSerializable(), - getRef()); + Props props = DataChangeListener.props(mockListener, TEST_PATH); + ActorRef subject = getSystem().actorOf(props, "testDataChangedWithListenerRuntimeEx"); - expectNoMsg(); - } + // Let the DataChangeListener know that notifications should be + // enabled + subject.tell(new EnableNotification(true), getRef()); + + 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); + } + }; } }