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%2FThreePhaseCommitCohortProxyTest.java;h=75c93dd5d2fd2de9521d7540da7169fe656c71e9;hb=00ed1c5bf9e630c9f1111deddd118547e479115f;hp=87231f08849ed02398472bb792a40a48753a96be;hpb=30faeb35260541c273a81b8f126b40da94daa825;p=controller.git diff --git a/opendaylight/md-sal/sal-distributed-datastore/src/test/java/org/opendaylight/controller/cluster/datastore/ThreePhaseCommitCohortProxyTest.java b/opendaylight/md-sal/sal-distributed-datastore/src/test/java/org/opendaylight/controller/cluster/datastore/ThreePhaseCommitCohortProxyTest.java index 87231f0884..75c93dd5d2 100644 --- a/opendaylight/md-sal/sal-distributed-datastore/src/test/java/org/opendaylight/controller/cluster/datastore/ThreePhaseCommitCohortProxyTest.java +++ b/opendaylight/md-sal/sal-distributed-datastore/src/test/java/org/opendaylight/controller/cluster/datastore/ThreePhaseCommitCohortProxyTest.java @@ -1,21 +1,21 @@ package org.opendaylight.controller.cluster.datastore; +import static org.junit.Assert.assertEquals; +import static org.junit.Assert.fail; +import static org.mockito.Matchers.any; +import static org.mockito.Matchers.isA; +import static org.mockito.Mockito.doReturn; +import static org.mockito.Mockito.times; +import static org.mockito.Mockito.verify; import akka.actor.ActorPath; import akka.actor.ActorSelection; import akka.actor.Props; import akka.dispatch.Futures; - import com.google.common.collect.Lists; import com.google.common.util.concurrent.ListenableFuture; -import static org.junit.Assert.assertEquals; -import static org.junit.Assert.assertNotNull; -import static org.mockito.Mockito.doReturn; -import static org.mockito.Mockito.any; -import static org.mockito.Mockito.isA; -import static org.mockito.Mockito.mock; -import static org.mockito.Mockito.verify; -import static org.mockito.Mockito.times; - +import java.util.List; +import java.util.concurrent.ExecutionException; +import java.util.concurrent.TimeUnit; import org.junit.Before; import org.junit.Test; import org.mockito.Mock; @@ -31,14 +31,15 @@ import org.opendaylight.controller.cluster.datastore.messages.PreCommitTransacti import org.opendaylight.controller.cluster.datastore.messages.PreCommitTransactionReply; import org.opendaylight.controller.cluster.datastore.messages.SerializableMessage; import org.opendaylight.controller.cluster.datastore.utils.ActorContext; -import org.opendaylight.controller.cluster.datastore.utils.MessageCollectorActor; -import scala.concurrent.duration.FiniteDuration; - -import java.util.List; -import java.util.concurrent.ExecutionException; +import org.opendaylight.controller.cluster.datastore.utils.DoNothingActor; +import scala.concurrent.Future; public class ThreePhaseCommitCohortProxyTest extends AbstractActorTest { + @SuppressWarnings("serial") + static class TestException extends RuntimeException { + } + @Mock private ActorContext actorContext; @@ -49,15 +50,28 @@ public class ThreePhaseCommitCohortProxyTest extends AbstractActorTest { doReturn(getSystem()).when(actorContext).getActorSystem(); } - private ThreePhaseCommitCohortProxy setupProxy(int nCohorts) { - List cohorts = Lists.newArrayList(); + private Future newCohort() { + ActorPath path = getSystem().actorOf(Props.create(DoNothingActor.class)).path(); + ActorSelection actorSelection = getSystem().actorSelection(path); + return Futures.successful(actorSelection); + } + + private final ThreePhaseCommitCohortProxy setupProxy(int nCohorts) throws Exception { + List> cohortFutures = Lists.newArrayList(); for(int i = 1; i <= nCohorts; i++) { - ActorPath path = getSystem().actorOf(Props.create(MessageCollectorActor.class)).path(); - cohorts.add(path); - doReturn(mock(ActorSelection.class)).when(actorContext).actorSelection(path); + cohortFutures.add(newCohort()); } - return new ThreePhaseCommitCohortProxy(actorContext, cohorts, "txn-1"); + return new ThreePhaseCommitCohortProxy(actorContext, cohortFutures, "txn-1"); + } + + private ThreePhaseCommitCohortProxy setupProxyWithFailedCohortPath() + throws Exception { + List> cohortFutures = Lists.newArrayList(); + cohortFutures.add(newCohort()); + cohortFutures.add(Futures.failed(new TestException())); + + return new ThreePhaseCommitCohortProxy(actorContext, cohortFutures, "txn-1"); } private void setupMockActorContext(Class requestType, Object... responses) { @@ -71,13 +85,23 @@ public class ThreePhaseCommitCohortProxyTest extends AbstractActorTest { .successful(((SerializableMessage) responses[i]).toSerializable())); } - stubber.when(actorContext).executeRemoteOperationAsync(any(ActorSelection.class), - isA(requestType), any(FiniteDuration.class)); + stubber.when(actorContext).executeOperationAsync(any(ActorSelection.class), + isA(requestType)); } private void verifyCohortInvocations(int nCohorts, Class requestType) { - verify(actorContext, times(nCohorts)).executeRemoteOperationAsync( - any(ActorSelection.class), isA(requestType), any(FiniteDuration.class)); + verify(actorContext, times(nCohorts)).executeOperationAsync( + any(ActorSelection.class), isA(requestType)); + } + + private void propagateExecutionExceptionCause(ListenableFuture future) throws Throwable { + + try { + future.get(5, TimeUnit.SECONDS); + fail("Expected ExecutionException"); + } catch(ExecutionException e) { + throw e.getCause(); + } } @Test @@ -86,18 +110,18 @@ public class ThreePhaseCommitCohortProxyTest extends AbstractActorTest { ThreePhaseCommitCohortProxy proxy = setupProxy(1); setupMockActorContext(CanCommitTransaction.SERIALIZABLE_CLASS, - new CanCommitTransactionReply(true)); + CanCommitTransactionReply.YES); ListenableFuture future = proxy.canCommit(); - assertEquals("canCommit", true, future.get()); + assertEquals("canCommit", true, future.get(5, TimeUnit.SECONDS)); setupMockActorContext(CanCommitTransaction.SERIALIZABLE_CLASS, - new CanCommitTransactionReply(false)); + CanCommitTransactionReply.NO); future = proxy.canCommit(); - assertEquals("canCommit", false, future.get()); + assertEquals("canCommit", false, future.get(5, TimeUnit.SECONDS)); verifyCohortInvocations(2, CanCommitTransaction.SERIALIZABLE_CLASS); } @@ -108,11 +132,11 @@ public class ThreePhaseCommitCohortProxyTest extends AbstractActorTest { ThreePhaseCommitCohortProxy proxy = setupProxy(2); setupMockActorContext(CanCommitTransaction.SERIALIZABLE_CLASS, - new CanCommitTransactionReply(true), new CanCommitTransactionReply(true)); + CanCommitTransactionReply.YES, CanCommitTransactionReply.YES); ListenableFuture future = proxy.canCommit(); - assertEquals("canCommit", true, future.get()); + assertEquals("canCommit", true, future.get(5, TimeUnit.SECONDS)); verifyCohortInvocations(2, CanCommitTransaction.SERIALIZABLE_CLASS); } @@ -123,24 +147,23 @@ public class ThreePhaseCommitCohortProxyTest extends AbstractActorTest { ThreePhaseCommitCohortProxy proxy = setupProxy(3); setupMockActorContext(CanCommitTransaction.SERIALIZABLE_CLASS, - new CanCommitTransactionReply(true), new CanCommitTransactionReply(false), - new CanCommitTransactionReply(true)); + CanCommitTransactionReply.YES, CanCommitTransactionReply.NO, CanCommitTransactionReply.YES); ListenableFuture future = proxy.canCommit(); - assertEquals("canCommit", false, future.get()); + assertEquals("canCommit", false, future.get(5, TimeUnit.SECONDS)); verifyCohortInvocations(3, CanCommitTransaction.SERIALIZABLE_CLASS); } - @Test(expected = ExecutionException.class) - public void testCanCommitWithExceptionFailure() throws Exception { + @Test(expected = TestException.class) + public void testCanCommitWithExceptionFailure() throws Throwable { ThreePhaseCommitCohortProxy proxy = setupProxy(1); - setupMockActorContext(CanCommitTransaction.SERIALIZABLE_CLASS, new RuntimeException("mock")); + setupMockActorContext(CanCommitTransaction.SERIALIZABLE_CLASS, new TestException()); - proxy.canCommit().get(); + propagateExecutionExceptionCause(proxy.canCommit()); } @Test(expected = ExecutionException.class) @@ -151,29 +174,30 @@ public class ThreePhaseCommitCohortProxyTest extends AbstractActorTest { setupMockActorContext(CanCommitTransaction.SERIALIZABLE_CLASS, new PreCommitTransactionReply()); - proxy.canCommit().get(); + proxy.canCommit().get(5, TimeUnit.SECONDS); } - @Test - public void testPreCommit() throws Exception { - ThreePhaseCommitCohortProxy proxy = setupProxy(1); + @Test(expected = TestException.class) + public void testCanCommitWithFailedCohortPath() throws Throwable { - setupMockActorContext(PreCommitTransaction.SERIALIZABLE_CLASS, - new PreCommitTransactionReply()); - - proxy.preCommit().get(); + ThreePhaseCommitCohortProxy proxy = setupProxyWithFailedCohortPath(); - verifyCohortInvocations(1, PreCommitTransaction.SERIALIZABLE_CLASS); + try { + propagateExecutionExceptionCause(proxy.canCommit()); + } finally { + verifyCohortInvocations(0, CanCommitTransaction.SERIALIZABLE_CLASS); + } } - @Test(expected = ExecutionException.class) - public void testPreCommitWithFailure() throws Exception { - ThreePhaseCommitCohortProxy proxy = setupProxy(2); + @Test + public void testPreCommit() throws Exception { + // Precommit is currently a no-op + ThreePhaseCommitCohortProxy proxy = setupProxy(1); setupMockActorContext(PreCommitTransaction.SERIALIZABLE_CLASS, - new PreCommitTransactionReply(), new RuntimeException("mock")); + new PreCommitTransactionReply()); - proxy.preCommit().get(); + proxy.preCommit().get(5, TimeUnit.SECONDS); } @Test @@ -182,7 +206,7 @@ public class ThreePhaseCommitCohortProxyTest extends AbstractActorTest { setupMockActorContext(AbortTransaction.SERIALIZABLE_CLASS, new AbortTransactionReply()); - proxy.abort().get(); + proxy.abort().get(5, TimeUnit.SECONDS); verifyCohortInvocations(1, AbortTransaction.SERIALIZABLE_CLASS); } @@ -194,11 +218,22 @@ public class ThreePhaseCommitCohortProxyTest extends AbstractActorTest { setupMockActorContext(AbortTransaction.SERIALIZABLE_CLASS, new RuntimeException("mock")); // The exception should not get propagated. - proxy.abort().get(); + proxy.abort().get(5, TimeUnit.SECONDS); verifyCohortInvocations(1, AbortTransaction.SERIALIZABLE_CLASS); } + @Test + public void testAbortWithFailedCohortPath() throws Throwable { + + ThreePhaseCommitCohortProxy proxy = setupProxyWithFailedCohortPath(); + + // The exception should not get propagated. + proxy.abort().get(5, TimeUnit.SECONDS); + + verifyCohortInvocations(0, AbortTransaction.SERIALIZABLE_CLASS); + } + @Test public void testCommit() throws Exception { @@ -207,39 +242,63 @@ public class ThreePhaseCommitCohortProxyTest extends AbstractActorTest { setupMockActorContext(CommitTransaction.SERIALIZABLE_CLASS, new CommitTransactionReply(), new CommitTransactionReply()); - proxy.commit().get(); + proxy.commit().get(5, TimeUnit.SECONDS); verifyCohortInvocations(2, CommitTransaction.SERIALIZABLE_CLASS); } - @Test(expected = ExecutionException.class) - public void testCommitWithFailure() throws Exception { + @Test(expected = TestException.class) + public void testCommitWithFailure() throws Throwable { ThreePhaseCommitCohortProxy proxy = setupProxy(2); setupMockActorContext(CommitTransaction.SERIALIZABLE_CLASS, new CommitTransactionReply(), - new RuntimeException("mock")); + new TestException()); - proxy.commit().get(); + propagateExecutionExceptionCause(proxy.commit()); } @Test(expected = ExecutionException.class) - public void teseCommitWithInvalidResponseType() throws Exception { + public void testCommitWithInvalidResponseType() throws Exception { ThreePhaseCommitCohortProxy proxy = setupProxy(1); setupMockActorContext(CommitTransaction.SERIALIZABLE_CLASS, new PreCommitTransactionReply()); - proxy.commit().get(); + proxy.commit().get(5, TimeUnit.SECONDS); + } + + @Test(expected = TestException.class) + public void testCommitWithFailedCohortPath() throws Throwable { + + ThreePhaseCommitCohortProxy proxy = setupProxyWithFailedCohortPath(); + + try { + propagateExecutionExceptionCause(proxy.commit()); + } finally { + verifyCohortInvocations(0, CommitTransaction.SERIALIZABLE_CLASS); + } } @Test - public void testGetCohortPaths() { + public void testAllThreePhasesSuccessful() throws Exception { ThreePhaseCommitCohortProxy proxy = setupProxy(2); - List paths = proxy.getCohortPaths(); - assertNotNull("getCohortPaths returned null", paths); - assertEquals("getCohortPaths size", 2, paths.size()); + setupMockActorContext(CanCommitTransaction.SERIALIZABLE_CLASS, + CanCommitTransactionReply.YES, CanCommitTransactionReply.YES); + + setupMockActorContext(PreCommitTransaction.SERIALIZABLE_CLASS, + new PreCommitTransactionReply(), new PreCommitTransactionReply()); + + setupMockActorContext(CommitTransaction.SERIALIZABLE_CLASS, + new CommitTransactionReply(), new CommitTransactionReply()); + + proxy.canCommit().get(5, TimeUnit.SECONDS); + proxy.preCommit().get(5, TimeUnit.SECONDS); + proxy.commit().get(5, TimeUnit.SECONDS); + + verifyCohortInvocations(2, CanCommitTransaction.SERIALIZABLE_CLASS); + verifyCohortInvocations(2, CommitTransaction.SERIALIZABLE_CLASS); } }