Repository: james-project Updated Branches: refs/heads/master 45b566136 -> 713a7c545
JAMES-2659 remove useless mock library usage Project: http://git-wip-us.apache.org/repos/asf/james-project/repo Commit: http://git-wip-us.apache.org/repos/asf/james-project/commit/9797d163 Tree: http://git-wip-us.apache.org/repos/asf/james-project/tree/9797d163 Diff: http://git-wip-us.apache.org/repos/asf/james-project/diff/9797d163 Branch: refs/heads/master Commit: 9797d1631444bf61c1f5035944b3e265076aa8d7 Parents: 45b5661 Author: Matthieu Baechler <[email protected]> Authored: Thu Feb 7 15:24:51 2019 +0100 Committer: Matthieu Baechler <[email protected]> Committed: Thu Feb 7 17:10:30 2019 +0100 ---------------------------------------------------------------------- .../events/MailboxListenerRegistryTest.java | 41 ++++++++++---------- 1 file changed, 20 insertions(+), 21 deletions(-) ---------------------------------------------------------------------- http://git-wip-us.apache.org/repos/asf/james-project/blob/9797d163/mailbox/event/event-rabbitmq/src/test/java/org/apache/james/mailbox/events/MailboxListenerRegistryTest.java ---------------------------------------------------------------------- diff --git a/mailbox/event/event-rabbitmq/src/test/java/org/apache/james/mailbox/events/MailboxListenerRegistryTest.java b/mailbox/event/event-rabbitmq/src/test/java/org/apache/james/mailbox/events/MailboxListenerRegistryTest.java index 4033f6f..a63dd21 100644 --- a/mailbox/event/event-rabbitmq/src/test/java/org/apache/james/mailbox/events/MailboxListenerRegistryTest.java +++ b/mailbox/event/event-rabbitmq/src/test/java/org/apache/james/mailbox/events/MailboxListenerRegistryTest.java @@ -20,7 +20,6 @@ package org.apache.james.mailbox.events; import static org.assertj.core.api.Assertions.assertThat; -import static org.mockito.Mockito.mock; import java.time.Duration; import java.util.concurrent.atomic.AtomicBoolean; @@ -52,7 +51,7 @@ class MailboxListenerRegistryTest { @Test void getLocalMailboxListenersShouldReturnPreviouslyAddedListener() { - MailboxListener listener = mock(MailboxListener.class); + MailboxListener listener = event -> {}; testee.addListener(KEY_1, listener, NOOP); assertThat(testee.getLocalMailboxListeners(KEY_1).collectList().block()) @@ -61,8 +60,8 @@ class MailboxListenerRegistryTest { @Test void getLocalMailboxListenersShouldReturnPreviouslyAddedListeners() { - MailboxListener listener1 = mock(MailboxListener.class); - MailboxListener listener2 = mock(MailboxListener.class); + MailboxListener listener1 = event -> {}; + MailboxListener listener2 = event -> {}; testee.addListener(KEY_1, listener1, NOOP); testee.addListener(KEY_1, listener2, NOOP); @@ -72,8 +71,8 @@ class MailboxListenerRegistryTest { @Test void getLocalMailboxListenersShouldNotReturnRemovedListeners() { - MailboxListener listener1 = mock(MailboxListener.class); - MailboxListener listener2 = mock(MailboxListener.class); + MailboxListener listener1 = event -> {}; + MailboxListener listener2 = event -> {}; testee.addListener(KEY_1, listener1, NOOP); testee.addListener(KEY_1, listener2, NOOP); @@ -85,7 +84,7 @@ class MailboxListenerRegistryTest { @Test void addListenerShouldRunTaskWhenNoPreviouslyRegisteredListeners() { - MailboxListener listener = mock(MailboxListener.class); + MailboxListener listener = event -> {}; AtomicBoolean atomicBoolean = new AtomicBoolean(false); testee.addListener(KEY_1, listener, () -> atomicBoolean.set(true)); @@ -95,7 +94,7 @@ class MailboxListenerRegistryTest { @Test void addListenerShouldNotRunTaskWhenPreviouslyRegisteredListeners() { - MailboxListener listener = mock(MailboxListener.class); + MailboxListener listener = event -> {}; AtomicBoolean atomicBoolean = new AtomicBoolean(false); testee.addListener(KEY_1, listener, NOOP); @@ -106,7 +105,7 @@ class MailboxListenerRegistryTest { @Test void removeListenerShouldNotRunTaskWhenNoListener() { - MailboxListener listener = mock(MailboxListener.class); + MailboxListener listener = event -> {}; AtomicBoolean atomicBoolean = new AtomicBoolean(false); testee.removeListener(KEY_1, listener, () -> atomicBoolean.set(true)); @@ -116,8 +115,8 @@ class MailboxListenerRegistryTest { @Test void removeListenerShouldNotRunTaskWhenSeveralListener() { - MailboxListener listener = mock(MailboxListener.class); - MailboxListener listener2 = mock(MailboxListener.class); + MailboxListener listener = event -> {}; + MailboxListener listener2 = event -> {}; AtomicBoolean atomicBoolean = new AtomicBoolean(false); testee.addListener(KEY_1, listener, NOOP); @@ -129,7 +128,7 @@ class MailboxListenerRegistryTest { @Test void removeListenerShouldRunTaskWhenOneListener() { - MailboxListener listener = mock(MailboxListener.class); + MailboxListener listener = event -> {}; AtomicBoolean atomicBoolean = new AtomicBoolean(false); testee.addListener(KEY_1, listener, NOOP); @@ -144,7 +143,7 @@ class MailboxListenerRegistryTest { @Test void getLocalMailboxListenersShouldReturnPreviousAddedListener() throws Exception { - MailboxListener listener = mock(MailboxListener.class); + MailboxListener listener = event -> {}; ConcurrentTestRunner.builder() .operation((threadNumber, operationNumber) -> testee.addListener(KEY_1, listener, NOOP)) @@ -158,9 +157,9 @@ class MailboxListenerRegistryTest { @Test void getLocalMailboxListenersShouldReturnAllPreviousAddedListeners() throws Exception { - MailboxListener listener1 = mock(MailboxListener.class); - MailboxListener listener2 = mock(MailboxListener.class); - MailboxListener listener3 = mock(MailboxListener.class); + MailboxListener listener1 = event -> {}; + MailboxListener listener2 = event -> {}; + MailboxListener listener3 = event -> {}; ConcurrentTestRunner.builder() .operation((threadNumber, operationNumber) -> { @@ -182,7 +181,7 @@ class MailboxListenerRegistryTest { @Test void getLocalMailboxListenersShouldReturnEmptyWhenRemoveAddedListener() throws Exception { - MailboxListener listener1 = mock(MailboxListener.class); + MailboxListener listener1 = event -> {}; testee.addListener(KEY_1, listener1, NOOP); @@ -199,9 +198,9 @@ class MailboxListenerRegistryTest { @Test void addListenerOnlyRunTaskOnceForEmptyRegistry() throws Exception { - MailboxListener listener1 = mock(MailboxListener.class); - MailboxListener listener2 = mock(MailboxListener.class); - MailboxListener listener3 = mock(MailboxListener.class); + MailboxListener listener1 = event -> {}; + MailboxListener listener2 = event -> {}; + MailboxListener listener3 = event -> {}; AtomicInteger runIfEmptyCount = new AtomicInteger(0); @@ -224,7 +223,7 @@ class MailboxListenerRegistryTest { @Test void removeListenerOnlyRunTaskOnceForEmptyRegistry() throws Exception { - MailboxListener listener1 = mock(MailboxListener.class); + MailboxListener listener1 = event -> {}; AtomicInteger runIfEmptyCount = new AtomicInteger(0); testee.addListener(KEY_1, listener1, NOOP); --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
