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%2Fdatabroker%2Factors%2Fdds%2FClientTransactionCommitCohortTest.java;fp=opendaylight%2Fmd-sal%2Fsal-distributed-datastore%2Fsrc%2Ftest%2Fjava%2Forg%2Fopendaylight%2Fcontroller%2Fcluster%2Fdatabroker%2Factors%2Fdds%2FClientTransactionCommitCohortTest.java;h=f9eb6ed6aff1b260e53341a5b454a6a53466c943;hp=294c62a23ca70426c6879d28fdd8ae32c776c31d;hb=808a02471eef0f975c13fc948f933fac34ada387;hpb=85c23944a273c338098997700b14f3a802025bb5 diff --git a/opendaylight/md-sal/sal-distributed-datastore/src/test/java/org/opendaylight/controller/cluster/databroker/actors/dds/ClientTransactionCommitCohortTest.java b/opendaylight/md-sal/sal-distributed-datastore/src/test/java/org/opendaylight/controller/cluster/databroker/actors/dds/ClientTransactionCommitCohortTest.java index 294c62a23c..f9eb6ed6af 100644 --- a/opendaylight/md-sal/sal-distributed-datastore/src/test/java/org/opendaylight/controller/cluster/databroker/actors/dds/ClientTransactionCommitCohortTest.java +++ b/opendaylight/md-sal/sal-distributed-datastore/src/test/java/org/opendaylight/controller/cluster/databroker/actors/dds/ClientTransactionCommitCohortTest.java @@ -56,7 +56,7 @@ public class ClientTransactionCommitCohortTest { @Mock private AbstractClientHistory history; private ActorSystem system; - private List transactions; + private List> transactions; private ClientTransactionCommitCohort cohort; @Before @@ -130,43 +130,43 @@ public class ClientTransactionCommitCohortTest { Assert.assertEquals(PersistenceProtocol.THREE_PHASE, request.getPersistenceProtocol().get()); } - void expectPreCommit(final TransactionTester tester) { + void expectPreCommit(final TransactionTester tester) { tester.expectTransactionRequest(TransactionPreCommitRequest.class); } - void expectCommit(final TransactionTester tester) { + void expectCommit(final TransactionTester tester) { tester.expectTransactionRequest(TransactionDoCommitRequest.class); } - void expectAbort(final TransactionTester tester) { + void expectAbort(final TransactionTester tester) { tester.expectTransactionRequest(TransactionAbortRequest.class); } - void replyCanCommitSuccess(final TransactionTester tester) { + void replyCanCommitSuccess(final TransactionTester tester) { final RequestSuccess success = new TransactionCanCommitSuccess(tester.getTransaction().getIdentifier(), tester.getLastReceivedMessage().getSequence()); tester.replySuccess(success); } - void replyPreCommitSuccess(final TransactionTester tester) { + void replyPreCommitSuccess(final TransactionTester tester) { final RequestSuccess success = new TransactionPreCommitSuccess(tester.getTransaction().getIdentifier(), tester.getLastReceivedMessage().getSequence()); tester.replySuccess(success); } - void replyCommitSuccess(final TransactionTester tester) { + void replyCommitSuccess(final TransactionTester tester) { final RequestSuccess success = new TransactionCommitSuccess(tester.getTransaction().getIdentifier(), tester.getLastReceivedMessage().getSequence()); tester.replySuccess(success); } - void replyAbortSuccess(final TransactionTester tester) { + void replyAbortSuccess(final TransactionTester tester) { final RequestSuccess success = new TransactionAbortSuccess(tester.getTransaction().getIdentifier(), tester.getLastReceivedMessage().getSequence()); tester.replySuccess(success); } - private static TransactionTester createTransactionTester(final TestProbe backendProbe, + private static TransactionTester createTransactionTester(final TestProbe backendProbe, final ClientActorContext context, final AbstractClientHistory history) { final ShardBackendInfo backend = new ShardBackendInfo(backendProbe.ref(), 0L, ABIVersion.BORON, @@ -176,13 +176,12 @@ public class ClientTransactionCommitCohortTest { final ProxyHistory proxyHistory = ProxyHistory.createClient(history, connection, HISTORY_ID); final RemoteProxyTransaction transaction = new RemoteProxyTransaction(proxyHistory, TRANSACTION_ID, false, false); - return new TransactionTester(transaction, connection, backendProbe); + return new TransactionTester<>(transaction, connection, backendProbe); } - private void replySuccess(final Collection transactions, - final Consumer expect, - final Consumer reply) { - for (final TransactionTester transaction : transactions) { + private static > void replySuccess(final Collection transactions, + final Consumer expect, final Consumer reply) { + for (final T transaction : transactions) { expect.accept(transaction); reply.accept(transaction); } @@ -201,8 +200,8 @@ public class ClientTransactionCommitCohortTest { * @throws Exception unexpected exception */ private void testOpSuccess(final Function> operation, - final Consumer expectFunction, - final Consumer replyFunction, + final Consumer> expectFunction, + final Consumer> replyFunction, final T expectedResult) throws Exception { final ListenableFuture result = operation.apply(cohort); replySuccess(transactions, expectFunction, replyFunction); @@ -221,13 +220,13 @@ public class ClientTransactionCommitCohortTest { * @throws Exception unexpected exception */ private void testOpFail(final Function> operation, - final Consumer expectFunction, - final Consumer replyFunction) throws Exception { + final Consumer> expectFunction, + final Consumer> replyFunction) throws Exception { final ListenableFuture canCommit = operation.apply(cohort); //reply success to all except last transaction replySuccess(transactions.subList(0, transactions.size() - 1), expectFunction, replyFunction); //reply fail to last transaction - final TransactionTester last = transactions.get(transactions.size() - 1); + final TransactionTester last = transactions.get(transactions.size() - 1); expectFunction.accept(last); final RuntimeRequestException cause = new RuntimeRequestException("fail", new RuntimeException()); last.replyFailure(cause); @@ -237,4 +236,4 @@ public class ClientTransactionCommitCohortTest { Assert.assertEquals(cause, exception.getCause()); } -} \ No newline at end of file +}