8:43 p.m.
Author: manik.surtani(a)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@yahoo.com">Cavin Song</a>
April 22, 2004
* @version 1.0
*/
-@Test(groups = {"functional"}, invocationCount = 30, enabled = false)
-// todo: look into intermittent failures in this test; probably due to LockMap changes.
+@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@jboss.org">manik@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);