[james-project] 35/43: JAMES-2982 more error messages in Username.of()

[bt...@apache.org]

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 feb8ae3297602583bfa56a76bde90976e11fcc1f
Author: Tran Tien Duc <dt...@linagora.com>
AuthorDate: Fri Nov 15 09:51:21 2019 +0700

    JAMES-2982 more error messages in Username.of()
---
 core/src/main/java/org/apache/james/core/Username.java | 5 +++--
 1 file changed, 3 insertions(+), 2 deletions(-)

diff --git a/core/src/main/java/org/apache/james/core/Username.java b/core/src/main/java/org/apache/james/core/Username.java
index b29e9b8..308bc69 100644
--- a/core/src/main/java/org/apache/james/core/Username.java
+++ b/core/src/main/java/org/apache/james/core/Username.java
@@ -36,8 +36,9 @@ public class Username {
     public static final int MAXIMUM_MAIL_ADDRESS_LENGTH = 255;
 
     public static Username of(String username) {
-        Preconditions.checkArgument(!Strings.isNullOrEmpty(username));
-        Preconditions.checkArgument(username.length() <= MAXIMUM_MAIL_ADDRESS_LENGTH);
+        Preconditions.checkArgument(!Strings.isNullOrEmpty(username), "username should not be null or empty");
+        Preconditions.checkArgument(username.length() <= MAXIMUM_MAIL_ADDRESS_LENGTH,
+            String.format("username length should not be longer than %d characters", MAXIMUM_MAIL_ADDRESS_LENGTH));
 
         List<String> parts = ImmutableList.copyOf(Splitter.on('@').split(username));
         switch (parts.size()) {


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