formatting of pull request #28

src/main/java/com/tngtech/java/junit/dataprovider/DataProviderRunner.java

@@ -63,7 +63,6 @@ public class DataProviderRunner extends BlockJUnit4ClassRunner {
      */
     TestGenerator testGenerator;
 
-
     /**
      * Cached result of {@link #computeTestMethods()}.
      * <p>

src/main/java/com/tngtech/java/junit/dataprovider/internal/TestGenerator.java

@@ -63,12 +63,12 @@ public List<FrameworkMethod> generateExplodedTestMethodsFor(FrameworkMethod test
     List<FrameworkMethod> explodeTestMethod(FrameworkMethod testMethod, FrameworkMethod dataProviderMethod) {
         Object data;
         try {
-          Class<?>[] parameterTypes = dataProviderMethod.getMethod().getParameterTypes();
-          if (parameterTypes.length > 0) {
-              data = dataProviderMethod.invokeExplosively(null, testMethod);
-          } else {
-              data = dataProviderMethod.invokeExplosively(null);
-          }
+            Class<?>[] parameterTypes = dataProviderMethod.getMethod().getParameterTypes();
+            if (parameterTypes.length > 0) {
+                data = dataProviderMethod.invokeExplosively(null, testMethod);
+            } else {
+                data = dataProviderMethod.invokeExplosively(null);
+            }
         } catch (Throwable t) {
             throw new Error(String.format("Exception while invoking data provider method '%s': %s",
                     dataProviderMethod.getName(), t.getMessage()), t);

src/test/java/com/tngtech/test/java/junit/dataprovider/DataProviderJavaAcceptanceTest.java

@@ -10,11 +10,11 @@
 
 import org.junit.Test;
 import org.junit.runner.RunWith;
+import org.junit.runners.model.FrameworkMethod;
 
 import com.tngtech.java.junit.dataprovider.DataProvider;
 import com.tngtech.java.junit.dataprovider.DataProviderRunner;
 import com.tngtech.java.junit.dataprovider.UseDataProvider;
-import org.junit.runners.model.FrameworkMethod;
 
 @RunWith(DataProviderRunner.class)
 public class DataProviderJavaAcceptanceTest {
@@ -232,18 +232,17 @@ public void testIsEmptyString2(String str) {
         assertThat(isEmpty).isTrue();
     }
 
+    @DataProvider
+    public static Object[][] loadFromExternalFile(FrameworkMethod testMethod) {
+        String testDataFile = testMethod.getAnnotation(ExternalFile.class).value();
+        return new Object[][] { { testDataFile } };
+    }
+
     @Test
     @UseDataProvider("loadFromExternalFile")
     @ExternalFile(format = ExternalFile.Format.CSV, value = "testdata.csv")
     public void testThatUsesUniversalDataProvider(String testData) {
+        // Expect:
         assertThat(testData).isEqualTo("testdata.csv");
     }
-
-    @DataProvider
-    public static Object[][] loadFromExternalFile(FrameworkMethod testMethod) {
-        String testDataFile = testMethod.getAnnotation(ExternalFile.class).value();
-        return new Object[][]{
-            { testDataFile }
-        };
-    }
 }

src/test/java/com/tngtech/test/java/junit/dataprovider/ExternalFile.java

@@ -1,15 +1,21 @@
 package com.tngtech.test.java.junit.dataprovider;
 
-import java.lang.annotation.*;
+import java.lang.annotation.ElementType;
+import java.lang.annotation.Retention;
+import java.lang.annotation.RetentionPolicy;
+import java.lang.annotation.Target;
 
 @Retention(RetentionPolicy.RUNTIME)
 @Target(ElementType.METHOD)
 public @interface ExternalFile {
 
-  public enum Format {
-    CSV, XML, XLS
-  }
+    public enum Format {
+        CSV,
+        XML,
+        XLS
+    }
 
-  Format format();
-  String value();
+    Format format();
+
+    String value();
 }