This is an automated email from the ASF dual-hosted git repository. rcordier pushed a commit to branch master in repository https://gitbox.apache.org/repos/asf/james-project.git
commit 4d2045b7b9b43c2f6228a6d09d05acd4a56b6683 Author: Rene Cordier <rcord...@linagora.com> AuthorDate: Thu Dec 5 17:01:57 2019 +0700 [Refactoring] Move MessageBatcherTest to JUnit 5 --- .../james/mailbox/store/MessageBatcherTest.java | 31 ++++++++++++---------- 1 file changed, 17 insertions(+), 14 deletions(-) diff --git a/mailbox/store/src/test/java/org/apache/james/mailbox/store/MessageBatcherTest.java b/mailbox/store/src/test/java/org/apache/james/mailbox/store/MessageBatcherTest.java index 1da24d5..a2f214c 100644 --- a/mailbox/store/src/test/java/org/apache/james/mailbox/store/MessageBatcherTest.java +++ b/mailbox/store/src/test/java/org/apache/james/mailbox/store/MessageBatcherTest.java @@ -20,15 +20,16 @@ package org.apache.james.mailbox.store; import static org.assertj.core.api.Assertions.assertThat; +import static org.assertj.core.api.Assertions.assertThatThrownBy; import org.apache.james.mailbox.MessageUid; import org.apache.james.mailbox.exception.MailboxException; import org.apache.james.mailbox.model.MessageRange; -import org.junit.Test; +import org.junit.jupiter.api.Test; import com.google.common.collect.Lists; -public class MessageBatcherTest { +class MessageBatcherTest { private MessageBatcher.BatchedOperation incrementBatcher = messageRange -> Lists.<MessageRange>newArrayList(MessageRange.range( @@ -36,7 +37,7 @@ public class MessageBatcherTest { messageRange.getUidTo().next())); @Test - public void batchMessagesShouldWorkOnSingleRangeMode() throws Exception { + void batchMessagesShouldWorkOnSingleRangeMode() throws Exception { MessageBatcher messageBatcher = new MessageBatcher(0); assertThat(messageBatcher.batchMessages(MessageRange.range(MessageUid.of(1), MessageUid.of(10)), incrementBatcher)) @@ -44,30 +45,32 @@ public class MessageBatcherTest { } @Test - public void batchMessagesShouldWorkWithNonZeroBatchedSize() throws Exception { + void batchMessagesShouldWorkWithNonZeroBatchedSize() throws Exception { MessageBatcher messageBatcher = new MessageBatcher(5); assertThat(messageBatcher.batchMessages(MessageRange.range(MessageUid.of(1), MessageUid.of(10)), incrementBatcher)) .containsOnly(MessageRange.range(MessageUid.of(2), MessageUid.of(6)), MessageRange.range(MessageUid.of(7), MessageUid.of(11))); } - @Test(expected = MailboxException.class) - public void batchMessagesShouldPropagateExceptions() throws Exception { + @Test + void batchMessagesShouldPropagateExceptions() { MessageBatcher messageBatcher = new MessageBatcher(0); - messageBatcher.batchMessages(MessageRange.range(MessageUid.of(1), MessageUid.of(10)), - messageRange -> { - throw new MailboxException(); - }); + assertThatThrownBy(() -> messageBatcher.batchMessages(MessageRange.range(MessageUid.of(1), MessageUid.of(10)), + messageRange -> { + throw new MailboxException(); + })) + .isInstanceOf(MailboxException.class); } - @Test(expected = IllegalArgumentException.class) - public void messageBatcherShouldThrowOnNegativeBatchSize() throws Exception { - new MessageBatcher(-1); + @Test + void messageBatcherShouldThrowOnNegativeBatchSize() { + assertThatThrownBy(() -> new MessageBatcher(-1)) + .isInstanceOf(IllegalArgumentException.class); } @Test - public void getBatchSizeShouldReturnTheBatchSize() { + void getBatchSizeShouldReturnTheBatchSize() { int batchSize = 123; MessageBatcher messageBatcher = new MessageBatcher(batchSize); assertThat(messageBatcher.getBatchSize()).isEqualTo(batchSize); --------------------------------------------------------------------- To unsubscribe, e-mail: server-dev-unsubscr...@james.apache.org For additional commands, e-mail: server-dev-h...@james.apache.org