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 dda56cc95205afdf138ef2fa35604becf7bae7d8
Author: Benoit Tellier <[email protected]>
AuthorDate: Mon Nov 18 17:16:25 2019 +0700

    [Refactoring] MockMailProcessor::newState can be final
---
 .../org/apache/james/mailetcontainer/api/mock/MockMailProcessor.java   | 2 +-
 .../james/mailetcontainer/lib/AbstractStateCompositeProcessorTest.java | 3 +--
 2 files changed, 2 insertions(+), 3 deletions(-)

diff --git 
a/server/mailet/mailetcontainer-api/src/test/java/org/apache/james/mailetcontainer/api/mock/MockMailProcessor.java
 
b/server/mailet/mailetcontainer-api/src/test/java/org/apache/james/mailetcontainer/api/mock/MockMailProcessor.java
index 507a3bc..c1e9cbc 100644
--- 
a/server/mailet/mailetcontainer-api/src/test/java/org/apache/james/mailetcontainer/api/mock/MockMailProcessor.java
+++ 
b/server/mailet/mailetcontainer-api/src/test/java/org/apache/james/mailetcontainer/api/mock/MockMailProcessor.java
@@ -22,7 +22,7 @@ import org.apache.james.mailetcontainer.api.MailProcessor;
 import org.apache.mailet.Mail;
 
 public class MockMailProcessor implements MailProcessor {
-    private String newState;
+    private final String newState;
 
     protected MockMailProcessor(String newState) {
         this.newState = newState;
diff --git 
a/server/mailet/mailetcontainer-camel/src/test/java/org/apache/james/mailetcontainer/lib/AbstractStateCompositeProcessorTest.java
 
b/server/mailet/mailetcontainer-camel/src/test/java/org/apache/james/mailetcontainer/lib/AbstractStateCompositeProcessorTest.java
index fbe8000..ffa1258 100644
--- 
a/server/mailet/mailetcontainer-camel/src/test/java/org/apache/james/mailetcontainer/lib/AbstractStateCompositeProcessorTest.java
+++ 
b/server/mailet/mailetcontainer-camel/src/test/java/org/apache/james/mailetcontainer/lib/AbstractStateCompositeProcessorTest.java
@@ -48,8 +48,7 @@ public abstract class AbstractStateCompositeProcessorTest {
         AbstractStateCompositeProcessor processor = new 
AbstractStateCompositeProcessor() {
 
             @Override
-            protected MailProcessor createMailProcessor(final String state, 
HierarchicalConfiguration<ImmutableNode> config) throws
-                Exception {
+            protected MailProcessor createMailProcessor(String state, 
HierarchicalConfiguration<ImmutableNode> config) {
                 return new MockMailProcessor("") {
                     @Override
                     public void service(Mail mail) {


---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to