Use JUnit 5 convention for test method visibility

Author: garydgregory

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

@@ -272,7 +272,7 @@ static Stream<Arguments> duplicateHeaderData() {
      */
     @ParameterizedTest
     @MethodSource(value = {"duplicateHeaderAllowsMissingColumnsNamesData"})
-    public void testCSVFormat(final DuplicateHeaderMode duplicateHeaderMode,
+    void testCSVFormat(final DuplicateHeaderMode duplicateHeaderMode,
                               final boolean allowMissingColumnNames,
                               final boolean ignoreHeaderCase,
                               final String[] headers,
@@ -305,7 +305,7 @@ public void testCSVFormat(final DuplicateHeaderMode duplicateHeaderMode,
      */
     @ParameterizedTest
     @MethodSource(value = {"duplicateHeaderData"})
-    public void testCSVParser(final DuplicateHeaderMode duplicateHeaderMode,
+    void testCSVParser(final DuplicateHeaderMode duplicateHeaderMode,
                               final boolean allowMissingColumnNames,
                               final boolean ignoreHeaderCase,
                               final String[] headers,

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

@@ -59,7 +59,7 @@ private String readTestData(final BufferedReader reader) throws IOException {
 
     @ParameterizedTest
     @MethodSource("generateData")
-    public void testCSVFile(final File testFile) throws Exception {
+    void testCSVFile(final File testFile) throws Exception {
         try (FileReader fr = new FileReader(testFile); BufferedReader testDataReader = new BufferedReader(fr)) {
             String line = readTestData(testDataReader);
             assertNotNull("file must contain config line", line);
@@ -104,7 +104,7 @@ public void testCSVFile(final File testFile) throws Exception {
 
     @ParameterizedTest
     @MethodSource("generateData")
-    public void testCSVUrl(final File testFile) throws Exception {
+    void testCSVUrl(final File testFile) throws Exception {
         try (FileReader fr = new FileReader(testFile); BufferedReader testData = new BufferedReader(fr)) {
             String line = readTestData(testData);
             assertNotNull("file must contain config line", line);

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

@@ -34,52 +34,52 @@ private void test(final CSVFormat format, final String enumName) {
     }
 
     @Test
-    public void testDefault() {
+    void testDefault() {
         test(CSVFormat.DEFAULT, "Default");
     }
 
     @Test
-    public void testExcel() {
+    void testExcel() {
         test(CSVFormat.EXCEL, "Excel");
     }
 
     @Test
-    public void testMongoDbCsv() {
+    void testMongoDbCsv() {
         test(CSVFormat.MONGODB_CSV, "MongoDBCsv");
     }
 
     @Test
-    public void testMongoDbTsv() {
+    void testMongoDbTsv() {
         test(CSVFormat.MONGODB_TSV, "MongoDBTsv");
     }
 
     @Test
-    public void testMySQL() {
+    void testMySQL() {
         test(CSVFormat.MYSQL, "MySQL");
     }
 
     @Test
-    public void testOracle() {
+    void testOracle() {
         test(CSVFormat.ORACLE, "Oracle");
     }
 
     @Test
-    public void testPostgreSqlCsv() {
+    void testPostgreSqlCsv() {
         test(CSVFormat.POSTGRESQL_CSV, "PostgreSQLCsv");
     }
 
     @Test
-    public void testPostgreSqlText() {
+    void testPostgreSqlText() {
         test(CSVFormat.POSTGRESQL_TEXT, "PostgreSQLText");
     }
 
     @Test
-    public void testRFC4180() {
+    void testRFC4180() {
         test(CSVFormat.RFC4180, "RFC4180");
     }
 
     @Test
-    public void testTDF() {
+    void testTDF() {
         test(CSVFormat.TDF, "TDF");
     }
 }