[infinispan-commits] Infinispan SVN: r1686 - trunk/core/src/test/java/org/infinispan/test.

[infinispan-commits at lists.jboss.org]

Author: mircea.markus
Date: 2010-04-14 06:48:45 -0400 (Wed, 14 Apr 2010)
New Revision: 1686

Modified:
   trunk/core/src/test/java/org/infinispan/test/MultipleCacheManagersTest.java
Log:
javadoc

Modified: trunk/core/src/test/java/org/infinispan/test/MultipleCacheManagersTest.java
===================================================================
--- trunk/core/src/test/java/org/infinispan/test/MultipleCacheManagersTest.java	2010-04-13 06:56:41 UTC (rev 1685)
+++ trunk/core/src/test/java/org/infinispan/test/MultipleCacheManagersTest.java	2010-04-14 10:48:45 UTC (rev 1686)
@@ -89,6 +89,12 @@
       }
    }
 
+   /**
+    * Reason: after a tm.commit is run, multiple tests assert that the new value (as within the committing transaction)
+    * is present on a remote cache (i.e. not on the cache on which tx originated). If we don't use sync commit,
+    * than this (i.e. actual commit of the tx on the remote cache) might happen after the tm.commit() returns,
+    * and result in an intermittent failure for the assertion
+    */
    protected void assertSupportedConfig() {
       for (CacheManager cm : cacheManagers) {
          for (Cache cache : TestingUtil.getRunningCaches(cm)) {