svn commit: r612307 - /commons/proper/scxml/branches/J5/src/test/java/org/apache/commons/scxml/SCXMLExecutorTest.java

[rahul]

Author: rahul
Date: Tue Jan 15 15:23:01 2008
New Revision: 612307

URL: http://svn.apache.org/viewvc?rev=612307&view=rev
Log:
Use for-each to reduce clutter.

Modified:
    
commons/proper/scxml/branches/J5/src/test/java/org/apache/commons/scxml/SCXMLExecutorTest.java

Modified: 
commons/proper/scxml/branches/J5/src/test/java/org/apache/commons/scxml/SCXMLExecutorTest.java
URL: 
http://svn.apache.org/viewvc/commons/proper/scxml/branches/J5/src/test/java/org/apache/commons/scxml/SCXMLExecutorTest.java?rev=612307&r1=612306&r2=612307&view=diff
==============================================================================
--- 
commons/proper/scxml/branches/J5/src/test/java/org/apache/commons/scxml/SCXMLExecutorTest.java
 (original)
+++ 
commons/proper/scxml/branches/J5/src/test/java/org/apache/commons/scxml/SCXMLExecutorTest.java
 Tue Jan 15 15:23:01 2008
@@ -18,7 +18,6 @@
 
 import java.net.URL;
 import java.util.HashSet;
-import java.util.Iterator;
 import java.util.Set;
 
 import junit.framework.Test;
@@ -201,14 +200,13 @@
         exec = SCXMLTestHelper.getExecutor(transitions03);
         assertNotNull(exec);
         try {
-            Set currentStates = SCXMLTestHelper.fireEvent(exec, "ten.done");
+            Set<TransitionTarget> currentStates = 
SCXMLTestHelper.fireEvent(exec, "ten.done");
             assertEquals(3, currentStates.size());
             Set<String> expected = new HashSet<String>();
             expected.add("twenty_one_2");
             expected.add("twenty_two_2");
             expected.add("twenty_three_2");
-            for (Iterator i = currentStates.iterator(); i.hasNext(); ) {
-                TransitionTarget tt = (TransitionTarget) i.next();
+            for (TransitionTarget tt : currentStates) {
                 if (!expected.remove(tt.getId())) {
                     fail("'" + tt.getId()
                         + "' is not an expected current state ID");
@@ -226,14 +224,13 @@
         exec = SCXMLTestHelper.getExecutor(scxml);
         assertNotNull(exec);
         try {
-            Set currentStates = SCXMLTestHelper.fireEvent(exec, "ten.done");
+            Set<TransitionTarget> currentStates = 
SCXMLTestHelper.fireEvent(exec, "ten.done");
             assertEquals(3, currentStates.size());
             Set<String> expected = new HashSet<String>();
             expected.add("twenty_one_1");
             expected.add("twenty_two_1");
             expected.add("twenty_three_1");
-            for (Iterator i = currentStates.iterator(); i.hasNext(); ) {
-                TransitionTarget tt = (TransitionTarget) i.next();
+            for (TransitionTarget tt : currentStates) {
                 if (!expected.remove(tt.getId())) {
                     fail("'" + tt.getId()
                         + "' is not an expected current state ID");