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 e6a2bd5bab879f0754ab8bcc7b0458d648b6c07c Author: Benoit Tellier <[email protected]> AuthorDate: Thu Apr 4 11:18:17 2019 +0700 JAMES-2723 Get rid of another useless Mailbox sub class --- .../store/mail/model/ListMailboxAssertTest.java | 77 ++-------------------- 1 file changed, 4 insertions(+), 73 deletions(-) diff --git a/mailbox/store/src/test/java/org/apache/james/mailbox/store/mail/model/ListMailboxAssertTest.java b/mailbox/store/src/test/java/org/apache/james/mailbox/store/mail/model/ListMailboxAssertTest.java index edd584f..f275673 100644 --- a/mailbox/store/src/test/java/org/apache/james/mailbox/store/mail/model/ListMailboxAssertTest.java +++ b/mailbox/store/src/test/java/org/apache/james/mailbox/store/mail/model/ListMailboxAssertTest.java @@ -24,12 +24,8 @@ import static org.assertj.core.api.Assertions.assertThat; import java.util.List; -import org.apache.commons.lang3.NotImplementedException; -import org.apache.james.mailbox.MailboxSession; import org.apache.james.mailbox.SimpleMailbox; import org.apache.james.mailbox.model.Mailbox; -import org.apache.james.mailbox.model.MailboxACL; -import org.apache.james.mailbox.model.MailboxId; import org.apache.james.mailbox.model.MailboxPath; import org.junit.Before; import org.junit.Test; @@ -55,78 +51,13 @@ public class ListMailboxAssertTest { } @Test - public void initListMailboxAssertShouldWork() throws Exception { + public void initListMailboxAssertShouldWork() { assertThat(listMaiboxAssert).isNotNull(); } @Test - public void assertListMailboxShouldWork() throws Exception { - assertMailboxes(actualMailbox).containOnly(createMailbox(NAMESPACE, USER, NAME, UID_VALIDITY), - createMailbox(OTHER_NAMESPACE, USER, NAME, UID_VALIDITY)); - } - - private Mailbox createMailbox(final String namespace, final String user, final String name, final long uid_validity) { - return new Mailbox() { - - @Override - public MailboxPath generateAssociatedPath() { - return new MailboxPath(getNamespace(), getUser(), getName()); - } - - @Override - public void setUser(String user) { - } - - @Override - public void setNamespace(String namespace) { - } - - @Override - public void setName(String name) { - } - - @Override - public void setMailboxId(MailboxId id) { - } - - @Override - public void setACL(MailboxACL acl) { - } - - @Override - public String getUser() { - return user; - } - - @Override - public long getUidValidity() { - return uid_validity; - } - - @Override - public String getNamespace() { - return namespace; - } - - @Override - public String getName() { - return name; - } - - @Override - public MailboxId getMailboxId() { - return null; - } - - @Override - public MailboxACL getACL() { - return null; - } - - @Override - public boolean isChildOf(Mailbox potentialParent, MailboxSession mailboxSession) { - throw new NotImplementedException("Not implemented"); - } - }; + public void assertListMailboxShouldWork() { + assertMailboxes(actualMailbox).containOnly(new SimpleMailbox(new MailboxPath(NAMESPACE, USER, NAME), UID_VALIDITY), + new SimpleMailbox(new MailboxPath(OTHER_NAMESPACE, USER, NAME), UID_VALIDITY)); } } --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
