You are viewing a plain text version of this content. The canonical link for it is here.
Posted to server-dev@james.apache.org by bt...@apache.org on 2019/11/28 02:12:35 UTC

[james-project] 18/23: JAMES-2989 Rework toString for MimePathImpl

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 be28f6eb6be6783986d970515dfed63cf08770ed
Author: Benoit Tellier <bt...@linagora.com>
AuthorDate: Tue Nov 26 13:46:55 2019 +0700

    JAMES-2989 Rework toString for MimePathImpl
---
 .../api/src/main/java/org/apache/james/mailbox/model/MimePath.java | 7 ++++++-
 .../src/test/java/org/apache/james/mailbox/model/MimePathTest.java | 6 +++---
 2 files changed, 9 insertions(+), 4 deletions(-)

diff --git a/mailbox/api/src/main/java/org/apache/james/mailbox/model/MimePath.java b/mailbox/api/src/main/java/org/apache/james/mailbox/model/MimePath.java
index f77a5ea..b41301e 100644
--- a/mailbox/api/src/main/java/org/apache/james/mailbox/model/MimePath.java
+++ b/mailbox/api/src/main/java/org/apache/james/mailbox/model/MimePath.java
@@ -66,7 +66,12 @@ public final class MimePath {
 
     @Override
     public final String toString() {
+        List<Integer> parts = Arrays.stream(positions)
+            .boxed()
+            .collect(Guavate.toImmutableList());
+
         return "MIMEPath:"
-            + Arrays.toString(positions);
+            + Joiner.on('.')
+            .join(parts);
     }
 }
\ No newline at end of file
diff --git a/mailbox/api/src/test/java/org/apache/james/mailbox/model/MimePathTest.java b/mailbox/api/src/test/java/org/apache/james/mailbox/model/MimePathTest.java
index 3715a40..91e2651 100644
--- a/mailbox/api/src/test/java/org/apache/james/mailbox/model/MimePathTest.java
+++ b/mailbox/api/src/test/java/org/apache/james/mailbox/model/MimePathTest.java
@@ -36,20 +36,20 @@ class MimePathTest {
     void toStringWhenEmpty() {
         int[] empty = {};
         assertThat(new MimePath(empty).toString())
-            .isEqualTo("MIMEPath:[]");
+            .isEqualTo("MIMEPath:");
     }
 
     @Test
     void toStringWhenSingle() {
         int[] single = {1};
         assertThat(new MimePath(single).toString())
-            .isEqualTo("MIMEPath:[1]");
+            .isEqualTo("MIMEPath:1");
     }
 
     @Test
     void toStringWhenMany() {
         int[] many = {1, 2, 3};
         assertThat(new MimePath(many).toString())
-            .isEqualTo("MIMEPath:[1, 2, 3]");
+            .isEqualTo("MIMEPath:1.2.3");
     }
 }
\ No newline at end of file


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