svn commit: r1403167 - in /commons/proper/vfs/trunk/core/src/test/java/org/apache/commons/vfs2: provider/tar/test/LargeTarTestCase.java util/DelegatingFileSystemOptionsBuilderTest.java

[gg...@apache.org]

Author: ggregory
Date: Mon Oct 29 04:00:41 2012
New Revision: 1403167

URL: http://svn.apache.org/viewvc?rev=1403167&view=rev
Log:
Convert to enhanced for loop.

Modified:
    commons/proper/vfs/trunk/core/src/test/java/org/apache/commons/vfs2/provider/tar/test/LargeTarTestCase.java
    commons/proper/vfs/trunk/core/src/test/java/org/apache/commons/vfs2/util/DelegatingFileSystemOptionsBuilderTest.java

Modified: commons/proper/vfs/trunk/core/src/test/java/org/apache/commons/vfs2/provider/tar/test/LargeTarTestCase.java
URL: http://svn.apache.org/viewvc/commons/proper/vfs/trunk/core/src/test/java/org/apache/commons/vfs2/provider/tar/test/LargeTarTestCase.java?rev=1403167&r1=1403166&r2=1403167&view=diff
==============================================================================
--- commons/proper/vfs/trunk/core/src/test/java/org/apache/commons/vfs2/provider/tar/test/LargeTarTestCase.java (original)
+++ commons/proper/vfs/trunk/core/src/test/java/org/apache/commons/vfs2/provider/tar/test/LargeTarTestCase.java Mon Oct 29 04:00:41 2012
@@ -112,18 +112,18 @@ public void setUp() throws Exception {
     List<FileObject> files = Arrays.asList(file.getChildren());
 
     assertNotNull(files);
-    for(int i=0; i < expectedFiles.length; ++i) {
-      String expectedFile = expectedFiles[i];
+    for (String expectedFile : expectedFiles)
+    {
       assertTrue("Expected file not found: " + expectedFile, fileExists(expectedFile, files));
     }
   }
 
   /**
    * Search for the expected file in a given list, without using the full path.
-   * 
+   *
    * @param expectedFile the expected file.
    * @param files a list of files to search.
-   * @return {@code true} if {@code expectedFile} is in {@code files}. 
+   * @return {@code true} if {@code expectedFile} is in {@code files}.
    */
     protected boolean fileExists(String expectedFile, List<FileObject> files)
     {

Modified: commons/proper/vfs/trunk/core/src/test/java/org/apache/commons/vfs2/util/DelegatingFileSystemOptionsBuilderTest.java
URL: http://svn.apache.org/viewvc/commons/proper/vfs/trunk/core/src/test/java/org/apache/commons/vfs2/util/DelegatingFileSystemOptionsBuilderTest.java?rev=1403167&r1=1403166&r2=1403167&view=diff
==============================================================================
--- commons/proper/vfs/trunk/core/src/test/java/org/apache/commons/vfs2/util/DelegatingFileSystemOptionsBuilderTest.java (original)
+++ commons/proper/vfs/trunk/core/src/test/java/org/apache/commons/vfs2/util/DelegatingFileSystemOptionsBuilderTest.java Mon Oct 29 04:00:41 2012
@@ -128,9 +128,9 @@ public class DelegatingFileSystemOptions
     @Test
     public void testConfiguration() throws Exception
     {
-        for (int i=0; i < schemes.length; ++i)
+        for (String scheme : schemes)
         {
-            assertTrue("Missing " + schemes[i] + " provider", fsm.hasProvider(schemes[i]));
+            assertTrue("Missing " + scheme + " provider", fsm.hasProvider(scheme));
         }
     }
 }