You are viewing a plain text version of this content. The canonical link for it is here.
Posted to server-dev@james.apache.org by ro...@apache.org on 2018/07/12 15:37:42 UTC

[04/11] james-project git commit: JAMES-2465 Migrate MailboxAppenderTest to Junit5

JAMES-2465 Migrate MailboxAppenderTest to Junit5


Project: http://git-wip-us.apache.org/repos/asf/james-project/repo
Commit: http://git-wip-us.apache.org/repos/asf/james-project/commit/f23a1848
Tree: http://git-wip-us.apache.org/repos/asf/james-project/tree/f23a1848
Diff: http://git-wip-us.apache.org/repos/asf/james-project/diff/f23a1848

Branch: refs/heads/master
Commit: f23a18489bcac26f044ee40f4300f2b90febec47
Parents: 68e03ed
Author: benwa <bt...@linagora.com>
Authored: Thu Jul 12 10:57:41 2018 +0700
Committer: benwa <bt...@linagora.com>
Committed: Thu Jul 12 10:57:41 2018 +0700

----------------------------------------------------------------------
 .../mailets/delivery/MailboxAppenderTest.java   | 34 +++++++++-----------
 1 file changed, 15 insertions(+), 19 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/james-project/blob/f23a1848/server/mailet/mailets/src/test/java/org/apache/james/transport/mailets/delivery/MailboxAppenderTest.java
----------------------------------------------------------------------
diff --git a/server/mailet/mailets/src/test/java/org/apache/james/transport/mailets/delivery/MailboxAppenderTest.java b/server/mailet/mailets/src/test/java/org/apache/james/transport/mailets/delivery/MailboxAppenderTest.java
index 866a70e..22e2e9f 100644
--- a/server/mailet/mailets/src/test/java/org/apache/james/transport/mailets/delivery/MailboxAppenderTest.java
+++ b/server/mailet/mailets/src/test/java/org/apache/james/transport/mailets/delivery/MailboxAppenderTest.java
@@ -20,6 +20,7 @@
 package org.apache.james.transport.mailets.delivery;
 
 import static org.assertj.core.api.Assertions.assertThat;
+import static org.assertj.core.api.Assertions.assertThatThrownBy;
 
 import javax.mail.MessagingException;
 import javax.mail.internet.MimeMessage;
@@ -34,11 +35,9 @@ import org.apache.james.mailbox.model.MailboxPath;
 import org.apache.james.mailbox.model.MessageRange;
 import org.apache.james.mailbox.model.MessageResult;
 import org.apache.james.mailbox.model.MessageResultIterator;
-import org.junit.After;
-import org.junit.Before;
-import org.junit.Rule;
-import org.junit.Test;
-import org.junit.rules.ExpectedException;
+import org.junit.jupiter.api.AfterEach;
+import org.junit.jupiter.api.BeforeEach;
+import org.junit.jupiter.api.Test;
 
 public class MailboxAppenderTest {
 
@@ -46,16 +45,14 @@ public class MailboxAppenderTest {
     public static final String FOLDER = "folder";
     public static final String EMPTY_FOLDER = "";
 
-    @Rule public ExpectedException expectedException = ExpectedException.none();
-
     private MailboxAppender testee;
     private MailboxManager mailboxManager;
     private MimeMessage mimeMessage;
     private InMemoryIntegrationResources integrationResources;
     private MailboxSession session;
 
-    @Before
-    public void setUp() throws Exception {
+    @BeforeEach
+    void setUp() throws Exception {
         mimeMessage = MimeMessageBuilder.mimeMessageBuilder()
             .setMultipartWithBodyParts(
                 MimeMessageBuilder.bodyPartBuilder()
@@ -70,13 +67,13 @@ public class MailboxAppenderTest {
         session = mailboxManager.createSystemSession("TEST");
     }
 
-    @After
-    public void cleanUp() throws Exception {
+    @AfterEach
+    void cleanUp() {
         integrationResources.clean();
     }
 
     @Test
-    public void appendShouldAddMessageToDesiredMailbox() throws Exception {
+    void appendShouldAddMessageToDesiredMailbox() throws Exception {
         testee.append(mimeMessage, USER, FOLDER);
 
         MessageResultIterator messages = mailboxManager.getMailbox(MailboxPath.forUser(USER, FOLDER), session)
@@ -86,7 +83,7 @@ public class MailboxAppenderTest {
     }
 
     @Test
-    public void appendShouldAddMessageToDesiredMailboxWhenMailboxExists() throws Exception {
+    void appendShouldAddMessageToDesiredMailboxWhenMailboxExists() throws Exception {
         MailboxPath mailboxPath = MailboxPath.forUser(USER, FOLDER);
         mailboxManager.createMailbox(mailboxPath, session);
 
@@ -99,14 +96,13 @@ public class MailboxAppenderTest {
     }
 
     @Test
-    public void appendShouldNotAppendToEmptyFolder() throws Exception {
-        expectedException.expect(MessagingException.class);
-
-        testee.append(mimeMessage, USER, EMPTY_FOLDER);
+    void appendShouldNotAppendToEmptyFolder() {
+        assertThatThrownBy(() -> testee.append(mimeMessage, USER, EMPTY_FOLDER))
+            .isInstanceOf(MessagingException.class);
     }
 
     @Test
-    public void appendShouldRemovePathSeparatorAsFirstChar() throws Exception {
+    void appendShouldRemovePathSeparatorAsFirstChar() throws Exception {
         testee.append(mimeMessage, USER, "." + FOLDER);
 
         MessageResultIterator messages = mailboxManager.getMailbox(MailboxPath.forUser(USER, FOLDER), session)
@@ -116,7 +112,7 @@ public class MailboxAppenderTest {
     }
 
     @Test
-    public void appendShouldReplaceSlashBySeparator() throws Exception {
+    void appendShouldReplaceSlashBySeparator() throws Exception {
         testee.append(mimeMessage, USER, FOLDER + "/any");
 
         MessageResultIterator messages = mailboxManager.getMailbox(MailboxPath.forUser(USER, FOLDER + ".any"), session)


---------------------------------------------------------------------
To unsubscribe, e-mail: server-dev-unsubscribe@james.apache.org
For additional commands, e-mail: server-dev-help@james.apache.org