X-Git-Url: https://git.opendaylight.org/gerrit/gitweb?a=blobdiff_plain;f=opendaylight%2Fmd-sal%2Fcds-access-client%2Fsrc%2Ftest%2Fjava%2Forg%2Fopendaylight%2Fcontroller%2Fcluster%2Faccess%2Fclient%2FActorBehaviorTest.java;h=0b630e2da8fd7b0769bf2c7e88da51184ac87bbd;hb=refs%2Fchanges%2F49%2F78449%2F2;hp=7431813010bcf633f4002805882254fa0cfad21d;hpb=9e6714ce7693d8e3c00adf72dafce78041d3a7cc;p=controller.git diff --git a/opendaylight/md-sal/cds-access-client/src/test/java/org/opendaylight/controller/cluster/access/client/ActorBehaviorTest.java b/opendaylight/md-sal/cds-access-client/src/test/java/org/opendaylight/controller/cluster/access/client/ActorBehaviorTest.java index 7431813010..0b630e2da8 100644 --- a/opendaylight/md-sal/cds-access-client/src/test/java/org/opendaylight/controller/cluster/access/client/ActorBehaviorTest.java +++ b/opendaylight/md-sal/cds-access-client/src/test/java/org/opendaylight/controller/cluster/access/client/ActorBehaviorTest.java @@ -7,7 +7,7 @@ */ package org.opendaylight.controller.cluster.access.client; -import static org.mockito.Matchers.any; +import static org.mockito.ArgumentMatchers.any; import static org.mockito.Mockito.mock; import static org.mockito.Mockito.timeout; import static org.mockito.Mockito.verify; @@ -19,8 +19,9 @@ import akka.actor.Props; import akka.persistence.Persistence; import akka.persistence.SelectedSnapshot; import akka.persistence.SnapshotMetadata; -import akka.testkit.JavaTestKit; import akka.testkit.TestProbe; +import akka.testkit.javadsl.TestKit; +import com.typesafe.config.ConfigFactory; import java.lang.reflect.Field; import java.util.Optional; import java.util.concurrent.TimeUnit; @@ -32,13 +33,12 @@ import org.opendaylight.controller.cluster.access.concepts.ClientIdentifier; import org.opendaylight.controller.cluster.access.concepts.FrontendIdentifier; import org.opendaylight.controller.cluster.access.concepts.FrontendType; import org.opendaylight.controller.cluster.access.concepts.MemberName; -import scala.concurrent.duration.Duration; import scala.concurrent.duration.FiniteDuration; public class ActorBehaviorTest { private static final String MEMBER_1_FRONTEND_TYPE_1 = "member-1-frontend-type-1"; - private static final FiniteDuration TIMEOUT = Duration.apply(5, TimeUnit.SECONDS); + private static final FiniteDuration TIMEOUT = FiniteDuration.create(5, TimeUnit.SECONDS); private ActorSystem system; private TestProbe probe; @@ -51,7 +51,8 @@ public class ActorBehaviorTest { public void setUp() throws Exception { initialBehavior = createInitialBehaviorMock(); system = ActorSystem.apply("system1"); - final ActorRef storeRef = system.registerExtension(Persistence.lookup()).snapshotStoreFor(null); + final ActorRef storeRef = system.registerExtension(Persistence.lookup()).snapshotStoreFor(null, + ConfigFactory.empty()); probe = new TestProbe(system); storeRef.tell(probe.ref(), ActorRef.noSender()); final MemberName name = MemberName.forName("member-1"); @@ -62,12 +63,12 @@ public class ActorBehaviorTest { } @After - public void tearDown() throws Exception { - JavaTestKit.shutdownActorSystem(system); + public void tearDown() { + TestKit.shutdownActorSystem(system); } @Test - public void testInitialBehavior() throws Exception { + public void testInitialBehavior() { final InternalCommand cmd = mock(InternalCommand.class); when(cmd.execute(any())).thenReturn(initialBehavior); mockedActor.tell(cmd, ActorRef.noSender()); @@ -75,7 +76,7 @@ public class ActorBehaviorTest { } @Test - public void testCommandStashing() throws Exception { + public void testCommandStashing() { system.stop(mockedActor); mockedActor = system.actorOf(MockedActor.props(id, initialBehavior)); final InternalCommand cmd = mock(InternalCommand.class); @@ -90,7 +91,7 @@ public class ActorBehaviorTest { } @Test - public void testRecoveryAfterRestart() throws Exception { + public void testRecoveryAfterRestart() { system.stop(mockedActor); mockedActor = system.actorOf(MockedActor.props(id, initialBehavior)); final MockedSnapshotStore.SaveRequest newSaveRequest = @@ -99,7 +100,7 @@ public class ActorBehaviorTest { } @Test - public void testRecoveryAfterRestartFrontendIdMismatch() throws Exception { + public void testRecoveryAfterRestartFrontendIdMismatch() { system.stop(mockedActor); //start actor again mockedActor = system.actorOf(MockedActor.props(id, initialBehavior)); @@ -116,7 +117,7 @@ public class ActorBehaviorTest { } @Test - public void testRecoveryAfterRestartSaveSnapshotFail() throws Exception { + public void testRecoveryAfterRestartSaveSnapshotFail() { system.stop(mockedActor); mockedActor = system.actorOf(MockedActor.props(id, initialBehavior)); probe.watch(mockedActor); @@ -129,7 +130,7 @@ public class ActorBehaviorTest { } @Test - public void testRecoveryAfterRestartDeleteSnapshotsFail() throws Exception { + public void testRecoveryAfterRestartDeleteSnapshotsFail() { system.stop(mockedActor); mockedActor = system.actorOf(MockedActor.props(id, initialBehavior)); probe.watch(mockedActor); @@ -140,7 +141,7 @@ public class ActorBehaviorTest { probe.expectMsgClass(MockedSnapshotStore.DeleteByCriteriaRequest.class); probe.reply(new RuntimeException("delete failed")); //actor shouldn't terminate - probe.expectNoMsg(); + probe.expectNoMessage(); } @SuppressWarnings("unchecked") @@ -174,12 +175,13 @@ public class ActorBehaviorTest { private static class MockedActor extends AbstractClientActor { private final ClientActorBehavior initialBehavior; + private final ClientActorConfig mockConfig = AccessClientUtil.newMockClientActorConfig(); private static Props props(final FrontendIdentifier frontendId, final ClientActorBehavior initialBehavior) { return Props.create(MockedActor.class, () -> new MockedActor(frontendId, initialBehavior)); } - private MockedActor(final FrontendIdentifier frontendId, final ClientActorBehavior initialBehavior) { + MockedActor(final FrontendIdentifier frontendId, final ClientActorBehavior initialBehavior) { super(frontendId); this.initialBehavior = initialBehavior; } @@ -189,6 +191,10 @@ public class ActorBehaviorTest { return initialBehavior; } + @Override + protected ClientActorConfig getClientActorConfig() { + return mockConfig; + } } -} \ No newline at end of file +}