[jbosscache-commits] JBoss Cache SVN: r5392 - core/trunk/src/test/java/org/jboss/cache/notifications.

[jbosscache-commits at lists.jboss.org]

Author: mircea.markus
Date: 2008-03-06 16:51:14 -0500 (Thu, 06 Mar 2008)
New Revision: 5392

Modified:
   core/trunk/src/test/java/org/jboss/cache/notifications/RemoteCacheListenerTest.java
Log:
fixed testStateTransfer in the case of optimistic lock:
- in case of optimistic locking an transaction registered event was expected. This is incorrect as locking in case of state transfer is always pessimistic . The assesment was introduced as a workaround in 2.1.0CR3, when a non-direct cache call was performed in DefaultStateTransferIntegrator.retainInternalNodes. 


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-06 02:01:44 UTC (rev 5391)
+++ core/trunk/src/test/java/org/jboss/cache/notifications/RemoteCacheListenerTest.java	2008-03-06 21:51:14 UTC (rev 5392)
@@ -659,11 +659,6 @@
       List<Event> expected = new ArrayList<Event>();
       // public EventImpl(boolean pre, Cache cache, ModificationType modificationType, Map data, Fqn fqn,
       // Transaction transaction, boolean originLocal, Fqn targetFqn, boolean successful, View newView, Type type)
-      if (optLocking)
-         expected.add(new EventImpl(false, cache2, null, null, null, null, true, null, false, null, TRANSACTION_REGISTERED));
-      if (optLocking)
-         expected.add(new EventImpl(false, cache2, null, null, null, null, true, null, true, null, TRANSACTION_COMPLETED));
-
       expected.add(new EventImpl(true, cache2, null, null, Fqn.ROOT, null, false, null, false, null, NODE_CREATED));
       expected.add(new EventImpl(false, cache2, null, null, Fqn.ROOT, null, false, null, false, null, NODE_CREATED));