[Junit-devel] CVS: junit/src/org/junit/internal/runners TestClassRunner.java, 1.2, 1.3 MethodValidator.java, 1.3, 1.4

[David S. <ds...@us...>]

Update of /cvsroot/junit/junit/src/org/junit/internal/runners
In directory sc8-pr-cvs6.sourceforge.net:/tmp/cvs-serv12633/src/org/junit/internal/runners

Modified Files:
	TestClassRunner.java MethodValidator.java 
Log Message:
Rename validateAllMethods to validateMethodsForDefaultRunner, which is clearer

Index: TestClassRunner.java
===================================================================
RCS file: /cvsroot/junit/junit/src/org/junit/internal/runners/TestClassRunner.java,v
retrieving revision 1.2
retrieving revision 1.3
diff -u -d -r1.2 -r1.3
--- TestClassRunner.java	6 Dec 2006 01:22:48 -0000	1.2
+++ TestClassRunner.java	21 Feb 2007 14:46:51 -0000	1.3
@@ -30,7 +30,7 @@
 
 	// TODO: this is parallel to passed-in runner
 	protected void validate(MethodValidator methodValidator) {
-		methodValidator.validateAllMethods();
+		methodValidator.validateMethodsForDefaultRunner();
 	}
 
 	@Override

Index: MethodValidator.java
===================================================================
RCS file: /cvsroot/junit/junit/src/org/junit/internal/runners/MethodValidator.java,v
retrieving revision 1.3
retrieving revision 1.4
diff -u -d -r1.3 -r1.4
--- MethodValidator.java	24 Jan 2007 16:32:59 -0000	1.3
+++ MethodValidator.java	21 Feb 2007 14:46:51 -0000	1.4
@@ -35,8 +35,7 @@
 		validateTestMethods(AfterClass.class, true);
 	}
 	
-	// TODO Ugly API--one method should do both
-	public List<Throwable> validateAllMethods() {
+	public List<Throwable> validateMethodsForDefaultRunner() {
 		validateNoArgConstructor();
 		validateStaticMethods();
 		validateInstanceMethods();