Repository: james-project Updated Branches: refs/heads/master f2cca215c -> 3a70ae82d
MPT-39 Handle issue tracking for too long mailboxes names Project: http://git-wip-us.apache.org/repos/asf/james-project/repo Commit: http://git-wip-us.apache.org/repos/asf/james-project/commit/ae336763 Tree: http://git-wip-us.apache.org/repos/asf/james-project/tree/ae336763 Diff: http://git-wip-us.apache.org/repos/asf/james-project/diff/ae336763 Branch: refs/heads/master Commit: ae3367632aac3c4a91fe23105b5e3dcea80e4787 Parents: 6a23a1b Author: benwa <btell...@linagora.com> Authored: Fri Jun 23 10:33:48 2017 +0700 Committer: Matthieu Baechler <matth...@apache.org> Committed: Thu Jul 6 10:32:17 2017 -0400 ---------------------------------------------------------------------- .../CassandraMailboxWithLongNameSuccess.java | 34 ------------------- .../suite/MailboxWithLongNameSuccess.java | 3 -- .../hbase/HBaseMailboxWithLongNameSuccess.java | 34 ------------------- .../InMemoryMailboxWithLongNameError.java | 34 ------------------- .../jpa/JpaMailboxWithLongNameError.java | 2 +- .../jpa/JpaMailboxWithLongNameSuccess.java | 34 ------------------- .../LuceneMailboxWithLongNameError.java | 2 +- .../LuceneMailboxWithLongNameSuccess.java | 35 -------------------- .../MaildirMailboxWithLongNameError.java | 2 +- .../MaildirMailboxWithLongNameSuccess.java | 34 ------------------- 10 files changed, 3 insertions(+), 211 deletions(-) ---------------------------------------------------------------------- http://git-wip-us.apache.org/repos/asf/james-project/blob/ae336763/mpt/impl/imap-mailbox/cassandra/src/test/java/org/apache/james/mpt/imapmailbox/cassandra/CassandraMailboxWithLongNameSuccess.java ---------------------------------------------------------------------- diff --git a/mpt/impl/imap-mailbox/cassandra/src/test/java/org/apache/james/mpt/imapmailbox/cassandra/CassandraMailboxWithLongNameSuccess.java b/mpt/impl/imap-mailbox/cassandra/src/test/java/org/apache/james/mpt/imapmailbox/cassandra/CassandraMailboxWithLongNameSuccess.java deleted file mode 100644 index 1be29c0..0000000 --- a/mpt/impl/imap-mailbox/cassandra/src/test/java/org/apache/james/mpt/imapmailbox/cassandra/CassandraMailboxWithLongNameSuccess.java +++ /dev/null @@ -1,34 +0,0 @@ -package org.apache.james.mpt.imapmailbox.cassandra; - -import org.apache.james.mpt.api.ImapHostSystem; -import org.apache.james.mpt.imapmailbox.suite.MailboxWithLongNameSuccess; -import org.junit.After; -import org.junit.Before; -import org.junit.Ignore; - -import com.google.inject.Guice; -import com.google.inject.Injector; - -@Ignore("do we have a JIRA for that ?") -public class CassandraMailboxWithLongNameSuccess extends MailboxWithLongNameSuccess { - - private ImapHostSystem system; - - @Before - public void setUp() throws Exception { - Injector injector = Guice.createInjector(new CassandraMailboxTestModule()); - system = injector.getInstance(ImapHostSystem.class); - super.setUp(); - } - - @Override - protected ImapHostSystem createImapHostSystem() { - return system; - } - - @After - public void tearDown() throws Exception { - system.afterTest(); - } - -} http://git-wip-us.apache.org/repos/asf/james-project/blob/ae336763/mpt/impl/imap-mailbox/core/src/main/java/org/apache/james/mpt/imapmailbox/suite/MailboxWithLongNameSuccess.java ---------------------------------------------------------------------- diff --git a/mpt/impl/imap-mailbox/core/src/main/java/org/apache/james/mpt/imapmailbox/suite/MailboxWithLongNameSuccess.java b/mpt/impl/imap-mailbox/core/src/main/java/org/apache/james/mpt/imapmailbox/suite/MailboxWithLongNameSuccess.java index 92c4e53..8366ea1 100644 --- a/mpt/impl/imap-mailbox/core/src/main/java/org/apache/james/mpt/imapmailbox/suite/MailboxWithLongNameSuccess.java +++ b/mpt/impl/imap-mailbox/core/src/main/java/org/apache/james/mpt/imapmailbox/suite/MailboxWithLongNameSuccess.java @@ -26,10 +26,8 @@ import org.apache.james.mpt.imapmailbox.ImapTestConstants; import org.apache.james.mpt.imapmailbox.suite.base.BasicImapCommands; import org.apache.james.mpt.script.SimpleScriptedTestProtocol; import org.junit.Before; -import org.junit.Ignore; import org.junit.Test; -@Ignore("why ?") public abstract class MailboxWithLongNameSuccess implements ImapTestConstants { protected abstract ImapHostSystem createImapHostSystem(); @@ -45,7 +43,6 @@ public abstract class MailboxWithLongNameSuccess implements ImapTestConstants { .withLocale(Locale.US); BasicImapCommands.welcome(simpleScriptedTestProtocol); BasicImapCommands.authenticate(simpleScriptedTestProtocol); - BasicImapCommands.prepareMailbox(simpleScriptedTestProtocol); } @Test http://git-wip-us.apache.org/repos/asf/james-project/blob/ae336763/mpt/impl/imap-mailbox/hbase/src/test/java/org/apache/james/mpt/imapmailbox/hbase/HBaseMailboxWithLongNameSuccess.java ---------------------------------------------------------------------- diff --git a/mpt/impl/imap-mailbox/hbase/src/test/java/org/apache/james/mpt/imapmailbox/hbase/HBaseMailboxWithLongNameSuccess.java b/mpt/impl/imap-mailbox/hbase/src/test/java/org/apache/james/mpt/imapmailbox/hbase/HBaseMailboxWithLongNameSuccess.java deleted file mode 100644 index 6c76d12..0000000 --- a/mpt/impl/imap-mailbox/hbase/src/test/java/org/apache/james/mpt/imapmailbox/hbase/HBaseMailboxWithLongNameSuccess.java +++ /dev/null @@ -1,34 +0,0 @@ -package org.apache.james.mpt.imapmailbox.hbase; - -import org.apache.james.mpt.api.ImapHostSystem; -import org.apache.james.mpt.imapmailbox.suite.MailboxWithLongNameSuccess; -import org.junit.After; -import org.junit.Before; -import org.junit.Ignore; - -import com.google.inject.Guice; -import com.google.inject.Injector; - -@Ignore("do we have a JIRA for that ?") -public class HBaseMailboxWithLongNameSuccess extends MailboxWithLongNameSuccess { - - private ImapHostSystem system; - - @Before - public void setUp() throws Exception { - Injector injector = Guice.createInjector(new HBaseMailboxTestModule()); - system = injector.getInstance(ImapHostSystem.class); - super.setUp(); - } - - @Override - protected ImapHostSystem createImapHostSystem() { - return system; - } - - @After - public void tearDown() throws Exception { - system.afterTest(); - } - -} http://git-wip-us.apache.org/repos/asf/james-project/blob/ae336763/mpt/impl/imap-mailbox/inmemory/src/test/java/org/apache/james/mpt/imapmailbox/inmemory/InMemoryMailboxWithLongNameError.java ---------------------------------------------------------------------- diff --git a/mpt/impl/imap-mailbox/inmemory/src/test/java/org/apache/james/mpt/imapmailbox/inmemory/InMemoryMailboxWithLongNameError.java b/mpt/impl/imap-mailbox/inmemory/src/test/java/org/apache/james/mpt/imapmailbox/inmemory/InMemoryMailboxWithLongNameError.java deleted file mode 100644 index 7971a6e..0000000 --- a/mpt/impl/imap-mailbox/inmemory/src/test/java/org/apache/james/mpt/imapmailbox/inmemory/InMemoryMailboxWithLongNameError.java +++ /dev/null @@ -1,34 +0,0 @@ -package org.apache.james.mpt.imapmailbox.inmemory; - -import org.apache.james.mpt.api.ImapHostSystem; -import org.apache.james.mpt.imapmailbox.suite.MailboxWithLongNameError; -import org.junit.After; -import org.junit.Before; -import org.junit.Ignore; - -import com.google.inject.Guice; -import com.google.inject.Injector; - -@Ignore("why ?") -public class InMemoryMailboxWithLongNameError extends MailboxWithLongNameError { - - private ImapHostSystem system; - - @Before - public void setUp() throws Exception { - Injector injector = Guice.createInjector(new InMemoryMailboxTestModule()); - system = injector.getInstance(ImapHostSystem.class); - super.setUp(); - } - - @Override - protected ImapHostSystem createImapHostSystem() { - return system; - } - - @After - public void tearDown() throws Exception { - system.afterTest(); - } - -} http://git-wip-us.apache.org/repos/asf/james-project/blob/ae336763/mpt/impl/imap-mailbox/jpa/src/test/java/org/apache/james/mpt/imapmailbox/jpa/JpaMailboxWithLongNameError.java ---------------------------------------------------------------------- diff --git a/mpt/impl/imap-mailbox/jpa/src/test/java/org/apache/james/mpt/imapmailbox/jpa/JpaMailboxWithLongNameError.java b/mpt/impl/imap-mailbox/jpa/src/test/java/org/apache/james/mpt/imapmailbox/jpa/JpaMailboxWithLongNameError.java index bfea2cb..4c6bea9 100644 --- a/mpt/impl/imap-mailbox/jpa/src/test/java/org/apache/james/mpt/imapmailbox/jpa/JpaMailboxWithLongNameError.java +++ b/mpt/impl/imap-mailbox/jpa/src/test/java/org/apache/james/mpt/imapmailbox/jpa/JpaMailboxWithLongNameError.java @@ -9,7 +9,7 @@ import org.junit.Ignore; import com.google.inject.Guice; import com.google.inject.Injector; -@Ignore("why ?") +@Ignore("MAILBOX-300 JPA should fail gracefully when too long mailbox name") public class JpaMailboxWithLongNameError extends MailboxWithLongNameError { private ImapHostSystem system; http://git-wip-us.apache.org/repos/asf/james-project/blob/ae336763/mpt/impl/imap-mailbox/jpa/src/test/java/org/apache/james/mpt/imapmailbox/jpa/JpaMailboxWithLongNameSuccess.java ---------------------------------------------------------------------- diff --git a/mpt/impl/imap-mailbox/jpa/src/test/java/org/apache/james/mpt/imapmailbox/jpa/JpaMailboxWithLongNameSuccess.java b/mpt/impl/imap-mailbox/jpa/src/test/java/org/apache/james/mpt/imapmailbox/jpa/JpaMailboxWithLongNameSuccess.java deleted file mode 100644 index 7ad407f..0000000 --- a/mpt/impl/imap-mailbox/jpa/src/test/java/org/apache/james/mpt/imapmailbox/jpa/JpaMailboxWithLongNameSuccess.java +++ /dev/null @@ -1,34 +0,0 @@ -package org.apache.james.mpt.imapmailbox.jpa; - -import org.apache.james.mpt.api.ImapHostSystem; -import org.apache.james.mpt.imapmailbox.suite.MailboxWithLongNameSuccess; -import org.junit.After; -import org.junit.Before; -import org.junit.Ignore; - -import com.google.inject.Guice; -import com.google.inject.Injector; - -@Ignore("why") -public class JpaMailboxWithLongNameSuccess extends MailboxWithLongNameSuccess { - - private ImapHostSystem system; - - @Before - public void setUp() throws Exception { - Injector injector = Guice.createInjector(new JpaMailboxTestModule()); - system = injector.getInstance(ImapHostSystem.class); - super.setUp(); - } - - @Override - protected ImapHostSystem createImapHostSystem() { - return system; - } - - @After - public void tearDown() throws Exception { - system.afterTest(); - } - -} http://git-wip-us.apache.org/repos/asf/james-project/blob/ae336763/mpt/impl/imap-mailbox/lucenesearch/src/test/java/org/apache/james/mpt/imapmailbox/lucenesearch/LuceneMailboxWithLongNameError.java ---------------------------------------------------------------------- diff --git a/mpt/impl/imap-mailbox/lucenesearch/src/test/java/org/apache/james/mpt/imapmailbox/lucenesearch/LuceneMailboxWithLongNameError.java b/mpt/impl/imap-mailbox/lucenesearch/src/test/java/org/apache/james/mpt/imapmailbox/lucenesearch/LuceneMailboxWithLongNameError.java index 3e92189..0530758 100644 --- a/mpt/impl/imap-mailbox/lucenesearch/src/test/java/org/apache/james/mpt/imapmailbox/lucenesearch/LuceneMailboxWithLongNameError.java +++ b/mpt/impl/imap-mailbox/lucenesearch/src/test/java/org/apache/james/mpt/imapmailbox/lucenesearch/LuceneMailboxWithLongNameError.java @@ -9,7 +9,7 @@ import org.junit.Ignore; import com.google.inject.Guice; import com.google.inject.Injector; -@Ignore("why ?") +@Ignore("MAILBOX-300 JPA should fail gracefully when too long mailbox name") public class LuceneMailboxWithLongNameError extends MailboxWithLongNameError { private ImapHostSystem system; http://git-wip-us.apache.org/repos/asf/james-project/blob/ae336763/mpt/impl/imap-mailbox/lucenesearch/src/test/java/org/apache/james/mpt/imapmailbox/lucenesearch/LuceneMailboxWithLongNameSuccess.java ---------------------------------------------------------------------- diff --git a/mpt/impl/imap-mailbox/lucenesearch/src/test/java/org/apache/james/mpt/imapmailbox/lucenesearch/LuceneMailboxWithLongNameSuccess.java b/mpt/impl/imap-mailbox/lucenesearch/src/test/java/org/apache/james/mpt/imapmailbox/lucenesearch/LuceneMailboxWithLongNameSuccess.java deleted file mode 100644 index 355b0a6..0000000 --- a/mpt/impl/imap-mailbox/lucenesearch/src/test/java/org/apache/james/mpt/imapmailbox/lucenesearch/LuceneMailboxWithLongNameSuccess.java +++ /dev/null @@ -1,35 +0,0 @@ -package org.apache.james.mpt.imapmailbox.lucenesearch; - -import org.apache.james.mpt.api.ImapHostSystem; -import org.apache.james.mpt.imapmailbox.suite.MailboxWithLongNameSuccess; -import org.junit.After; -import org.junit.Before; -import org.junit.Ignore; - -import com.google.inject.Guice; -import com.google.inject.Injector; - -@Ignore("why") -public class LuceneMailboxWithLongNameSuccess extends MailboxWithLongNameSuccess { - - private ImapHostSystem system; - - @Before - public void setUp() throws Exception { - Injector injector = Guice.createInjector(new LuceneSearchMailboxTestModule()); - system = injector.getInstance(ImapHostSystem.class); - system.beforeTest(); - super.setUp(); - } - - @Override - protected ImapHostSystem createImapHostSystem() { - return system; - } - - @After - public void tearDown() throws Exception { - system.afterTest(); - } - -} http://git-wip-us.apache.org/repos/asf/james-project/blob/ae336763/mpt/impl/imap-mailbox/maildir/src/test/java/org/apache/james/mpt/imapmailbox/maildir/MaildirMailboxWithLongNameError.java ---------------------------------------------------------------------- diff --git a/mpt/impl/imap-mailbox/maildir/src/test/java/org/apache/james/mpt/imapmailbox/maildir/MaildirMailboxWithLongNameError.java b/mpt/impl/imap-mailbox/maildir/src/test/java/org/apache/james/mpt/imapmailbox/maildir/MaildirMailboxWithLongNameError.java index e67b8cc..854fdfc 100644 --- a/mpt/impl/imap-mailbox/maildir/src/test/java/org/apache/james/mpt/imapmailbox/maildir/MaildirMailboxWithLongNameError.java +++ b/mpt/impl/imap-mailbox/maildir/src/test/java/org/apache/james/mpt/imapmailbox/maildir/MaildirMailboxWithLongNameError.java @@ -9,7 +9,7 @@ import org.junit.Ignore; import com.google.inject.Guice; import com.google.inject.Injector; -@Ignore("why ?") +@Ignore("MAILBOX-299 Maildir should fail gracefully when too long mailbox name") public class MaildirMailboxWithLongNameError extends MailboxWithLongNameError { private ImapHostSystem system; http://git-wip-us.apache.org/repos/asf/james-project/blob/ae336763/mpt/impl/imap-mailbox/maildir/src/test/java/org/apache/james/mpt/imapmailbox/maildir/MaildirMailboxWithLongNameSuccess.java ---------------------------------------------------------------------- diff --git a/mpt/impl/imap-mailbox/maildir/src/test/java/org/apache/james/mpt/imapmailbox/maildir/MaildirMailboxWithLongNameSuccess.java b/mpt/impl/imap-mailbox/maildir/src/test/java/org/apache/james/mpt/imapmailbox/maildir/MaildirMailboxWithLongNameSuccess.java deleted file mode 100644 index 35b1176..0000000 --- a/mpt/impl/imap-mailbox/maildir/src/test/java/org/apache/james/mpt/imapmailbox/maildir/MaildirMailboxWithLongNameSuccess.java +++ /dev/null @@ -1,34 +0,0 @@ -package org.apache.james.mpt.imapmailbox.maildir; - -import org.apache.james.mpt.api.ImapHostSystem; -import org.apache.james.mpt.imapmailbox.suite.MailboxWithLongNameSuccess; -import org.junit.After; -import org.junit.Before; -import org.junit.Ignore; - -import com.google.inject.Guice; -import com.google.inject.Injector; - -@Ignore("why") -public class MaildirMailboxWithLongNameSuccess extends MailboxWithLongNameSuccess { - - private ImapHostSystem system; - - @Before - public void setUp() throws Exception { - Injector injector = Guice.createInjector(new MaildirMailboxTestModule()); - system = injector.getInstance(ImapHostSystem.class); - super.setUp(); - } - - @Override - protected ImapHostSystem createImapHostSystem() { - return system; - } - - @After - public void tearDown() throws Exception { - system.afterTest(); - } - -} --------------------------------------------------------------------- To unsubscribe, e-mail: server-dev-unsubscr...@james.apache.org For additional commands, e-mail: server-dev-h...@james.apache.org
