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 ma...@apache.org on 2019/11/12 21:37:34 UTC

[james-project] 12/15: [Refactoring] migrate tests to junit5

This is an automated email from the ASF dual-hosted git repository.

matthieu pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/james-project.git

commit 055e487b8f7616b01878ae899a2f394e8bf415dd
Author: Matthieu Baechler <ma...@apache.org>
AuthorDate: Thu Nov 7 22:53:25 2019 +0100

    [Refactoring] migrate tests to junit5
---
 .../james/smtpserver/SetMimeHeaderHandlerTest.java | 26 +++++++++-------------
 1 file changed, 11 insertions(+), 15 deletions(-)

diff --git a/server/protocols/protocols-smtp/src/test/java/org/apache/james/smtpserver/SetMimeHeaderHandlerTest.java b/server/protocols/protocols-smtp/src/test/java/org/apache/james/smtpserver/SetMimeHeaderHandlerTest.java
index ba47a3d..7e018b3 100644
--- a/server/protocols/protocols-smtp/src/test/java/org/apache/james/smtpserver/SetMimeHeaderHandlerTest.java
+++ b/server/protocols/protocols-smtp/src/test/java/org/apache/james/smtpserver/SetMimeHeaderHandlerTest.java
@@ -29,23 +29,19 @@ import org.apache.james.core.builder.MimeMessageBuilder;
 import org.apache.james.protocols.smtp.SMTPSession;
 import org.apache.james.protocols.smtp.utils.BaseFakeSMTPSession;
 import org.apache.james.server.core.MailImpl;
-import org.junit.Before;
-import org.junit.Test;
+import org.junit.jupiter.api.BeforeEach;
+import org.junit.jupiter.api.Test;
 
-public class SetMimeHeaderHandlerTest {
+class SetMimeHeaderHandlerTest {
 
     private static final String HEADER_NAME = "JUNIT";
     private static final String HEADER_VALUE = "test-value";
 
-    private SMTPSession mockedSMTPSession;
+    private SMTPSession fakeSMTPSession;
 
-    @Before
-    public void setUp() throws Exception {
-        setupMockedSMTPSession();
-    }
-
-    private void setupMockedSMTPSession() {
-        mockedSMTPSession = new BaseFakeSMTPSession() {
+    @BeforeEach
+    void setUp() {
+        fakeSMTPSession = new BaseFakeSMTPSession() {
 
             @Override
             public int getRcptCount() {
@@ -55,7 +51,7 @@ public class SetMimeHeaderHandlerTest {
     }
 
     @Test
-    public void setMimeHeaderHandlerShouldAddSpecifiedHeader() throws MessagingException {
+    void setMimeHeaderHandlerShouldAddSpecifiedHeader() throws MessagingException {
         MimeMessage mimeMessage = MimeMessageBuilder.mimeMessageBuilder()
             .setSubject("testmail")
             .setText("testtext")
@@ -73,13 +69,13 @@ public class SetMimeHeaderHandlerTest {
         header.setHeaderName(HEADER_NAME);
         header.setHeaderValue(HEADER_VALUE);
 
-        header.onMessage(mockedSMTPSession, mail);
+        header.onMessage(fakeSMTPSession, mail);
 
         assertThat(mail.getMessage().getHeader(HEADER_NAME)).containsOnly(HEADER_VALUE);
     }
 
     @Test
-    public void setMimeHeaderHandlerShouldReplaceSpecifiedHeader() throws MessagingException {
+    void setMimeHeaderHandlerShouldReplaceSpecifiedHeader() throws MessagingException {
 
         MimeMessage mimeMessage = MimeMessageBuilder.mimeMessageBuilder()
             .addHeader(HEADER_NAME, "defaultHeaderValue")
@@ -98,7 +94,7 @@ public class SetMimeHeaderHandlerTest {
         header.setHeaderName(HEADER_NAME);
         header.setHeaderValue(HEADER_VALUE);
 
-        header.onMessage(mockedSMTPSession, mail);
+        header.onMessage(fakeSMTPSession, mail);
 
         assertThat(mail.getMessage().getHeader(HEADER_NAME)).containsOnly(HEADER_VALUE);
     }


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