[jboss-svn-commits] JBL Code SVN: r32331 - in labs/jbossrules/trunk/drools-core/src: test/resources/org/drools/reteoo/test and 1 other directory.

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

Author: tirelli
Date: 2010-03-31 17:08:56 -0400 (Wed, 31 Mar 2010)
New Revision: 32331

Modified:
   labs/jbossrules/trunk/drools-core/src/main/java/org/drools/reteoo/AccumulateNode.java
   labs/jbossrules/trunk/drools-core/src/test/resources/org/drools/reteoo/test/AccumulateNodeAssertRetractTest.nodeTestCase
   labs/jbossrules/trunk/drools-core/src/test/resources/org/drools/reteoo/test/CollectNodeAssertRetractTest.nodeTestCase
   labs/jbossrules/trunk/drools-core/src/test/resources/org/drools/reteoo/test/JoinNodeModifyTest.nodeTestCase
Log:
Fixing tests

Modified: labs/jbossrules/trunk/drools-core/src/main/java/org/drools/reteoo/AccumulateNode.java
===================================================================
--- labs/jbossrules/trunk/drools-core/src/main/java/org/drools/reteoo/AccumulateNode.java	2010-03-31 18:45:48 UTC (rev 32330)
+++ labs/jbossrules/trunk/drools-core/src/main/java/org/drools/reteoo/AccumulateNode.java	2010-03-31 21:08:56 UTC (rev 32331)
@@ -820,7 +820,7 @@
                                            accctx,
                                            false );
         for ( LeftTuple match = matchings[0]; match != null; match = match.getLeftParentNext() ) {
-            match.unlinkFromLeftParent();
+            // can't unlink from the left parent as it was already unlinked during the splitList call above
             match.unlinkFromRightParent();
         }
         // since there are no more matches, the following call will just re-initialize the accumulation

Modified: labs/jbossrules/trunk/drools-core/src/test/resources/org/drools/reteoo/test/AccumulateNodeAssertRetractTest.nodeTestCase
===================================================================
--- labs/jbossrules/trunk/drools-core/src/test/resources/org/drools/reteoo/test/AccumulateNodeAssertRetractTest.nodeTestCase	2010-03-31 18:45:48 UTC (rev 32330)
+++ labs/jbossrules/trunk/drools-core/src/test/resources/org/drools/reteoo/test/AccumulateNodeAssertRetractTest.nodeTestCase	2010-03-31 21:08:56 UTC (rev 32331)
@@ -57,7 +57,7 @@
 /**        
  *  check single left then right assertion
  */
- Test "single left then right assertion" 
+ Test "accumulate node single left then right assertion" 
 	assert:
 	    acc, [[h0]];  
 	sink:

Modified: labs/jbossrules/trunk/drools-core/src/test/resources/org/drools/reteoo/test/CollectNodeAssertRetractTest.nodeTestCase
===================================================================
--- labs/jbossrules/trunk/drools-core/src/test/resources/org/drools/reteoo/test/CollectNodeAssertRetractTest.nodeTestCase	2010-03-31 18:45:48 UTC (rev 32330)
+++ labs/jbossrules/trunk/drools-core/src/test/resources/org/drools/reteoo/test/CollectNodeAssertRetractTest.nodeTestCase	2010-03-31 21:08:56 UTC (rev 32331)
@@ -59,7 +59,7 @@
 /**        
  *  check single left then right assertion
  */
- Test "single left then right assertion" 
+ Test "collect node single left then right assertion" 
 	assert:
 	    otnLeft1, [h0];  
 	sink1:

Modified: labs/jbossrules/trunk/drools-core/src/test/resources/org/drools/reteoo/test/JoinNodeModifyTest.nodeTestCase
===================================================================
--- labs/jbossrules/trunk/drools-core/src/test/resources/org/drools/reteoo/test/JoinNodeModifyTest.nodeTestCase	2010-03-31 18:45:48 UTC (rev 32330)
+++ labs/jbossrules/trunk/drools-core/src/test/resources/org/drools/reteoo/test/JoinNodeModifyTest.nodeTestCase	2010-03-31 21:08:56 UTC (rev 32331)
@@ -22,7 +22,7 @@
 	    new org.drools.Person('yoda', 36), new org.drools.Person('luke', 37),
 	    new org.drools.Person('dave', 38), new org.drools.Person('bob', 39); 
 
-Test "Modify test"	     
+Test "Join Node Modify test"	     
 	/**    
 	 * insert one left and try modify
 	 */