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 2020/02/27 03:32:12 UTC
[james-project] 05/08: JAMES-3058 Fix invalid MailboxPath::AsString
formatting
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 cfa46093e6158e66c9430a6cf857cb4fc2d4b44f
Author: Benoit Tellier <bt...@linagora.com>
AuthorDate: Thu Feb 13 15:37:36 2020 +0700
JAMES-3058 Fix invalid MailboxPath::AsString formatting
---
.../src/main/java/org/apache/james/mailbox/model/MailboxPath.java | 2 +-
.../test/java/org/apache/james/mailbox/model/MailboxPathTest.java | 6 ++++++
2 files changed, 7 insertions(+), 1 deletion(-)
diff --git a/mailbox/api/src/main/java/org/apache/james/mailbox/model/MailboxPath.java b/mailbox/api/src/main/java/org/apache/james/mailbox/model/MailboxPath.java
index e922503..f2617b7 100644
--- a/mailbox/api/src/main/java/org/apache/james/mailbox/model/MailboxPath.java
+++ b/mailbox/api/src/main/java/org/apache/james/mailbox/model/MailboxPath.java
@@ -179,7 +179,7 @@ public class MailboxPath {
}
public String asString() {
- return namespace + ":" + user + ":" + name;
+ return namespace + ":" + user.asString() + ":" + name;
}
public boolean isInbox() {
diff --git a/mailbox/api/src/test/java/org/apache/james/mailbox/model/MailboxPathTest.java b/mailbox/api/src/test/java/org/apache/james/mailbox/model/MailboxPathTest.java
index 0e5e887..453c91f 100644
--- a/mailbox/api/src/test/java/org/apache/james/mailbox/model/MailboxPathTest.java
+++ b/mailbox/api/src/test/java/org/apache/james/mailbox/model/MailboxPathTest.java
@@ -45,6 +45,12 @@ class MailboxPathTest {
}
@Test
+ void asStringShouldFormatUser() {
+ assertThat(MailboxPath.forUser(USER, "inbox.folder.subfolder").asString())
+ .isEqualTo("#private:user:inbox.folder.subfolder");
+ }
+
+ @Test
void getHierarchyLevelsShouldBeOrdered() {
assertThat(MailboxPath.forUser(USER, "inbox.folder.subfolder")
.getHierarchyLevels('.'))
---------------------------------------------------------------------
To unsubscribe, e-mail: server-dev-unsubscribe@james.apache.org
For additional commands, e-mail: server-dev-help@james.apache.org