X-Git-Url: https://git.opendaylight.org/gerrit/gitweb?a=blobdiff_plain;ds=sidebyside;f=opendaylight%2Fmd-sal%2Fsal-akka-raft%2Fsrc%2Ftest%2Fjava%2Forg%2Fopendaylight%2Fcontroller%2Fcluster%2Fraft%2FReplicatedLogImplTest.java;h=8e9c6c9271f70d8a2becdcdb89f8cb236a3cdb3b;hb=f33beecf2a10955a9219757529ba3017079816cc;hp=ff4d097010081de5fcf8814fc4e12a9b44fbf396;hpb=913ae866cd0cc82991e1f66ac80f6a42b0daaa48;p=controller.git diff --git a/opendaylight/md-sal/sal-akka-raft/src/test/java/org/opendaylight/controller/cluster/raft/ReplicatedLogImplTest.java b/opendaylight/md-sal/sal-akka-raft/src/test/java/org/opendaylight/controller/cluster/raft/ReplicatedLogImplTest.java index ff4d097010..8e9c6c9271 100644 --- a/opendaylight/md-sal/sal-akka-raft/src/test/java/org/opendaylight/controller/cluster/raft/ReplicatedLogImplTest.java +++ b/opendaylight/md-sal/sal-akka-raft/src/test/java/org/opendaylight/controller/cluster/raft/ReplicatedLogImplTest.java @@ -8,23 +8,23 @@ package org.opendaylight.controller.cluster.raft; import static org.junit.Assert.assertEquals; -import static org.mockito.Matchers.same; +import static org.mockito.ArgumentMatchers.argThat; +import static org.mockito.ArgumentMatchers.same; import static org.mockito.Mockito.doReturn; +import static org.mockito.Mockito.mock; import static org.mockito.Mockito.reset; import static org.mockito.Mockito.verify; import static org.mockito.Mockito.verifyNoMoreInteractions; import akka.japi.Procedure; +import com.google.common.util.concurrent.MoreExecutors; import java.util.Collections; -import org.hamcrest.BaseMatcher; -import org.hamcrest.Description; -import org.hamcrest.Matcher; +import java.util.function.Consumer; import org.junit.Before; import org.junit.Test; import org.mockito.ArgumentCaptor; -import org.mockito.Matchers; +import org.mockito.ArgumentMatcher; import org.mockito.Mock; -import org.mockito.Mockito; import org.mockito.MockitoAnnotations; import org.mockito.internal.matchers.Same; import org.opendaylight.controller.cluster.DataPersistenceProvider; @@ -58,7 +58,7 @@ public class ReplicatedLogImplTest { context = new RaftActorContextImpl(null, null, "test", new ElectionTermImpl(mockPersistence, "test", LOG), -1, -1, Collections.emptyMap(), - configParams, mockPersistence, applyState -> { }, LOG); + configParams, mockPersistence, applyState -> { }, LOG, MoreExecutors.directExecutor()); } private void verifyPersist(Object message) throws Exception { @@ -66,12 +66,12 @@ public class ReplicatedLogImplTest { } @SuppressWarnings({ "unchecked", "rawtypes" }) - private void verifyPersist(Object message, Matcher matcher, boolean async) throws Exception { + private void verifyPersist(Object message, ArgumentMatcher matcher, boolean async) throws Exception { ArgumentCaptor procedure = ArgumentCaptor.forClass(Procedure.class); if (async) { - verify(mockPersistence).persistAsync(Matchers.argThat(matcher), procedure.capture()); + verify(mockPersistence).persistAsync(argThat(matcher), procedure.capture()); } else { - verify(mockPersistence).persist(Matchers.argThat(matcher), procedure.capture()); + verify(mockPersistence).persist(argThat(matcher), procedure.capture()); } procedure.getValue().apply(message); @@ -93,12 +93,12 @@ public class ReplicatedLogImplTest { reset(mockPersistence); ReplicatedLogEntry logEntry2 = new SimpleReplicatedLogEntry(2, 1, new MockPayload("2")); - Procedure mockCallback = Mockito.mock(Procedure.class); + Consumer mockCallback = mock(Consumer.class); log.appendAndPersist(logEntry2, mockCallback, true); verifyPersist(logEntry2); - verify(mockCallback).apply(same(logEntry2)); + verify(mockCallback).accept(same(logEntry2)); assertEquals("size", 2, log.size()); } @@ -108,7 +108,7 @@ public class ReplicatedLogImplTest { public void testAppendAndPersisWithDuplicateEntry() throws Exception { ReplicatedLog log = ReplicatedLogImpl.newInstance(context); - Procedure mockCallback = Mockito.mock(Procedure.class); + Consumer mockCallback = mock(Consumer.class); ReplicatedLogEntry logEntry = new SimpleReplicatedLogEntry(1, 1, new MockPayload("1")); log.appendAndPersist(logEntry, mockCallback, true); @@ -196,18 +196,7 @@ public class ReplicatedLogImplTest { verifyNoMoreInteractions(mockPersistence); } - public Matcher match(final DeleteEntries actual) { - return new BaseMatcher() { - @Override - public boolean matches(Object obj) { - DeleteEntries other = (DeleteEntries) obj; - return actual.getFromIndex() == other.getFromIndex(); - } - - @Override - public void describeTo(Description description) { - description.appendText("DeleteEntries: fromIndex: " + actual.getFromIndex()); - } - }; + public ArgumentMatcher match(final DeleteEntries actual) { + return other -> actual.getFromIndex() == other.getFromIndex(); } }