From jbosscache-commits at lists.jboss.org Tue Mar  4 06:43:07 2008
Content-Type: multipart/mixed; boundary="===============6446864193523599689=="
MIME-Version: 1.0
From: jbosscache-commits at lists.jboss.org
To: jbosscache-commits at lists.jboss.org
Subject: [jbosscache-commits] JBoss Cache SVN: r5383 - in
 core/trunk/src/test/java/org/jboss/cache: api and 4 other directories.
Date: Tue, 04 Mar 2008 06:43:07 -0500
Message-ID: <E1JWVY3-0001Ly-Ax@committer01.frg.pub.inap.atl.jboss.com>

--===============6446864193523599689==
Content-Type: text/plain; charset="utf-8"
MIME-Version: 1.0
Content-Transfer-Encoding: quoted-printable

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/RemoteCacheListen=
erTest.java
   core/trunk/src/test/java/org/jboss/cache/profiling/AbstractProfileTest.j=
ava
   core/trunk/src/test/java/org/jboss/cache/statetransfer/StateTransferConc=
urrencyTest.java
Log:
Reinstated disabled tests

Modified: core/trunk/src/test/java/org/jboss/cache/LifeCycleTest.java
=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=
=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=
=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D
--- 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 =3D 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
=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=
=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=
=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D
--- 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 =3D {"functional"})
    public void testConcurrency() throws InterruptedException
    {
-      // TODO: investigate intermittent failure when in optimistic mode.
-      if (optimistic) return;
-
       final int N =3D 3;// number of threads
       final int loops =3D 1 << 6;// number of loops
       // tests a tree structure as such:

Modified: core/trunk/src/test/java/org/jboss/cache/lock/NonBlockingWriterLo=
ckTest.java
=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=
=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=
=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D
--- 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=3D"mailto:cavin_song(a)yahoo.com">Cavin Song</a> April =
22, 2004
  * @version 1.0
  */
-(a)Test(groups =3D {"functional"}, invocationCount =3D 30, enabled =3D fal=
se)
-// todo: look into intermittent failures in this test; probably due to Loc=
kMap changes.
+(a)Test(groups =3D {"functional"}, invocationCount =3D 30)
 public class NonBlockingWriterLockTest
 {
    static final NonBlockingWriterLock lock_ =3D new NonBlockingWriterLock(=
);

Modified: core/trunk/src/test/java/org/jboss/cache/notifications/RemoteCach=
eListenerTest.java
=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=
=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=
=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D
--- core/trunk/src/test/java/org/jboss/cache/notifications/RemoteCacheListe=
nerTest.java	2008-03-04 10:32:35 UTC (rev 5382)
+++ core/trunk/src/test/java/org/jboss/cache/notifications/RemoteCacheListe=
nerTest.java	2008-03-04 11:43:07 UTC (rev 5383)
@@ -630,8 +630,7 @@
       assertEquals(expected, eventLog2.events);
    }
 =

-   @Test(enabled =3D false)
-   // TODO: Enable and fix after CR4.  This needs to be addressed!!  Why i=
s the state transfer process doing a get() on an internal node?1??
+   @Test(enabled =3D true)
    public void testStateTransfer() throws Exception
    {
       // first stop cache2

Modified: core/trunk/src/test/java/org/jboss/cache/profiling/AbstractProfil=
eTest.java
=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=
=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=
=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D
--- 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=3D"mailto:manik(a)jboss.org">manik(a)jbo=
ss.org</a>)
- * @since 2.1.0
- */
 @Test(groups =3D "profiling")
 public abstract class AbstractProfileTest
 {

Modified: core/trunk/src/test/java/org/jboss/cache/statetransfer/StateTrans=
ferConcurrencyTest.java
=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=
=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=
=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D
--- core/trunk/src/test/java/org/jboss/cache/statetransfer/StateTransferCon=
currencyTest.java	2008-03-04 10:32:35 UTC (rev 5382)
+++ core/trunk/src/test/java/org/jboss/cache/statetransfer/StateTransferCon=
currencyTest.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=3D"mailto://brian.stansberry(a)jboss.com">Brian Stansbe=
rry</a>
  * @version $Id$
@@ -52,8 +51,7 @@
     *
     * @throws Exception
     */
-   @Test(invocationCount =3D 25, enabled =3D false)
-   // TODO: Fix this before 2.1.0.GA !! - Manik
+   @Test(invocationCount =3D 25, enabled =3D true)
    public void testConcurrentActivationSync() throws Exception
    {
       concurrentActivationTest(true);
@@ -66,8 +64,7 @@
     *
     * @throws Exception
     */
-   @Test(invocationCount =3D 25, enabled =3D false)
-   // TODO: Fix this before 2.1.0.GA !! - Manik
+   @Test(invocationCount =3D 25, enabled =3D 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 s=
tate
     * of all caches is not consistent.
     */
-   @Test(invocationCount =3D 25, enabled =3D false)
-   // TODO: Fix this before 2.1.0.GA !! - Manik
+   @Test(invocationCount =3D 25, enabled =3D 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 s=
tate
     * of all caches is not consistent.
     */
-   @Test(invocationCount =3D 25, enabled =3D false)
-   // TODO: Fix this before 2.1.0.GA !! - Manik
+   @Test(invocationCount =3D 25, enabled =3D true)
    public void testConcurrentStartupActivationSync() throws Exception
    {
       concurrentActivationTest2(true);


--===============6446864193523599689==--