move validation

Author: panchenko

src/main/java/org/junit/runners/Parameterized.java

@@ -1,5 +1,6 @@
 package org.junit.runners;
 
+import java.lang.annotation.Annotation;
 import java.lang.annotation.ElementType;
 import java.lang.annotation.Inherited;
 import java.lang.annotation.Retention;
@@ -267,6 +268,21 @@ public Parameterized(Class<?> klass) throws Throwable {
         super(klass, RunnersFactory.createRunnersForClass(klass));
     }
 
+    @Override
+    protected void collectInitializationErrors(List<Throwable> errors) {
+        super.collectInitializationErrors(errors);
+        validatePublicStaticVoidMethods(Parameterized.BeforeParam.class, errors);
+        validatePublicStaticVoidMethods(Parameterized.AfterParam.class, errors);
+    }
+
+    private void validatePublicStaticVoidMethods(Class<? extends Annotation> annotation,
+                                                 List<Throwable> errors) {
+        final List<FrameworkMethod> methods = getTestClass().getAnnotatedMethods(annotation);
+        for (FrameworkMethod eachTestMethod : methods) {
+            eachTestMethod.validatePublicVoid(true, errors);
+        }
+    }
+
     private static class RunnersFactory {
         private static final ParametersRunnerFactory DEFAULT_FACTORY = new BlockJUnit4ClassRunnerWithParametersFactory();
 

src/main/java/org/junit/runners/parameterized/BlockJUnit4ClassRunnerWithParameters.java

@@ -96,21 +96,6 @@ protected String testName(FrameworkMethod method) {
         return method.getName() + getName();
     }
 
-    @Override
-    protected void collectInitializationErrors(List<Throwable> errors) {
-        super.collectInitializationErrors(errors);
-        validatePublicStaticVoidMethods(Parameterized.BeforeParam.class, errors);
-        validatePublicStaticVoidMethods(Parameterized.AfterParam.class, errors);
-    }
-
-    private void validatePublicStaticVoidMethods(Class<? extends Annotation> annotation,
-                                                 List<Throwable> errors) {
-        final List<FrameworkMethod> methods = getTestClass().getAnnotatedMethods(annotation);
-        for (FrameworkMethod eachTestMethod : methods) {
-            eachTestMethod.validatePublicVoid(true, errors);
-        }
-    }
-
     @Override
     protected void validateConstructor(List<Throwable> errors) {
         validateOnlyOneConstructor(errors);