You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@commons.apache.org by jb...@apache.org on 2010/06/25 14:11:42 UTC

svn commit: r957911 - in /commons/sandbox/gsoc/2010/scxml-js/branches/core-module.SCXML-137/test/conditional_transition: TestConditionalTransition.xml scripts/unitTest.js

Author: jbeard
Date: Fri Jun 25 12:11:42 2010
New Revision: 957911

URL: http://svn.apache.org/viewvc?rev=957911&view=rev
Log:
Changed state id from 'final' to 'last' because 'final' is a reserved word in js. More robust filtering of identifiers is needed to prevent future issues like this.

Modified:
    commons/sandbox/gsoc/2010/scxml-js/branches/core-module.SCXML-137/test/conditional_transition/TestConditionalTransition.xml
    commons/sandbox/gsoc/2010/scxml-js/branches/core-module.SCXML-137/test/conditional_transition/scripts/unitTest.js

Modified: commons/sandbox/gsoc/2010/scxml-js/branches/core-module.SCXML-137/test/conditional_transition/TestConditionalTransition.xml
URL: http://svn.apache.org/viewvc/commons/sandbox/gsoc/2010/scxml-js/branches/core-module.SCXML-137/test/conditional_transition/TestConditionalTransition.xml?rev=957911&r1=957910&r2=957911&view=diff
==============================================================================
--- commons/sandbox/gsoc/2010/scxml-js/branches/core-module.SCXML-137/test/conditional_transition/TestConditionalTransition.xml (original)
+++ commons/sandbox/gsoc/2010/scxml-js/branches/core-module.SCXML-137/test/conditional_transition/TestConditionalTransition.xml Fri Jun 25 12:11:42 2010
@@ -93,10 +93,10 @@ transitions originating from the same st
 
 		<state id="j"/>
 
-		<transition target="final" event="t5" cond="true"/>
+		<transition target="last" event="t5" cond="true"/>
 	</state>
 
-	<state id="final"/>
+	<state id="last"/>
 </scxml>
 
 

Modified: commons/sandbox/gsoc/2010/scxml-js/branches/core-module.SCXML-137/test/conditional_transition/scripts/unitTest.js
URL: http://svn.apache.org/viewvc/commons/sandbox/gsoc/2010/scxml-js/branches/core-module.SCXML-137/test/conditional_transition/scripts/unitTest.js?rev=957911&r1=957910&r2=957911&view=diff
==============================================================================
--- commons/sandbox/gsoc/2010/scxml-js/branches/core-module.SCXML-137/test/conditional_transition/scripts/unitTest.js (original)
+++ commons/sandbox/gsoc/2010/scxml-js/branches/core-module.SCXML-137/test/conditional_transition/scripts/unitTest.js Fri Jun 25 12:11:42 2010
@@ -74,7 +74,7 @@ dojo.require("doh.runner");
 					sc.t5()
 					var conf6 = sc.getCurrentConfiguration();
 					console.log($(conf6));
-					doh.assertTrue(setwiseEqual(conf6,[sc._states.final]));
+					doh.assertTrue(setwiseEqual(conf6,[sc._states.last]));
 				},
 				tearDown:function(){
 					 sc.destroy();