This is an automated email from the ASF dual-hosted git repository.
rcordier pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/james-project.git
commit 07252a99b1406d69be50a5823e96d968c141921c
Author: Rene Cordier <rcord...@linagora.com>
AuthorDate: Mon Jan 13 14:27:42 2020 +0700
JAMES-2993 Add missing test cases in MailboxManagerTest
---
.../apache/james/mailbox/MailboxManagerTest.java | 410 ++++++++++++++++++++-
.../DomainUserMaildirMailboxManagerTest.java | 5 +
.../maildir/FullUserMaildirMailboxManagerTest.java | 5 +
3 files changed, 419 insertions(+), 1 deletion(-)
diff --git
a/mailbox/api/src/test/java/org/apache/james/mailbox/MailboxManagerTest.java
b/mailbox/api/src/test/java/org/apache/james/mailbox/MailboxManagerTest.java
index 1c30e4f..55d00f4 100644
--- a/mailbox/api/src/test/java/org/apache/james/mailbox/MailboxManagerTest.java
+++ b/mailbox/api/src/test/java/org/apache/james/mailbox/MailboxManagerTest.java
@@ -83,6 +83,7 @@ import org.apache.james.util.concurrency.ConcurrentTestRunner;
import org.assertj.core.api.SoftAssertions;
import org.junit.jupiter.api.AfterEach;
import org.junit.jupiter.api.BeforeEach;
+import org.junit.jupiter.api.Disabled;
import org.junit.jupiter.api.Nested;
import org.junit.jupiter.api.Test;
import org.mockito.ArgumentCaptor;
@@ -1565,6 +1566,84 @@ public abstract class MailboxManagerTest<T extends
MailboxManager> {
}
@Test
+ void renameMailboxShouldThrowWhenMailboxPathsDoNotBelongToUser()
throws Exception {
+ MailboxSession sessionUser1 =
mailboxManager.createSystemSession(USER_1);
+ MailboxSession sessionUser2 =
mailboxManager.createSystemSession(USER_2);
+
+ MailboxPath mailboxPath1 = MailboxPath.forUser(USER_1, "mbx1");
+ MailboxPath mailboxPath2 = MailboxPath.forUser(USER_1, "mbx2");
+ mailboxManager.createMailbox(mailboxPath1, sessionUser1);
+
+ assertThatThrownBy(() ->
mailboxManager.renameMailbox(mailboxPath1, mailboxPath2, sessionUser2))
+ .isInstanceOf(MailboxException.class);
+ }
+
+ @Test
+ void renameMailboxByIdShouldThrowWhenMailboxPathsDoNotBelongToUser()
throws Exception {
+ MailboxSession sessionUser1 =
mailboxManager.createSystemSession(USER_1);
+ MailboxSession sessionUser2 =
mailboxManager.createSystemSession(USER_2);
+
+ MailboxPath mailboxPath1 = MailboxPath.forUser(USER_1, "mbx1");
+ MailboxPath mailboxPath2 = MailboxPath.forUser(USER_1, "mbx2");
+ Optional<MailboxId> mailboxId =
mailboxManager.createMailbox(mailboxPath1, sessionUser1);
+
+ assertThatThrownBy(() ->
mailboxManager.renameMailbox(mailboxId.get(), mailboxPath2, sessionUser2))
+ .isInstanceOf(MailboxException.class);
+ }
+
+ @Test
+ void renameMailboxShouldThrowWhenFromMailboxPathDoesNotBelongToUser()
throws Exception {
+ MailboxSession sessionUser1 =
mailboxManager.createSystemSession(USER_1);
+ MailboxSession sessionUser2 =
mailboxManager.createSystemSession(USER_2);
+
+ MailboxPath mailboxPath1 = MailboxPath.forUser(USER_1, "mbx1");
+ MailboxPath mailboxPath2 = MailboxPath.forUser(USER_2, "mbx2");
+ mailboxManager.createMailbox(mailboxPath1, sessionUser1);
+
+ assertThatThrownBy(() ->
mailboxManager.renameMailbox(mailboxPath1, mailboxPath2, sessionUser2))
+ .isInstanceOf(MailboxNotFoundException.class);
+ }
+
+ @Test
+ void
renameMailboxByIdShouldThrowWhenFromMailboxPathDoesNotBelongToUser() throws
Exception {
+ MailboxSession sessionUser1 =
mailboxManager.createSystemSession(USER_1);
+ MailboxSession sessionUser2 =
mailboxManager.createSystemSession(USER_2);
+
+ MailboxPath mailboxPath1 = MailboxPath.forUser(USER_1, "mbx1");
+ MailboxPath mailboxPath2 = MailboxPath.forUser(USER_2, "mbx2");
+ Optional<MailboxId> mailboxId =
mailboxManager.createMailbox(mailboxPath1, sessionUser1);
+
+ assertThatThrownBy(() ->
mailboxManager.renameMailbox(mailboxId.get(), mailboxPath2, sessionUser2))
+ .isInstanceOf(MailboxNotFoundException.class);
+ }
+
+ @Test
+ @Disabled("JAMES-2933 renameMailbox does not assert that user is the
owner of destination mailbox")
+ void renameMailboxShouldThrowWhenToMailboxPathDoesNotBelongToUser()
throws Exception {
+ session = mailboxManager.createSystemSession(USER_1);
+
+ MailboxPath mailboxPath1 = MailboxPath.forUser(USER_1, "mbx1");
+ MailboxPath mailboxPath2 = MailboxPath.forUser(USER_2, "mbx2");
+ mailboxManager.createMailbox(mailboxPath1, session);
+
+ assertThatThrownBy(() ->
mailboxManager.renameMailbox(mailboxPath1, mailboxPath2, session))
+ .isInstanceOf(MailboxNotFoundException.class);
+ }
+
+ @Test
+ @Disabled("JAMES-2933 renameMailbox does not assert that user is the
owner of destination mailbox")
+ void
renameMailboxByIdShouldThrowWhenToMailboxPathDoesNotBelongToUser() throws
Exception {
+ session = mailboxManager.createSystemSession(USER_1);
+
+ MailboxPath mailboxPath1 = MailboxPath.forUser(USER_1, "mbx1");
+ MailboxPath mailboxPath2 = MailboxPath.forUser(USER_2, "mbx2");
+ Optional<MailboxId> mailboxId =
mailboxManager.createMailbox(mailboxPath1, session);
+
+ assertThatThrownBy(() ->
mailboxManager.renameMailbox(mailboxId.get(), mailboxPath2, session))
+ .isInstanceOf(MailboxNotFoundException.class);
+ }
+
+ @Test
void user1ShouldNotBeAbleToCreateInboxTwice() throws Exception {
session = mailboxManager.createSystemSession(USER_1);
mailboxManager.startProcessingRequest(session);
@@ -1633,6 +1712,31 @@ public abstract class MailboxManagerTest<T extends
MailboxManager> {
}
@Test
+ void user2ShouldNotBeAbleToDeleteUser1Mailbox() throws Exception {
+ MailboxSession sessionUser1 =
mailboxManager.createSystemSession(USER_1);
+ MailboxSession sessionUser2 =
mailboxManager.createSystemSession(USER_2);
+
+ MailboxPath inbox = MailboxPath.inbox(sessionUser1);
+ mailboxManager.createMailbox(inbox, sessionUser1);
+
+ assertThatThrownBy(() -> mailboxManager.deleteMailbox(inbox,
sessionUser2))
+ .isInstanceOf(MailboxException.class);
+ }
+
+
+ @Test
+ void user2ShouldNotBeAbleToDeleteUser1MailboxById() throws Exception {
+ MailboxSession sessionUser1 =
mailboxManager.createSystemSession(USER_1);
+ MailboxSession sessionUser2 =
mailboxManager.createSystemSession(USER_2);
+
+ MailboxPath inbox = MailboxPath.inbox(sessionUser1);
+ MailboxId inboxId = mailboxManager.createMailbox(inbox,
sessionUser1).get();
+
+ assertThatThrownBy(() -> mailboxManager.deleteMailbox(inboxId,
sessionUser2))
+ .isInstanceOf(MailboxException.class);
+ }
+
+ @Test
void user1ShouldBeAbleToDeleteSubmailbox() throws Exception {
session = mailboxManager.createSystemSession(USER_1);
mailboxManager.startProcessingRequest(session);
@@ -1675,6 +1779,14 @@ public abstract class MailboxManagerTest<T extends
MailboxManager> {
}
@Test
+ void listShouldReturnEmptyListWhenNoMailboxes() throws Exception {
+ session =
mailboxManager.createSystemSession(Username.of("manager"));
+
+ assertThat(mailboxManager.list(session))
+ .isEmpty();
+ }
+
+ @Test
void user2ShouldBeAbleToCreateRootlessFolder() throws MailboxException
{
session = mailboxManager.createSystemSession(USER_2);
MailboxPath trash = MailboxPath.forUser(USER_2, "Trash");
@@ -1706,6 +1818,125 @@ public abstract class MailboxManagerTest<T extends
MailboxManager> {
}
@Test
+ void
moveMessagesShouldMoveAllMessagesFromOneMailboxToAnOtherOfASameUser() throws
Exception {
+ assumeTrue(mailboxManager.hasCapability(MailboxCapabilities.ACL));
+ assumeTrue(mailboxManager.hasCapability(MailboxCapabilities.Move));
+
+ session = mailboxManager.createSystemSession(USER_1);
+
+ MailboxPath inbox = MailboxPath.inbox(session);
+ MailboxId inboxId = mailboxManager.createMailbox(inbox,
session).get();
+
+ MailboxPath otherMailbox = MailboxPath.forUser(USER_1,
"otherMailbox");
+ MailboxId otherMailboxId =
mailboxManager.createMailbox(otherMailbox, session).get();
+
+ MessageManager inboxMessageManager =
mailboxManager.getMailbox(inbox, session);
+
+ MessageId messageId1 = inboxMessageManager
+ .appendMessage(AppendCommand.from(message), session)
+ .getMessageId();
+ MessageId messageId2 = inboxMessageManager
+ .appendMessage(AppendCommand.from(message), session)
+ .getMessageId();
+
+ mailboxManager.moveMessages(MessageRange.all(), inbox,
otherMailbox, session);
+
+ MultimailboxesSearchQuery inboxQuery = MultimailboxesSearchQuery
+ .from(new SearchQuery())
+ .inMailboxes(inboxId)
+ .build();
+
+ MultimailboxesSearchQuery otherMailboxQuery =
MultimailboxesSearchQuery
+ .from(new SearchQuery())
+ .inMailboxes(otherMailboxId)
+ .build();
+
+ assertThat(mailboxManager.search(inboxQuery, session,
DEFAULT_MAXIMUM_LIMIT))
+ .isEmpty();
+ assertThat(mailboxManager.search(otherMailboxQuery, session,
DEFAULT_MAXIMUM_LIMIT))
+ .containsExactly(messageId1, messageId2);
+ }
+
+ @Test
+ void
moveMessagesShouldMoveOnlyOneMessageFromOneMailboxToAnOtherOfASameUser() throws
Exception {
+ assumeTrue(mailboxManager.hasCapability(MailboxCapabilities.ACL));
+ assumeTrue(mailboxManager.hasCapability(MailboxCapabilities.Move));
+
+ session = mailboxManager.createSystemSession(USER_1);
+
+ MailboxPath inbox = MailboxPath.inbox(session);
+ MailboxId inboxId = mailboxManager.createMailbox(inbox,
session).get();
+
+ MailboxPath otherMailbox = MailboxPath.forUser(USER_1,
"otherMailbox");
+ MailboxId otherMailboxId =
mailboxManager.createMailbox(otherMailbox, session).get();
+
+ MessageManager inboxMessageManager =
mailboxManager.getMailbox(inbox, session);
+
+ ComposedMessageId composedMessageId1 = inboxMessageManager
+ .appendMessage(AppendCommand.from(message), session);
+ MessageId messageId2 = inboxMessageManager
+ .appendMessage(AppendCommand.from(message), session)
+ .getMessageId();
+
+
mailboxManager.moveMessages(MessageRange.one(composedMessageId1.getUid()),
inbox, otherMailbox, session);
+
+ MultimailboxesSearchQuery inboxQuery = MultimailboxesSearchQuery
+ .from(new SearchQuery())
+ .inMailboxes(inboxId)
+ .build();
+
+ MultimailboxesSearchQuery otherMailboxQuery =
MultimailboxesSearchQuery
+ .from(new SearchQuery())
+ .inMailboxes(otherMailboxId)
+ .build();
+
+ assertThat(mailboxManager.search(inboxQuery, session,
DEFAULT_MAXIMUM_LIMIT))
+ .containsExactly(messageId2);
+ assertThat(mailboxManager.search(otherMailboxQuery, session,
DEFAULT_MAXIMUM_LIMIT))
+ .containsExactly(composedMessageId1.getMessageId());
+ }
+
+ @Test
+ void
moveMessagesShouldThrowWhenMovingMessageFromMailboxNotBelongingToSameUser()
throws Exception {
+ MailboxSession sessionUser1 =
mailboxManager.createSystemSession(USER_1);
+ MailboxSession sessionUser2 =
mailboxManager.createSystemSession(USER_2);
+
+ MailboxPath inbox1 = MailboxPath.inbox(sessionUser1);
+ mailboxManager.createMailbox(inbox1, sessionUser1);
+
+ MailboxPath inbox2 = MailboxPath.inbox(sessionUser2);
+ mailboxManager.createMailbox(inbox2, sessionUser2);
+
+ MessageManager inboxMessageManager =
mailboxManager.getMailbox(inbox1, sessionUser1);
+
+ inboxMessageManager
+ .appendMessage(AppendCommand.from(message), sessionUser1);
+
+ assertThatThrownBy(() ->
mailboxManager.moveMessages(MessageRange.all(), inbox1, inbox2, sessionUser2))
+ .isInstanceOf(MailboxException.class);
+ }
+
+ @Test
+ void
moveMessagesShouldThrowWhenMovingMessageToMailboxNotBelongingToSameUser()
throws Exception {
+ MailboxSession sessionUser1 =
mailboxManager.createSystemSession(USER_1);
+ MailboxSession sessionUser2 =
mailboxManager.createSystemSession(USER_2);
+
+ MailboxPath inbox1 = MailboxPath.inbox(sessionUser1);
+ mailboxManager.createMailbox(inbox1, sessionUser1);
+
+ MailboxPath inbox2 = MailboxPath.inbox(sessionUser2);
+ mailboxManager.createMailbox(inbox2, sessionUser2);
+
+ MessageManager inboxMessageManager =
mailboxManager.getMailbox(inbox1, sessionUser1);
+
+ inboxMessageManager
+ .appendMessage(AppendCommand.from(message), sessionUser1);
+
+ assertThatThrownBy(() ->
mailboxManager.moveMessages(MessageRange.all(), inbox1, inbox2, sessionUser1))
+ .isInstanceOf(MailboxException.class);
+ }
+
+ @Test
void copyMessagesShouldNotThrowWhenMovingAllMessagesOfAnEmptyMailbox()
throws Exception {
session = mailboxManager.createSystemSession(USER_1);
@@ -1717,6 +1948,127 @@ public abstract class MailboxManagerTest<T extends
MailboxManager> {
}
@Test
+ void
copyMessagesShouldCopyAllMessagesFromOneMailboxToAnOtherOfASameUser() throws
Exception {
+ assumeTrue(mailboxManager.hasCapability(MailboxCapabilities.ACL));
+
+ session = mailboxManager.createSystemSession(USER_1);
+
+ MailboxPath inbox = MailboxPath.inbox(session);
+ MailboxId inboxId = mailboxManager.createMailbox(inbox,
session).get();
+
+ MailboxPath otherMailbox = MailboxPath.forUser(USER_1,
"otherMailbox");
+ MailboxId otherMailboxId =
mailboxManager.createMailbox(otherMailbox, session).get();
+
+ MessageManager inboxMessageManager =
mailboxManager.getMailbox(inbox, session);
+
+ MessageId messageId1 = inboxMessageManager
+ .appendMessage(AppendCommand.from(message), session)
+ .getMessageId();
+ MessageId messageId2 = inboxMessageManager
+ .appendMessage(AppendCommand.from(message), session)
+ .getMessageId();
+
+ mailboxManager.copyMessages(MessageRange.all(), inbox,
otherMailbox, session);
+
+ MultimailboxesSearchQuery inboxQuery = MultimailboxesSearchQuery
+ .from(new SearchQuery())
+ .inMailboxes(inboxId)
+ .build();
+
+ MultimailboxesSearchQuery otherMailboxQuery =
MultimailboxesSearchQuery
+ .from(new SearchQuery())
+ .inMailboxes(otherMailboxId)
+ .build();
+
+ assertThat(mailboxManager.search(inboxQuery, session,
DEFAULT_MAXIMUM_LIMIT))
+ .containsExactly(messageId1, messageId2);
+ assertThat(mailboxManager.search(otherMailboxQuery, session,
DEFAULT_MAXIMUM_LIMIT))
+ .containsExactly(messageId1, messageId2);
+ }
+
+ @Test
+ void
copyMessagesShouldCopyOnlyOneMessageFromOneMailboxToAnOtherOfASameUser() throws
Exception {
+ assumeTrue(mailboxManager.hasCapability(MailboxCapabilities.ACL));
+
+ session = mailboxManager.createSystemSession(USER_1);
+
+ MailboxPath inbox = MailboxPath.inbox(session);
+ MailboxId inboxId = mailboxManager.createMailbox(inbox,
session).get();
+
+ MailboxPath otherMailbox = MailboxPath.forUser(USER_1,
"otherMailbox");
+ MailboxId otherMailboxId =
mailboxManager.createMailbox(otherMailbox, session).get();
+
+ MessageManager inboxMessageManager =
mailboxManager.getMailbox(inbox, session);
+
+ ComposedMessageId composedMessageId1 = inboxMessageManager
+ .appendMessage(AppendCommand.from(message), session);
+ MessageId messageId2 = inboxMessageManager
+ .appendMessage(AppendCommand.from(message), session)
+ .getMessageId();
+
+ MessageId messageId1 = composedMessageId1.getMessageId();
+
+
mailboxManager.copyMessages(MessageRange.one(composedMessageId1.getUid()),
inbox, otherMailbox, session);
+
+ MultimailboxesSearchQuery inboxQuery = MultimailboxesSearchQuery
+ .from(new SearchQuery())
+ .inMailboxes(inboxId)
+ .build();
+
+ MultimailboxesSearchQuery otherMailboxQuery =
MultimailboxesSearchQuery
+ .from(new SearchQuery())
+ .inMailboxes(otherMailboxId)
+ .build();
+
+ assertThat(mailboxManager.search(inboxQuery, session,
DEFAULT_MAXIMUM_LIMIT))
+ .containsExactly(messageId1, messageId2);
+ assertThat(mailboxManager.search(otherMailboxQuery, session,
DEFAULT_MAXIMUM_LIMIT))
+ .containsExactly(messageId1);
+ }
+
+ @Test
+ @Disabled("JAMES-2993 It seems that getMailbox by path in
MailboxManager does not check that the mailbox belongs to user, "
+ + "compared to getMailbox by id that actually does assert it")
+ void
copyMessagesShouldThrowWhenCopyingMessageFromMailboxNotBelongingToSameUser()
throws Exception {
+ MailboxSession sessionUser1 =
mailboxManager.createSystemSession(USER_1);
+ MailboxSession sessionUser2 =
mailboxManager.createSystemSession(USER_2);
+
+ MailboxPath inbox1 = MailboxPath.inbox(sessionUser1);
+ mailboxManager.createMailbox(inbox1, sessionUser1);
+
+ MailboxPath inbox2 = MailboxPath.inbox(sessionUser2);
+ mailboxManager.createMailbox(inbox2, sessionUser2);
+
+ MessageManager inboxMessageManager =
mailboxManager.getMailbox(inbox1, sessionUser1);
+
+ inboxMessageManager
+ .appendMessage(AppendCommand.from(message), sessionUser1);
+
+ assertThatThrownBy(() ->
mailboxManager.copyMessages(MessageRange.all(), inbox1, inbox2, sessionUser2))
+ .isInstanceOf(MailboxException.class);
+ }
+
+ @Test
+ void
copyMessagesShouldThrowWhenCopyingMessageToMailboxNotBelongingToSameUser()
throws Exception {
+ MailboxSession sessionUser1 =
mailboxManager.createSystemSession(USER_1);
+ MailboxSession sessionUser2 =
mailboxManager.createSystemSession(USER_2);
+
+ MailboxPath inbox1 = MailboxPath.inbox(sessionUser1);
+ mailboxManager.createMailbox(inbox1, sessionUser1);
+
+ MailboxPath inbox2 = MailboxPath.inbox(sessionUser2);
+ mailboxManager.createMailbox(inbox2, sessionUser2);
+
+ MessageManager inboxMessageManager =
mailboxManager.getMailbox(inbox1, sessionUser1);
+
+ inboxMessageManager
+ .appendMessage(AppendCommand.from(message), sessionUser1);
+
+ assertThatThrownBy(() ->
mailboxManager.copyMessages(MessageRange.all(), inbox1, inbox2, sessionUser1))
+ .isInstanceOf(MailboxException.class);
+ }
+
+ @Test
void createMailboxShouldNotThrowWhenMailboxPathBelongsToUser() throws
MailboxException {
session = mailboxManager.createSystemSession(USER_1);
Optional<MailboxId> mailboxId = mailboxManager
@@ -1726,13 +2078,69 @@ public abstract class MailboxManagerTest<T extends
MailboxManager> {
}
@Test
- void createMailboxShouldThrowWhenMailboxPathBelongsToAnotherUser()
throws MailboxException {
+ void createMailboxShouldThrowWhenMailboxPathBelongsToAnotherUser() {
session = mailboxManager.createSystemSession(USER_1);
assertThatThrownBy(() -> mailboxManager
.createMailbox(MailboxPath.forUser(USER_2, "mailboxName"),
session))
.isInstanceOf(MailboxException.class);
}
+
+ @Test
+ void getMailboxShouldThrowWhenMailboxDoesNotExist() {
+ session = mailboxManager.createSystemSession(USER_1);
+
+ assertThatThrownBy(() ->
mailboxManager.getMailbox(MailboxPath.forUser(USER_1, "mailboxName"), session))
+ .isInstanceOf(MailboxException.class);
+ }
+
+ @Test
+ void getMailboxByPathShouldReturnMailboxWhenBelongingToUser() throws
Exception {
+ session = mailboxManager.createSystemSession(USER_1);
+
+ MailboxPath mailboxPath = MailboxPath.forUser(USER_1,
"mailboxName");
+ Optional<MailboxId> mailboxId =
mailboxManager.createMailbox(mailboxPath, session);
+
+ assertThat(mailboxManager.getMailbox(mailboxPath, session).getId())
+ .isEqualTo(mailboxId.get());
+ }
+
+ @Test
+ protected void getMailboxByIdShouldReturnMailboxWhenBelongingToUser()
throws Exception {
+ session = mailboxManager.createSystemSession(USER_1);
+
+ MailboxPath mailboxPath = MailboxPath.forUser(USER_1,
"mailboxName");
+ Optional<MailboxId> mailboxId =
mailboxManager.createMailbox(mailboxPath, session);
+
+ assertThat(mailboxManager.getMailbox(mailboxId.get(),
session).getId())
+ .isEqualTo(mailboxId.get());
+ }
+
+ @Test
+ @Disabled("JAMES-2993 It seems that getMailbox by path in
MailboxManager does not check that the mailbox belongs to user, "
+ + "compared to getMailbox by id that actually does assert it")
+ void getMailboxByPathShouldThrowWhenMailboxNotBelongingToUser() throws
Exception {
+ MailboxSession sessionUser1 =
mailboxManager.createSystemSession(USER_1);
+ MailboxSession sessionUser2 =
mailboxManager.createSystemSession(USER_2);
+
+ MailboxPath mailboxPath = MailboxPath.forUser(USER_1,
"mailboxName");
+ mailboxManager.createMailbox(mailboxPath, sessionUser1);
+
+ assertThatThrownBy(() -> mailboxManager.getMailbox(mailboxPath,
sessionUser2))
+ .isInstanceOf(MailboxException.class);
+ }
+
+ @Test
+ void getMailboxByIdShouldThrowWhenMailboxNotBelongingToUser() throws
Exception {
+ MailboxSession sessionUser1 =
mailboxManager.createSystemSession(USER_1);
+ MailboxSession sessionUser2 =
mailboxManager.createSystemSession(USER_2);
+
+ MailboxPath mailboxPath = MailboxPath.forUser(USER_1,
"mailboxName");
+ Optional<MailboxId> mailboxId =
mailboxManager.createMailbox(mailboxPath, sessionUser1);
+
+ assertThatThrownBy(() ->
mailboxManager.getMailbox(mailboxId.get(), sessionUser2))
+ .isInstanceOf(MailboxException.class);
+ }
}
@Nested
diff --git
a/mailbox/maildir/src/test/java/org/apache/james/mailbox/maildir/DomainUserMaildirMailboxManagerTest.java
b/mailbox/maildir/src/test/java/org/apache/james/mailbox/maildir/DomainUserMaildirMailboxManagerTest.java
index 2bcc35c..e7f5a42 100644
---
a/mailbox/maildir/src/test/java/org/apache/james/mailbox/maildir/DomainUserMaildirMailboxManagerTest.java
+++
b/mailbox/maildir/src/test/java/org/apache/james/mailbox/maildir/DomainUserMaildirMailboxManagerTest.java
@@ -56,6 +56,11 @@ class DomainUserMaildirMailboxManagerTest extends
MailboxManagerTest<StoreMailbo
@Test
protected void user1ShouldBeAbleToDeleteInboxById() {
}
+
+ @Disabled("JAMES-2993 mailboxId support for Maildir is partial")
+ @Test
+ protected void getMailboxByIdShouldReturnMailboxWhenBelongingToUser() {
+ }
}
@Nested
diff --git
a/mailbox/maildir/src/test/java/org/apache/james/mailbox/maildir/FullUserMaildirMailboxManagerTest.java
b/mailbox/maildir/src/test/java/org/apache/james/mailbox/maildir/FullUserMaildirMailboxManagerTest.java
index d1f24c5..080cc37 100644
---
a/mailbox/maildir/src/test/java/org/apache/james/mailbox/maildir/FullUserMaildirMailboxManagerTest.java
+++
b/mailbox/maildir/src/test/java/org/apache/james/mailbox/maildir/FullUserMaildirMailboxManagerTest.java
@@ -56,6 +56,11 @@ class FullUserMaildirMailboxManagerTest extends
MailboxManagerTest<StoreMailboxM
@Test
protected void user1ShouldBeAbleToDeleteInboxById() {
}
+
+ @Disabled("JAMES-2993 mailboxId support for Maildir is partial")
+ @Test
+ protected void getMailboxByIdShouldReturnMailboxWhenBelongingToUser() {
+ }
}
@Nested
---------------------------------------------------------------------
To unsubscribe, e-mail: server-dev-unsubscr...@james.apache.org
For additional commands, e-mail: server-dev-h...@james.apache.org
