You are viewing a plain text version of this content. The canonical link for it is here.
Posted to notifications@james.apache.org by bt...@apache.org on 2021/01/21 06:42:28 UTC

[james-project] 02/02: JAMES-3431 Migrate Mailet integration tests: JUNIT 5 related visibility changes

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

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

commit ecd8f2e68245f7c167f9e998fc48fbcec26d29bc
Author: Benoit Tellier <bt...@linagora.com>
AuthorDate: Wed Jan 20 11:43:53 2021 +0700

    JAMES-3431 Migrate Mailet integration tests: JUNIT 5 related visibility changes
---
 .../james/mailets/AddDeliveredToHeaderTest.java    |  8 ++--
 .../james/mailets/BounceIntegrationTest.java       | 22 +++++-----
 .../mailets/CommonMailetConfigurationTest.java     | 12 +++---
 .../apache/james/mailets/DKIMIntegrationTest.java  |  8 ++--
 ...ectResolutionRemoteDeliveryIntegrationTest.java | 12 +++---
 .../GatewayRemoteDeliveryIntegrationTest.java      | 20 ++++-----
 .../org/apache/james/mailets/MailetErrorsTest.java | 50 +++++++++++-----------
 .../mailets/NetworkMatcherIntegrationTest.java     | 20 ++++-----
 .../RecipientRewriteTableIntegrationTest.java      | 22 +++++-----
 .../james/mailets/RemoteDeliveryErrorTest.java     | 30 ++++++-------
 .../mailets/SenderIsLocalIntegrationTest.java      | 12 +++---
 .../org/apache/james/mailets/SieveDelivery.java    |  8 ++--
 .../mailets/SizeGreaterThanIntegrationTest.java    |  4 +-
 .../james/mailets/SmtpAuthIntegrationTest.java     | 12 +++---
 .../java/org/apache/james/smtp/DSNRelayTest.java   | 19 ++++----
 .../james/smtp/SmtpAuthorizedAddressesTest.java    | 12 +++---
 .../james/smtp/SmtpBracketEnforcementTest.java     | 12 +++---
 .../org/apache/james/smtp/SmtpContentTypeTest.java |  8 ++--
 .../james/smtp/SmtpIdentityVerificationTest.java   | 11 +++--
 .../org/apache/james/smtp/SmtpNullSenderTest.java  | 10 ++---
 .../apache/james/smtp/SmtpRandomStoringTest.java   | 10 ++---
 .../apache/james/smtp/SmtpSizeLimitationTest.java  |  8 ++--
 .../james/transport/mailets/AddFooterTest.java     | 10 ++---
 .../james/transport/mailets/AliasMappingTest.java  | 38 ++++++++--------
 .../mailets/AmqpForwardAttachmentTest.java         |  8 ++--
 .../transport/mailets/ContactExtractorTest.java    |  8 ++--
 .../transport/mailets/DlpIntegrationTest.java      | 14 +++---
 .../james/transport/mailets/DomainMappingTest.java | 18 ++++----
 .../transport/mailets/GroupMappingRelayTest.java   |  8 ++--
 .../james/transport/mailets/GroupMappingTest.java  | 42 +++++++++---------
 .../mailets/ICSAttachmentWorkflowTest.java         | 24 +++++------
 .../transport/mailets/IsOverQuotaMatcherTest.java  | 18 ++++----
 .../mailets/MailReprocessingIntegrationTest.java   | 12 +++---
 .../mailets/RemoteDeliveryErrorHandlingTest.java   |  2 +-
 .../mailets/SpamAssassinIntegrationTest.java       | 14 +++---
 .../transport/mailets/StripAttachmentTest.java     |  8 ++--
 .../mailets/ToRepositoryIntegrationTest.java       | 12 +++---
 .../ToSenderDomainRepositoryIntegrationTest.java   | 17 ++++----
 38 files changed, 289 insertions(+), 294 deletions(-)

diff --git a/server/mailet/integration-testing/src/test/java/org/apache/james/mailets/AddDeliveredToHeaderTest.java b/server/mailet/integration-testing/src/test/java/org/apache/james/mailets/AddDeliveredToHeaderTest.java
index f56dca4..9215550 100644
--- a/server/mailet/integration-testing/src/test/java/org/apache/james/mailets/AddDeliveredToHeaderTest.java
+++ b/server/mailet/integration-testing/src/test/java/org/apache/james/mailets/AddDeliveredToHeaderTest.java
@@ -42,7 +42,7 @@ import org.junit.jupiter.api.BeforeEach;
 import org.junit.jupiter.api.extension.RegisterExtension;
 import org.junit.jupiter.api.io.TempDir;
 
-public class AddDeliveredToHeaderTest {
+class AddDeliveredToHeaderTest {
     @RegisterExtension
     public TestIMAPClient testIMAPClient = new TestIMAPClient();
     @RegisterExtension
@@ -51,7 +51,7 @@ public class AddDeliveredToHeaderTest {
     private TemporaryJamesServer jamesServer;
 
     @BeforeEach
-    public void setup(@TempDir File temporaryFolder) throws Exception {
+    void setup(@TempDir File temporaryFolder) throws Exception {
         jamesServer = TemporaryJamesServer.builder().build(temporaryFolder);
         jamesServer.start();
 
@@ -61,12 +61,12 @@ public class AddDeliveredToHeaderTest {
     }
 
     @AfterEach
-    public void tearDown() {
+    void tearDown() {
         jamesServer.shutdown();
     }
 
     @Test
-    public void receivedMessagesShouldContainDeliveredToHeaders() throws Exception {
+    void receivedMessagesShouldContainDeliveredToHeaders() throws Exception {
         messageSender.connect(LOCALHOST_IP, jamesServer.getProbe(SmtpGuiceProbe.class).getSmtpPort())
             .sendMessage(FROM, RECIPIENT);
 
diff --git a/server/mailet/integration-testing/src/test/java/org/apache/james/mailets/BounceIntegrationTest.java b/server/mailet/integration-testing/src/test/java/org/apache/james/mailets/BounceIntegrationTest.java
index 43e337b..b38a372 100644
--- a/server/mailet/integration-testing/src/test/java/org/apache/james/mailets/BounceIntegrationTest.java
+++ b/server/mailet/integration-testing/src/test/java/org/apache/james/mailets/BounceIntegrationTest.java
@@ -58,7 +58,7 @@ import org.junit.jupiter.api.Test;
 import org.junit.jupiter.api.extension.RegisterExtension;
 import org.junit.jupiter.api.io.TempDir;
 
-public class BounceIntegrationTest {
+class BounceIntegrationTest {
     public static final String POSTMASTER = "postmaster@" + DEFAULT_DOMAIN;
     public static final String POSTMASTER_PASSWORD = "postmasterSecret";
     public static final String SENDER = "bounce.receiver@" + DEFAULT_DOMAIN;
@@ -74,12 +74,12 @@ public class BounceIntegrationTest {
     private TemporaryJamesServer jamesServer;
 
     @AfterEach
-    public void tearDown() {
+    void tearDown() {
         jamesServer.shutdown();
     }
 
     @Test
-    public void dsnBounceMailetShouldDeliverBounce(@TempDir File temporaryFolder) throws Exception {
+    void dsnBounceMailetShouldDeliverBounce(@TempDir File temporaryFolder) throws Exception {
         setup(temporaryFolder, DSNBounce.class);
 
         messageSender.connect(LOCALHOST_IP, jamesServer.getProbe(SmtpGuiceProbe.class).getSmtpPort())
@@ -92,7 +92,7 @@ public class BounceIntegrationTest {
     }
 
     @Test
-    public void dsnBounceMailetShouldDeliverBounceToTheMailFromAddress(@TempDir File temporaryFolder) throws Exception {
+    void dsnBounceMailetShouldDeliverBounceToTheMailFromAddress(@TempDir File temporaryFolder) throws Exception {
         setup(temporaryFolder, DSNBounce.class);
 
         messageSender.connect(LOCALHOST_IP, jamesServer.getProbe(SmtpGuiceProbe.class).getSmtpPort())
@@ -111,7 +111,7 @@ public class BounceIntegrationTest {
     }
 
     @Test
-    public void dsnBounceMailetBouncedMailShouldBeAdressedToTheSenderInEnvelopeAndHeader(@TempDir File temporaryFolder) throws Exception {
+    void dsnBounceMailetBouncedMailShouldBeAdressedToTheSenderInEnvelopeAndHeader(@TempDir File temporaryFolder) throws Exception {
         setup(temporaryFolder, DSNBounce.class);
 
         messageSender.connect(LOCALHOST_IP, jamesServer.getProbe(SmtpGuiceProbe.class).getSmtpPort())
@@ -157,7 +157,7 @@ public class BounceIntegrationTest {
     }
 
     @Test
-    public void bounceMailetShouldDeliverBounce(@TempDir File temporaryFolder) throws Exception {
+    void bounceMailetShouldDeliverBounce(@TempDir File temporaryFolder) throws Exception {
         setup(temporaryFolder, Bounce.class);
 
         messageSender.connect(LOCALHOST_IP, jamesServer.getProbe(SmtpGuiceProbe.class).getSmtpPort())
@@ -170,7 +170,7 @@ public class BounceIntegrationTest {
     }
 
     @Test
-    public void forwardMailetShouldDeliverBounce(@TempDir File temporaryFolder) throws Exception {
+    void forwardMailetShouldDeliverBounce(@TempDir File temporaryFolder) throws Exception {
         setup(temporaryFolder, Forward.class, Pair.of("forwardTo", SENDER));
 
         messageSender.connect(LOCALHOST_IP, jamesServer.getProbe(SmtpGuiceProbe.class).getSmtpPort())
@@ -183,7 +183,7 @@ public class BounceIntegrationTest {
     }
 
     @Test
-    public void redirectMailetShouldDeliverBounce(@TempDir File temporaryFolder) throws Exception {
+    void redirectMailetShouldDeliverBounce(@TempDir File temporaryFolder) throws Exception {
         setup(temporaryFolder, Redirect.class, Pair.of("recipients", SENDER));
 
         messageSender.connect(LOCALHOST_IP, jamesServer.getProbe(SmtpGuiceProbe.class).getSmtpPort())
@@ -196,7 +196,7 @@ public class BounceIntegrationTest {
     }
 
     @Test
-    public void resendMailetShouldDeliverBounce(@TempDir File temporaryFolder) throws Exception {
+    void resendMailetShouldDeliverBounce(@TempDir File temporaryFolder) throws Exception {
         setup(temporaryFolder, Resend.class, Pair.of("recipients", SENDER));
 
         messageSender.connect(LOCALHOST_IP, jamesServer.getProbe(SmtpGuiceProbe.class).getSmtpPort())
@@ -209,7 +209,7 @@ public class BounceIntegrationTest {
     }
 
     @Test
-    public void notifySenderMailetShouldDeliverBounce(@TempDir File temporaryFolder) throws Exception {
+    void notifySenderMailetShouldDeliverBounce(@TempDir File temporaryFolder) throws Exception {
         setup(temporaryFolder, NotifySender.class);
 
         messageSender.connect(LOCALHOST_IP, jamesServer.getProbe(SmtpGuiceProbe.class).getSmtpPort())
@@ -222,7 +222,7 @@ public class BounceIntegrationTest {
     }
 
     @Test
-    public void notifyPostmasterMailetShouldDeliverBounce(@TempDir File temporaryFolder) throws Exception {
+    void notifyPostmasterMailetShouldDeliverBounce(@TempDir File temporaryFolder) throws Exception {
         setup(temporaryFolder, NotifyPostmaster.class);
 
         messageSender.connect(LOCALHOST_IP, jamesServer.getProbe(SmtpGuiceProbe.class).getSmtpPort())
diff --git a/server/mailet/integration-testing/src/test/java/org/apache/james/mailets/CommonMailetConfigurationTest.java b/server/mailet/integration-testing/src/test/java/org/apache/james/mailets/CommonMailetConfigurationTest.java
index 26984c2..2a10803 100644
--- a/server/mailet/integration-testing/src/test/java/org/apache/james/mailets/CommonMailetConfigurationTest.java
+++ b/server/mailet/integration-testing/src/test/java/org/apache/james/mailets/CommonMailetConfigurationTest.java
@@ -41,7 +41,7 @@ import org.junit.jupiter.api.Test;
 import org.junit.jupiter.api.extension.RegisterExtension;
 import org.junit.jupiter.api.io.TempDir;
 
-public class CommonMailetConfigurationTest {
+class CommonMailetConfigurationTest {
     @RegisterExtension
     public TestIMAPClient testIMAPClient = new TestIMAPClient();
     @RegisterExtension
@@ -50,7 +50,7 @@ public class CommonMailetConfigurationTest {
     private TemporaryJamesServer jamesServer;
 
     @BeforeEach
-    public void setup(@TempDir File temporaryFolder) throws Exception {
+    void setup(@TempDir File temporaryFolder) throws Exception {
         jamesServer = TemporaryJamesServer.builder().build(temporaryFolder);
         jamesServer.start();
 
@@ -60,16 +60,16 @@ public class CommonMailetConfigurationTest {
     }
 
     @AfterEach
-    public void tearDown() {
+    void tearDown() {
         jamesServer.shutdown();
     }
 
     @Test
-    public void startingJamesWithCommonMailetConfigurationShouldWork() throws Exception {
+    void startingJamesWithCommonMailetConfigurationShouldWork() {
     }
 
     @Test
-    public void simpleMailShouldBeSent() throws Exception {
+    void simpleMailShouldBeSent() throws Exception {
         messageSender.connect(LOCALHOST_IP, jamesServer.getProbe(SmtpGuiceProbe.class).getSmtpPort())
             .sendMessage(FROM, RECIPIENT);
 
@@ -80,7 +80,7 @@ public class CommonMailetConfigurationTest {
     }
 
     @Test
-    public void simpleMailShouldBeSentToUpperCaseRecipient() throws Exception {
+    void simpleMailShouldBeSentToUpperCaseRecipient() throws Exception {
         messageSender.connect(LOCALHOST_IP, jamesServer.getProbe(SmtpGuiceProbe.class).getSmtpPort())
             .sendMessage(FROM, RECIPIENT.toUpperCase(Locale.US));
 
diff --git a/server/mailet/integration-testing/src/test/java/org/apache/james/mailets/DKIMIntegrationTest.java b/server/mailet/integration-testing/src/test/java/org/apache/james/mailets/DKIMIntegrationTest.java
index 19ce6e3..bd833da 100644
--- a/server/mailet/integration-testing/src/test/java/org/apache/james/mailets/DKIMIntegrationTest.java
+++ b/server/mailet/integration-testing/src/test/java/org/apache/james/mailets/DKIMIntegrationTest.java
@@ -53,7 +53,7 @@ import org.junit.jupiter.api.Test;
 import org.junit.jupiter.api.extension.RegisterExtension;
 import org.junit.jupiter.api.io.TempDir;
 
-public class DKIMIntegrationTest {
+class DKIMIntegrationTest {
 
     private static final String FROM_LOCAL_PART = "fromUser";
     private static final String FROM = FROM_LOCAL_PART + "@" + DEFAULT_DOMAIN;
@@ -109,7 +109,7 @@ public class DKIMIntegrationTest {
     private List<Optional<String>> dkimAuthResults;
 
     @BeforeEach
-    public void setup(@TempDir File temporaryFolder) throws Exception {
+    void setup(@TempDir File temporaryFolder) throws Exception {
         dkimAuthResults = new ArrayList<>();
         ExtractAttributeStub.setDkimAuthResultInspector(value -> dkimAuthResults.add(value.map(result -> (String) result)));
         MailetContainer.Builder mailetContainer = TemporaryJamesServer.simpleMailetContainerConfiguration()
@@ -134,12 +134,12 @@ public class DKIMIntegrationTest {
     }
 
     @AfterEach
-    public void tearDown() {
+    void tearDown() {
         jamesServer.shutdown();
     }
 
     @Test
-    public void incomingMessageShouldBeReceivedSignedAndChecked() throws Exception {
+    void incomingMessageShouldBeReceivedSignedAndChecked() throws Exception {
         messageSender.connect(LOCALHOST_IP, jamesServer.getProbe(SmtpGuiceProbe.class).getSmtpPort())
             .sendMessage(FROM, RECIPIENT);
 
diff --git a/server/mailet/integration-testing/src/test/java/org/apache/james/mailets/DirectResolutionRemoteDeliveryIntegrationTest.java b/server/mailet/integration-testing/src/test/java/org/apache/james/mailets/DirectResolutionRemoteDeliveryIntegrationTest.java
index 90693a8..a6260ea 100644
--- a/server/mailet/integration-testing/src/test/java/org/apache/james/mailets/DirectResolutionRemoteDeliveryIntegrationTest.java
+++ b/server/mailet/integration-testing/src/test/java/org/apache/james/mailets/DirectResolutionRemoteDeliveryIntegrationTest.java
@@ -80,14 +80,14 @@ public class DirectResolutionRemoteDeliveryIntegrationTest {
     private DataProbe dataProbe;
 
     @AfterEach
-    public void tearDown() {
+    void tearDown() {
         if (jamesServer != null) {
             jamesServer.shutdown();
         }
     }
 
     @Test
-    public void directResolutionShouldBeWellPerformed(@TempDir File temporaryFolder) throws Exception {
+    void directResolutionShouldBeWellPerformed(@TempDir File temporaryFolder) throws Exception {
         InMemoryDNSService inMemoryDNSService = new InMemoryDNSService()
             .registerMxRecord(JAMES_ANOTHER_DOMAIN, fakeSmtp.getContainer().getContainerIp());
 
@@ -112,7 +112,7 @@ public class DirectResolutionRemoteDeliveryIntegrationTest {
     }
 
     @Test
-    public void directResolutionShouldFailoverOnSecondMxWhenFirstMxFailed(@TempDir File temporaryFolder) throws Exception {
+    void directResolutionShouldFailoverOnSecondMxWhenFirstMxFailed(@TempDir File temporaryFolder) throws Exception {
         InMemoryDNSService inMemoryDNSService = new InMemoryDNSService()
             .registerRecord(JAMES_ANOTHER_DOMAIN, ADDRESS_EMPTY_LIST, JAMES_ANOTHER_MX_DOMAINS, RECORD_EMPTY_LIST)
             .registerMxRecord(JAMES_ANOTHER_MX_DOMAIN_1, fakeSmtpOnPort26.getContainer().getContainerIp())
@@ -139,7 +139,7 @@ public class DirectResolutionRemoteDeliveryIntegrationTest {
     }
 
     @Test
-    public void directResolutionShouldBounceUponUnreachableMxRecords(@TempDir File temporaryFolder) throws Exception {
+    void directResolutionShouldBounceUponUnreachableMxRecords(@TempDir File temporaryFolder) throws Exception {
         InMemoryDNSService inMemoryDNSService = new InMemoryDNSService()
             .registerRecord(JAMES_ANOTHER_DOMAIN, ADDRESS_EMPTY_LIST, ImmutableList.of("unknown"), RECORD_EMPTY_LIST);
 
@@ -166,7 +166,7 @@ public class DirectResolutionRemoteDeliveryIntegrationTest {
     }
 
     @Test
-    public void directResolutionShouldBounceWhenNoMxRecord(@TempDir File temporaryFolder) throws Exception {
+    void directResolutionShouldBounceWhenNoMxRecord(@TempDir File temporaryFolder) throws Exception {
         InMemoryDNSService inMemoryDNSService = new InMemoryDNSService()
             .registerRecord(JAMES_ANOTHER_DOMAIN, ADDRESS_EMPTY_LIST, RECORD_EMPTY_LIST, RECORD_EMPTY_LIST);
 
@@ -194,7 +194,7 @@ public class DirectResolutionRemoteDeliveryIntegrationTest {
 
     @Disabled("JAMES-2913 PerRecipientHeaders are not handled by RemoteDelivery")
     @Test
-    public void remoteDeliveryShouldAddPerRecipientHeaders(@TempDir File temporaryFolder) throws Exception {
+    void remoteDeliveryShouldAddPerRecipientHeaders(@TempDir File temporaryFolder) throws Exception {
         InMemoryDNSService inMemoryDNSService = new InMemoryDNSService()
             .registerMxRecord(JAMES_ANOTHER_DOMAIN, fakeSmtp.getContainer().getContainerIp());
 
diff --git a/server/mailet/integration-testing/src/test/java/org/apache/james/mailets/GatewayRemoteDeliveryIntegrationTest.java b/server/mailet/integration-testing/src/test/java/org/apache/james/mailets/GatewayRemoteDeliveryIntegrationTest.java
index a9f3dbf..943396b 100644
--- a/server/mailet/integration-testing/src/test/java/org/apache/james/mailets/GatewayRemoteDeliveryIntegrationTest.java
+++ b/server/mailet/integration-testing/src/test/java/org/apache/james/mailets/GatewayRemoteDeliveryIntegrationTest.java
@@ -50,7 +50,7 @@ import org.junit.jupiter.api.Test;
 import org.junit.jupiter.api.extension.RegisterExtension;
 import org.junit.jupiter.api.io.TempDir;
 
-public class GatewayRemoteDeliveryIntegrationTest {
+class GatewayRemoteDeliveryIntegrationTest {
     private static final String JAMES_ANOTHER_DOMAIN = "james.com";
 
     private static final String FROM = "from@" + DEFAULT_DOMAIN;
@@ -68,13 +68,13 @@ public class GatewayRemoteDeliveryIntegrationTest {
     private InMemoryDNSService inMemoryDNSService;
 
     @BeforeEach
-    public void setup() throws Exception {
+    void setup() throws Exception {
         inMemoryDNSService = new InMemoryDNSService()
             .registerMxRecord(JAMES_ANOTHER_DOMAIN, fakeSmtp.getContainer().getContainerIp());
     }
 
     @AfterEach
-    public void tearDown() {
+    void tearDown() {
         fakeSmtp.clean();
         if (jamesServer != null) {
             jamesServer.shutdown();
@@ -82,7 +82,7 @@ public class GatewayRemoteDeliveryIntegrationTest {
     }
 
     @Test
-    public void outgoingMailShouldTransitThroughGatewayWhenNoPort(@TempDir File temporaryFolder) throws Exception {
+    void outgoingMailShouldTransitThroughGatewayWhenNoPort(@TempDir File temporaryFolder) throws Exception {
         String gatewayProperty = fakeSmtp.getContainer().getContainerIp();
 
         jamesServer = TemporaryJamesServer.builder()
@@ -103,7 +103,7 @@ public class GatewayRemoteDeliveryIntegrationTest {
     }
 
     @Test
-    public void outgoingMailShouldTransitThroughGatewayWhenPort(@TempDir File temporaryFolder) throws Exception {
+    void outgoingMailShouldTransitThroughGatewayWhenPort(@TempDir File temporaryFolder) throws Exception {
         String gatewayProperty = fakeSmtp.getContainer().getContainerIp() + ":25";
 
         jamesServer = TemporaryJamesServer.builder()
@@ -123,7 +123,7 @@ public class GatewayRemoteDeliveryIntegrationTest {
     }
 
     @Test
-    public void outgoingMailShouldTransitThroughGatewayWhenSeveralIps(@TempDir File temporaryFolder) throws Exception {
+    void outgoingMailShouldTransitThroughGatewayWhenSeveralIps(@TempDir File temporaryFolder) throws Exception {
         String gatewayProperty = fakeSmtp.getContainer().getContainerIp() + ",invalid.domain";
 
         jamesServer = TemporaryJamesServer.builder()
@@ -144,7 +144,7 @@ public class GatewayRemoteDeliveryIntegrationTest {
     }
 
     @Test
-    public void outgoingMailShouldFallbackToSecondGatewayWhenFirstInvalid(@TempDir File temporaryFolder) throws Exception {
+    void outgoingMailShouldFallbackToSecondGatewayWhenFirstInvalid(@TempDir File temporaryFolder) throws Exception {
         String gatewayProperty = "invalid.domain," + fakeSmtp.getContainer().getContainerIp();
 
         jamesServer = TemporaryJamesServer.builder()
@@ -165,7 +165,7 @@ public class GatewayRemoteDeliveryIntegrationTest {
     }
 
     @Test
-    public void outgoingMailShouldNotBeSentDirectlyToTheHostWhenGatewayFails(@TempDir File temporaryFolder) throws Exception {
+    void outgoingMailShouldNotBeSentDirectlyToTheHostWhenGatewayFails(@TempDir File temporaryFolder) throws Exception {
         String gatewayProperty = "invalid.domain";
 
         jamesServer = TemporaryJamesServer.builder()
@@ -191,7 +191,7 @@ public class GatewayRemoteDeliveryIntegrationTest {
     }
 
     @Test
-    public void remoteDeliveryShouldBounceUponFailure(@TempDir File temporaryFolder) throws Exception {
+    void remoteDeliveryShouldBounceUponFailure(@TempDir File temporaryFolder) throws Exception {
         String gatewayProperty = "invalid.domain";
 
         jamesServer = TemporaryJamesServer.builder()
@@ -215,7 +215,7 @@ public class GatewayRemoteDeliveryIntegrationTest {
     }
 
     @Test
-    public void remoteDeliveryShouldBounceUponFailureWhenNoBounceProcessor(@TempDir File temporaryFolder) throws Exception {
+    void remoteDeliveryShouldBounceUponFailureWhenNoBounceProcessor(@TempDir File temporaryFolder) throws Exception {
         String gatewayProperty = "invalid.domain";
 
         jamesServer = TemporaryJamesServer.builder()
diff --git a/server/mailet/integration-testing/src/test/java/org/apache/james/mailets/MailetErrorsTest.java b/server/mailet/integration-testing/src/test/java/org/apache/james/mailets/MailetErrorsTest.java
index fe83f2d..15fd7ec 100644
--- a/server/mailet/integration-testing/src/test/java/org/apache/james/mailets/MailetErrorsTest.java
+++ b/server/mailet/integration-testing/src/test/java/org/apache/james/mailets/MailetErrorsTest.java
@@ -55,7 +55,7 @@ import org.junit.jupiter.api.AfterEach;
 import org.junit.jupiter.api.extension.RegisterExtension;
 import org.junit.jupiter.api.io.TempDir;
 
-public class MailetErrorsTest {
+class MailetErrorsTest {
     public static final String CUSTOM_PROCESSOR = "custom";
     public static final MailRepositoryUrl CUSTOM_REPOSITORY = MailRepositoryUrl.from("memory://var/mail/custom/");
 
@@ -65,14 +65,14 @@ public class MailetErrorsTest {
     private TemporaryJamesServer jamesServer;
 
     @AfterEach
-    public void tearDown() {
+    void tearDown() {
         if (jamesServer != null) {
             jamesServer.shutdown();
         }
     }
 
     @Test
-    public void mailetProcessorsShouldHandleMessagingException(@TempDir File temporaryFolder) throws Exception {
+    void mailetProcessorsShouldHandleMessagingException(@TempDir File temporaryFolder) throws Exception {
         jamesServer = TemporaryJamesServer.builder()
             .withBase(SMTP_ONLY_MODULE)
             .withMailetContainer(MailetContainer.builder()
@@ -92,7 +92,7 @@ public class MailetErrorsTest {
     }
 
     @Test
-    public void mailetProcessingShouldHandleClassNotFoundException(@TempDir File temporaryFolder) throws Exception {
+    void mailetProcessingShouldHandleClassNotFoundException(@TempDir File temporaryFolder) throws Exception {
         jamesServer = TemporaryJamesServer.builder()
             .withBase(SMTP_ONLY_MODULE)
             .withMailetContainer(MailetContainer.builder()
@@ -112,7 +112,7 @@ public class MailetErrorsTest {
     }
 
     @Test
-    public void matcherProcessingShouldHandleClassNotFoundException(@TempDir File temporaryFolder) throws Exception {
+    void matcherProcessingShouldHandleClassNotFoundException(@TempDir File temporaryFolder) throws Exception {
         jamesServer = TemporaryJamesServer.builder()
             .withBase(SMTP_ONLY_MODULE)
             .withMailetContainer(MailetContainer.builder()
@@ -132,7 +132,7 @@ public class MailetErrorsTest {
     }
 
     @Test
-    public void mailetProcessorsShouldHandleRuntimeException(@TempDir File temporaryFolder) throws Exception {
+    void mailetProcessorsShouldHandleRuntimeException(@TempDir File temporaryFolder) throws Exception {
         jamesServer = TemporaryJamesServer.builder()
             .withBase(SMTP_ONLY_MODULE)
             .withMailetContainer(MailetContainer.builder()
@@ -152,7 +152,7 @@ public class MailetErrorsTest {
     }
 
     @Test
-    public void spoolerShouldEventuallyProcessUponTemporaryError(@TempDir File temporaryFolder) throws Exception {
+    void spoolerShouldEventuallyProcessUponTemporaryError(@TempDir File temporaryFolder) throws Exception {
         jamesServer = TemporaryJamesServer.builder()
             .withBase(SMTP_ONLY_MODULE)
             .withMailetContainer(MailetContainer.builder()
@@ -176,7 +176,7 @@ public class MailetErrorsTest {
     }
 
     @Test
-    public void spoolerShouldEventuallyProcessMailsAfterThreadSuicide(@TempDir File temporaryFolder) throws Exception {
+    void spoolerShouldEventuallyProcessMailsAfterThreadSuicide(@TempDir File temporaryFolder) throws Exception {
         jamesServer = TemporaryJamesServer.builder()
             .withBase(SMTP_ONLY_MODULE)
             .withMailetContainer(MailetContainer.builder()
@@ -200,7 +200,7 @@ public class MailetErrorsTest {
     }
 
     @Test
-    public void spoolerShouldNotInfinitLoopUponPermanentError(@TempDir File temporaryFolder) throws Exception {
+    void spoolerShouldNotInfinitLoopUponPermanentError(@TempDir File temporaryFolder) throws Exception {
         jamesServer = TemporaryJamesServer.builder()
             .withBase(SMTP_ONLY_MODULE)
             .withMailetContainer(MailetContainer.builder()
@@ -220,7 +220,7 @@ public class MailetErrorsTest {
     }
 
     @Test
-    public void mailetProcessorsShouldHandleMessagingExceptionWhenSpecificErrorHandlingSpecified(@TempDir File temporaryFolder) throws Exception {
+    void mailetProcessorsShouldHandleMessagingExceptionWhenSpecificErrorHandlingSpecified(@TempDir File temporaryFolder) throws Exception {
         jamesServer = TemporaryJamesServer.builder()
             .withBase(SMTP_ONLY_MODULE)
             .withMailetContainer(MailetContainer.builder()
@@ -243,7 +243,7 @@ public class MailetErrorsTest {
     }
 
     @Test
-    public void mailetProcessorsShouldHandleRuntimeExceptionWhenSpecificErrorHandlingSpecified(@TempDir File temporaryFolder) throws Exception {
+    void mailetProcessorsShouldHandleRuntimeExceptionWhenSpecificErrorHandlingSpecified(@TempDir File temporaryFolder) throws Exception {
         jamesServer = TemporaryJamesServer.builder()
             .withBase(SMTP_ONLY_MODULE)
             .withMailetContainer(MailetContainer.builder()
@@ -265,7 +265,7 @@ public class MailetErrorsTest {
     }
 
     @Test
-    public void onExceptionIgnoreShouldContinueProcessingWhenRuntimeException(@TempDir File temporaryFolder) throws Exception {
+    void onExceptionIgnoreShouldContinueProcessingWhenRuntimeException(@TempDir File temporaryFolder) throws Exception {
         jamesServer = TemporaryJamesServer.builder()
             .withBase(SMTP_ONLY_MODULE)
             .withMailetContainer(MailetContainer.builder()
@@ -291,7 +291,7 @@ public class MailetErrorsTest {
     }
 
     @Test
-    public void onExceptionIgnoreShouldContinueProcessingWhenMessagingException(@TempDir File temporaryFolder) throws Exception {
+    void onExceptionIgnoreShouldContinueProcessingWhenMessagingException(@TempDir File temporaryFolder) throws Exception {
         jamesServer = TemporaryJamesServer.builder()
             .withBase(SMTP_ONLY_MODULE)
             .withMailetContainer(MailetContainer.builder()
@@ -317,7 +317,7 @@ public class MailetErrorsTest {
     }
 
     @Test
-    public void matcherProcessorsShouldHandleMessagingException(@TempDir File temporaryFolder) throws Exception {
+    void matcherProcessorsShouldHandleMessagingException(@TempDir File temporaryFolder) throws Exception {
         jamesServer = TemporaryJamesServer.builder()
             .withBase(SMTP_ONLY_MODULE)
             .withMailetContainer(MailetContainer.builder()
@@ -337,7 +337,7 @@ public class MailetErrorsTest {
     }
 
     @Test
-    public void matcherProcessorsShouldHandleRuntimeException(@TempDir File temporaryFolder) throws Exception {
+    void matcherProcessorsShouldHandleRuntimeException(@TempDir File temporaryFolder) throws Exception {
         jamesServer = TemporaryJamesServer.builder()
             .withBase(SMTP_ONLY_MODULE)
             .withMailetContainer(MailetContainer.builder()
@@ -357,7 +357,7 @@ public class MailetErrorsTest {
     }
 
     @Test
-    public void matcherProcessorsShouldHandleMessagingExceptionWhenSpecificErrorHandlingSpecified(@TempDir File temporaryFolder) throws Exception {
+    void matcherProcessorsShouldHandleMessagingExceptionWhenSpecificErrorHandlingSpecified(@TempDir File temporaryFolder) throws Exception {
         jamesServer = TemporaryJamesServer.builder()
             .withBase(SMTP_ONLY_MODULE)
             .withMailetContainer(MailetContainer.builder()
@@ -379,7 +379,7 @@ public class MailetErrorsTest {
     }
 
     @Test
-    public void matcherProcessorsShouldHandleRuntimeExceptionWhenSpecificErrorHandlingSpecified(@TempDir File temporaryFolder) throws Exception {
+    void matcherProcessorsShouldHandleRuntimeExceptionWhenSpecificErrorHandlingSpecified(@TempDir File temporaryFolder) throws Exception {
         jamesServer = TemporaryJamesServer.builder()
             .withBase(SMTP_ONLY_MODULE)
             .withMailetContainer(MailetContainer.builder()
@@ -401,7 +401,7 @@ public class MailetErrorsTest {
     }
 
     @Test
-    public void onMatcherExceptionIgnoreShouldNotMatchWhenRuntimeExceptionAndNoMatchConfigured(@TempDir File temporaryFolder) throws Exception {
+    void onMatcherExceptionIgnoreShouldNotMatchWhenRuntimeExceptionAndNoMatchConfigured(@TempDir File temporaryFolder) throws Exception {
         jamesServer = TemporaryJamesServer.builder()
             .withBase(SMTP_ONLY_MODULE)
             .withMailetContainer(MailetContainer.builder()
@@ -427,7 +427,7 @@ public class MailetErrorsTest {
     }
 
     @Test
-    public void onMatcherExceptionIgnoreShouldNotMatchWhenMessagingExceptionAndNoMatchConfigured(@TempDir File temporaryFolder) throws Exception {
+    void onMatcherExceptionIgnoreShouldNotMatchWhenMessagingExceptionAndNoMatchConfigured(@TempDir File temporaryFolder) throws Exception {
         jamesServer = TemporaryJamesServer.builder()
             .withBase(SMTP_ONLY_MODULE)
             .withMailetContainer(MailetContainer.builder()
@@ -453,7 +453,7 @@ public class MailetErrorsTest {
     }
 
     @Test
-    public void onMatcherExceptionIgnoreShouldMatchWhenRuntimeExceptionAndAllMatchConfigured(@TempDir File temporaryFolder) throws Exception {
+    void onMatcherExceptionIgnoreShouldMatchWhenRuntimeExceptionAndAllMatchConfigured(@TempDir File temporaryFolder) throws Exception {
         jamesServer = TemporaryJamesServer.builder()
             .withBase(SMTP_ONLY_MODULE)
             .withMailetContainer(MailetContainer.builder()
@@ -479,7 +479,7 @@ public class MailetErrorsTest {
     }
 
     @Test
-    public void onMatcherExceptionIgnoreShouldMatchWhenMessagingExceptionAndAllMatchConfigured(@TempDir File temporaryFolder) throws Exception {
+    void onMatcherExceptionIgnoreShouldMatchWhenMessagingExceptionAndAllMatchConfigured(@TempDir File temporaryFolder) throws Exception {
         jamesServer = TemporaryJamesServer.builder()
             .withBase(SMTP_ONLY_MODULE)
             .withMailetContainer(MailetContainer.builder()
@@ -505,7 +505,7 @@ public class MailetErrorsTest {
     }
 
     @Test
-    public void hasExceptionMatcherShouldMatchWhenMatcherThrowsExceptionSpecified(@TempDir File temporaryFolder) throws Exception {
+    void hasExceptionMatcherShouldMatchWhenMatcherThrowsExceptionSpecified(@TempDir File temporaryFolder) throws Exception {
         jamesServer = TemporaryJamesServer.builder()
             .withBase(SMTP_ONLY_MODULE)
             .withMailetContainer(MailetContainer.builder()
@@ -535,7 +535,7 @@ public class MailetErrorsTest {
     }
     
     @Test
-    public void hasExceptionMatcherShouldNotMatchWhenMatcherThrowsExceptionNotSpecified(@TempDir File temporaryFolder) throws Exception {
+    void hasExceptionMatcherShouldNotMatchWhenMatcherThrowsExceptionNotSpecified(@TempDir File temporaryFolder) throws Exception {
         jamesServer = TemporaryJamesServer.builder()
             .withBase(SMTP_ONLY_MODULE)
             .withMailetContainer(MailetContainer.builder()
@@ -565,7 +565,7 @@ public class MailetErrorsTest {
     }
     
     @Test
-    public void hasExceptionMatcherShouldMatchWhenMailetThrowsExceptionSpecified(@TempDir File temporaryFolder) throws Exception {
+    void hasExceptionMatcherShouldMatchWhenMailetThrowsExceptionSpecified(@TempDir File temporaryFolder) throws Exception {
         jamesServer = TemporaryJamesServer.builder()
             .withBase(SMTP_ONLY_MODULE)
             .withMailetContainer(MailetContainer.builder()
@@ -595,7 +595,7 @@ public class MailetErrorsTest {
     }
     
     @Test
-    public void hasExceptionMatcherShouldNotMatchWhenMailetThrowsExceptionNotSpecified(@TempDir File temporaryFolder) throws Exception {
+    void hasExceptionMatcherShouldNotMatchWhenMailetThrowsExceptionNotSpecified(@TempDir File temporaryFolder) throws Exception {
         jamesServer = TemporaryJamesServer.builder()
             .withBase(SMTP_ONLY_MODULE)
             .withMailetContainer(MailetContainer.builder()
diff --git a/server/mailet/integration-testing/src/test/java/org/apache/james/mailets/NetworkMatcherIntegrationTest.java b/server/mailet/integration-testing/src/test/java/org/apache/james/mailets/NetworkMatcherIntegrationTest.java
index c42e2ef..5bf023b 100644
--- a/server/mailet/integration-testing/src/test/java/org/apache/james/mailets/NetworkMatcherIntegrationTest.java
+++ b/server/mailet/integration-testing/src/test/java/org/apache/james/mailets/NetworkMatcherIntegrationTest.java
@@ -49,7 +49,7 @@ import org.junit.jupiter.api.Test;
 import org.junit.jupiter.api.extension.RegisterExtension;
 import org.junit.jupiter.api.io.TempDir;
 
-public class NetworkMatcherIntegrationTest {
+class NetworkMatcherIntegrationTest {
     private static final String FROM = "fromuser@" + DEFAULT_DOMAIN;
     private static final MailRepositoryUrl DROPPED_MAILS = MailRepositoryUrl.from("memory://var/mail/dropped-mails/");
 
@@ -83,12 +83,12 @@ public class NetworkMatcherIntegrationTest {
     }
 
     @AfterEach
-    public void tearDown() {
+    void tearDown() {
         jamesServer.shutdown();
     }
 
     @Test
-    public void mailsFromAuthorizedNetworksShouldBeDeliveredWithRemoteAddrInNetwork(@TempDir File temporaryFolder) throws Exception {
+    void mailsFromAuthorizedNetworksShouldBeDeliveredWithRemoteAddrInNetwork(@TempDir File temporaryFolder) throws Exception {
         jamesServer = createJamesServerWithRootProcessor(temporaryFolder, ProcessorConfiguration.root()
             .addMailet(MailetConfiguration.builder()
                 .matcher(RemoteAddrInNetwork.class)
@@ -108,7 +108,7 @@ public class NetworkMatcherIntegrationTest {
     }
 
     @Test
-    public void mailsFromAuthorizedNetworksShouldBeDeliveredWithRemoteAddrNotInNetwork(@TempDir File temporaryFolder) throws Exception {
+    void mailsFromAuthorizedNetworksShouldBeDeliveredWithRemoteAddrNotInNetwork(@TempDir File temporaryFolder) throws Exception {
         jamesServer = createJamesServerWithRootProcessor(temporaryFolder, ProcessorConfiguration.root()
             .addMailet(MailetConfiguration.builder()
                 .matcher(RemoteAddrNotInNetwork.class)
@@ -128,7 +128,7 @@ public class NetworkMatcherIntegrationTest {
     }
 
     @Test
-    public void remoteAddrInNetworkShouldSupportLargerMask(@TempDir File temporaryFolder) throws Exception {
+    void remoteAddrInNetworkShouldSupportLargerMask(@TempDir File temporaryFolder) throws Exception {
         jamesServer = createJamesServerWithRootProcessor(temporaryFolder, ProcessorConfiguration.root()
             .addMailet(MailetConfiguration.builder()
                 .matcher(RemoteAddrInNetwork.class)
@@ -148,7 +148,7 @@ public class NetworkMatcherIntegrationTest {
     }
 
     @Test
-    public void remoteAddrInNetworkShouldSupportRangesDefinedByAMiddleIp(@TempDir File temporaryFolder) throws Exception {
+    void remoteAddrInNetworkShouldSupportRangesDefinedByAMiddleIp(@TempDir File temporaryFolder) throws Exception {
         jamesServer = createJamesServerWithRootProcessor(temporaryFolder, ProcessorConfiguration.root()
             .addMailet(MailetConfiguration.builder()
                 .matcher(RemoteAddrInNetwork.class)
@@ -168,7 +168,7 @@ public class NetworkMatcherIntegrationTest {
     }
 
     @Test
-    public void remoteAddrInNetworkShouldSupportRangesDefinedByEndingIp(@TempDir File temporaryFolder) throws Exception {
+    void remoteAddrInNetworkShouldSupportRangesDefinedByEndingIp(@TempDir File temporaryFolder) throws Exception {
         jamesServer = createJamesServerWithRootProcessor(temporaryFolder, ProcessorConfiguration.root()
             .addMailet(MailetConfiguration.builder()
                 .matcher(RemoteAddrInNetwork.class)
@@ -188,7 +188,7 @@ public class NetworkMatcherIntegrationTest {
     }
 
     @Test
-    public void remoteAddrInNetworkShouldSupportRangesWithNonEightMultipleSubMasks(@TempDir File temporaryFolder) throws Exception {
+    void remoteAddrInNetworkShouldSupportRangesWithNonEightMultipleSubMasks(@TempDir File temporaryFolder) throws Exception {
         jamesServer = createJamesServerWithRootProcessor(temporaryFolder, ProcessorConfiguration.root()
             .addMailet(MailetConfiguration.builder()
                 .matcher(RemoteAddrInNetwork.class)
@@ -208,7 +208,7 @@ public class NetworkMatcherIntegrationTest {
     }
 
     @Test
-    public void mailsFromNonAuthorizedNetworksShouldNotBeDeliveredWithRemoteAddrInNetwork(@TempDir File temporaryFolder) throws Exception {
+    void mailsFromNonAuthorizedNetworksShouldNotBeDeliveredWithRemoteAddrInNetwork(@TempDir File temporaryFolder) throws Exception {
         jamesServer = createJamesServerWithRootProcessor(temporaryFolder, ProcessorConfiguration.root()
             .addMailet(MailetConfiguration.builder()
                 .matcher(RemoteAddrInNetwork.class)
@@ -232,7 +232,7 @@ public class NetworkMatcherIntegrationTest {
     }
 
     @Test
-    public void mailsFromNonAuthorizedNetworksShouldNotBeDeliveredWithRemoteAddrNotInNetwork(@TempDir File temporaryFolder) throws Exception {
+    void mailsFromNonAuthorizedNetworksShouldNotBeDeliveredWithRemoteAddrNotInNetwork(@TempDir File temporaryFolder) throws Exception {
         jamesServer = createJamesServerWithRootProcessor(temporaryFolder, ProcessorConfiguration.root()
             .addMailet(MailetConfiguration.builder()
                 .matcher(RemoteAddrNotInNetwork.class)
diff --git a/server/mailet/integration-testing/src/test/java/org/apache/james/mailets/RecipientRewriteTableIntegrationTest.java b/server/mailet/integration-testing/src/test/java/org/apache/james/mailets/RecipientRewriteTableIntegrationTest.java
index b0453e1..3981a12 100644
--- a/server/mailet/integration-testing/src/test/java/org/apache/james/mailets/RecipientRewriteTableIntegrationTest.java
+++ b/server/mailet/integration-testing/src/test/java/org/apache/james/mailets/RecipientRewriteTableIntegrationTest.java
@@ -69,7 +69,7 @@ public class RecipientRewriteTableIntegrationTest {
     private RequestSpecification webAdminApi;
 
     @BeforeEach
-    public void setup(@TempDir File temporaryFolder) throws Exception {
+    void setup(@TempDir File temporaryFolder) throws Exception {
         jamesServer = TemporaryJamesServer.builder().build(temporaryFolder);
         jamesServer.start();
 
@@ -85,12 +85,12 @@ public class RecipientRewriteTableIntegrationTest {
     }
 
     @AfterEach
-    public void tearDown() {
+    void tearDown() {
         jamesServer.shutdown();
     }
 
     @Test
-    public void rrtServiceShouldNotImpactRecipientsNotMatchingAnyRRT() throws Exception {
+    void rrtServiceShouldNotImpactRecipientsNotMatchingAnyRRT() throws Exception {
         messageSender.connect(LOCALHOST_IP, jamesServer.getProbe(SmtpGuiceProbe.class).getSmtpPort())
             .sendMessage(FROM, RECIPIENT);
 
@@ -102,7 +102,7 @@ public class RecipientRewriteTableIntegrationTest {
     }
 
     @Test
-    public void rrtServiceShouldDeliverEmailToMappingRecipients() throws Exception {
+    void rrtServiceShouldDeliverEmailToMappingRecipients() throws Exception {
         dataProbe.addAddressMapping(RECIPIENT_LOCAL_PART, DEFAULT_DOMAIN, ANY_AT_JAMES);
         dataProbe.addAddressMapping(RECIPIENT_LOCAL_PART, DEFAULT_DOMAIN, OTHER_AT_JAMES);
 
@@ -122,7 +122,7 @@ public class RecipientRewriteTableIntegrationTest {
     }
 
     @Test
-    public void rrtServiceShouldNotDeliverEmailToRecipientWhenHaveMappingRecipients() throws Exception {
+    void rrtServiceShouldNotDeliverEmailToRecipientWhenHaveMappingRecipients() throws Exception {
         dataProbe.addAddressMapping(RECIPIENT_LOCAL_PART, DEFAULT_DOMAIN, ANY_AT_JAMES);
         dataProbe.addAddressMapping(RECIPIENT_LOCAL_PART, DEFAULT_DOMAIN, OTHER_AT_JAMES);
 
@@ -137,7 +137,7 @@ public class RecipientRewriteTableIntegrationTest {
     }
 
     @Test
-    public void rrtServiceShouldDeliverEmailToRecipientOnLocalWhenMappingContainsNonDomain() throws Exception {
+    void rrtServiceShouldDeliverEmailToRecipientOnLocalWhenMappingContainsNonDomain() throws Exception {
         String nonDomainUser = "nondomain";
         String localUser = nonDomainUser + "@" + dataProbe.getDefaultDomain();
         dataProbe.addUser(localUser, PASSWORD);
@@ -161,7 +161,7 @@ public class RecipientRewriteTableIntegrationTest {
     }
 
     @Test
-    public void messageShouldRedirectToTheSameUserWhenDomainMapping() throws Exception {
+    void messageShouldRedirectToTheSameUserWhenDomainMapping() throws Exception {
         dataProbe.addDomainAliasMapping(DEFAULT_DOMAIN, JAMES_ANOTHER_DOMAIN);
         dataProbe.addUser(ANY_AT_ANOTHER_DOMAIN, PASSWORD);
 
@@ -176,7 +176,7 @@ public class RecipientRewriteTableIntegrationTest {
     }
 
     @Test
-    public void messageShouldNotSendToRecipientWhenDomainMapping() throws Exception {
+    void messageShouldNotSendToRecipientWhenDomainMapping() throws Exception {
         dataProbe.addDomainAliasMapping(DEFAULT_DOMAIN, JAMES_ANOTHER_DOMAIN);
 
         messageSender.connect(LOCALHOST_IP, jamesServer.getProbe(SmtpGuiceProbe.class).getSmtpPort())
@@ -190,7 +190,7 @@ public class RecipientRewriteTableIntegrationTest {
     }
 
     @Test
-    public void rrtServiceShouldDeliverEmailToForwardRecipients() throws Exception {
+    void rrtServiceShouldDeliverEmailToForwardRecipients() throws Exception {
         webAdminApi.put(ForwardRoutes.ROOT_PATH + "/" + RECIPIENT + "/targets/" + ANY_AT_JAMES);
         webAdminApi.put(ForwardRoutes.ROOT_PATH + "/" + RECIPIENT + "/targets/" + OTHER_AT_JAMES);
 
@@ -209,7 +209,7 @@ public class RecipientRewriteTableIntegrationTest {
     }
 
     @Test
-    public void rrtServiceShouldFollowForwardWhenSendingToAGroup() throws Exception {
+    void rrtServiceShouldFollowForwardWhenSendingToAGroup() throws Exception {
         dataProbe.addAddressMapping(GROUP_LOCAL_PART, DEFAULT_DOMAIN, ANY_AT_JAMES);
 
         webAdminApi.put(ForwardRoutes.ROOT_PATH + "/" + ANY_AT_JAMES + "/targets/" + OTHER_AT_JAMES);
@@ -225,7 +225,7 @@ public class RecipientRewriteTableIntegrationTest {
     }
 
     @Test
-    public void domainAliasMappingShouldNotCreateNonExistedUserWhenReRouting() throws Exception {
+    void domainAliasMappingShouldNotCreateNonExistedUserWhenReRouting() throws Exception {
         dataProbe.addDomain("domain1.com");
         dataProbe.addDomain("domain2.com");
 
diff --git a/server/mailet/integration-testing/src/test/java/org/apache/james/mailets/RemoteDeliveryErrorTest.java b/server/mailet/integration-testing/src/test/java/org/apache/james/mailets/RemoteDeliveryErrorTest.java
index 1b40895..aee9e52 100644
--- a/server/mailet/integration-testing/src/test/java/org/apache/james/mailets/RemoteDeliveryErrorTest.java
+++ b/server/mailet/integration-testing/src/test/java/org/apache/james/mailets/RemoteDeliveryErrorTest.java
@@ -68,7 +68,7 @@ import org.slf4j.LoggerFactory;
 
 import com.google.common.collect.ImmutableList;
 
-public class RemoteDeliveryErrorTest {
+class RemoteDeliveryErrorTest {
 
     private static final Logger LOGGER = LoggerFactory.getLogger(RemoteDeliveryErrorTest.class);
 
@@ -116,7 +116,7 @@ public class RemoteDeliveryErrorTest {
     private TemporaryJamesServer jamesServer;
 
     @BeforeAll
-    public static void setUpClass() throws AddressException {
+    static void setUpClass() throws AddressException {
         FROM_ADDRESS = new MailAddress(FROM);
         RECIPIENT_ADDRESS = new MailAddress(RECIPIENT);
         RECIPIENT1_ADDRESS = new MailAddress(RECIPIENT1);
@@ -128,7 +128,7 @@ public class RemoteDeliveryErrorTest {
     }
 
     @BeforeEach
-    public void setUp(@TempDir File temporaryFolder) throws Exception {
+    void setUp(@TempDir File temporaryFolder) throws Exception {
         inMemoryDNSService = new InMemoryDNSService()
             .registerMxRecord(DEFAULT_DOMAIN, LOCALHOST_IP)
             .registerMxRecord(ANOTHER_DOMAIN, mockSmtp.getContainerIp());
@@ -157,7 +157,7 @@ public class RemoteDeliveryErrorTest {
     }
 
     @AfterEach
-    public void tearDown() {
+    void tearDown() {
         jamesServer.shutdown();
 
         mockSMTP1Configuration.cleanServer();
@@ -165,7 +165,7 @@ public class RemoteDeliveryErrorTest {
     }
 
     @Test
-    public void remoteDeliveryShouldBounceWhenAlwaysRCPT421() throws Exception {
+    void remoteDeliveryShouldBounceWhenAlwaysRCPT421() throws Exception {
         mockSMTP1Configuration
             .addNewBehavior()
                 .expect(SMTPCommand.RCPT_TO)
@@ -186,7 +186,7 @@ public class RemoteDeliveryErrorTest {
     }
 
     @Test
-    public void remoteDeliveryShouldBounceWhenAlwaysFROM421() throws Exception {
+    void remoteDeliveryShouldBounceWhenAlwaysFROM421() throws Exception {
         mockSMTP1Configuration
             .addNewBehavior()
                 .expect(SMTPCommand.MAIL_FROM)
@@ -207,7 +207,7 @@ public class RemoteDeliveryErrorTest {
     }
 
     @Test
-    public void remoteDeliveryShouldBounceWhenAlwaysDATA421() throws Exception {
+    void remoteDeliveryShouldBounceWhenAlwaysDATA421() throws Exception {
         mockSMTP1Configuration
             .addNewBehavior()
                 .expect(SMTPCommand.DATA)
@@ -228,7 +228,7 @@ public class RemoteDeliveryErrorTest {
     }
 
     @Test
-    public void remoteDeliveryShouldNotRetryWhenRCPT500() throws Exception {
+    void remoteDeliveryShouldNotRetryWhenRCPT500() throws Exception {
         mockSMTP1Configuration
             .addNewBehavior()
                 .expect(SMTPCommand.RCPT_TO)
@@ -249,7 +249,7 @@ public class RemoteDeliveryErrorTest {
     }
 
     @Test
-    public void remoteDeliveryShouldNotRetryWhenFROM500() throws Exception {
+    void remoteDeliveryShouldNotRetryWhenFROM500() throws Exception {
         mockSMTP1Configuration
             .addNewBehavior()
                 .expect(SMTPCommand.MAIL_FROM)
@@ -270,7 +270,7 @@ public class RemoteDeliveryErrorTest {
     }
 
     @Test
-    public void remoteDeliveryShouldNotRetryWhenDATA500() throws Exception {
+    void remoteDeliveryShouldNotRetryWhenDATA500() throws Exception {
         mockSMTP1Configuration
             .addNewBehavior()
                 .expect(SMTPCommand.DATA)
@@ -291,7 +291,7 @@ public class RemoteDeliveryErrorTest {
     }
 
     @Test
-    public void remoteDeliveryShouldRetryWhenRCPT421() throws Exception {
+    void remoteDeliveryShouldRetryWhenRCPT421() throws Exception {
         mockSMTP1Configuration
             .addNewBehavior()
                 .expect(SMTPCommand.RCPT_TO)
@@ -313,7 +313,7 @@ public class RemoteDeliveryErrorTest {
     }
 
     @Test
-    public void remoteDeliveryShouldRetryWhenFROM421() throws Exception {
+    void remoteDeliveryShouldRetryWhenFROM421() throws Exception {
         mockSMTP1Configuration
             .addNewBehavior()
                 .expect(SMTPCommand.MAIL_FROM)
@@ -335,7 +335,7 @@ public class RemoteDeliveryErrorTest {
     }
 
     @Test
-    public void remoteDeliveryShouldRetryWhenDATA421() throws Exception {
+    void remoteDeliveryShouldRetryWhenDATA421() throws Exception {
         mockSMTP1Configuration
             .addNewBehavior()
                 .expect(SMTPCommand.DATA)
@@ -357,7 +357,7 @@ public class RemoteDeliveryErrorTest {
     }
 
     @Test
-    public void remoteDeliveryShouldNotDuplicateContentWhenSendPartial() throws Exception {
+    void remoteDeliveryShouldNotDuplicateContentWhenSendPartial() throws Exception {
         mockSMTP1Configuration
             .addNewBehavior()
                 .expect(SMTPCommand.RCPT_TO)
@@ -383,7 +383,7 @@ public class RemoteDeliveryErrorTest {
     }
 
     @Test
-    public void remoteDeliveryShouldNotDuplicateContentWhenSendPartialWhenFailover() throws Exception {
+    void remoteDeliveryShouldNotDuplicateContentWhenSendPartialWhenFailover() throws Exception {
         ImmutableList<InetAddress> addresses = ImmutableList.of(InetAddress.getByName(mockSmtp.getContainerIp()));
         ImmutableList<String> mxs = ImmutableList.of(mockSmtp.getContainerIp(), mockSmtp2.getContainerIp());
         ImmutableList<String> txtRecords = ImmutableList.of();
diff --git a/server/mailet/integration-testing/src/test/java/org/apache/james/mailets/SenderIsLocalIntegrationTest.java b/server/mailet/integration-testing/src/test/java/org/apache/james/mailets/SenderIsLocalIntegrationTest.java
index ab96740..fbcb82e 100644
--- a/server/mailet/integration-testing/src/test/java/org/apache/james/mailets/SenderIsLocalIntegrationTest.java
+++ b/server/mailet/integration-testing/src/test/java/org/apache/james/mailets/SenderIsLocalIntegrationTest.java
@@ -53,7 +53,7 @@ import org.junit.jupiter.api.io.TempDir;
 
 import io.restassured.specification.RequestSpecification;
 
-public class SenderIsLocalIntegrationTest {
+class SenderIsLocalIntegrationTest {
     private static final String POSTMASTER = "postmaster@" + DEFAULT_DOMAIN;
     private static final MailRepositoryUrl LOCAL_SENDER_REPOSITORY = MailRepositoryUrl.from("memory://var/mail/local/sender/");
     private static final MailRepositoryUrl REMOTE_SENDER_REPOSITORY = MailRepositoryUrl.from("memory://var/mail/remote/sender/");
@@ -68,7 +68,7 @@ public class SenderIsLocalIntegrationTest {
     private RequestSpecification webAdminApi;
 
     @BeforeEach
-    public void setUp(@TempDir File temporaryFolder) throws Exception {
+    void setUp(@TempDir File temporaryFolder) throws Exception {
         jamesServer = TemporaryJamesServer.builder()
             .withBase(MemoryJamesServerMain.IN_MEMORY_SERVER_AGGREGATE_MODULE)
             .withMailetContainer(TemporaryJamesServer.defaultMailetContainerConfiguration()
@@ -85,12 +85,12 @@ public class SenderIsLocalIntegrationTest {
     }
 
     @AfterEach
-    public void tearDown() {
+    void tearDown() {
         jamesServer.shutdown();
     }
 
     @Test
-    public void shouldMatchLocalSender() throws Exception {
+    void shouldMatchLocalSender() throws Exception {
         messageSender.connect(LOCALHOST_IP, jamesServer.getProbe(SmtpGuiceProbe.class).getSmtpPort())
             .sendMessage(RECIPIENT, RECIPIENT);
 
@@ -98,7 +98,7 @@ public class SenderIsLocalIntegrationTest {
     }
 
     @Test
-    public void shouldMatchLocalSenderAlias() throws Exception {
+    void shouldMatchLocalSenderAlias() throws Exception {
         webAdminApi.put(AliasRoutes.ROOT_PATH + "/" + RECIPIENT + "/sources/" + ALIAS);
 
         messageSender.connect(LOCALHOST_IP, jamesServer.getProbe(SmtpGuiceProbe.class).getSmtpPort())
@@ -108,7 +108,7 @@ public class SenderIsLocalIntegrationTest {
     }
 
     @Test
-    public void shouldNotMatchRemoteSender() throws Exception {
+    void shouldNotMatchRemoteSender() throws Exception {
         messageSender.connect(LOCALHOST_IP, jamesServer.getProbe(SmtpGuiceProbe.class).getSmtpPort())
             .sendMessage("sender@domain.com", RECIPIENT);
 
diff --git a/server/mailet/integration-testing/src/test/java/org/apache/james/mailets/SieveDelivery.java b/server/mailet/integration-testing/src/test/java/org/apache/james/mailets/SieveDelivery.java
index 85cabbe..522d9d5 100644
--- a/server/mailet/integration-testing/src/test/java/org/apache/james/mailets/SieveDelivery.java
+++ b/server/mailet/integration-testing/src/test/java/org/apache/james/mailets/SieveDelivery.java
@@ -43,7 +43,7 @@ import org.junit.jupiter.api.Test;
 import org.junit.jupiter.api.extension.RegisterExtension;
 import org.junit.jupiter.api.io.TempDir;
 
-public class SieveDelivery {
+class SieveDelivery {
     private static final String TARGETED_MAILBOX = "INBOX.any";
 
     @RegisterExtension
@@ -54,7 +54,7 @@ public class SieveDelivery {
     private TemporaryJamesServer jamesServer;
 
     @BeforeEach
-    public void setup(@TempDir File temporaryFolder) throws Exception {
+    void setup(@TempDir File temporaryFolder) throws Exception {
         jamesServer = TemporaryJamesServer.builder().build(temporaryFolder);
         jamesServer.start();
 
@@ -67,12 +67,12 @@ public class SieveDelivery {
     }
 
     @AfterEach
-    public void tearDown() {
+    void tearDown() {
         jamesServer.shutdown();
     }
 
     @Test
-    public void simpleMailShouldBeSent() throws Exception {
+    void simpleMailShouldBeSent() throws Exception {
         jamesServer.getProbe(SieveProbeImpl.class).addActiveSieveScript(RECIPIENT, "myscript.sieve",
             "require \"fileinto\";\n" +
             "\n" +
diff --git a/server/mailet/integration-testing/src/test/java/org/apache/james/mailets/SizeGreaterThanIntegrationTest.java b/server/mailet/integration-testing/src/test/java/org/apache/james/mailets/SizeGreaterThanIntegrationTest.java
index 62af8f8..fec80ae 100644
--- a/server/mailet/integration-testing/src/test/java/org/apache/james/mailets/SizeGreaterThanIntegrationTest.java
+++ b/server/mailet/integration-testing/src/test/java/org/apache/james/mailets/SizeGreaterThanIntegrationTest.java
@@ -47,7 +47,7 @@ import org.junit.jupiter.api.Test;
 import org.junit.jupiter.api.extension.RegisterExtension;
 import org.junit.jupiter.api.io.TempDir;
 
-public class SizeGreaterThanIntegrationTest {
+class SizeGreaterThanIntegrationTest {
     public static final String POSTMASTER = "postmaster@" + DEFAULT_DOMAIN;
     public static final String SENDER = "sender@" + DEFAULT_DOMAIN;
 
@@ -59,7 +59,7 @@ public class SizeGreaterThanIntegrationTest {
     private TemporaryJamesServer jamesServer;
 
     @BeforeEach
-    public void setup(@TempDir File temporaryFolder) throws Exception {
+    void setup(@TempDir File temporaryFolder) throws Exception {
         jamesServer = TemporaryJamesServer.builder()
             .withBase(MemoryJamesServerMain.SMTP_AND_IMAP_MODULE)
             .withMailetContainer(
diff --git a/server/mailet/integration-testing/src/test/java/org/apache/james/mailets/SmtpAuthIntegrationTest.java b/server/mailet/integration-testing/src/test/java/org/apache/james/mailets/SmtpAuthIntegrationTest.java
index d6a877b..13f0c79 100644
--- a/server/mailet/integration-testing/src/test/java/org/apache/james/mailets/SmtpAuthIntegrationTest.java
+++ b/server/mailet/integration-testing/src/test/java/org/apache/james/mailets/SmtpAuthIntegrationTest.java
@@ -50,7 +50,7 @@ import org.junit.jupiter.api.Test;
 import org.junit.jupiter.api.extension.RegisterExtension;
 import org.junit.jupiter.api.io.TempDir;
 
-public class SmtpAuthIntegrationTest {
+class SmtpAuthIntegrationTest {
     private static final String FROM = "fromuser@" + DEFAULT_DOMAIN;
     private static final MailRepositoryUrl DROPPED_MAILS = MailRepositoryUrl.from("memory://var/mail/dropped-mails/");
 
@@ -63,7 +63,7 @@ public class SmtpAuthIntegrationTest {
     private MailRepositoryProbeImpl repositoryProbe;
 
     @BeforeEach
-    public void setup(@TempDir File temporaryFolder) throws Exception {
+    void setup(@TempDir File temporaryFolder) throws Exception {
         ProcessorConfiguration.Builder rootProcessor = ProcessorConfiguration.root()
             .addMailet(MailetConfiguration.builder()
                 .matcher(SMTPAuthSuccessful.class)
@@ -97,12 +97,12 @@ public class SmtpAuthIntegrationTest {
     }
 
     @AfterEach
-    public void tearDown() {
+    void tearDown() {
         jamesServer.shutdown();
     }
 
     @Test
-    public void authenticatedSmtpSessionsShouldBeDelivered() throws Exception {
+    void authenticatedSmtpSessionsShouldBeDelivered() throws Exception {
         messageSender.connect(LOCALHOST_IP, jamesServer.getProbe(SmtpGuiceProbe.class).getSmtpPort())
             .authenticate(FROM, PASSWORD)
             .sendMessage(FROM, FROM);
@@ -114,7 +114,7 @@ public class SmtpAuthIntegrationTest {
     }
 
     @Test
-    public void nonAuthenticatedSmtpSessionsShouldNotBeDelivered() throws Exception {
+    void nonAuthenticatedSmtpSessionsShouldNotBeDelivered() throws Exception {
         messageSender.connect(LOCALHOST_IP, jamesServer.getProbe(SmtpGuiceProbe.class).getSmtpPort())
             .sendMessage(FROM, FROM);
 
@@ -128,7 +128,7 @@ public class SmtpAuthIntegrationTest {
     }
 
     @Test
-    public void mixedCaseSenderMailShouldBeDelivered() throws Exception {
+    void mixedCaseSenderMailShouldBeDelivered() throws Exception {
         messageSender.connect(LOCALHOST_IP, jamesServer.getProbe(SmtpGuiceProbe.class).getSmtpPort())
             .authenticate(FROM, PASSWORD)
             .sendMessage("FROMUSER@" + DEFAULT_DOMAIN, FROM);
diff --git a/server/mailet/integration-testing/src/test/java/org/apache/james/smtp/DSNRelayTest.java b/server/mailet/integration-testing/src/test/java/org/apache/james/smtp/DSNRelayTest.java
index 2e49c37..c7afc79 100644
--- a/server/mailet/integration-testing/src/test/java/org/apache/james/smtp/DSNRelayTest.java
+++ b/server/mailet/integration-testing/src/test/java/org/apache/james/smtp/DSNRelayTest.java
@@ -66,7 +66,7 @@ import org.junit.jupiter.api.io.TempDir;
 import org.slf4j.Logger;
 import org.slf4j.LoggerFactory;
 
-public class DSNRelayTest {
+class DSNRelayTest {
     private static final Logger LOGGER = LoggerFactory.getLogger(DSNRelayTest.class);
 
     private static final String ANOTHER_DOMAIN = "other.com";
@@ -89,7 +89,7 @@ public class DSNRelayTest {
     private TemporaryJamesServer jamesServer;
 
     @BeforeEach
-    public void setUp(@TempDir File temporaryFolder) throws Exception {
+    void setUp(@TempDir File temporaryFolder) throws Exception {
         inMemoryDNSService = new InMemoryDNSService()
             .registerMxRecord(DEFAULT_DOMAIN, LOCALHOST_IP)
             .registerMxRecord(ANOTHER_DOMAIN, mockSmtp.getContainerIp());
@@ -122,7 +122,7 @@ public class DSNRelayTest {
     }
 
     @AfterEach
-    public void tearDown() {
+    void tearDown() {
         jamesServer.shutdown();
 
         mockSMTPConfiguration.cleanServer();
@@ -130,7 +130,7 @@ public class DSNRelayTest {
 
     @Disabled("JAMES-3431 No javax.mail support for ORCPT DSN parameter...")
     @Test
-    public void orcptIsUnsupported() throws Exception {
+    void orcptIsUnsupported() throws Exception {
         AuthenticatingSMTPClient smtpClient = new AuthenticatingSMTPClient("TLS", "UTF-8");
 
         try {
@@ -171,7 +171,7 @@ public class DSNRelayTest {
     }
 
     @Test
-    public void remoteDeliveryShouldCarryOverDSNParameters() throws Exception {
+    void remoteDeliveryShouldCarryOverDSNParameters() throws Exception {
         AuthenticatingSMTPClient smtpClient = new AuthenticatingSMTPClient("TLS", "UTF-8");
 
         try {
@@ -242,7 +242,7 @@ public class DSNRelayTest {
     }
 
     @Test
-    public void remoteDeliveryShouldDeliverSimilarDsnNotifyParametersTogether() throws Exception {
+    void remoteDeliveryShouldDeliverSimilarDsnNotifyParametersTogether() throws Exception {
         AuthenticatingSMTPClient smtpClient = new AuthenticatingSMTPClient("TLS", "UTF-8");
 
         try {
@@ -290,7 +290,7 @@ public class DSNRelayTest {
     }
 
     @Test
-    public void remoteDeliveryShouldCarryOverDSNParametersWhenSingleRecipient() throws Exception {
+    void remoteDeliveryShouldCarryOverDSNParametersWhenSingleRecipient() throws Exception {
         AuthenticatingSMTPClient smtpClient = new AuthenticatingSMTPClient("TLS", "UTF-8");
 
         try {
@@ -327,7 +327,7 @@ public class DSNRelayTest {
     }
 
     @Test
-    public void dsnShouldBeCarriedAfterRRT() throws Exception {
+    void dsnShouldBeCarriedAfterRRT() throws Exception {
         DataProbeImpl dataProbe = jamesServer.getProbe(DataProbeImpl.class);
         dataProbe.addDomain(ANOTHER_DOMAIN);
         dataProbe.addAddressMapping("touser", ANOTHER_DOMAIN, "touser-alias@other.com");
@@ -368,7 +368,7 @@ public class DSNRelayTest {
     }
 
     @Test
-    public void remoteDeliveryShouldCarryOverDSNMailParameters() throws Exception {
+    void remoteDeliveryShouldCarryOverDSNMailParameters() throws Exception {
         AuthenticatingSMTPClient smtpClient = new AuthenticatingSMTPClient("TLS", "UTF-8");
 
         try {
@@ -400,7 +400,6 @@ public class DSNRelayTest {
                 .build()));
     }
 
-
     private ProcessorConfiguration.Builder directResolutionTransport() {
         return ProcessorConfiguration.transport()
             .addMailet(MailetConfiguration.BCC_STRIPPER)
diff --git a/server/mailet/integration-testing/src/test/java/org/apache/james/smtp/SmtpAuthorizedAddressesTest.java b/server/mailet/integration-testing/src/test/java/org/apache/james/smtp/SmtpAuthorizedAddressesTest.java
index 256bead..6019431 100644
--- a/server/mailet/integration-testing/src/test/java/org/apache/james/smtp/SmtpAuthorizedAddressesTest.java
+++ b/server/mailet/integration-testing/src/test/java/org/apache/james/smtp/SmtpAuthorizedAddressesTest.java
@@ -51,7 +51,7 @@ import org.junit.jupiter.api.Test;
 import org.junit.jupiter.api.extension.RegisterExtension;
 import org.junit.jupiter.api.io.TempDir;
 
-public class SmtpAuthorizedAddressesTest {
+class SmtpAuthorizedAddressesTest {
     private static final String FROM = "fromuser@" + DEFAULT_DOMAIN;
     private static final String TO = "to@any.com";
 
@@ -86,7 +86,7 @@ public class SmtpAuthorizedAddressesTest {
     }
 
     @AfterEach
-    public void tearDown() {
+    void tearDown() {
         fakeSmtp.clean();
         if (jamesServer != null) {
             jamesServer.shutdown();
@@ -94,7 +94,7 @@ public class SmtpAuthorizedAddressesTest {
     }
 
     @Test
-    public void userShouldBeAbleToRelayMessagesWhenInAcceptedNetwork(@TempDir File temporaryFolder) throws Exception {
+    void userShouldBeAbleToRelayMessagesWhenInAcceptedNetwork(@TempDir File temporaryFolder) throws Exception {
         createJamesServer(temporaryFolder, SmtpConfiguration.builder()
             .requireAuthentication()
             .withAutorizedAddresses("127.0.0.0/8"));
@@ -110,7 +110,7 @@ public class SmtpAuthorizedAddressesTest {
     }
 
     @Test
-    public void userShouldNotBeAbleToRelayMessagesWhenOutOfAcceptedNetwork(@TempDir File temporaryFolder) throws Exception {
+    void userShouldNotBeAbleToRelayMessagesWhenOutOfAcceptedNetwork(@TempDir File temporaryFolder) throws Exception {
         createJamesServer(temporaryFolder, SmtpConfiguration.builder()
             .requireAuthentication()
             .withAutorizedAddresses("172.0.0.0/8"));
@@ -122,7 +122,7 @@ public class SmtpAuthorizedAddressesTest {
     }
 
     @Test
-    public void userShouldBeAbleToRelayMessagesWhenOutOfAcceptedNetworkButAuthenticated(@TempDir File temporaryFolder) throws Exception {
+    void userShouldBeAbleToRelayMessagesWhenOutOfAcceptedNetworkButAuthenticated(@TempDir File temporaryFolder) throws Exception {
         createJamesServer(temporaryFolder, SmtpConfiguration.builder()
             .requireAuthentication()
             .withAutorizedAddresses("172.0.0.0/8"));
@@ -139,7 +139,7 @@ public class SmtpAuthorizedAddressesTest {
     }
 
     @Test
-    public void localDeliveryShouldBePossibleFromNonAuthenticatedNonAuthorizedSender(@TempDir File temporaryFolder) throws Exception {
+    void localDeliveryShouldBePossibleFromNonAuthenticatedNonAuthorizedSender(@TempDir File temporaryFolder) throws Exception {
         createJamesServer(temporaryFolder, SmtpConfiguration.builder()
             .requireAuthentication()
             .withAutorizedAddresses("172.0.0.0/8"));
diff --git a/server/mailet/integration-testing/src/test/java/org/apache/james/smtp/SmtpBracketEnforcementTest.java b/server/mailet/integration-testing/src/test/java/org/apache/james/smtp/SmtpBracketEnforcementTest.java
index f95cfa3..915cc78 100644
--- a/server/mailet/integration-testing/src/test/java/org/apache/james/smtp/SmtpBracketEnforcementTest.java
+++ b/server/mailet/integration-testing/src/test/java/org/apache/james/smtp/SmtpBracketEnforcementTest.java
@@ -39,7 +39,7 @@ import org.junit.jupiter.api.Test;
 import org.junit.jupiter.api.extension.RegisterExtension;
 import org.junit.jupiter.api.io.TempDir;
 
-public class SmtpBracketEnforcementTest {
+class SmtpBracketEnforcementTest {
     private static final String USER = "user@" + DEFAULT_DOMAIN;
 
     @RegisterExtension
@@ -59,14 +59,14 @@ public class SmtpBracketEnforcementTest {
     }
 
     @AfterEach
-    public void tearDown() {
+    void tearDown() {
         if (jamesServer != null) {
             jamesServer.shutdown();
         }
     }
 
     @Test
-    public void recipientWithBracketsShouldBeAcceptedWhenNoBracketRequired(@TempDir File temporaryFolder) throws Exception {
+    void recipientWithBracketsShouldBeAcceptedWhenNoBracketRequired(@TempDir File temporaryFolder) throws Exception {
         createJamesServer(temporaryFolder, SmtpConfiguration.builder()
             .doNotRequireBracketEnforcement());
 
@@ -76,7 +76,7 @@ public class SmtpBracketEnforcementTest {
     }
 
     @Test
-    public void recipientWithNoBracketsShouldBeAcceptedWhenNoBracketRequired(@TempDir File temporaryFolder) throws Exception {
+    void recipientWithNoBracketsShouldBeAcceptedWhenNoBracketRequired(@TempDir File temporaryFolder) throws Exception {
         createJamesServer(temporaryFolder, SmtpConfiguration.builder()
             .doNotRequireBracketEnforcement());
 
@@ -86,7 +86,7 @@ public class SmtpBracketEnforcementTest {
     }
 
     @Test
-    public void recipientWithBracketsShouldBeAcceptedWhenBracketRequired(@TempDir File temporaryFolder) throws Exception {
+    void recipientWithBracketsShouldBeAcceptedWhenBracketRequired(@TempDir File temporaryFolder) throws Exception {
         createJamesServer(temporaryFolder, SmtpConfiguration.builder()
             .requireBracketEnforcement());
 
@@ -96,7 +96,7 @@ public class SmtpBracketEnforcementTest {
     }
 
     @Test
-    public void recipientWithNoBracketsShouldBeRejectedWhenBracketRequired(@TempDir File temporaryFolder) throws Exception {
+    void recipientWithNoBracketsShouldBeRejectedWhenBracketRequired(@TempDir File temporaryFolder) throws Exception {
         createJamesServer(temporaryFolder, SmtpConfiguration.builder()
             .requireBracketEnforcement());
 
diff --git a/server/mailet/integration-testing/src/test/java/org/apache/james/smtp/SmtpContentTypeTest.java b/server/mailet/integration-testing/src/test/java/org/apache/james/smtp/SmtpContentTypeTest.java
index e6b8898..1191990 100644
--- a/server/mailet/integration-testing/src/test/java/org/apache/james/smtp/SmtpContentTypeTest.java
+++ b/server/mailet/integration-testing/src/test/java/org/apache/james/smtp/SmtpContentTypeTest.java
@@ -54,7 +54,7 @@ import org.junit.jupiter.api.Test;
 import org.junit.jupiter.api.extension.RegisterExtension;
 import org.junit.jupiter.api.io.TempDir;
 
-public class SmtpContentTypeTest {
+class SmtpContentTypeTest {
     private static final String FROM = "fromuser@" + DEFAULT_DOMAIN;
     private static final String TO = "to@any.com";
     public static final String SUBJECT = "test";
@@ -90,7 +90,7 @@ public class SmtpContentTypeTest {
     }
 
     @AfterEach
-    public void tearDown() {
+    void tearDown() {
         fakeSmtp.clean();
         if (jamesServer != null) {
             jamesServer.shutdown();
@@ -98,7 +98,7 @@ public class SmtpContentTypeTest {
     }
 
     @Test
-    public void userShouldBeAbleToReceiveMessagesWithGoodContentType(@TempDir File temporaryFolder) throws Exception {
+    void userShouldBeAbleToReceiveMessagesWithGoodContentType(@TempDir File temporaryFolder) throws Exception {
         createJamesServer(temporaryFolder, SmtpConfiguration.builder()
             .requireAuthentication()
             .withAutorizedAddresses("172.0.0.0/8"));
@@ -116,7 +116,7 @@ public class SmtpContentTypeTest {
     }
 
     @Test
-    public void userShouldBeAbleToReceiveMessagesWithBadContentType(@TempDir File temporaryFolder) throws Exception {
+    void userShouldBeAbleToReceiveMessagesWithBadContentType(@TempDir File temporaryFolder) throws Exception {
         createJamesServer(temporaryFolder, SmtpConfiguration.builder()
             .requireAuthentication()
             .withAutorizedAddresses("172.0.0.0/8"));
diff --git a/server/mailet/integration-testing/src/test/java/org/apache/james/smtp/SmtpIdentityVerificationTest.java b/server/mailet/integration-testing/src/test/java/org/apache/james/smtp/SmtpIdentityVerificationTest.java
index cb9173b..61b21c7 100644
--- a/server/mailet/integration-testing/src/test/java/org/apache/james/smtp/SmtpIdentityVerificationTest.java
+++ b/server/mailet/integration-testing/src/test/java/org/apache/james/smtp/SmtpIdentityVerificationTest.java
@@ -39,7 +39,7 @@ import org.junit.jupiter.api.Test;
 import org.junit.jupiter.api.extension.RegisterExtension;
 import org.junit.jupiter.api.io.TempDir;
 
-public class SmtpIdentityVerificationTest {
+class SmtpIdentityVerificationTest {
     private static final String ATTACKER_PASSWORD = "secret";
 
     private static final String ATTACKER = "attacker@" + DEFAULT_DOMAIN;
@@ -63,14 +63,14 @@ public class SmtpIdentityVerificationTest {
     }
 
     @AfterEach
-    public void tearDown() {
+    void tearDown() {
         if (jamesServer != null) {
             jamesServer.shutdown();
         }
     }
 
     @Test
-    public void smtpShouldAcceptMessageWhenIdentityIsMatching(@TempDir File temporaryFolder) throws Exception {
+    void smtpShouldAcceptMessageWhenIdentityIsMatching(@TempDir File temporaryFolder) throws Exception {
         createJamesServer(temporaryFolder, SmtpConfiguration.builder()
             .requireAuthentication()
             .verifyIdentity());
@@ -80,7 +80,7 @@ public class SmtpIdentityVerificationTest {
     }
 
     @Test
-    public void smtpShouldAcceptMessageWhenIdentityIsNotMatchingButNotChecked(@TempDir File temporaryFolder) throws Exception {
+    void smtpShouldAcceptMessageWhenIdentityIsNotMatchingButNotChecked(@TempDir File temporaryFolder) throws Exception {
         createJamesServer(temporaryFolder, SmtpConfiguration.builder()
             .requireAuthentication()
             .doNotVerifyIdentity());
@@ -91,7 +91,7 @@ public class SmtpIdentityVerificationTest {
     }
 
     @Test
-    public void smtpShouldRejectMessageWhenIdentityIsNotMatching(@TempDir File temporaryFolder) throws Exception {
+    void smtpShouldRejectMessageWhenIdentityIsNotMatching(@TempDir File temporaryFolder) throws Exception {
         createJamesServer(temporaryFolder, SmtpConfiguration.builder()
             .requireAuthentication()
             .verifyIdentity());
@@ -102,5 +102,4 @@ public class SmtpIdentityVerificationTest {
                 .sendMessage(USER, USER))
             .isEqualTo(new SMTPSendingException(SmtpSendingStep.RCPT, "503 5.7.1 Incorrect Authentication for Specified Email Address\n"));
     }
-
 }
diff --git a/server/mailet/integration-testing/src/test/java/org/apache/james/smtp/SmtpNullSenderTest.java b/server/mailet/integration-testing/src/test/java/org/apache/james/smtp/SmtpNullSenderTest.java
index 5099f2c..01c6968 100644
--- a/server/mailet/integration-testing/src/test/java/org/apache/james/smtp/SmtpNullSenderTest.java
+++ b/server/mailet/integration-testing/src/test/java/org/apache/james/smtp/SmtpNullSenderTest.java
@@ -39,7 +39,7 @@ import org.junit.jupiter.api.Test;
 import org.junit.jupiter.api.extension.RegisterExtension;
 import org.junit.jupiter.api.io.TempDir;
 
-public class SmtpNullSenderTest {
+class SmtpNullSenderTest {
     private static final String USER = "user@" + DEFAULT_DOMAIN;
 
     @RegisterExtension
@@ -50,7 +50,7 @@ public class SmtpNullSenderTest {
     private TemporaryJamesServer jamesServer;
 
     @BeforeEach
-    public void setUp(@TempDir File temporaryFolder) throws Exception {
+    void setUp(@TempDir File temporaryFolder) throws Exception {
         jamesServer = TemporaryJamesServer.builder()
             .build(temporaryFolder);
         jamesServer.start();
@@ -61,18 +61,18 @@ public class SmtpNullSenderTest {
     }
 
     @AfterEach
-    public void tearDown() {
+    void tearDown() {
         jamesServer.shutdown();
     }
 
     @Test
-    public void smtpLayerShouldAcceptEmptySenders() throws Exception {
+    void smtpLayerShouldAcceptEmptySenders() throws Exception {
         messageSender.connect(LOCALHOST_IP, jamesServer.getProbe(SmtpGuiceProbe.class).getSmtpPort())
             .sendMessageWithHeaders("", USER,"Short message");
     }
 
     @Test
-    public void mailProcessingShouldDeliverNullSender() throws Exception {
+    void mailProcessingShouldDeliverNullSender() throws Exception {
         messageSender.connect(LOCALHOST_IP, jamesServer.getProbe(SmtpGuiceProbe.class).getSmtpPort())
             .sendMessageWithHeaders("", USER,"Short message");
 
diff --git a/server/mailet/integration-testing/src/test/java/org/apache/james/smtp/SmtpRandomStoringTest.java b/server/mailet/integration-testing/src/test/java/org/apache/james/smtp/SmtpRandomStoringTest.java
index 753bbef..675a192 100644
--- a/server/mailet/integration-testing/src/test/java/org/apache/james/smtp/SmtpRandomStoringTest.java
+++ b/server/mailet/integration-testing/src/test/java/org/apache/james/smtp/SmtpRandomStoringTest.java
@@ -66,7 +66,7 @@ import com.github.fge.lambdas.Throwing;
 import com.github.steveash.guavate.Guavate;
 import com.google.common.collect.ImmutableList;
 
-public class SmtpRandomStoringTest {
+class SmtpRandomStoringTest {
     private static final String FROM = "from@" + DEFAULT_DOMAIN;
     private static final String TO = "to@any.com";
     private static final Long USERS_NUMBERS = 10L;
@@ -93,7 +93,7 @@ public class SmtpRandomStoringTest {
     private Collection<TestIMAPClient> connections;
 
     @BeforeEach
-    public void setUp(@TempDir File temporaryFolder) throws Exception {
+    void setUp(@TempDir File temporaryFolder) throws Exception {
         createJamesServer(temporaryFolder);
 
         createUsersAndMailboxes();
@@ -155,13 +155,13 @@ public class SmtpRandomStoringTest {
     }
 
     @AfterEach
-    public void tearDown() {
+    void tearDown() {
         connections.forEach(Throwing.consumer(TestIMAPClient::close).sneakyThrow());
         jamesServer.shutdown();
     }
 
     @Test
-    public void oneHundredMailsShouldHaveBeenStoredBetweenFourAndEightTimes() {
+    void oneHundredMailsShouldHaveBeenStoredBetweenFourAndEightTimes() {
         connections = USERS
             .stream()
             .map(this::createIMAPConnection)
@@ -172,7 +172,7 @@ public class SmtpRandomStoringTest {
     }
 
     @Test
-    public void messagesShouldBeRandomlyAssignedToEveryMailboxesOfEveryUsers() {
+    void messagesShouldBeRandomlyAssignedToEveryMailboxesOfEveryUsers() {
         connections = USERS
             .stream()
             .map(this::createIMAPConnection)
diff --git a/server/mailet/integration-testing/src/test/java/org/apache/james/smtp/SmtpSizeLimitationTest.java b/server/mailet/integration-testing/src/test/java/org/apache/james/smtp/SmtpSizeLimitationTest.java
index a41c288..9b73aaf 100644
--- a/server/mailet/integration-testing/src/test/java/org/apache/james/smtp/SmtpSizeLimitationTest.java
+++ b/server/mailet/integration-testing/src/test/java/org/apache/james/smtp/SmtpSizeLimitationTest.java
@@ -41,7 +41,7 @@ import org.junit.jupiter.api.io.TempDir;
 
 import com.google.common.base.Strings;
 
-public class SmtpSizeLimitationTest {
+class SmtpSizeLimitationTest {
     private static final String USER = "user@" + DEFAULT_DOMAIN;
 
     @RegisterExtension
@@ -61,14 +61,14 @@ public class SmtpSizeLimitationTest {
     }
 
     @AfterEach
-    public void tearDown() {
+    void tearDown() {
         if (jamesServer != null) {
             jamesServer.shutdown();
         }
     }
 
     @Test
-    public void messageShouldNotBeAcceptedWhenOverSized(@TempDir File temporaryFolder) throws Exception {
+    void messageShouldNotBeAcceptedWhenOverSized(@TempDir File temporaryFolder) throws Exception {
         createJamesServer(temporaryFolder, SmtpConfiguration.builder()
             .doNotVerifyIdentity()
             .withMaxMessageSizeInKb(10));
@@ -81,7 +81,7 @@ public class SmtpSizeLimitationTest {
     }
 
     @Test
-    public void messageShouldBeAcceptedWhenNotOverSized(@TempDir File temporaryFolder) throws Exception {
+    void messageShouldBeAcceptedWhenNotOverSized(@TempDir File temporaryFolder) throws Exception {
         createJamesServer(temporaryFolder, SmtpConfiguration.builder()
             .doNotVerifyIdentity()
             .withMaxMessageSizeInKb(10));
diff --git a/server/mailet/integration-testing/src/test/java/org/apache/james/transport/mailets/AddFooterTest.java b/server/mailet/integration-testing/src/test/java/org/apache/james/transport/mailets/AddFooterTest.java
index 02c8bfb..e2c1cee 100644
--- a/server/mailet/integration-testing/src/test/java/org/apache/james/transport/mailets/AddFooterTest.java
+++ b/server/mailet/integration-testing/src/test/java/org/apache/james/transport/mailets/AddFooterTest.java
@@ -49,7 +49,7 @@ import org.junit.jupiter.api.Test;
 import org.junit.jupiter.api.extension.RegisterExtension;
 import org.junit.jupiter.api.io.TempDir;
 
-public class AddFooterTest {
+class AddFooterTest {
     public static final String MATCH_ME = "Match me";
 
     @RegisterExtension
@@ -60,7 +60,7 @@ public class AddFooterTest {
     private TemporaryJamesServer jamesServer;
 
     @BeforeEach
-    public void setup(@TempDir File temporaryFolder) throws Exception {
+    void setup(@TempDir File temporaryFolder) throws Exception {
         jamesServer = TemporaryJamesServer.builder()
             .withOverrides(new ActiveMQQueueModule())
             .withMailetContainer(TemporaryJamesServer.simpleMailetContainerConfiguration()
@@ -79,12 +79,12 @@ public class AddFooterTest {
     }
 
     @AfterEach
-    public void tearDown() {
+    void tearDown() {
         jamesServer.shutdown();
     }
 
     @Test
-    public void shouldAddFooterWhenSimpleMessage() throws Exception {
+    void shouldAddFooterWhenSimpleMessage() throws Exception {
         smtpMessageSender.connect(LOCALHOST_IP, jamesServer.getProbe(SmtpGuiceProbe.class).getSmtpPort())
             .sendMessage(FakeMail.builder()
                 .name("name")
@@ -103,7 +103,7 @@ public class AddFooterTest {
     }
 
     @Test
-    public void shouldAddFooterWhenMultipartMessage() throws Exception {
+    void shouldAddFooterWhenMultipartMessage() throws Exception {
         smtpMessageSender.connect(LOCALHOST_IP, jamesServer.getProbe(SmtpGuiceProbe.class).getSmtpPort())
             .sendMessage(FakeMail.builder()
                 .name("name")
diff --git a/server/mailet/integration-testing/src/test/java/org/apache/james/transport/mailets/AliasMappingTest.java b/server/mailet/integration-testing/src/test/java/org/apache/james/transport/mailets/AliasMappingTest.java
index 7a4ba04..9fb9b07 100644
--- a/server/mailet/integration-testing/src/test/java/org/apache/james/transport/mailets/AliasMappingTest.java
+++ b/server/mailet/integration-testing/src/test/java/org/apache/james/transport/mailets/AliasMappingTest.java
@@ -58,7 +58,7 @@ import org.junit.jupiter.api.io.TempDir;
 
 import io.restassured.specification.RequestSpecification;
 
-public class AliasMappingTest {
+class AliasMappingTest {
     private static final String DOMAIN = "domain.tld";
     private static final String DOMAIN_2 = "domain2.tld";
 
@@ -93,7 +93,7 @@ public class AliasMappingTest {
     public SMTPMessageSender messageSender = new SMTPMessageSender(DEFAULT_DOMAIN);
 
     @BeforeEach
-    public void setup(@TempDir File temporaryFolder) throws Exception {
+    void setup(@TempDir File temporaryFolder) throws Exception {
         MailetContainer.Builder mailetContainer = TemporaryJamesServer.simpleMailetContainerConfiguration()
             .putProcessor(CommonProcessors.rrtErrorEnabledTransport())
             .putProcessor(CommonProcessors.rrtErrorProcessor());
@@ -124,12 +124,12 @@ public class AliasMappingTest {
     }
 
     @AfterEach
-    public void tearDown() {
+    void tearDown() {
         jamesServer.shutdown();
     }
 
     @Test
-    public void messageShouldRedirectToUserWhenSentToHisAlias() throws Exception {
+    void messageShouldRedirectToUserWhenSentToHisAlias() throws Exception {
         webAdminApi.put(AliasRoutes.ROOT_PATH + "/" + BOB_ADDRESS + "/sources/" + BOB_ALIAS);
 
         messageSender.connect(LOCALHOST_IP, jamesServer.getProbe(SmtpGuiceProbe.class).getSmtpPort())
@@ -147,7 +147,7 @@ public class AliasMappingTest {
     }
 
     @Test
-    public void messageShouldRedirectToUserWhenSentToHisUpperCaseAlias() throws Exception {
+    void messageShouldRedirectToUserWhenSentToHisUpperCaseAlias() throws Exception {
         webAdminApi.put(AliasRoutes.ROOT_PATH + "/" + BOB_ADDRESS + "/sources/" + BOB_ALIAS);
 
         messageSender.connect(LOCALHOST_IP, jamesServer.getProbe(SmtpGuiceProbe.class).getSmtpPort())
@@ -165,7 +165,7 @@ public class AliasMappingTest {
     }
 
     @Test
-    public void messageShouldRedirectToUserWhenLowerCaseAliasMappedToUpperCaseUser() throws Exception {
+    void messageShouldRedirectToUserWhenLowerCaseAliasMappedToUpperCaseUser() throws Exception {
         webAdminApi.put(AliasRoutes.ROOT_PATH + "/" + UPPER_BOB_ADDRESS + "/sources/" + BOB_ALIAS);
 
         messageSender.connect(LOCALHOST_IP, jamesServer.getProbe(SmtpGuiceProbe.class).getSmtpPort())
@@ -183,7 +183,7 @@ public class AliasMappingTest {
     }
 
     @Test
-    public void messageShouldRedirectToUserWithUpperCaseDefinedAliasWhenSentToHisLowerCaseAlias() throws Exception {
+    void messageShouldRedirectToUserWithUpperCaseDefinedAliasWhenSentToHisLowerCaseAlias() throws Exception {
         webAdminApi.put(AliasRoutes.ROOT_PATH + "/" + BOB_ADDRESS + "/sources/" + UPPER_BOB_ALIAS);
 
         messageSender.connect(LOCALHOST_IP, jamesServer.getProbe(SmtpGuiceProbe.class).getSmtpPort())
@@ -201,7 +201,7 @@ public class AliasMappingTest {
     }
 
     @Test
-    public void messageShouldRedirectToForwardOfUserWhenSentToHisAlias() throws Exception {
+    void messageShouldRedirectToForwardOfUserWhenSentToHisAlias() throws Exception {
         webAdminApi.put(AliasRoutes.ROOT_PATH + "/" + BOB_ADDRESS + "/sources/" + BOB_ALIAS);
         webAdminApi.put(ForwardRoutes.ROOT_PATH + "/" + BOB_ADDRESS + "/targets/" + CEDRIC_ADDRESS);
 
@@ -220,7 +220,7 @@ public class AliasMappingTest {
     }
 
     @Test
-    public void messageShouldRedirectToUserWhenForwardedToHisAlias() throws Exception {
+    void messageShouldRedirectToUserWhenForwardedToHisAlias() throws Exception {
         webAdminApi.put(AliasRoutes.ROOT_PATH + "/" + BOB_ADDRESS + "/sources/" + BOB_ALIAS);
         webAdminApi.put(ForwardRoutes.ROOT_PATH + "/" + ALICE_ADDRESS + "/targets/" + BOB_ALIAS);
 
@@ -239,7 +239,7 @@ public class AliasMappingTest {
     }
 
     @Test
-    public void messageShouldRedirectToUserWhenHisAliasIsPartOfGroup() throws Exception {
+    void messageShouldRedirectToUserWhenHisAliasIsPartOfGroup() throws Exception {
         webAdminApi.put(AliasRoutes.ROOT_PATH + "/" + BOB_ADDRESS + "/sources/" + BOB_ALIAS);
         webAdminApi.put(GroupsRoutes.ROOT_PATH + "/" + GROUP_ADDRESS + "/" + BOB_ALIAS);
 
@@ -258,7 +258,7 @@ public class AliasMappingTest {
     }
 
     @Test
-    public void messageShouldRedirectToMembersWhenSentToGroupAlias() throws Exception {
+    void messageShouldRedirectToMembersWhenSentToGroupAlias() throws Exception {
         webAdminApi.put(AliasRoutes.ROOT_PATH + "/" + GROUP_ADDRESS + "/sources/" + GROUP_ALIAS);
         webAdminApi.put(GroupsRoutes.ROOT_PATH + "/" + GROUP_ADDRESS + "/" + BOB_ADDRESS);
 
@@ -277,7 +277,7 @@ public class AliasMappingTest {
     }
 
     @Test
-    public void messageShouldRedirectToUserWithAliasesCascading() throws Exception {
+    void messageShouldRedirectToUserWithAliasesCascading() throws Exception {
         webAdminApi.put(AliasRoutes.ROOT_PATH + "/" + BOB_ADDRESS + "/sources/" + BOB_ALIAS);
         webAdminApi.put(AliasRoutes.ROOT_PATH + "/" + BOB_ALIAS + "/sources/" + BOB_ALIAS_2);
 
@@ -296,7 +296,7 @@ public class AliasMappingTest {
     }
 
     @Test
-    public void messageShouldRedirectToUsersSharingSameAlias() throws Exception {
+    void messageShouldRedirectToUsersSharingSameAlias() throws Exception {
         webAdminApi.put(AliasRoutes.ROOT_PATH + "/" + BOB_ADDRESS + "/sources/" + BOB_ALIAS);
         webAdminApi.put(AliasRoutes.ROOT_PATH + "/" + ALICE_ADDRESS + "/sources/" + BOB_ALIAS);
 
@@ -320,9 +320,8 @@ public class AliasMappingTest {
         assertThat(testIMAPClient.readFirstMessage()).contains(MESSAGE_CONTENT);
     }
 
-
     @Test
-    public void messageShouldRedirectFromAliasContainingSlash() throws Exception {
+    void messageShouldRedirectFromAliasContainingSlash() throws Exception {
         String aliasWithSlash = "bob/alias@" + DOMAIN;
         String aliasWithEncodedSlash = "bob%2Falias@" + DOMAIN;
         webAdminApi.put(AliasRoutes.ROOT_PATH + "/" + BOB_ADDRESS + "/sources/" + aliasWithEncodedSlash);
@@ -341,7 +340,7 @@ public class AliasMappingTest {
     }
 
     @Test
-    public void messageShouldRedirectToUserContainingSlash() throws Exception {
+    void messageShouldRedirectToUserContainingSlash() throws Exception {
         String userWithSlash = "bob/a@" + DOMAIN;
         dataProbe.addUser(userWithSlash, PASSWORD);
         String userWithEncodedSlash = "bob%2Fa@" + DOMAIN;
@@ -361,7 +360,7 @@ public class AliasMappingTest {
     }
 
     @Test
-    public void messageShouldRedirectToUserWhenEncodingAt() throws Exception {
+    void messageShouldRedirectToUserWhenEncodingAt() throws Exception {
         String userWithEncodedAt = "bob%40" + DOMAIN;
         String aliasWithEncodedAt = "bob-alias%40" + DOMAIN;
         webAdminApi.put(AliasRoutes.ROOT_PATH + "/" + userWithEncodedAt + "/sources/" + aliasWithEncodedAt);
@@ -380,7 +379,7 @@ public class AliasMappingTest {
     }
 
     @Test
-    public void messageShouldBeStoredInRepositoryWhenAliasLoopMapping() throws Exception {
+    void messageShouldBeStoredInRepositoryWhenAliasLoopMapping() throws Exception {
         String bobAlias2 = BOB_USER + "@" + DOMAIN_2;
 
         webAdminApi.put(AliasRoutes.ROOT_PATH + "/" + BOB_ADDRESS + "/sources/" + bobAlias2);
@@ -399,7 +398,7 @@ public class AliasMappingTest {
     }
 
     @Test
-    public void userShouldNotReceiveDuplicatesWhenUserAndAliasRegisteredToAGroup() throws Exception {
+    void userShouldNotReceiveDuplicatesWhenUserAndAliasRegisteredToAGroup() throws Exception {
         webAdminApi.put(AliasRoutes.ROOT_PATH + "/" + BOB_ADDRESS + "/sources/" + BOB_ALIAS);
         webAdminApi.put(GroupsRoutes.ROOT_PATH + "/" + GROUP_ADDRESS + "/" + BOB_ADDRESS);
         webAdminApi.put(GroupsRoutes.ROOT_PATH + "/" + GROUP_ADDRESS + "/" + BOB_ALIAS);
@@ -416,5 +415,4 @@ public class AliasMappingTest {
             .select(TestIMAPClient.INBOX)
             .awaitMessageCount(awaitAtMostOneMinute, 1);
     }
-
 }
diff --git a/server/mailet/integration-testing/src/test/java/org/apache/james/transport/mailets/AmqpForwardAttachmentTest.java b/server/mailet/integration-testing/src/test/java/org/apache/james/transport/mailets/AmqpForwardAttachmentTest.java
index d03be06..eea1afe 100644
--- a/server/mailet/integration-testing/src/test/java/org/apache/james/transport/mailets/AmqpForwardAttachmentTest.java
+++ b/server/mailet/integration-testing/src/test/java/org/apache/james/transport/mailets/AmqpForwardAttachmentTest.java
@@ -50,7 +50,7 @@ import org.junit.jupiter.api.Test;
 import org.junit.jupiter.api.extension.RegisterExtension;
 import org.junit.jupiter.api.io.TempDir;
 
-public class AmqpForwardAttachmentTest {
+class AmqpForwardAttachmentTest {
     private static final String FROM = "fromUser@" + DEFAULT_DOMAIN;
     private static final String RECIPIENT = "touser@" + DEFAULT_DOMAIN;
     
@@ -70,7 +70,7 @@ public class AmqpForwardAttachmentTest {
     private TemporaryJamesServer jamesServer;
 
     @BeforeEach
-    public void setup(@TempDir File temporaryFolder) throws Exception {
+    void setup(@TempDir File temporaryFolder) throws Exception {
         MailetContainer.Builder mailetContainer = TemporaryJamesServer.defaultMailetContainerConfiguration()
             .putProcessor(ProcessorConfiguration.transport()
                 .addMailet(MailetConfiguration.builder()
@@ -103,12 +103,12 @@ public class AmqpForwardAttachmentTest {
     }
 
     @AfterEach
-    public void tearDown() throws Exception {
+    void tearDown() {
         jamesServer.shutdown();
     }
 
     @Test
-    public void stripAttachmentShouldPutAttachmentsInMailAttributeWhenConfiguredForIt() throws Exception {
+    void stripAttachmentShouldPutAttachmentsInMailAttributeWhenConfiguredForIt() throws Exception {
         MimeMessageBuilder message = MimeMessageBuilder.mimeMessageBuilder()
             .setMultipartWithBodyParts(
                 MimeMessageBuilder.bodyPartBuilder()
diff --git a/server/mailet/integration-testing/src/test/java/org/apache/james/transport/mailets/ContactExtractorTest.java b/server/mailet/integration-testing/src/test/java/org/apache/james/transport/mailets/ContactExtractorTest.java
index 864eb6a..8ac3e3d 100644
--- a/server/mailet/integration-testing/src/test/java/org/apache/james/transport/mailets/ContactExtractorTest.java
+++ b/server/mailet/integration-testing/src/test/java/org/apache/james/transport/mailets/ContactExtractorTest.java
@@ -50,7 +50,7 @@ import org.junit.jupiter.api.Test;
 import org.junit.jupiter.api.extension.RegisterExtension;
 import org.junit.jupiter.api.io.TempDir;
 
-public class ContactExtractorTest {
+class ContactExtractorTest {
     public static final String SENDER = "sender@" + DEFAULT_DOMAIN;
     public static final String TO = "to@" + DEFAULT_DOMAIN;
     public static final String TO2 = "to2@" + DEFAULT_DOMAIN;
@@ -71,7 +71,7 @@ public class ContactExtractorTest {
     private TemporaryJamesServer jamesServer;
 
     @BeforeEach
-    public void setup(@TempDir File temporaryFolder) throws Exception {
+    void setup(@TempDir File temporaryFolder) throws Exception {
         String attribute = "ExtractedContacts";
         MailetContainer.Builder mailets = TemporaryJamesServer.defaultMailetContainerConfiguration()
             .postmaster(SENDER)
@@ -108,12 +108,12 @@ public class ContactExtractorTest {
     }
 
     @AfterEach
-    public void tearDown() {
+    void tearDown() {
         jamesServer.shutdown();
     }
 
     @Test
-    public void recipientsShouldBePublishedToAmqpWhenSendingEmail() throws Exception {
+    void recipientsShouldBePublishedToAmqpWhenSendingEmail() throws Exception {
         MimeMessageBuilder message = MimeMessageBuilder.mimeMessageBuilder()
             .setSender(SENDER)
             .addToRecipient(TO, "John To2 <" + TO2 + ">")
diff --git a/server/mailet/integration-testing/src/test/java/org/apache/james/transport/mailets/DlpIntegrationTest.java b/server/mailet/integration-testing/src/test/java/org/apache/james/transport/mailets/DlpIntegrationTest.java
index f83a088..0f231be 100644
--- a/server/mailet/integration-testing/src/test/java/org/apache/james/transport/mailets/DlpIntegrationTest.java
+++ b/server/mailet/integration-testing/src/test/java/org/apache/james/transport/mailets/DlpIntegrationTest.java
@@ -58,7 +58,7 @@ import com.google.inject.util.Modules;
 
 import io.restassured.specification.RequestSpecification;
 
-public class DlpIntegrationTest {
+class DlpIntegrationTest {
     public static final String REPOSITORY_PREFIX = "memory://var/mail/dlp/quarantine/";
 
     @RegisterExtension
@@ -93,12 +93,12 @@ public class DlpIntegrationTest {
     }
 
     @AfterEach
-    public void tearDown() {
+    void tearDown() {
         jamesServer.shutdown();
     }
 
     @Test
-    public void dlpShouldStoreMatchingEmails(@TempDir File temporaryFolder) throws Exception {
+    void dlpShouldStoreMatchingEmails(@TempDir File temporaryFolder) throws Exception {
         createJamesServer(temporaryFolder, MailetConfiguration.builder()
             .matcher(Dlp.class)
             .mailet(ToSenderDomainRepository.class)
@@ -130,7 +130,7 @@ public class DlpIntegrationTest {
     }
 
     @Test
-    public void dlpShouldNotCreateRepositoryWhenNotAllowed(@TempDir File temporaryFolder) throws Exception {
+    void dlpShouldNotCreateRepositoryWhenNotAllowed(@TempDir File temporaryFolder) throws Exception {
         createJamesServer(temporaryFolder, MailetConfiguration.builder()
             .matcher(Dlp.class)
             .mailet(ToSenderDomainRepository.class)
@@ -168,7 +168,7 @@ public class DlpIntegrationTest {
     }
 
     @Test
-    public void dlpShouldCreateRepositoryWhenAllowed(@TempDir File temporaryFolder) throws Exception {
+    void dlpShouldCreateRepositoryWhenAllowed(@TempDir File temporaryFolder) throws Exception {
         createJamesServer(temporaryFolder, MailetConfiguration.builder()
             .matcher(Dlp.class)
             .mailet(ToSenderDomainRepository.class)
@@ -202,7 +202,7 @@ public class DlpIntegrationTest {
     }
 
     @Test
-    public void dlpShouldStoreMailWhenNotAllowedButRepositoryExists(@TempDir File temporaryFolder) throws Exception {
+    void dlpShouldStoreMailWhenNotAllowedButRepositoryExists(@TempDir File temporaryFolder) throws Exception {
         createJamesServer(temporaryFolder, MailetConfiguration.builder()
             .matcher(Dlp.class)
             .mailet(ToSenderDomainRepository.class)
@@ -241,7 +241,7 @@ public class DlpIntegrationTest {
     }
 
     @Test
-    public void dlpShouldBeAbleToReadMailContentWithAttachments(@TempDir File temporaryFolder) throws Exception {
+    void dlpShouldBeAbleToReadMailContentWithAttachments(@TempDir File temporaryFolder) throws Exception {
         createJamesServer(temporaryFolder, MailetConfiguration.builder()
             .matcher(Dlp.class)
             .mailet(ToSenderDomainRepository.class)
diff --git a/server/mailet/integration-testing/src/test/java/org/apache/james/transport/mailets/DomainMappingTest.java b/server/mailet/integration-testing/src/test/java/org/apache/james/transport/mailets/DomainMappingTest.java
index 1b3e804..b67537c 100644
--- a/server/mailet/integration-testing/src/test/java/org/apache/james/transport/mailets/DomainMappingTest.java
+++ b/server/mailet/integration-testing/src/test/java/org/apache/james/transport/mailets/DomainMappingTest.java
@@ -55,7 +55,7 @@ import org.junit.jupiter.api.io.TempDir;
 
 import io.restassured.specification.RequestSpecification;
 
-public class DomainMappingTest {
+class DomainMappingTest {
     private static final String DOMAIN1 = "domain1.com";
     private static final String DOMAIN2 = "domain2.com";
 
@@ -77,7 +77,7 @@ public class DomainMappingTest {
     public SMTPMessageSender messageSender = new SMTPMessageSender(DEFAULT_DOMAIN);
 
     @BeforeEach
-    public void setup(@TempDir File temporaryFolder) throws Exception {
+    void setup(@TempDir File temporaryFolder) throws Exception {
         MailetContainer.Builder mailetContainer = TemporaryJamesServer.simpleMailetContainerConfiguration()
             .putProcessor(CommonProcessors.rrtErrorEnabledTransport())
             .putProcessor(CommonProcessors.rrtErrorProcessor());
@@ -105,12 +105,12 @@ public class DomainMappingTest {
     }
 
     @AfterEach
-    public void tearDown() {
+    void tearDown() {
         jamesServer.shutdown();
     }
 
     @Test
-    public void messageShouldRedirectToUserWhenBelongingToGroup() throws Exception {
+    void messageShouldRedirectToUserWhenBelongingToGroup() throws Exception {
         webAdminApi.body(DOMAIN1).put("/domainMappings/" + DOMAIN2);
 
         messageSender.connect(LOCALHOST_IP, jamesServer.getProbe(SmtpGuiceProbe.class).getSmtpPort())
@@ -128,7 +128,7 @@ public class DomainMappingTest {
     }
 
     @Test
-    public void messageShouldRedirectToUserOfTheDestinationDomainWhenSentToTheAliasDomain() throws Exception {
+    void messageShouldRedirectToUserOfTheDestinationDomainWhenSentToTheAliasDomain() throws Exception {
         webAdminApi.put("/domains/" + DOMAIN1 + "/aliases/" + DOMAIN2);
 
         messageSender.connect(LOCALHOST_IP, jamesServer.getProbe(SmtpGuiceProbe.class).getSmtpPort())
@@ -146,7 +146,7 @@ public class DomainMappingTest {
     }
 
     @Test
-    public void mailShouldGoToRRTErrorMailRepositoryUponDomainLoop() throws Exception {
+    void mailShouldGoToRRTErrorMailRepositoryUponDomainLoop() throws Exception {
         webAdminApi.put("/domains/" + DOMAIN1 + "/aliases/" + DOMAIN2);
         webAdminApi.put("/domains/" + DOMAIN2 + "/aliases/" + DOMAIN1);
 
@@ -163,7 +163,7 @@ public class DomainMappingTest {
     }
 
     @Test
-    public void mailShouldGoToRRTErrorMailRepositoryUponLoopCombiningDomainAndAlias() throws Exception {
+    void mailShouldGoToRRTErrorMailRepositoryUponLoopCombiningDomainAndAlias() throws Exception {
         jamesServer.getProbe(DataProbeImpl.class).addUser(BOB_DOMAIN2, PASSWORD);
 
         webAdminApi.put("/address/aliases/" + BOB_DOMAIN2 + "/sources/" + BOB_DOMAIN1);
@@ -182,7 +182,7 @@ public class DomainMappingTest {
     }
 
     @Test
-    public void domainAliasShouldBeIgnoredWhenUserAlias() throws Exception {
+    void domainAliasShouldBeIgnoredWhenUserAlias() throws Exception {
         jamesServer.getProbe(DataProbeImpl.class).addUser(BOB_DOMAIN1, PASSWORD);
 
         webAdminApi.put("/address/aliases/" + BOB_DOMAIN1 + "/sources/" + USER_DOMAIN2);
@@ -203,7 +203,7 @@ public class DomainMappingTest {
     }
 
     @Test
-    public void domainAliasShouldBeChainedIfApplicableAfterUserAliasRewrite() throws Exception {
+    void domainAliasShouldBeChainedIfApplicableAfterUserAliasRewrite() throws Exception {
         jamesServer.getProbe(DataProbeImpl.class).addUser(BOB_DOMAIN1, PASSWORD);
 
         webAdminApi.put("/address/aliases/" + BOB_DOMAIN2 + "/sources/" + USER_DOMAIN2);
diff --git a/server/mailet/integration-testing/src/test/java/org/apache/james/transport/mailets/GroupMappingRelayTest.java b/server/mailet/integration-testing/src/test/java/org/apache/james/transport/mailets/GroupMappingRelayTest.java
index 7ad4f44..36a56eb 100644
--- a/server/mailet/integration-testing/src/test/java/org/apache/james/transport/mailets/GroupMappingRelayTest.java
+++ b/server/mailet/integration-testing/src/test/java/org/apache/james/transport/mailets/GroupMappingRelayTest.java
@@ -53,7 +53,7 @@ import org.junit.jupiter.api.io.TempDir;
 
 import io.restassured.specification.RequestSpecification;
 
-public class GroupMappingRelayTest {
+class GroupMappingRelayTest {
     private static final String DOMAIN1 = "domain1.com";
 
     public static final String SENDER_LOCAL_PART = "fromuser";
@@ -74,7 +74,7 @@ public class GroupMappingRelayTest {
     public SMTPMessageSender messageSender = new SMTPMessageSender(DEFAULT_DOMAIN);
 
     @BeforeEach
-    public void setup(@TempDir File temporaryFolder) throws Exception {
+    void setup(@TempDir File temporaryFolder) throws Exception {
         MailetContainer.Builder mailetContainer = TemporaryJamesServer.simpleMailetContainerConfiguration()
             .putProcessor(CommonProcessors.rrtErrorEnabledTransport()
                 .addMailet(MailetConfiguration.remoteDeliveryBuilder()
@@ -101,12 +101,12 @@ public class GroupMappingRelayTest {
     }
 
     @AfterEach
-    public void tearDown() {
+    void tearDown() {
         jamesServer.shutdown();
     }
 
     @Test
-    public void sendMessageShouldSendAMessageToAnExternalGroupMember() throws Exception {
+    void sendMessageShouldSendAMessageToAnExternalGroupMember() throws Exception {
         String externalMail = "ray@yopmail.com";
         webAdminApi.put(GroupsRoutes.ROOT_PATH + "/" + GROUP_ON_DOMAIN1 + "/" + externalMail);
 
diff --git a/server/mailet/integration-testing/src/test/java/org/apache/james/transport/mailets/GroupMappingTest.java b/server/mailet/integration-testing/src/test/java/org/apache/james/transport/mailets/GroupMappingTest.java
index 6822f11..6225eab 100644
--- a/server/mailet/integration-testing/src/test/java/org/apache/james/transport/mailets/GroupMappingTest.java
+++ b/server/mailet/integration-testing/src/test/java/org/apache/james/transport/mailets/GroupMappingTest.java
@@ -57,7 +57,7 @@ import org.junit.jupiter.api.io.TempDir;
 
 import io.restassured.specification.RequestSpecification;
 
-public class GroupMappingTest {
+class GroupMappingTest {
     private static final String DOMAIN1 = "domain1.com";
     private static final String DOMAIN2 = "domain2.com";
 
@@ -81,7 +81,7 @@ public class GroupMappingTest {
     public SMTPMessageSender messageSender = new SMTPMessageSender(DEFAULT_DOMAIN);
 
     @BeforeEach
-    public void setup(@TempDir File temporaryFolder) throws Exception {
+    void setup(@TempDir File temporaryFolder) throws Exception {
         MailetContainer.Builder mailetContainer = TemporaryJamesServer.simpleMailetContainerConfiguration()
             .putProcessor(CommonProcessors.rrtErrorEnabledTransport())
             .putProcessor(CommonProcessors.rrtErrorProcessor());
@@ -112,12 +112,12 @@ public class GroupMappingTest {
     }
 
     @AfterEach
-    public void tearDown() {
+    void tearDown() {
         jamesServer.shutdown();
     }
 
     @Test
-    public void messageShouldRedirectToUserWhenBelongingToGroup() throws Exception {
+    void messageShouldRedirectToUserWhenBelongingToGroup() throws Exception {
         webAdminApi.put(GroupsRoutes.ROOT_PATH + "/" + GROUP_ON_DOMAIN1 + "/" + USER_DOMAIN1);
 
         messageSender.connect(LOCALHOST_IP, jamesServer.getProbe(SmtpGuiceProbe.class).getSmtpPort())
@@ -135,7 +135,7 @@ public class GroupMappingTest {
     }
 
     @Test
-    public void messageShouldRedirectToUserDoesNotHaveSameDomainWhenBelongingToGroup() throws Exception {
+    void messageShouldRedirectToUserDoesNotHaveSameDomainWhenBelongingToGroup() throws Exception {
         webAdminApi.put(GroupsRoutes.ROOT_PATH + "/" + GROUP_ON_DOMAIN1 + "/" + USER_DOMAIN2);
 
         messageSender.connect(LOCALHOST_IP, jamesServer.getProbe(SmtpGuiceProbe.class).getSmtpPort())
@@ -153,7 +153,7 @@ public class GroupMappingTest {
     }
 
     @Test
-    public void messageShouldRedirectToAllUsersBelongingToGroup() throws Exception {
+    void messageShouldRedirectToAllUsersBelongingToGroup() throws Exception {
         webAdminApi.put(GroupsRoutes.ROOT_PATH + "/" + GROUP_ON_DOMAIN1 + "/" + USER_DOMAIN1);
 
         webAdminApi.put(GroupsRoutes.ROOT_PATH + "/" + GROUP_ON_DOMAIN1 + "/" + USER_DOMAIN2);
@@ -176,7 +176,7 @@ public class GroupMappingTest {
     }
 
     @Test
-    public void messageShouldRedirectWhenGroupBelongingToAnotherGroup() throws Exception {
+    void messageShouldRedirectWhenGroupBelongingToAnotherGroup() throws Exception {
         webAdminApi.put(GroupsRoutes.ROOT_PATH + "/" + GROUP_ON_DOMAIN2 + "/" + USER_DOMAIN2);
 
         webAdminApi.put(GroupsRoutes.ROOT_PATH + "/" + GROUP_ON_DOMAIN1 + "/" + GROUP_ON_DOMAIN2);
@@ -196,7 +196,7 @@ public class GroupMappingTest {
     }
 
     @Test
-    public void messageShouldNotBeDuplicatedWhenUserBelongingToTwoGroups() throws Exception {
+    void messageShouldNotBeDuplicatedWhenUserBelongingToTwoGroups() throws Exception {
         webAdminApi.put(GroupsRoutes.ROOT_PATH + "/" + GROUP_ON_DOMAIN1 + "/" + USER_DOMAIN1);
 
         webAdminApi.put(GroupsRoutes.ROOT_PATH + "/" + GROUP_ON_DOMAIN2 + "/" + USER_DOMAIN1);
@@ -217,7 +217,7 @@ public class GroupMappingTest {
     }
 
     @Test
-    public void messageShouldNotBeDuplicatedWhenRecipientIsAlsoPartOfGroup() throws Exception {
+    void messageShouldNotBeDuplicatedWhenRecipientIsAlsoPartOfGroup() throws Exception {
         webAdminApi.put(GroupsRoutes.ROOT_PATH + "/" + GROUP_ON_DOMAIN1 + "/" + USER_DOMAIN1);
 
         messageSender.connect(LOCALHOST_IP, jamesServer.getProbe(SmtpGuiceProbe.class).getSmtpPort())
@@ -234,7 +234,7 @@ public class GroupMappingTest {
     }
 
     @Test
-    public void groupMappingShouldSupportTreeStructure() throws Exception {
+    void groupMappingShouldSupportTreeStructure() throws Exception {
         webAdminApi.put(GroupsRoutes.ROOT_PATH + "/" + GROUP_ON_DOMAIN1 + "/" + USER_DOMAIN1);
 
         webAdminApi.put(GroupsRoutes.ROOT_PATH + "/" + GROUP_ON_DOMAIN2 + "/" + USER_DOMAIN2);
@@ -260,7 +260,7 @@ public class GroupMappingTest {
     }
 
     @Test
-    public void messageShouldBeStoredInRepositoryWhenGroupLoopMapping() throws Exception {
+    void messageShouldBeStoredInRepositoryWhenGroupLoopMapping() throws Exception {
         webAdminApi.put(GroupsRoutes.ROOT_PATH + "/" + GROUP_ON_DOMAIN1 + "/" + USER_DOMAIN1);
 
         webAdminApi.put(GroupsRoutes.ROOT_PATH + "/" + GROUP_ON_DOMAIN2 + "/" + USER_DOMAIN2);
@@ -282,7 +282,7 @@ public class GroupMappingTest {
     }
 
     @Test
-    public void messageShouldBeWellDeliveredToRecipientNotPartOfTheLoop() throws Exception {
+    void messageShouldBeWellDeliveredToRecipientNotPartOfTheLoop() throws Exception {
         webAdminApi.put(GroupsRoutes.ROOT_PATH + "/" + GROUP_ON_DOMAIN1 + "/" + GROUP_ON_DOMAIN2);
 
         webAdminApi.put(GroupsRoutes.ROOT_PATH + "/" + GROUP_ON_DOMAIN2 + "/" + GROUP_ON_DOMAIN1);
@@ -301,7 +301,7 @@ public class GroupMappingTest {
     }
 
     @Test
-    public void senderShouldReceiveABounceUponRRTFailure() throws Exception {
+    void senderShouldReceiveABounceUponRRTFailure() throws Exception {
         webAdminApi.put(GroupsRoutes.ROOT_PATH + "/" + GROUP_ON_DOMAIN1 + "/" + GROUP_ON_DOMAIN2);
         jamesServer.getProbe(DataProbeImpl.class).addDomainAliasMapping(DOMAIN2, DOMAIN1);
 
@@ -319,7 +319,7 @@ public class GroupMappingTest {
     }
 
     @Test
-    public void senderShouldNotReceiveABounceUponRRTFailureWhenPartOfTheLoop() throws Exception {
+    void senderShouldNotReceiveABounceUponRRTFailureWhenPartOfTheLoop() throws Exception {
         webAdminApi.put(GroupsRoutes.ROOT_PATH + "/" + GROUP_ON_DOMAIN1 + "/" + GROUP_ON_DOMAIN2);
         jamesServer.getProbe(DataProbeImpl.class).addDomainAliasMapping(DOMAIN2, DOMAIN1);
 
@@ -337,7 +337,7 @@ public class GroupMappingTest {
     }
 
     @Test
-    public void avoidInfiniteBouncingLoopWhenSenderIsPartOfRRTLoop() throws Exception {
+    void avoidInfiniteBouncingLoopWhenSenderIsPartOfRRTLoop() throws Exception {
         webAdminApi.put(GroupsRoutes.ROOT_PATH + "/" + GROUP_ON_DOMAIN1 + "/" + GROUP_ON_DOMAIN2);
         jamesServer.getProbe(DataProbeImpl.class).addDomainAliasMapping(DOMAIN2, DOMAIN1);
 
@@ -354,7 +354,7 @@ public class GroupMappingTest {
     }
 
     @Test
-    public void messageShouldRedirectToUserWhenDomainMapping() throws Exception {
+    void messageShouldRedirectToUserWhenDomainMapping() throws Exception {
         dataProbe.addDomainAliasMapping(DOMAIN1, DOMAIN2);
 
         webAdminApi.put(GroupsRoutes.ROOT_PATH + "/" + GROUP_ON_DOMAIN1 + "/" + USER_DOMAIN1);
@@ -373,7 +373,7 @@ public class GroupMappingTest {
     }
 
     @Test
-    public void messageShouldNotSendToUserBelongingToGroupWhenDomainMapping() throws Exception {
+    void messageShouldNotSendToUserBelongingToGroupWhenDomainMapping() throws Exception {
         dataProbe.addDomainAliasMapping(DOMAIN1, DOMAIN2);
 
         webAdminApi.put(GroupsRoutes.ROOT_PATH + "/" + GROUP_ON_DOMAIN1 + "/" + USER_DOMAIN1);
@@ -392,7 +392,7 @@ public class GroupMappingTest {
     }
 
     @Test
-    public void messageShouldRedirectToGroupWhenDomainMapping() throws Exception {
+    void messageShouldRedirectToGroupWhenDomainMapping() throws Exception {
         dataProbe.addDomainAliasMapping(DOMAIN1, DOMAIN2);
 
         webAdminApi.put(GroupsRoutes.ROOT_PATH + "/" + GROUP_ON_DOMAIN2 + "/" + USER_DOMAIN2);
@@ -411,7 +411,7 @@ public class GroupMappingTest {
     }
 
     @Test
-    public void messageShouldRedirectToGroupContainingSlash() throws Exception {
+    void messageShouldRedirectToGroupContainingSlash() throws Exception {
         String groupWithSlash = "a/a@" + DOMAIN1;
         String groupWithEncodedSlash = "a%2Fa@" + DOMAIN1;
         webAdminApi.put(GroupsRoutes.ROOT_PATH + "/" + groupWithEncodedSlash + "/" + USER_DOMAIN1);
@@ -430,7 +430,7 @@ public class GroupMappingTest {
     }
 
     @Test
-    public void messageShouldRedirectToUserContainingSlash() throws Exception {
+    void messageShouldRedirectToUserContainingSlash() throws Exception {
         String userWithSlash = "a/a@" + DOMAIN1;
         dataProbe.addUser(userWithSlash, PASSWORD);
         String userWithEncodedSlash = "a%2Fa@" + DOMAIN1;
@@ -450,7 +450,7 @@ public class GroupMappingTest {
     }
 
     @Test
-    public void messageShouldRedirectToUserWhenEncodingAt() throws Exception {
+    void messageShouldRedirectToUserWhenEncodingAt() throws Exception {
         String userWithEncodedAt = "user%40" + DOMAIN1;
         String groupWithEncodedAt = "group%40" + DOMAIN1;
         webAdminApi.put(GroupsRoutes.ROOT_PATH + "/" + groupWithEncodedAt + "/" + userWithEncodedAt);
diff --git a/server/mailet/integration-testing/src/test/java/org/apache/james/transport/mailets/ICSAttachmentWorkflowTest.java b/server/mailet/integration-testing/src/test/java/org/apache/james/transport/mailets/ICSAttachmentWorkflowTest.java
index 08d79a1..338883d 100644
--- a/server/mailet/integration-testing/src/test/java/org/apache/james/transport/mailets/ICSAttachmentWorkflowTest.java
+++ b/server/mailet/integration-testing/src/test/java/org/apache/james/transport/mailets/ICSAttachmentWorkflowTest.java
@@ -440,7 +440,7 @@ public class ICSAttachmentWorkflowTest {
     private MimeMessage yahooInvitationMessage;
 
     @BeforeEach
-    public void setup(@TempDir File temporaryFolder) throws Exception {
+    void setup(@TempDir File temporaryFolder) throws Exception {
         MailetContainer.Builder mailetContainer = TemporaryJamesServer.defaultMailetContainerConfiguration()
             .putProcessor(ProcessorConfiguration.transport()
                 .addMailet(MailetConfiguration.builder()
@@ -543,12 +543,12 @@ public class ICSAttachmentWorkflowTest {
     }
 
     @AfterEach
-    public void tearDown() {
+    void tearDown() {
         jamesServer.shutdown();
     }
 
     @Test
-    public void calendarAttachmentShouldNotBePublishedInMQWhenNoICalAttachment() throws Exception {
+    void calendarAttachmentShouldNotBePublishedInMQWhenNoICalAttachment() throws Exception {
         messageSender.connect(LOCALHOST_IP, jamesServer.getProbe(SmtpGuiceProbe.class).getSmtpPort())
             .sendMessage(FakeMail.builder()
                 .name("name")
@@ -565,7 +565,7 @@ public class ICSAttachmentWorkflowTest {
     }
 
     @Test
-    public void calendarAttachmentShouldBePublishedInMQWhenMatchingWorkflowConfiguration() throws Exception {
+    void calendarAttachmentShouldBePublishedInMQWhenMatchingWorkflowConfiguration() throws Exception {
         messageSender.connect(LOCALHOST_IP, jamesServer.getProbe(SmtpGuiceProbe.class).getSmtpPort())
             .sendMessage(FakeMail.builder()
                 .name("name")
@@ -598,7 +598,7 @@ public class ICSAttachmentWorkflowTest {
     }
 
     @Test
-    public void headersShouldNotBeAddedInMailWhenNoICalAttachment() throws Exception {
+    void headersShouldNotBeAddedInMailWhenNoICalAttachment() throws Exception {
         messageSender.connect(LOCALHOST_IP, jamesServer.getProbe(SmtpGuiceProbe.class).getSmtpPort())
             .sendMessage(FakeMail.builder()
                 .name("name")
@@ -620,7 +620,7 @@ public class ICSAttachmentWorkflowTest {
     }
 
     @Test
-    public void headersShouldBeAddedInMailWhenOneICalAttachment() throws Exception {
+    void headersShouldBeAddedInMailWhenOneICalAttachment() throws Exception {
         messageSender.connect(LOCALHOST_IP, jamesServer.getProbe(SmtpGuiceProbe.class).getSmtpPort())
             .sendMessage(FakeMail.builder()
                 .name("name")
@@ -641,7 +641,7 @@ public class ICSAttachmentWorkflowTest {
     }
 
     @Test
-    public void headersShouldBeAddedInMailWhenOneBase64ICalAttachment() throws Exception {
+    void headersShouldBeAddedInMailWhenOneBase64ICalAttachment() throws Exception {
         messageSender.connect(LOCALHOST_IP, jamesServer.getProbe(SmtpGuiceProbe.class).getSmtpPort())
             .sendMessage(FakeMail.builder()
                 .name("name")
@@ -662,7 +662,7 @@ public class ICSAttachmentWorkflowTest {
     }
 
     @Test
-    public void base64CalendarAttachmentShouldBePublishedInMQWhenMatchingWorkflowConfiguration() throws Exception {
+    void base64CalendarAttachmentShouldBePublishedInMQWhenMatchingWorkflowConfiguration() throws Exception {
         messageSender.connect(LOCALHOST_IP, jamesServer.getProbe(SmtpGuiceProbe.class).getSmtpPort())
             .sendMessage(FakeMail.builder()
                 .name("name")
@@ -688,7 +688,7 @@ public class ICSAttachmentWorkflowTest {
     }
 
     @Test
-    public void yahooBase64CalendarAttachmentShouldBePublishedInMQWhenMatchingWorkflowConfiguration() throws Exception {
+    void yahooBase64CalendarAttachmentShouldBePublishedInMQWhenMatchingWorkflowConfiguration() throws Exception {
         messageSender.connect(LOCALHOST_IP, jamesServer.getProbe(SmtpGuiceProbe.class).getSmtpPort())
             .sendMessage(FakeMail.builder()
                 .name("name")
@@ -715,7 +715,7 @@ public class ICSAttachmentWorkflowTest {
     }
 
     @Test
-    public void headersShouldBeFilledOnlyWithOneICalAttachmentWhenMailHasSeveral() throws Exception {
+    void headersShouldBeFilledOnlyWithOneICalAttachmentWhenMailHasSeveral() throws Exception {
         messageSender.connect(LOCALHOST_IP, jamesServer.getProbe(SmtpGuiceProbe.class).getSmtpPort())
             .sendMessage(FakeMail.builder()
                 .name("name")
@@ -736,7 +736,7 @@ public class ICSAttachmentWorkflowTest {
     }
 
     @Test
-    public void pipelineShouldSendSeveralJSONOverRabbitMQWhenSeveralAttachments() throws Exception {
+    void pipelineShouldSendSeveralJSONOverRabbitMQWhenSeveralAttachments() throws Exception {
         messageSender.connect(LOCALHOST_IP, jamesServer.getProbe(SmtpGuiceProbe.class).getSmtpPort())
             .sendMessage(FakeMail.builder()
                 .name("name")
@@ -773,7 +773,7 @@ public class ICSAttachmentWorkflowTest {
     }
 
     @Test
-    public void mailShouldNotContainCalendarContentInTextBodyButAttachment() throws Exception {
+    void mailShouldNotContainCalendarContentInTextBodyButAttachment() throws Exception {
         MimeMessage calendarMessage = MimeMessageUtil.mimeMessageFromStream(ClassLoader.getSystemResourceAsStream("eml/calendar.eml"));
 
         messageSender.connect(LOCALHOST_IP, jamesServer.getProbe(SmtpGuiceProbe.class).getSmtpPort())
diff --git a/server/mailet/integration-testing/src/test/java/org/apache/james/transport/mailets/IsOverQuotaMatcherTest.java b/server/mailet/integration-testing/src/test/java/org/apache/james/transport/mailets/IsOverQuotaMatcherTest.java
index be89608..790c7f9 100644
--- a/server/mailet/integration-testing/src/test/java/org/apache/james/transport/mailets/IsOverQuotaMatcherTest.java
+++ b/server/mailet/integration-testing/src/test/java/org/apache/james/transport/mailets/IsOverQuotaMatcherTest.java
@@ -52,7 +52,7 @@ import org.junit.jupiter.api.io.TempDir;
 
 import io.restassured.specification.RequestSpecification;
 
-public class IsOverQuotaMatcherTest {
+class IsOverQuotaMatcherTest {
 
     private static final String OTHER_DOMAIN = "other.com";
     private static final String FROM = "fromuser@" + OTHER_DOMAIN;
@@ -74,7 +74,7 @@ public class IsOverQuotaMatcherTest {
     private RequestSpecification webAdminApi;
 
     @BeforeEach
-    public void setup(@TempDir File temporaryFolder) throws Exception {
+    void setup(@TempDir File temporaryFolder) throws Exception {
         MailetContainer.Builder mailetContainer = TemporaryJamesServer.defaultMailetContainerConfiguration()
                 .putProcessor(ProcessorConfiguration.transport()
                         .addMailet(MailetConfiguration.builder()
@@ -102,12 +102,12 @@ public class IsOverQuotaMatcherTest {
     }
 
     @AfterEach
-    public void tearDown() {
+    void tearDown() {
         jamesServer.shutdown();
     }
 
     @Test
-    public void aBounceMessageShouldBeSentToTheSenderWhenRecipientAsReachedHisSizeQuota() throws Exception {
+    void aBounceMessageShouldBeSentToTheSenderWhenRecipientAsReachedHisSizeQuota() throws Exception {
         webAdminApi.given()
             .body(SMALL_SIZE.asLong())
             .put("/quota/users/" + RECIPIENT + "/size");
@@ -125,7 +125,7 @@ public class IsOverQuotaMatcherTest {
     }
 
     @Test
-    public void aBounceMessageShouldBeSentToTheSenderWhenRecipientAsReachedHisDomainSizeQuota() throws Exception {
+    void aBounceMessageShouldBeSentToTheSenderWhenRecipientAsReachedHisDomainSizeQuota() throws Exception {
         webAdminApi.given()
             .body(SMALL_SIZE.asLong())
             .put("/quota/domains/" + DEFAULT_DOMAIN + "/size");
@@ -143,7 +143,7 @@ public class IsOverQuotaMatcherTest {
     }
 
     @Test
-    public void aBounceMessageShouldBeSentToTheRecipientWhenRecipientSizeQuotaIsNotExceeded() throws Exception {
+    void aBounceMessageShouldBeSentToTheRecipientWhenRecipientSizeQuotaIsNotExceeded() throws Exception {
         webAdminApi.given()
             .body(LARGE_SIZE.asLong())
             .put("/quota/users/" + RECIPIENT + "/size");
@@ -158,7 +158,7 @@ public class IsOverQuotaMatcherTest {
     }
 
     @Test
-    public void aBounceMessageShouldBeSentToTheSenderWhenRecipientAsReachedHisCountQuota() throws Exception {
+    void aBounceMessageShouldBeSentToTheSenderWhenRecipientAsReachedHisCountQuota() throws Exception {
         webAdminApi.given()
             .body(SMALL_COUNT.asLong())
             .put("/quota/users/" + RECIPIENT + "/count");
@@ -176,7 +176,7 @@ public class IsOverQuotaMatcherTest {
     }
 
     @Test
-    public void aBounceMessageShouldBeSentToTheSenderWhenRecipientAsReachedHisDomainCountQuota() throws Exception {
+    void aBounceMessageShouldBeSentToTheSenderWhenRecipientAsReachedHisDomainCountQuota() throws Exception {
         webAdminApi.given()
             .body(SMALL_COUNT.asLong())
             .put("/quota/domains/" + DEFAULT_DOMAIN + "/count");
@@ -194,7 +194,7 @@ public class IsOverQuotaMatcherTest {
     }
 
     @Test
-    public void aBounceMessageShouldBeSentToTheRecipientWhenRecipientCountQuotaIsNotExceeded() throws Exception {
+    void aBounceMessageShouldBeSentToTheRecipientWhenRecipientCountQuotaIsNotExceeded() throws Exception {
         webAdminApi.given()
             .body(LARGE_COUNT.asLong())
             .put("/quota/users/" + RECIPIENT + "/count");
diff --git a/server/mailet/integration-testing/src/test/java/org/apache/james/transport/mailets/MailReprocessingIntegrationTest.java b/server/mailet/integration-testing/src/test/java/org/apache/james/transport/mailets/MailReprocessingIntegrationTest.java
index 2ff4c8c..5b4d1af 100644
--- a/server/mailet/integration-testing/src/test/java/org/apache/james/transport/mailets/MailReprocessingIntegrationTest.java
+++ b/server/mailet/integration-testing/src/test/java/org/apache/james/transport/mailets/MailReprocessingIntegrationTest.java
@@ -58,7 +58,7 @@ import com.google.inject.util.Modules;
 
 import io.restassured.specification.RequestSpecification;
 
-public class MailReprocessingIntegrationTest {
+class MailReprocessingIntegrationTest {
     private static final MailRepositoryUrl REPOSITORY_A = MailRepositoryUrl.from("memory://var/mail/a");
     private static final MailRepositoryUrl REPOSITORY_B = MailRepositoryUrl.from("memory://var/mail/b");
 
@@ -69,7 +69,7 @@ public class MailReprocessingIntegrationTest {
     private RequestSpecification specification;
 
     @BeforeEach
-    public void createJamesServer(@TempDir File temporaryFolder) throws Exception {
+    void createJamesServer(@TempDir File temporaryFolder) throws Exception {
         MailetContainer.Builder mailets = TemporaryJamesServer.defaultMailetContainerConfiguration()
             .putProcessor(ProcessorConfiguration.transport()
                     .addMailet(MailetConfiguration.BCC_STRIPPER)
@@ -100,12 +100,12 @@ public class MailReprocessingIntegrationTest {
     }
 
     @AfterEach
-    public void tearDown() {
+    void tearDown() {
         jamesServer.shutdown();
     }
 
     @Test
-    public void reprocessingShouldAllowToTargetASpecificProcessor() throws Exception {
+    void reprocessingShouldAllowToTargetASpecificProcessor() throws Exception {
         // Given an incoming email
         messageSender.connect(LOCALHOST_IP, jamesServer.getProbe(SmtpGuiceProbe.class).getSmtpPort())
             .sendMessage(FakeMail.builder()
@@ -133,7 +133,7 @@ public class MailReprocessingIntegrationTest {
     }
 
     @Test
-    public void reprocessingShouldPreserveStateWhenProcessorIsNotSpecified() throws Exception {
+    void reprocessingShouldPreserveStateWhenProcessorIsNotSpecified() throws Exception {
         // Given an incoming email
         messageSender.connect(LOCALHOST_IP, jamesServer.getProbe(SmtpGuiceProbe.class).getSmtpPort())
             .sendMessage(FakeMail.builder()
@@ -177,7 +177,7 @@ public class MailReprocessingIntegrationTest {
     }
 
     @Test
-    public void reprocessingShouldProcessAsErrorWhenUnknownMailProcessor() throws Exception {
+    void reprocessingShouldProcessAsErrorWhenUnknownMailProcessor() throws Exception {
         // Given an incoming email
         messageSender.connect(LOCALHOST_IP, jamesServer.getProbe(SmtpGuiceProbe.class).getSmtpPort())
             .sendMessage(FakeMail.builder()
diff --git a/server/mailet/integration-testing/src/test/java/org/apache/james/transport/mailets/RemoteDeliveryErrorHandlingTest.java b/server/mailet/integration-testing/src/test/java/org/apache/james/transport/mailets/RemoteDeliveryErrorHandlingTest.java
index a818426..a85009b 100644
--- a/server/mailet/integration-testing/src/test/java/org/apache/james/transport/mailets/RemoteDeliveryErrorHandlingTest.java
+++ b/server/mailet/integration-testing/src/test/java/org/apache/james/transport/mailets/RemoteDeliveryErrorHandlingTest.java
@@ -66,7 +66,7 @@ import com.google.inject.util.Modules;
 
 import io.restassured.specification.RequestSpecification;
 
-public class RemoteDeliveryErrorHandlingTest {
+class RemoteDeliveryErrorHandlingTest {
     private static final String FROM = "from@" + DEFAULT_DOMAIN;
     private static final String RECIPIENT_DOMAIN = "test.com";
     private static final String RECIPIENT = "touser@" + RECIPIENT_DOMAIN;
diff --git a/server/mailet/integration-testing/src/test/java/org/apache/james/transport/mailets/SpamAssassinIntegrationTest.java b/server/mailet/integration-testing/src/test/java/org/apache/james/transport/mailets/SpamAssassinIntegrationTest.java
index e25d25f..1e79738 100644
--- a/server/mailet/integration-testing/src/test/java/org/apache/james/transport/mailets/SpamAssassinIntegrationTest.java
+++ b/server/mailet/integration-testing/src/test/java/org/apache/james/transport/mailets/SpamAssassinIntegrationTest.java
@@ -56,7 +56,7 @@ import org.junit.jupiter.api.extension.RegisterExtension;
 import org.junit.jupiter.api.io.TempDir;
 import org.testcontainers.containers.wait.strategy.HostPortWaitStrategy;
 
-public class SpamAssassinIntegrationTest {
+class SpamAssassinIntegrationTest {
     private static final String SPAM_CONTENT = "XJS*C4JDBQADN1.NSBN3*2IDNEN*GTUBE-STANDARD-ANTI-UBE-TEST-EMAIL*C.34X";
 
     @RegisterExtension
@@ -72,7 +72,7 @@ public class SpamAssassinIntegrationTest {
     private TemporaryJamesServer jamesServer;
 
     @BeforeEach
-    public void setup(@TempDir File temporaryFolder) throws Exception {
+    void setup(@TempDir File temporaryFolder) throws Exception {
         MailetContainer.Builder mailets = TemporaryJamesServer.defaultMailetContainerConfiguration()
             .putProcessor(
                 ProcessorConfiguration.transport()
@@ -97,12 +97,12 @@ public class SpamAssassinIntegrationTest {
     }
 
     @AfterEach
-    public void tearDown() {
+    void tearDown() {
         jamesServer.shutdown();
     }
 
     @Test
-    public void spamAssassinShouldAppendNewHeaderOnMessage() throws Exception {
+    void spamAssassinShouldAppendNewHeaderOnMessage() throws Exception {
         messageSender.connect(LOCALHOST_IP, jamesServer.getProbe(SmtpGuiceProbe.class).getSmtpPort())
             .sendMessage(mailWithContent("This is the content", RECIPIENT));
 
@@ -118,7 +118,7 @@ public class SpamAssassinIntegrationTest {
     }
 
     @Test
-    public void spamAssassinShouldAppendNewHeaderWhichDetectIsSpamWhenSpamMessage() throws Exception {
+    void spamAssassinShouldAppendNewHeaderWhichDetectIsSpamWhenSpamMessage() throws Exception {
         messageSender.connect(LOCALHOST_IP, jamesServer.getProbe(SmtpGuiceProbe.class).getSmtpPort())
             .sendMessage(mailWithContent(SPAM_CONTENT, RECIPIENT));
 
@@ -133,7 +133,7 @@ public class SpamAssassinIntegrationTest {
     }
 
     @Test
-    public void spamAssassinShouldAppendNewHeaderWhichNoWhenNonSpamMessage() throws Exception {
+    void spamAssassinShouldAppendNewHeaderWhichNoWhenNonSpamMessage() throws Exception {
         messageSender.connect(LOCALHOST_IP, jamesServer.getProbe(SmtpGuiceProbe.class).getSmtpPort())
             .sendMessage(mailWithContent("This is the content", RECIPIENT));
 
@@ -148,7 +148,7 @@ public class SpamAssassinIntegrationTest {
     }
 
     @Test
-    public void spamAssassinShouldAppendNewHeaderPerRecipientOnMessage() throws Exception {
+    void spamAssassinShouldAppendNewHeaderPerRecipientOnMessage() throws Exception {
         messageSender.connect(LOCALHOST_IP, jamesServer.getProbe(SmtpGuiceProbe.class).getSmtpPort())
             .sendMessage(mailWithContent("This is the content", RECIPIENT, RECIPIENT2));
 
diff --git a/server/mailet/integration-testing/src/test/java/org/apache/james/transport/mailets/StripAttachmentTest.java b/server/mailet/integration-testing/src/test/java/org/apache/james/transport/mailets/StripAttachmentTest.java
index 5458448..709361c 100644
--- a/server/mailet/integration-testing/src/test/java/org/apache/james/transport/mailets/StripAttachmentTest.java
+++ b/server/mailet/integration-testing/src/test/java/org/apache/james/transport/mailets/StripAttachmentTest.java
@@ -50,7 +50,7 @@ import org.junit.jupiter.api.Test;
 import org.junit.jupiter.api.extension.RegisterExtension;
 import org.junit.jupiter.api.io.TempDir;
 
-public class StripAttachmentTest {
+class StripAttachmentTest {
     private static final String FROM = "fromUser@" + DEFAULT_DOMAIN;
     private static final String RECIPIENT = "touser@" + DEFAULT_DOMAIN;
 
@@ -62,7 +62,7 @@ public class StripAttachmentTest {
     private TemporaryJamesServer jamesServer;
 
     @BeforeEach
-    public void setup(@TempDir File temporaryFolder) throws Exception {
+    void setup(@TempDir File temporaryFolder) throws Exception {
         MailetContainer.Builder mailetContainer = TemporaryJamesServer.defaultMailetContainerConfiguration()
             .putProcessor(ProcessorConfiguration.transport()
                 .addMailet(MailetConfiguration.builder()
@@ -92,7 +92,7 @@ public class StripAttachmentTest {
     }
 
     @AfterEach
-    public void tearDown() {
+    void tearDown() {
         jamesServer.shutdown();
     }
 
@@ -103,7 +103,7 @@ public class StripAttachmentTest {
      * - RecoverAttachment recovers attachments from attribute
      */
     @Test
-    public void stripAttachmentShouldPutAttachmentsInMailAttributeWhenConfiguredForIt() throws Exception {
+    void stripAttachmentShouldPutAttachmentsInMailAttributeWhenConfiguredForIt() throws Exception {
         MimeMessage message = MimeMessageBuilder.mimeMessageBuilder()
             .setMultipartWithBodyParts(
                 MimeMessageBuilder.bodyPartBuilder()
diff --git a/server/mailet/integration-testing/src/test/java/org/apache/james/transport/mailets/ToRepositoryIntegrationTest.java b/server/mailet/integration-testing/src/test/java/org/apache/james/transport/mailets/ToRepositoryIntegrationTest.java
index 79fb660..01ffac1 100644
--- a/server/mailet/integration-testing/src/test/java/org/apache/james/transport/mailets/ToRepositoryIntegrationTest.java
+++ b/server/mailet/integration-testing/src/test/java/org/apache/james/transport/mailets/ToRepositoryIntegrationTest.java
@@ -53,7 +53,7 @@ import org.junit.jupiter.api.io.TempDir;
 
 import io.restassured.specification.RequestSpecification;
 
-public class ToRepositoryIntegrationTest {
+class ToRepositoryIntegrationTest {
     private static final String RECIPIENT = "touser@" + DEFAULT_DOMAIN;
     public static final MailRepositoryUrl CUSTOM_REPOSITORY = MailRepositoryUrl.from("memory://var/mail/custom/");
 
@@ -67,7 +67,7 @@ public class ToRepositoryIntegrationTest {
     private RequestSpecification webAdminAPI;
 
     @BeforeEach
-    public void setup(@TempDir File temporaryFolder) throws Exception {
+    void setup(@TempDir File temporaryFolder) throws Exception {
         MailetContainer.Builder mailetContainer = TemporaryJamesServer.simpleMailetContainerConfiguration()
             .putProcessor(ProcessorConfiguration.root()
                 .addMailet(MailetConfiguration.builder()
@@ -92,12 +92,12 @@ public class ToRepositoryIntegrationTest {
     }
 
     @AfterEach
-    public void tearDown() {
+    void tearDown() {
         jamesServer.shutdown();
     }
 
     @Test
-    public void incomingShouldBeStoredInProcessorByDefault() throws Exception {
+    void incomingShouldBeStoredInProcessorByDefault() throws Exception {
         messageSender.connect(LOCALHOST_IP, jamesServer.getProbe(SmtpGuiceProbe.class).getSmtpPort())
             .sendMessage(RECIPIENT, RECIPIENT)
             .sendMessage(RECIPIENT, RECIPIENT);
@@ -106,7 +106,7 @@ public class ToRepositoryIntegrationTest {
     }
 
     @Test
-    public void userShouldBeAbleToAccessReprocessedMails() throws Exception {
+    void userShouldBeAbleToAccessReprocessedMails() throws Exception {
         messageSender.connect(LOCALHOST_IP, jamesServer.getProbe(SmtpGuiceProbe.class).getSmtpPort())
             .sendMessage(RECIPIENT, RECIPIENT)
             .sendMessage(RECIPIENT, RECIPIENT);
@@ -136,7 +136,7 @@ public class ToRepositoryIntegrationTest {
     }
 
     @Test
-    public void userShouldBeAbleToAccessReprocessedMail() throws Exception {
+    void userShouldBeAbleToAccessReprocessedMail() throws Exception {
         messageSender.connect(LOCALHOST_IP, jamesServer.getProbe(SmtpGuiceProbe.class).getSmtpPort())
             .sendMessage(RECIPIENT, RECIPIENT)
             .sendMessage(RECIPIENT, RECIPIENT);
diff --git a/server/mailet/integration-testing/src/test/java/org/apache/james/transport/mailets/ToSenderDomainRepositoryIntegrationTest.java b/server/mailet/integration-testing/src/test/java/org/apache/james/transport/mailets/ToSenderDomainRepositoryIntegrationTest.java
index 062a2b8..98653b9 100644
--- a/server/mailet/integration-testing/src/test/java/org/apache/james/transport/mailets/ToSenderDomainRepositoryIntegrationTest.java
+++ b/server/mailet/integration-testing/src/test/java/org/apache/james/transport/mailets/ToSenderDomainRepositoryIntegrationTest.java
@@ -43,8 +43,7 @@ import org.junit.jupiter.api.Test;
 import org.junit.jupiter.api.extension.RegisterExtension;
 import org.junit.jupiter.api.io.TempDir;
 
-public class ToSenderDomainRepositoryIntegrationTest {
-
+class ToSenderDomainRepositoryIntegrationTest {
     private static final String RECIPIENT = "touser@" + DEFAULT_DOMAIN;
     private static final String CUSTOM_REPOSITORY_PREFIX = "memory://var/mail/custom/";
     public static final MailRepositoryUrl DOMAIN_URL = MailRepositoryUrl.from(CUSTOM_REPOSITORY_PREFIX + DEFAULT_DOMAIN);
@@ -58,14 +57,14 @@ public class ToSenderDomainRepositoryIntegrationTest {
     private TemporaryJamesServer jamesServer;
 
     @AfterEach
-    public void tearDown() {
+    void tearDown() {
         if (jamesServer != null) {
             jamesServer.shutdown();
         }
     }
 
     @Test
-    public void incomingMailShouldBeStoredInCorrespondingMailRepository(@TempDir File temporaryFolder) throws Exception {
+    void incomingMailShouldBeStoredInCorrespondingMailRepository(@TempDir File temporaryFolder) throws Exception {
         startJamesServerWithMailetContainer(temporaryFolder, TemporaryJamesServer.simpleMailetContainerConfiguration()
             .putProcessor(ProcessorConfiguration.root()
                 .addMailet(MailetConfiguration.builder()
@@ -85,7 +84,7 @@ public class ToSenderDomainRepositoryIntegrationTest {
     }
 
     @Test
-    public void incomingMailShouldBeStoredWhenRepositoryDoesNotExistAndAllowedToCreateRepository(@TempDir File temporaryFolder) throws Exception {
+    void incomingMailShouldBeStoredWhenRepositoryDoesNotExistAndAllowedToCreateRepository(@TempDir File temporaryFolder) throws Exception {
         startJamesServerWithMailetContainer(temporaryFolder, TemporaryJamesServer.simpleMailetContainerConfiguration()
             .putProcessor(ProcessorConfiguration.root()
                 .addMailet(MailetConfiguration.builder()
@@ -106,7 +105,7 @@ public class ToSenderDomainRepositoryIntegrationTest {
     }
 
     @Test
-    public void incomingMailShouldBeStoredWhenRepositoryExistsAndAllowedToCreateRepository(@TempDir File temporaryFolder) throws Exception {
+    void incomingMailShouldBeStoredWhenRepositoryExistsAndAllowedToCreateRepository(@TempDir File temporaryFolder) throws Exception {
         startJamesServerWithMailetContainer(temporaryFolder, TemporaryJamesServer.simpleMailetContainerConfiguration()
             .putProcessor(ProcessorConfiguration.root()
                 .addMailet(MailetConfiguration.builder()
@@ -129,7 +128,7 @@ public class ToSenderDomainRepositoryIntegrationTest {
     }
 
     @Test
-    public void incomingMailShouldBeIgnoredWhenRepositoryDoesNotExistAndNotAllowedToCreateRepository(@TempDir File temporaryFolder) throws Exception {
+    void incomingMailShouldBeIgnoredWhenRepositoryDoesNotExistAndNotAllowedToCreateRepository(@TempDir File temporaryFolder) throws Exception {
         startJamesServerWithMailetContainer(temporaryFolder, TemporaryJamesServer.simpleMailetContainerConfiguration()
             .putProcessor(ProcessorConfiguration.root()
                 .addMailet(MailetConfiguration.builder()
@@ -155,7 +154,7 @@ public class ToSenderDomainRepositoryIntegrationTest {
     }
 
     @Test
-    public void incomingMailShouldBeStoredWhenRepositoryExistsAndNotAllowedToCreateRepository(@TempDir File temporaryFolder) throws Exception {
+    void incomingMailShouldBeStoredWhenRepositoryExistsAndNotAllowedToCreateRepository(@TempDir File temporaryFolder) throws Exception {
         startJamesServerWithMailetContainer(temporaryFolder, TemporaryJamesServer.simpleMailetContainerConfiguration()
             .putProcessor(ProcessorConfiguration.root()
                 .addMailet(MailetConfiguration.builder()
@@ -178,7 +177,7 @@ public class ToSenderDomainRepositoryIntegrationTest {
     }
 
     @Test
-    public void incomingMailsShouldBeStoredInCorrespondingMailRepository(@TempDir File temporaryFolder) throws Exception {
+    void incomingMailsShouldBeStoredInCorrespondingMailRepository(@TempDir File temporaryFolder) throws Exception {
         startJamesServerWithMailetContainer(temporaryFolder, TemporaryJamesServer.simpleMailetContainerConfiguration()
             .putProcessor(ProcessorConfiguration.root()
                 .addMailet(MailetConfiguration.builder()


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