[jboss-cvs] JBossAS SVN: r109272 - trunk/testsuite/src/main/org/jboss/test/cluster/defaultcfg/web/test.

[jboss-cvs-commits at lists.jboss.org]

Author: pferraro
Date: 2010-11-11 12:39:22 -0500 (Thu, 11 Nov 2010)
New Revision: 109272

Modified:
   trunk/testsuite/src/main/org/jboss/test/cluster/defaultcfg/web/test/SessionPassivationTestCase.java
Log:
[JBAS-8613] Don't ever expect sessions to survive restart.  We can't rely on previous JBC hack to prevent cache purging.  Don't see the point in doing this to begin with - as it is not a real world scenario.
Fixes SessionPassivationTestCase/AttributeBasedSessionPassivationTestCase testRedeploy() failures in REPL mode.

Modified: trunk/testsuite/src/main/org/jboss/test/cluster/defaultcfg/web/test/SessionPassivationTestCase.java
===================================================================
--- trunk/testsuite/src/main/org/jboss/test/cluster/defaultcfg/web/test/SessionPassivationTestCase.java	2010-11-11 17:17:01 UTC (rev 109271)
+++ trunk/testsuite/src/main/org/jboss/test/cluster/defaultcfg/web/test/SessionPassivationTestCase.java	2010-11-11 17:39:22 UTC (rev 109272)
@@ -225,14 +225,6 @@
        deployed1 = false;
        undeploy(adaptors[1]);
        
-       if (!isCleanCacheOnRedeploy())
-       {
-          // Make sure the cache is not stopped during redeploy due to
-          // no services using it. Tell CacheHelper to hold a ref.
-//          String cacheConfigName = System.getProperty(SessionTestUtil.CACHE_CONFIG_PROP, "standard-session-cache");
-//          SessionTestUtil.setCacheConfigName(adaptors[0], cacheConfigName);
-       }
-       
        sleep(2000);
           
        // Create an instance of HttpClient.
@@ -259,16 +251,8 @@
        // Get the Attribute using the same session ID
        SessionTestUtil.setCookieDomainToThisServer(client, servers_[0]);
        
-       if (isCleanCacheOnRedeploy())
-       {
-          // We don't expect data to survive a restart
-          makeGetFailed(client, baseURL0_ +getUrl);
-       }
-       else
-       {                 
-          String attr0 = makeGet(client, baseURL0_ +getUrl);          
-          assertEquals("attributeMatches after activation", attr0, attr);
-       }
+       // We don't expect data to survive a restart
+       makeGetFailed(client, baseURL0_ +getUrl);
        
        getLog().debug("Exit testRedeploy");