Author: eric Date: Sun Dec 26 10:30:21 2010 New Revision: 1052856 URL: http://svn.apache.org/viewvc?rev=1052856&view=rev Log: Use instance instead of static for before/after tests - fix the assert expected value - make setMailboxManager abstract.
Modified: james/mailbox/trunk/api/src/test/java/org/apache/james/mailbox/MailboxManagerTest.java james/mailbox/trunk/jcr/src/test/java/org/apache/james/mailbox/jcr/JCRMailboxManagerTest.java james/mailbox/trunk/jpa/src/test/java/org/apache/james/mailbox/jpa/JPAMailboxManagerTest.java james/mailbox/trunk/maildir/src/test/java/org/apache/james/mailbox/functional/maildir/MaildirMailboxManagerTest.java Modified: james/mailbox/trunk/api/src/test/java/org/apache/james/mailbox/MailboxManagerTest.java URL: http://svn.apache.org/viewvc/james/mailbox/trunk/api/src/test/java/org/apache/james/mailbox/MailboxManagerTest.java?rev=1052856&r1=1052855&r2=1052856&view=diff ============================================================================== --- james/mailbox/trunk/api/src/test/java/org/apache/james/mailbox/MailboxManagerTest.java (original) +++ james/mailbox/trunk/api/src/test/java/org/apache/james/mailbox/MailboxManagerTest.java Sun Dec 26 10:30:21 2010 @@ -50,27 +50,32 @@ public abstract class MailboxManagerTest * The mailboxManager that needs to get instanciated * by the mailbox implementations. */ - private static MailboxManager mailboxManager; + protected MailboxManager mailboxManager; /** * Number of Domains to be created in the Mailbox Manager. */ - private static final int DOMAIN_COUNT = 5; + private static final int DOMAIN_COUNT = 3; /** * Number of Users (with INBOX) to be created in the Mailbox Manager. */ - private static final int USER_COUNT = 5; + private static final int USER_COUNT = 3; /** - * Number of Sub Mailboxes (mailbox in another mailbox) to be created in the Mailbox Manager. + * Number of Sub Mailboxes (mailbox in INBOX) to be created in the Mailbox Manager. */ - private static final int SUB_MAILBOXES_COUNT = 5; + private static final int SUB_MAILBOXES_COUNT = 3; + + /** + * Number of Sub Sub Mailboxes (mailbox in a mailbox under INBOX) to be created in the Mailbox Manager. + */ + private static final int SUB_SUB_MAILBOXES_COUNT = 3; /** * Number of Messages per Mailbox to be created in the Mailbox Manager. */ - private static final int MESSAGE_PER_MAILBOX_COUNT = 5; + private static final int MESSAGE_PER_MAILBOX_COUNT = 3; /** * Create some INBOXes and their sub mailboxes and assert list() method. @@ -87,8 +92,8 @@ public abstract class MailboxManagerTest getMailboxManager().startProcessingRequest(mailboxSession); Assert.assertEquals(DOMAIN_COUNT * (USER_COUNT + // INBOX - USER_COUNT * MESSAGE_PER_MAILBOX_COUNT + // INBOX.SUB_FOLDER - USER_COUNT * MESSAGE_PER_MAILBOX_COUNT * MESSAGE_PER_MAILBOX_COUNT), // INBOX.SUB_FOLDER.SUBSUB_FOLDER + USER_COUNT * SUB_MAILBOXES_COUNT + // INBOX.SUB_FOLDER + USER_COUNT * SUB_MAILBOXES_COUNT * SUB_SUB_MAILBOXES_COUNT), // INBOX.SUB_FOLDER.SUBSUB_FOLDER getMailboxManager().list(mailboxSession).size()); } @@ -118,14 +123,14 @@ public abstract class MailboxManagerTest for (int k=0; k < SUB_MAILBOXES_COUNT; k++) { - folderName = folderName + ".SUB_FOLDER_" + k; - mailboxPath = new MailboxPath("#private", user, folderName); + String subFolderName = folderName + ".SUB_FOLDER_" + k; + mailboxPath = new MailboxPath("#private", user, subFolderName); createMailbox(mailboxSession, mailboxPath); - for (int l=0; l < SUB_MAILBOXES_COUNT; l++) { + for (int l=0; l < SUB_SUB_MAILBOXES_COUNT; l++) { - folderName = folderName + ".SUBSUB_FOLDER_" + l; - mailboxPath = new MailboxPath("#private", user, folderName); + String subSubfolderName = subFolderName + ".SUBSUB_FOLDER_" + l; + mailboxPath = new MailboxPath("#private", user, subSubfolderName); createMailbox(mailboxSession, mailboxPath); } @@ -163,9 +168,7 @@ public abstract class MailboxManagerTest /** * Setter to inject the mailboxManager. */ - protected static void setMailboxManager(MailboxManager mailboxManager) { - MailboxManagerTest.mailboxManager = mailboxManager; - } + protected abstract void setMailboxManager(MailboxManager mailboxManager); /** * Accessor to the mailboxManager. @@ -173,7 +176,7 @@ public abstract class MailboxManagerTest * @return the mailboxManager instance. * @throws IllegalStateException in case of null mailboxManager */ - protected static MailboxManager getMailboxManager() { + protected MailboxManager getMailboxManager() { if (mailboxManager == null) { throw new IllegalStateException("Please setMailboxManager with a non null value before requesting getMailboxManager()"); } Modified: james/mailbox/trunk/jcr/src/test/java/org/apache/james/mailbox/jcr/JCRMailboxManagerTest.java URL: http://svn.apache.org/viewvc/james/mailbox/trunk/jcr/src/test/java/org/apache/james/mailbox/jcr/JCRMailboxManagerTest.java?rev=1052856&r1=1052855&r2=1052856&view=diff ============================================================================== --- james/mailbox/trunk/jcr/src/test/java/org/apache/james/mailbox/jcr/JCRMailboxManagerTest.java (original) +++ james/mailbox/trunk/jcr/src/test/java/org/apache/james/mailbox/jcr/JCRMailboxManagerTest.java Sun Dec 26 10:30:21 2010 @@ -19,19 +19,18 @@ package org.apache.james.mailbox.jcr; import java.io.File; -import java.io.UnsupportedEncodingException; import org.apache.commons.logging.impl.SimpleLog; import org.apache.jackrabbit.core.RepositoryImpl; import org.apache.jackrabbit.core.config.RepositoryConfig; import org.apache.james.mailbox.BadCredentialsException; import org.apache.james.mailbox.MailboxException; +import org.apache.james.mailbox.MailboxManager; import org.apache.james.mailbox.MailboxManagerTest; import org.apache.james.mailbox.MailboxSession; import org.apache.james.mailbox.jcr.mail.JCRCachingUidProvider; -import org.junit.AfterClass; -import org.junit.BeforeClass; -import org.junit.Test; +import org.junit.After; +import org.junit.Before; import org.xml.sax.InputSource; /** @@ -50,8 +49,8 @@ public class JCRMailboxManagerTest exten * * @throws Exception */ - @BeforeClass - public static void setup() throws Exception { + @Before + public void setup() throws Exception { new File(JACKRABBIT_HOME).delete(); @@ -74,15 +73,21 @@ public class JCRMailboxManagerTest exten /** - * @throws BadCredentialsException - * @throws MailboxException + * Close system session and shutdown system repository. */ - @AfterClass - public static void tearDown() throws BadCredentialsException, MailboxException { + @After + public void tearDown() throws BadCredentialsException, MailboxException { MailboxSession session = getMailboxManager().createSystemSession("test", new SimpleLog("Test")); session.close(); repository.shutdown(); new File(JACKRABBIT_HOME).delete(); } + + /* (non-Javadoc) + * @see org.apache.james.mailbox.MailboxManagerTest#setMailboxManager(org.apache.james.mailbox.MailboxManager) + */ + protected void setMailboxManager(MailboxManager mailboxManager) { + this.mailboxManager = mailboxManager; + } } Modified: james/mailbox/trunk/jpa/src/test/java/org/apache/james/mailbox/jpa/JPAMailboxManagerTest.java URL: http://svn.apache.org/viewvc/james/mailbox/trunk/jpa/src/test/java/org/apache/james/mailbox/jpa/JPAMailboxManagerTest.java?rev=1052856&r1=1052855&r2=1052856&view=diff ============================================================================== --- james/mailbox/trunk/jpa/src/test/java/org/apache/james/mailbox/jpa/JPAMailboxManagerTest.java (original) +++ james/mailbox/trunk/jpa/src/test/java/org/apache/james/mailbox/jpa/JPAMailboxManagerTest.java Sun Dec 26 10:30:21 2010 @@ -25,6 +25,7 @@ import javax.persistence.EntityManagerFa import org.apache.commons.logging.impl.SimpleLog; import org.apache.james.mailbox.BadCredentialsException; import org.apache.james.mailbox.MailboxException; +import org.apache.james.mailbox.MailboxManager; import org.apache.james.mailbox.MailboxManagerTest; import org.apache.james.mailbox.MailboxSession; import org.apache.james.mailbox.jpa.mail.JPACachingUidProvider; @@ -38,8 +39,8 @@ import org.apache.james.mailbox.jpa.mail import org.apache.james.mailbox.jpa.openjpa.OpenJPAMailboxManager; import org.apache.james.mailbox.jpa.user.model.JPASubscription; import org.apache.openjpa.persistence.OpenJPAPersistence; -import org.junit.AfterClass; -import org.junit.BeforeClass; +import org.junit.After; +import org.junit.Before; /** * JPAMailboxManagerTest that extends the StoreMailboxManagerTest. @@ -56,8 +57,8 @@ public class JPAMailboxManagerTest exten * * @throws Exception */ - @BeforeClass - public static void setup() throws Exception { + @Before + public void setup() throws Exception { HashMap<String, String> properties = new HashMap<String, String>(); properties.put("openjpa.ConnectionDriverName", "org.h2.Driver"); @@ -89,11 +90,18 @@ public class JPAMailboxManagerTest exten * @throws MailboxException * @throws BadCredentialsException */ - @AfterClass - public static void tearDown() throws BadCredentialsException, MailboxException { + @After + public void tearDown() throws BadCredentialsException, MailboxException { MailboxSession session = getMailboxManager().createSystemSession("test", new SimpleLog("Test")); session.close(); entityManagerFactory.close(); } + /* (non-Javadoc) + * @see org.apache.james.mailbox.MailboxManagerTest#setMailboxManager(org.apache.james.mailbox.MailboxManager) + */ + protected void setMailboxManager(MailboxManager mailboxManager) { + this.mailboxManager = mailboxManager; + } + } Modified: james/mailbox/trunk/maildir/src/test/java/org/apache/james/mailbox/functional/maildir/MaildirMailboxManagerTest.java URL: http://svn.apache.org/viewvc/james/mailbox/trunk/maildir/src/test/java/org/apache/james/mailbox/functional/maildir/MaildirMailboxManagerTest.java?rev=1052856&r1=1052855&r2=1052856&view=diff ============================================================================== --- james/mailbox/trunk/maildir/src/test/java/org/apache/james/mailbox/functional/maildir/MaildirMailboxManagerTest.java (original) +++ james/mailbox/trunk/maildir/src/test/java/org/apache/james/mailbox/functional/maildir/MaildirMailboxManagerTest.java Sun Dec 26 10:30:21 2010 @@ -19,17 +19,19 @@ package org.apache.james.mailbox.functional.maildir; import java.io.File; +import java.io.IOException; import java.io.UnsupportedEncodingException; import org.apache.commons.io.FileUtils; import org.apache.james.mailbox.BadCredentialsException; import org.apache.james.mailbox.MailboxException; +import org.apache.james.mailbox.MailboxManager; import org.apache.james.mailbox.MailboxManagerTest; import org.apache.james.mailbox.maildir.MaildirMailboxManager; import org.apache.james.mailbox.maildir.MaildirMailboxSessionMapperFactory; import org.apache.james.mailbox.maildir.MaildirStore; -import org.junit.AfterClass; -import org.junit.BeforeClass; +import org.junit.After; +import org.junit.Before; import org.junit.Test; /** @@ -44,8 +46,8 @@ public class MaildirMailboxManagerTest e * * @throws Exception */ - @BeforeClass - public static void setup() throws Exception { + @Before + public void setup() throws Exception { FileUtils.deleteDirectory(new File(MAILDIR_HOME)); MaildirStore store = new MaildirStore(MAILDIR_HOME + "/%domain/%user"); MaildirMailboxSessionMapperFactory mf = new MaildirMailboxSessionMapperFactory(store); @@ -53,14 +55,13 @@ public class MaildirMailboxManagerTest e } /** - * Close the system session and entityManagerFactory + * Delete Maildir directory after test. * - * @throws MailboxException - * @throws BadCredentialsException + * @throws IOException */ - @AfterClass - public static void tearDown() throws BadCredentialsException, MailboxException { -// FileUtils.deleteDirectory(new File(MAILDIR_HOME)); + @After + public void tearDown() throws IOException { + FileUtils.deleteDirectory(new File(MAILDIR_HOME)); } /* (non-Javadoc) @@ -75,4 +76,11 @@ public class MaildirMailboxManagerTest e } } + /* (non-Javadoc) + * @see org.apache.james.mailbox.MailboxManagerTest#setMailboxManager(org.apache.james.mailbox.MailboxManager) + */ + protected void setMailboxManager(MailboxManager mailboxManager) { + this.mailboxManager = mailboxManager; + } + } --------------------------------------------------------------------- To unsubscribe, e-mail: server-dev-unsubscr...@james.apache.org For additional commands, e-mail: server-dev-h...@james.apache.org