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/06/21 09:16:33 UTC

[james-project] 10/12: JAMES-3167 Improve MailboxManagerTest coverage to avoid bug detection upon MPT IMAP tests run

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 2aefb065783bb07ea6090d896494032714a9a358
Author: Benoit Tellier <bt...@linagora.com>
AuthorDate: Tue Jun 9 09:53:53 2020 +0700

    JAMES-3167 Improve MailboxManagerTest coverage to avoid bug detection upon MPT IMAP tests run
---
 .../apache/james/mailbox/MailboxManagerTest.java    | 21 +++++++++++++++++++++
 1 file changed, 21 insertions(+)

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 603483b..3d92d48 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
@@ -1742,6 +1742,27 @@ public abstract class MailboxManagerTest<T extends MailboxManager> {
         }
 
         @Test
+        void renamingMailboxByPathShouldThrowWhenFromNotFound() throws Exception {
+            MailboxSession session = mailboxManager.createSystemSession(USER_1);
+
+            MailboxPath originPath = MailboxPath.forUser(USER_1, "origin");
+            MailboxPath destinationPath = MailboxPath.forUser(USER_1, "destination");
+
+            assertThatThrownBy(() -> mailboxManager.renameMailbox(originPath, destinationPath, session))
+                .isInstanceOf(MailboxNotFoundException.class);
+        }
+
+        @Test
+        void renamingMailboxByIdShouldThrowWhenFromNotFound() throws Exception {
+            MailboxSession session = mailboxManager.createSystemSession(USER_1);
+
+            MailboxPath notFound = MailboxPath.forUser(USER_1, "notFound");
+
+            assertThatThrownBy(() -> mailboxManager.deleteMailbox(notFound, session))
+                .isInstanceOf(MailboxNotFoundException.class);
+        }
+
+        @Test
         void user2ShouldNotBeAbleToDeleteUser1Mailbox() throws Exception {
             MailboxSession sessionUser1 = mailboxManager.createSystemSession(USER_1);
             MailboxSession sessionUser2 = mailboxManager.createSystemSession(USER_2);


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