Code Review
/
controller.git
/ blobdiff
commit
grep
author
committer
pickaxe
?
search:
re
summary
|
shortlog
|
log
|
commit
|
commitdiff
|
review
|
tree
raw
|
inline
| side by side
Add AbstractIdentifiablePayload unit tests
[controller.git]
/
opendaylight
/
md-sal
/
sal-distributed-datastore
/
src
/
test
/
java
/
org
/
opendaylight
/
controller
/
cluster
/
datastore
/
TransactionContextWrapperTest.java
diff --git
a/opendaylight/md-sal/sal-distributed-datastore/src/test/java/org/opendaylight/controller/cluster/datastore/TransactionContextWrapperTest.java
b/opendaylight/md-sal/sal-distributed-datastore/src/test/java/org/opendaylight/controller/cluster/datastore/TransactionContextWrapperTest.java
index 10fee5c6d5258e9440a4a79ede3505897016b161..058968c33992d27dc10e88cc0c76229dedd5d128 100644
(file)
--- a/
opendaylight/md-sal/sal-distributed-datastore/src/test/java/org/opendaylight/controller/cluster/datastore/TransactionContextWrapperTest.java
+++ b/
opendaylight/md-sal/sal-distributed-datastore/src/test/java/org/opendaylight/controller/cluster/datastore/TransactionContextWrapperTest.java
@@
-11,36
+11,33
@@
package org.opendaylight.controller.cluster.datastore;
import static org.junit.Assert.assertEquals;
import static org.mockito.Mockito.doReturn;
import static org.mockito.Mockito.mock;
import static org.junit.Assert.assertEquals;
import static org.mockito.Mockito.doReturn;
import static org.mockito.Mockito.mock;
+
import org.junit.Before;
import org.junit.Test;
import org.mockito.Mock;
import org.mockito.MockitoAnnotations;
import org.junit.Before;
import org.junit.Test;
import org.mockito.Mock;
import org.mockito.MockitoAnnotations;
-import org.opendaylight.controller.cluster.datastore.identifiers.TransactionIdentifier;
import org.opendaylight.controller.cluster.datastore.utils.ActorContext;
public class TransactionContextWrapperTest {
import org.opendaylight.controller.cluster.datastore.utils.ActorContext;
public class TransactionContextWrapperTest {
-
- @Mock
- TransactionIdentifier identifier;
-
@Mock
@Mock
- ActorContext actorContext;
+
private
ActorContext actorContext;
@Mock
@Mock
- TransactionContext transactionContext;
+
private
TransactionContext transactionContext;
- TransactionContextWrapper transactionContextWrapper;
+
private
TransactionContextWrapper transactionContextWrapper;
@Before
@Before
- public void setUp(){
+ public void setUp()
{
MockitoAnnotations.initMocks(this);
doReturn(DatastoreContext.newBuilder().build()).when(actorContext).getDatastoreContext();
MockitoAnnotations.initMocks(this);
doReturn(DatastoreContext.newBuilder().build()).when(actorContext).getDatastoreContext();
- transactionContextWrapper = new TransactionContextWrapper(identifier, actorContext);
+ transactionContextWrapper = new TransactionContextWrapper(MockIdentifiers.transactionIdentifier(
+ TransactionContextWrapperTest.class, "mock"), actorContext);
}
@Test
}
@Test
- public void testExecutePriorTransactionOperations(){
- for
(int i=0;i<100;
i++) {
+ public void testExecutePriorTransactionOperations()
{
+ for
(int i = 0; i < 100;
i++) {
transactionContextWrapper.maybeExecuteTransactionOperation(mock(TransactionOperation.class));
}
assertEquals(901, transactionContextWrapper.getLimiter().availablePermits());
transactionContextWrapper.maybeExecuteTransactionOperation(mock(TransactionOperation.class));
}
assertEquals(901, transactionContextWrapper.getLimiter().availablePermits());
@@
-49,4
+46,4
@@
public class TransactionContextWrapperTest {
assertEquals(1001, transactionContextWrapper.getLimiter().availablePermits());
}
assertEquals(1001, transactionContextWrapper.getLimiter().availablePermits());
}
-}
\ No newline at end of file
+}