Removing unnecessary suite() methods, object attributes and constructors

git-svn-id: https://svn.apache.org/repos/asf/commons/sandbox/csv/trunk@631847 13f79535-47bb-0310-9956-ffa450edef68

Author: hen

src/test/org/apache/commons/csv/CSVParserTest.java

@@ -59,23 +59,6 @@ public String testNextToken() throws IOException {
     }
   }
 
-  /**
-   * Constructor for JUnit.
-   * @param name Name to be used in JUnit Test Environment
-   */
-  public CSVParserTest(String name) {
-    super(name);
-  }
-
-  /**
-   * Returns a Test suite for JUnit.
-   * @return Test suite for JUnit
-   */
-  public static Test suite() {
-    return new TestSuite(CSVParserTest.class);
-  }
-
-
   // ======================================================
   //   lexer tests
   // ======================================================

src/test/org/apache/commons/csv/CSVPrinterTest.java

@@ -26,19 +26,7 @@
  */
 public class CSVPrinterTest extends TestCase {
 
-  String lineSeparator = null;
-  
-  /**
-   * Constructor for CSVPrinterTest.
-   */
-  public CSVPrinterTest(String name) {
-    super(name);
-    this.lineSeparator = System.getProperty("line.separator");
-  }
-
-  public static Test suite() {
-    return new TestSuite(CSVPrinterTest.class);
-  }
+  String lineSeparator = System.getProperty("line.separator");
 
   public void testPrinter1() {
     StringWriter sw = new StringWriter();

src/test/org/apache/commons/csv/CSVStrategyTest.java

@@ -32,23 +32,6 @@
  * has problems...).
  */
 public class CSVStrategyTest extends TestCase {
-  
-  /**
-   * Constructor for JUnit.
-   * @param name Name to be used in JUnit Test Environment
-   */
-  public CSVStrategyTest(String name) {
-    super(name);
-  }
-
-  /**
-   * Returns a Test suite for JUnit.
-   * @return Test suite for JUnit
-   */
-  public static Test suite() {
-    return new TestSuite(CSVStrategyTest.class);
-  }
-
 
   // ======================================================
   //   getters / setters

src/test/org/apache/commons/csv/ExtendedBufferedReaderTest.java

@@ -29,33 +29,19 @@
  */
 public class ExtendedBufferedReaderTest extends TestCase {
 
-  private ExtendedBufferedReader br = null;
-  
-  /**
-   * Constructor for ExtendedBufferedReaderTest.
-   * @param arg0
-   */
-  public ExtendedBufferedReaderTest(String arg0) {
-    super(arg0);
-  }
-
-  public static Test suite() {
-    return new TestSuite(ExtendedBufferedReaderTest.class);
-  }
-  
   // ======================================================
   //   the test cases
   // ======================================================
 
   public void testConstructors() {
-    br = new ExtendedBufferedReader(new StringReader(""));
+    ExtendedBufferedReader br = new ExtendedBufferedReader(new StringReader(""));
     br = new ExtendedBufferedReader(new StringReader(""), 10); 
   }
 
   public void testReadLookahead1() throws Exception {
 
     assertEquals(ExtendedBufferedReader.END_OF_STREAM, getEBR("").read());
-    br = getEBR("1\n2\r3\n");
+    ExtendedBufferedReader br = getEBR("1\n2\r3\n");
     assertEquals('1', br.lookAhead());
     assertEquals(ExtendedBufferedReader.UNDEFINED, br.readAgain());
     assertEquals('1', br.read());
@@ -109,7 +95,7 @@ public void testReadLookahead2() throws Exception {
     char[] ref = new char[5];
     char[] res = new char[5];  
 
-    br = getEBR("");
+    ExtendedBufferedReader br = getEBR("");
     assertEquals(0, br.read(res, 0, 0));
     assertTrue(Arrays.equals(res, ref)); 
 
@@ -134,7 +120,7 @@ public void testMarkSupported() {
   }
 
   public void testReadLine() throws Exception {
-    br = getEBR("");
+    ExtendedBufferedReader br = getEBR("");
     assertTrue(br.readLine() == null);
 
     br = getEBR("\n");
@@ -177,7 +163,7 @@ public void testReadLine() throws Exception {
 
   public void testSkip0() throws Exception {
 
-    br = getEBR("");
+    ExtendedBufferedReader br = getEBR("");
     assertEquals(0, br.skip(0));
     assertEquals(0, br.skip(1));
 
@@ -203,7 +189,7 @@ public void testSkip0() throws Exception {
   }
 
   public void testSkipUntil() throws Exception {   
-    br = getEBR("");
+    ExtendedBufferedReader br = getEBR("");
     assertEquals(0, br.skipUntil(';'));
     br = getEBR("ABCDEF,GHL,,MN");
     assertEquals(6, br.skipUntil(','));
@@ -217,7 +203,7 @@ public void testSkipUntil() throws Exception {
   }
 
   public void testReadUntil() throws Exception {
-    br = getEBR("");
+    ExtendedBufferedReader br = getEBR("");
     assertTrue(br.readUntil(';').equals(""));
     br = getEBR("ABCDEF;GHL;;MN");
     assertTrue(br.readUntil(';').equals("ABCDEF"));