[infinispan-commits] Infinispan SVN: r808 - trunk/cachestore/jdbc/src/test/java/org/infinispan/loaders/jdbc.

[infinispan-commits at lists.jboss.org]

Author: galder.zamarreno at jboss.com
Date: 2009-09-11 10:08:33 -0400 (Fri, 11 Sep 2009)
New Revision: 808

Modified:
   trunk/cachestore/jdbc/src/test/java/org/infinispan/loaders/jdbc/TableNameUniquenessTest.java
Log:
VAM.start() does get called. If it wasn't this was due to ISPN-148, but once VAM was converted to NonVolatile, the issue was gone, so remove the TODOs.

Modified: trunk/cachestore/jdbc/src/test/java/org/infinispan/loaders/jdbc/TableNameUniquenessTest.java
===================================================================
--- trunk/cachestore/jdbc/src/test/java/org/infinispan/loaders/jdbc/TableNameUniquenessTest.java	2009-09-11 14:03:55 UTC (rev 807)
+++ trunk/cachestore/jdbc/src/test/java/org/infinispan/loaders/jdbc/TableNameUniquenessTest.java	2009-09-11 14:08:33 UTC (rev 808)
@@ -11,8 +11,6 @@
 import org.infinispan.loaders.jdbc.stringbased.JdbcStringBasedCacheStoreConfig;
 import org.infinispan.manager.CacheManager;
 import org.infinispan.manager.DefaultCacheManager;
-import org.infinispan.marshall.Marshaller;
-import org.infinispan.marshall.VersionAwareMarshaller;
 import org.infinispan.test.TestingUtil;
 import org.testng.annotations.Test;
 
@@ -39,12 +37,6 @@
       assert firstCacheLoaderConfig instanceof JdbcStringBasedCacheStoreConfig;
       assert secondCacheLoaderConfig instanceof JdbcStringBasedCacheStoreConfig;
 
-      //TODO - this is a hack as VAM.start does not get called, for some reason. This should be removed.
-      VersionAwareMarshaller firstVam = (VersionAwareMarshaller) TestingUtil.extractComponent(first, Marshaller.class);
-      firstVam.start();
-      VersionAwareMarshaller secondVam = (VersionAwareMarshaller) TestingUtil.extractComponent(first, Marshaller.class);
-      secondVam.start();
-
       JdbcStringBasedCacheStore firstCs = (JdbcStringBasedCacheStore) TestingUtil.extractComponent(first, CacheLoaderManager.class).getCacheLoader();
       JdbcStringBasedCacheStore secondCs = (JdbcStringBasedCacheStore) TestingUtil.extractComponent(second, CacheLoaderManager.class).getCacheLoader();
 
@@ -58,12 +50,6 @@
       Cache<String, String> first = cm.getCache("first");
       Cache<String, String> second = cm.getCache("second");
 
-      //TODO - this is a hack as VAM.start does not get called, for some reason. This should be removed.
-      VersionAwareMarshaller firstVam = (VersionAwareMarshaller) TestingUtil.extractComponent(first, Marshaller.class);
-      firstVam.start();
-      VersionAwareMarshaller secondVam = (VersionAwareMarshaller) TestingUtil.extractComponent(first, Marshaller.class);
-      secondVam.start();
-
       JdbcBinaryCacheStore firstCs = (JdbcBinaryCacheStore) TestingUtil.extractComponent(first, CacheLoaderManager.class).getCacheLoader();
       JdbcBinaryCacheStore secondCs = (JdbcBinaryCacheStore) TestingUtil.extractComponent(second, CacheLoaderManager.class).getCacheLoader();
 
@@ -78,12 +64,6 @@
       Cache first = cm.getCache("first");
       Cache second = cm.getCache("second");
 
-      //TODO - this is a hack as VAM.start does not get called, for some reason. This should be removed.
-      VersionAwareMarshaller firstVam = (VersionAwareMarshaller) TestingUtil.extractComponent(first, Marshaller.class);
-      firstVam.start();
-      VersionAwareMarshaller secondVam = (VersionAwareMarshaller) TestingUtil.extractComponent(first, Marshaller.class);
-      secondVam.start();
-
       JdbcMixedCacheStore firstCs = (JdbcMixedCacheStore) TestingUtil.extractComponent(first, CacheLoaderManager.class).getCacheLoader();
       JdbcMixedCacheStore secondCs = (JdbcMixedCacheStore) TestingUtil.extractComponent(second, CacheLoaderManager.class).getCacheLoader();