Clean-up after review in github's compare view. Thanks, github.

lyh411201000 · Apr 8, 2010 · a30e87b · a30e87b
1 parent c84f84e
commit a30e87b
Show file tree

Hide file tree

Showing 3 changed files with 3 additions and 4 deletions.
diff --git a/src/main/java/junit/framework/JUnit4TestAdapter.java b/src/main/java/junit/framework/JUnit4TestAdapter.java
@@ -16,9 +16,9 @@
 public class JUnit4TestAdapter implements Test, Filterable, Sortable, Describable {
 	private final Class<?> fNewTestClass;
 
-	private Runner fRunner;
+	private final Runner fRunner;
 
-	private JUnit4TestAdapterCache fCache;
+	private final JUnit4TestAdapterCache fCache;
 
 	public JUnit4TestAdapter(Class<?> newTestClass) {
 		this(newTestClass, JUnit4TestAdapterCache.getDefault());

diff --git a/src/main/java/org/junit/rules/ExternalResource.java b/src/main/java/org/junit/rules/ExternalResource.java
@@ -58,7 +58,6 @@ protected void before() throws Throwable {
 
 	/**
 	 * Override to tear down your specific external resource.
-	 * @throws if setup fails (which will disable {@code after}
 	 */
 	protected void after() {
 		// do nothing

diff --git a/src/test/java/junit/tests/framework/SuiteTest.java b/src/test/java/junit/tests/framework/SuiteTest.java
@@ -82,7 +82,7 @@ public void testOneTestCase() {
 		assertTrue(fResult.wasSuccessful());
 	}
 	public void testOneTestCaseEclipseSeesSameStructureAs381() {
-		TestSuite t= new TestSuite(ThreeTestCases 	.class);
+		TestSuite t= new TestSuite(ThreeTestCases.class);
 		assertEquals(3, Collections.list(t.tests()).size());
 	}
 	public void testShadowedTests() {