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%2FAbstractShardManagerTest.java;h=283e6867bff94997861cd385db9b10d3a9eb78ba;hb=a6af137c30470b86d4bc624d4c48cb686495a182;hp=15e6b0364509f8521b58f2248ec2ee9eed97a410;hpb=5370e8be094b802caa732efb4da5a035c53dc9c6;p=controller.git diff --git a/opendaylight/md-sal/sal-distributed-datastore/src/test/java/org/opendaylight/controller/cluster/datastore/AbstractShardManagerTest.java b/opendaylight/md-sal/sal-distributed-datastore/src/test/java/org/opendaylight/controller/cluster/datastore/AbstractShardManagerTest.java index 15e6b03645..283e6867bf 100644 --- a/opendaylight/md-sal/sal-distributed-datastore/src/test/java/org/opendaylight/controller/cluster/datastore/AbstractShardManagerTest.java +++ b/opendaylight/md-sal/sal-distributed-datastore/src/test/java/org/opendaylight/controller/cluster/datastore/AbstractShardManagerTest.java @@ -8,16 +8,12 @@ package org.opendaylight.controller.cluster.datastore; +import static org.mockito.Mockito.mock; import static org.mockito.MockitoAnnotations.initMocks; import akka.actor.ActorRef; -import akka.actor.ActorSystem; import akka.actor.PoisonPill; import akka.actor.Props; -import akka.testkit.JavaTestKit; -import akka.testkit.TestActorRef; -import java.util.ArrayList; -import java.util.Collection; import java.util.concurrent.CountDownLatch; import java.util.concurrent.TimeUnit; import org.junit.After; @@ -32,17 +28,16 @@ import org.opendaylight.controller.cluster.raft.utils.InMemoryJournal; import org.opendaylight.controller.cluster.raft.utils.InMemorySnapshotStore; import org.opendaylight.controller.cluster.raft.utils.MessageCollectorActor; -public class AbstractShardManagerTest extends AbstractActorTest { +public class AbstractShardManagerTest extends AbstractClusterRefActorTest { protected static final MemberName MEMBER_1 = MemberName.forName("member-1"); protected static int ID_COUNTER = 1; - protected static TestActorRef mockShardActor; + protected static ActorRef mockShardActor; protected static ShardIdentifier mockShardName; protected final String shardMrgIDSuffix = "config" + ID_COUNTER++; protected final TestActorFactory actorFactory = new TestActorFactory(getSystem()); - protected final Collection actorSystems = new ArrayList<>(); protected final DatastoreContext.Builder datastoreContextBuilder = DatastoreContext.newBuilder() .dataStoreName(shardMrgIDSuffix).shardInitializationTimeout(600, TimeUnit.MILLISECONDS) .shardHeartbeatIntervalInMillis(100).shardElectionTimeoutFactor(6); @@ -51,11 +46,12 @@ public class AbstractShardManagerTest extends AbstractActorTest { protected static CountDownLatch ready; protected TestShardManager.Builder newTestShardMgrBuilder() { - return TestShardManager.builder(datastoreContextBuilder); + return TestShardManager.builder(datastoreContextBuilder).distributedDataStore(mock(DistributedDataStore.class)); } protected TestShardManager.Builder newTestShardMgrBuilder(final Configuration config) { - return TestShardManager.builder(datastoreContextBuilder).configuration(config); + return TestShardManager.builder(datastoreContextBuilder).configuration(config) + .distributedDataStore(mock(DistributedDataStore.class)); } protected Props newShardMgrProps(final Configuration config) { @@ -63,7 +59,7 @@ public class AbstractShardManagerTest extends AbstractActorTest { } @Before - public void setUp() throws Exception { + public void setUp() { initMocks(this); InMemoryJournal.clear(); @@ -71,11 +67,10 @@ public class AbstractShardManagerTest extends AbstractActorTest { if (mockShardActor == null) { mockShardName = ShardIdentifier.create(Shard.DEFAULT_NAME, MEMBER_1, "config"); - mockShardActor = TestActorRef.create(getSystem(), Props.create(MessageCollectorActor.class), - mockShardName.toString()); + mockShardActor = getSystem().actorOf(MessageCollectorActor.props(), mockShardName.toString()); } - mockShardActor.underlyingActor().clear(); + MessageCollectorActor.clearMessages(mockShardActor); } @After @@ -83,10 +78,6 @@ public class AbstractShardManagerTest extends AbstractActorTest { InMemoryJournal.clear(); InMemorySnapshotStore.clear(); - for (final ActorSystem system : actorSystems) { - JavaTestKit.shutdownActorSystem(system, null, Boolean.TRUE); - } - mockShardActor.tell(PoisonPill.getInstance(), ActorRef.noSender()); mockShardActor = null;