[jbosscache-commits] JBoss Cache SVN: r5383 - in core/trunk/src/test/java/org/jboss/cache: api and 4 other directories.

[jbosscache-commits at lists.jboss.org]

Author: manik.surtani at jboss.com
Date: 2008-03-04 06:43:07 -0500 (Tue, 04 Mar 2008)
New Revision: 5383

Modified:
   core/trunk/src/test/java/org/jboss/cache/LifeCycleTest.java
   core/trunk/src/test/java/org/jboss/cache/api/NodeMoveAPITest.java
   core/trunk/src/test/java/org/jboss/cache/lock/NonBlockingWriterLockTest.java
   core/trunk/src/test/java/org/jboss/cache/notifications/RemoteCacheListenerTest.java
   core/trunk/src/test/java/org/jboss/cache/profiling/AbstractProfileTest.java
   core/trunk/src/test/java/org/jboss/cache/statetransfer/StateTransferConcurrencyTest.java
Log:
Reinstated disabled tests

Modified: core/trunk/src/test/java/org/jboss/cache/LifeCycleTest.java
===================================================================
--- core/trunk/src/test/java/org/jboss/cache/LifeCycleTest.java	2008-03-04 10:32:35 UTC (rev 5382)
+++ core/trunk/src/test/java/org/jboss/cache/LifeCycleTest.java	2008-03-04 11:43:07 UTC (rev 5383)
@@ -282,8 +282,6 @@
       }
    }
 
-   @Test(enabled = false)
-   // TODO: needs investigation ... !
    public void testRemoteInvalidStateInvocations2() throws Exception
    {
       createAndRegisterCache(Configuration.CacheMode.REPL_SYNC, true);

Modified: core/trunk/src/test/java/org/jboss/cache/api/NodeMoveAPITest.java
===================================================================
--- core/trunk/src/test/java/org/jboss/cache/api/NodeMoveAPITest.java	2008-03-04 10:32:35 UTC (rev 5382)
+++ core/trunk/src/test/java/org/jboss/cache/api/NodeMoveAPITest.java	2008-03-04 11:43:07 UTC (rev 5383)
@@ -461,9 +461,6 @@
    @Test(groups = {"functional"})
    public void testConcurrency() throws InterruptedException
    {
-      // TODO: investigate intermittent failure when in optimistic mode.
-      if (optimistic) return;
-
       final int N = 3;// number of threads
       final int loops = 1 << 6;// number of loops
       // tests a tree structure as such:

Modified: core/trunk/src/test/java/org/jboss/cache/lock/NonBlockingWriterLockTest.java
===================================================================
--- core/trunk/src/test/java/org/jboss/cache/lock/NonBlockingWriterLockTest.java	2008-03-04 10:32:35 UTC (rev 5382)
+++ core/trunk/src/test/java/org/jboss/cache/lock/NonBlockingWriterLockTest.java	2008-03-04 11:43:07 UTC (rev 5383)
@@ -26,8 +26,7 @@
  * @author <a href="mailto:cavin_song at yahoo.com">Cavin Song</a> April 22, 2004
  * @version 1.0
  */
- at Test(groups = {"functional"}, invocationCount = 30, enabled = false)
-// todo: look into intermittent failures in this test; probably due to LockMap changes.
+ at Test(groups = {"functional"}, invocationCount = 30)
 public class NonBlockingWriterLockTest
 {
    static final NonBlockingWriterLock lock_ = new NonBlockingWriterLock();

Modified: core/trunk/src/test/java/org/jboss/cache/notifications/RemoteCacheListenerTest.java
===================================================================
--- core/trunk/src/test/java/org/jboss/cache/notifications/RemoteCacheListenerTest.java	2008-03-04 10:32:35 UTC (rev 5382)
+++ core/trunk/src/test/java/org/jboss/cache/notifications/RemoteCacheListenerTest.java	2008-03-04 11:43:07 UTC (rev 5383)
@@ -630,8 +630,7 @@
       assertEquals(expected, eventLog2.events);
    }
 
-   @Test(enabled = false)
-   // TODO: Enable and fix after CR4.  This needs to be addressed!!  Why is the state transfer process doing a get() on an internal node?1??
+   @Test(enabled = true)
    public void testStateTransfer() throws Exception
    {
       // first stop cache2

Modified: core/trunk/src/test/java/org/jboss/cache/profiling/AbstractProfileTest.java
===================================================================
--- core/trunk/src/test/java/org/jboss/cache/profiling/AbstractProfileTest.java	2008-03-04 10:32:35 UTC (rev 5382)
+++ core/trunk/src/test/java/org/jboss/cache/profiling/AbstractProfileTest.java	2008-03-04 11:43:07 UTC (rev 5383)
@@ -8,12 +8,6 @@
 import org.testng.annotations.BeforeTest;
 import org.testng.annotations.Test;
 
-/**
- * // TODO Document this
- *
- * @author Manik Surtani (<a href="mailto:manik at jboss.org">manik at jboss.org</a>)
- * @since 2.1.0
- */
 @Test(groups = "profiling")
 public abstract class AbstractProfileTest
 {

Modified: core/trunk/src/test/java/org/jboss/cache/statetransfer/StateTransferConcurrencyTest.java
===================================================================
--- core/trunk/src/test/java/org/jboss/cache/statetransfer/StateTransferConcurrencyTest.java	2008-03-04 10:32:35 UTC (rev 5382)
+++ core/trunk/src/test/java/org/jboss/cache/statetransfer/StateTransferConcurrencyTest.java	2008-03-04 11:43:07 UTC (rev 5383)
@@ -31,7 +31,6 @@
  * Abstract superclass of "StateTransferVersion"-specific tests
  * of CacheSPI's state transfer capability.
  * <p/>
- * TODO add tests with classloader regions
  *
  * @author <a href="mailto://brian.stansberry@jboss.com">Brian Stansberry</a>
  * @version $Id$
@@ -52,8 +51,7 @@
     *
     * @throws Exception
     */
-   @Test(invocationCount = 25, enabled = false)
-   // TODO: Fix this before 2.1.0.GA !! - Manik
+   @Test(invocationCount = 25, enabled = true)
    public void testConcurrentActivationSync() throws Exception
    {
       concurrentActivationTest(true);
@@ -66,8 +64,7 @@
     *
     * @throws Exception
     */
-   @Test(invocationCount = 25, enabled = false)
-   // TODO: Fix this before 2.1.0.GA !! - Manik
+   @Test(invocationCount = 25, enabled = true)
    public void testConcurrentActivationAsync() throws Exception
    {
       concurrentActivationTest(false);
@@ -286,8 +283,7 @@
     * Failure condition is if any node sees an exception or if the final state
     * of all caches is not consistent.
     */
-   @Test(invocationCount = 25, enabled = false)
-   // TODO: Fix this before 2.1.0.GA !! - Manik
+   @Test(invocationCount = 25, enabled = true)
    public void testConcurrentStartupActivationAsync() throws Exception
    {
       concurrentActivationTest2(false);
@@ -308,8 +304,7 @@
     * Failure condition is if any node sees an exception or if the final state
     * of all caches is not consistent.
     */
-   @Test(invocationCount = 25, enabled = false)
-   // TODO: Fix this before 2.1.0.GA !! - Manik
+   @Test(invocationCount = 25, enabled = true)
    public void testConcurrentStartupActivationSync() throws Exception
    {
       concurrentActivationTest2(true);