This is an automated email from the ASF dual-hosted git repository.
ggregory pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/commons-cli.git
The following commit(s) were added to refs/heads/master by this push:
new cd272c31 Use JUnit 5 convention for class visibility
cd272c31 is described below
commit cd272c310f84ea039ddc4824a810b0f57d41cc02
Author: Gary D. Gregory <garydgreg...@gmail.com>
AuthorDate: Sun Jun 8 13:43:49 2025 -0400
Use JUnit 5 convention for class visibility
---
src/test/java/org/apache/commons/cli/AlreadySelectedExceptionTest.java | 2 +-
src/test/java/org/apache/commons/cli/ApplicationTest.java | 2 +-
src/test/java/org/apache/commons/cli/ArgumentIsOptionTest.java | 2 +-
src/test/java/org/apache/commons/cli/BasicParserTest.java | 2 +-
src/test/java/org/apache/commons/cli/CommandLineTest.java | 2 +-
src/test/java/org/apache/commons/cli/DefaultParserTest.java | 2 +-
src/test/java/org/apache/commons/cli/DeprecatedAttributesTest.java | 2 +-
src/test/java/org/apache/commons/cli/DisablePartialMatchingTest.java | 2 +-
src/test/java/org/apache/commons/cli/GnuParserTest.java | 2 +-
src/test/java/org/apache/commons/cli/HelpFormatterTest.java | 2 +-
src/test/java/org/apache/commons/cli/OptionBuilderTest.java | 2 +-
src/test/java/org/apache/commons/cli/OptionGroupTest.java | 2 +-
src/test/java/org/apache/commons/cli/OptionTest.java | 2 +-
src/test/java/org/apache/commons/cli/OptionValidatorTest.java | 2 +-
src/test/java/org/apache/commons/cli/OptionsTest.java | 2 +-
src/test/java/org/apache/commons/cli/ParseExceptionTest.java | 2 +-
src/test/java/org/apache/commons/cli/PatternOptionBuilderTest.java | 2 +-
src/test/java/org/apache/commons/cli/PosixParserTest.java | 2 +-
src/test/java/org/apache/commons/cli/SolrCreateToolTest.java | 2 +-
src/test/java/org/apache/commons/cli/TypeHandlerTest.java | 2 +-
.../java/org/apache/commons/cli/UnrecognizedOptionExceptionTest.java | 2 +-
src/test/java/org/apache/commons/cli/UtilTest.java | 2 +-
src/test/java/org/apache/commons/cli/ValueTest.java | 2 +-
src/test/java/org/apache/commons/cli/ValuesTest.java | 2 +-
src/test/java/org/apache/commons/cli/bug/BugCLI133Test.java | 2 +-
src/test/java/org/apache/commons/cli/bug/BugCLI13Test.java | 2 +-
src/test/java/org/apache/commons/cli/bug/BugCLI148Test.java | 2 +-
src/test/java/org/apache/commons/cli/bug/BugCLI162Test.java | 2 +-
src/test/java/org/apache/commons/cli/bug/BugCLI18Test.java | 2 +-
src/test/java/org/apache/commons/cli/bug/BugCLI252Test.java | 2 +-
src/test/java/org/apache/commons/cli/bug/BugCLI265Test.java | 2 +-
src/test/java/org/apache/commons/cli/bug/BugCLI266Test.java | 2 +-
src/test/java/org/apache/commons/cli/bug/BugCLI312Test.java | 2 +-
src/test/java/org/apache/commons/cli/bug/BugCLI325Test.java | 2 +-
src/test/java/org/apache/commons/cli/bug/BugCLI71Test.java | 2 +-
src/test/java/org/apache/commons/cli/bug/BugsTest.java | 2 +-
src/test/java/org/apache/commons/cli/example/AptHelpAppendableTest.java | 2 +-
.../java/org/apache/commons/cli/example/XhtmlHelpAppendableTest.java | 2 +-
src/test/java/org/apache/commons/cli/help/HelpFormatterTest.java | 2 +-
src/test/java/org/apache/commons/cli/help/OptionFormatterTest.java | 2 +-
src/test/java/org/apache/commons/cli/help/UtilTest.java | 2 +-
41 files changed, 41 insertions(+), 41 deletions(-)
diff --git
a/src/test/java/org/apache/commons/cli/AlreadySelectedExceptionTest.java
b/src/test/java/org/apache/commons/cli/AlreadySelectedExceptionTest.java
index b4ffbb0c..03067e90 100644
--- a/src/test/java/org/apache/commons/cli/AlreadySelectedExceptionTest.java
+++ b/src/test/java/org/apache/commons/cli/AlreadySelectedExceptionTest.java
@@ -26,7 +26,7 @@ import org.junit.jupiter.api.Test;
/**
* Tests {@link AlreadySelectedException}.
*/
-public class AlreadySelectedExceptionTest {
+class AlreadySelectedExceptionTest {
@Test
void testConstructor() {
diff --git a/src/test/java/org/apache/commons/cli/ApplicationTest.java
b/src/test/java/org/apache/commons/cli/ApplicationTest.java
index 37d803cd..59b32917 100644
--- a/src/test/java/org/apache/commons/cli/ApplicationTest.java
+++ b/src/test/java/org/apache/commons/cli/ApplicationTest.java
@@ -39,7 +39,7 @@ import org.junit.jupiter.api.Test;
* </p>
*/
@SuppressWarnings("deprecation") // tests some deprecated classes
-public class ApplicationTest {
+class ApplicationTest {
/**
* Ant test
*/
diff --git a/src/test/java/org/apache/commons/cli/ArgumentIsOptionTest.java
b/src/test/java/org/apache/commons/cli/ArgumentIsOptionTest.java
index b16f9aa0..91ab4d18 100644
--- a/src/test/java/org/apache/commons/cli/ArgumentIsOptionTest.java
+++ b/src/test/java/org/apache/commons/cli/ArgumentIsOptionTest.java
@@ -25,7 +25,7 @@ import org.junit.jupiter.api.BeforeEach;
import org.junit.jupiter.api.Test;
@SuppressWarnings("deprecation") // tests some deprecated classes
-public class ArgumentIsOptionTest {
+class ArgumentIsOptionTest {
private Options options;
private CommandLineParser parser;
diff --git a/src/test/java/org/apache/commons/cli/BasicParserTest.java
b/src/test/java/org/apache/commons/cli/BasicParserTest.java
index 8f85bc63..420b609c 100644
--- a/src/test/java/org/apache/commons/cli/BasicParserTest.java
+++ b/src/test/java/org/apache/commons/cli/BasicParserTest.java
@@ -25,7 +25,7 @@ import org.junit.jupiter.api.Test;
* TODO Needs a rework using JUnit parameterized tests.
*/
@SuppressWarnings("deprecation") // tests some deprecated classes
-public class BasicParserTest extends AbstractParserTestCase {
+class BasicParserTest extends AbstractParserTestCase {
@Override
@BeforeEach
public void setUp() {
diff --git a/src/test/java/org/apache/commons/cli/CommandLineTest.java
b/src/test/java/org/apache/commons/cli/CommandLineTest.java
index 2da8975d..e79bb258 100644
--- a/src/test/java/org/apache/commons/cli/CommandLineTest.java
+++ b/src/test/java/org/apache/commons/cli/CommandLineTest.java
@@ -37,7 +37,7 @@ import org.junit.jupiter.params.ParameterizedTest;
import org.junit.jupiter.params.provider.Arguments;
import org.junit.jupiter.params.provider.MethodSource;
-public class CommandLineTest {
+class CommandLineTest {
private enum Count { ONE, TWO, THREE }
diff --git a/src/test/java/org/apache/commons/cli/DefaultParserTest.java
b/src/test/java/org/apache/commons/cli/DefaultParserTest.java
index 759700ad..02a8f489 100644
--- a/src/test/java/org/apache/commons/cli/DefaultParserTest.java
+++ b/src/test/java/org/apache/commons/cli/DefaultParserTest.java
@@ -35,7 +35,7 @@ import org.junit.jupiter.params.provider.Arguments;
import org.junit.jupiter.params.provider.ArgumentsProvider;
import org.junit.jupiter.params.provider.ArgumentsSource;
-public class DefaultParserTest extends AbstractParserTestCase {
+class DefaultParserTest extends AbstractParserTestCase {
static class ExternalArgumentsProvider implements ArgumentsProvider {
diff --git a/src/test/java/org/apache/commons/cli/DeprecatedAttributesTest.java
b/src/test/java/org/apache/commons/cli/DeprecatedAttributesTest.java
index 29e72352..6fb12869 100644
--- a/src/test/java/org/apache/commons/cli/DeprecatedAttributesTest.java
+++ b/src/test/java/org/apache/commons/cli/DeprecatedAttributesTest.java
@@ -21,7 +21,7 @@ import static org.junit.jupiter.api.Assertions.assertEquals;
import org.junit.jupiter.api.Test;
-public class DeprecatedAttributesTest {
+class DeprecatedAttributesTest {
@Test
void testBuilderNonDefaults() {
diff --git
a/src/test/java/org/apache/commons/cli/DisablePartialMatchingTest.java
b/src/test/java/org/apache/commons/cli/DisablePartialMatchingTest.java
index e0c5f4a5..64f86517 100644
--- a/src/test/java/org/apache/commons/cli/DisablePartialMatchingTest.java
+++ b/src/test/java/org/apache/commons/cli/DisablePartialMatchingTest.java
@@ -22,7 +22,7 @@ import static org.junit.jupiter.api.Assertions.assertTrue;
import org.junit.jupiter.api.Test;
-public class DisablePartialMatchingTest {
+class DisablePartialMatchingTest {
@Test
void testDisablePartialMatching() throws Exception {
final CommandLineParser parser = new DefaultParser(false);
diff --git a/src/test/java/org/apache/commons/cli/GnuParserTest.java
b/src/test/java/org/apache/commons/cli/GnuParserTest.java
index 9ac1a7a3..f928c50b 100644
--- a/src/test/java/org/apache/commons/cli/GnuParserTest.java
+++ b/src/test/java/org/apache/commons/cli/GnuParserTest.java
@@ -25,7 +25,7 @@ import org.junit.jupiter.api.Test;
* TODO Needs a rework using JUnit parameterized tests.
*/
@SuppressWarnings("deprecation") // tests some deprecated classes
-public class GnuParserTest extends AbstractParserTestCase {
+class GnuParserTest extends AbstractParserTestCase {
@Override
@BeforeEach
public void setUp() {
diff --git a/src/test/java/org/apache/commons/cli/HelpFormatterTest.java
b/src/test/java/org/apache/commons/cli/HelpFormatterTest.java
index 279ff688..83e84e6e 100644
--- a/src/test/java/org/apache/commons/cli/HelpFormatterTest.java
+++ b/src/test/java/org/apache/commons/cli/HelpFormatterTest.java
@@ -43,7 +43,7 @@ import org.junit.jupiter.params.provider.MethodSource;
/**
* Test case for the HelpFormatter class.
*/
-public class HelpFormatterTest {
+class HelpFormatterTest {
private static final String EOL = System.lineSeparator();
static Stream<Arguments> deprecatedOptionsProvider() {
diff --git a/src/test/java/org/apache/commons/cli/OptionBuilderTest.java
b/src/test/java/org/apache/commons/cli/OptionBuilderTest.java
index 31fab0d2..39c7e713 100644
--- a/src/test/java/org/apache/commons/cli/OptionBuilderTest.java
+++ b/src/test/java/org/apache/commons/cli/OptionBuilderTest.java
@@ -27,7 +27,7 @@ import org.junit.jupiter.api.Test;
import org.junit.jupiter.api.function.Executable;
@SuppressWarnings("deprecation") // OptionBuilder is marked deprecated
-public class OptionBuilderTest {
+class OptionBuilderTest {
@Test
void testBaseOptionCharOpt() {
final Option base = OptionBuilder.withDescription("option
description").create('o');
diff --git a/src/test/java/org/apache/commons/cli/OptionGroupTest.java
b/src/test/java/org/apache/commons/cli/OptionGroupTest.java
index be99b9aa..79653567 100644
--- a/src/test/java/org/apache/commons/cli/OptionGroupTest.java
+++ b/src/test/java/org/apache/commons/cli/OptionGroupTest.java
@@ -29,7 +29,7 @@ import org.junit.jupiter.api.BeforeEach;
import org.junit.jupiter.api.Test;
@SuppressWarnings("deprecation") // tests some deprecated classes
-public class OptionGroupTest {
+class OptionGroupTest {
private Options options;
private final Parser parser = new PosixParser();
diff --git a/src/test/java/org/apache/commons/cli/OptionTest.java
b/src/test/java/org/apache/commons/cli/OptionTest.java
index 4b134c3d..ec0cb100 100644
--- a/src/test/java/org/apache/commons/cli/OptionTest.java
+++ b/src/test/java/org/apache/commons/cli/OptionTest.java
@@ -33,7 +33,7 @@ import java.io.ObjectOutputStream;
import org.junit.jupiter.api.Test;
-public class OptionTest {
+class OptionTest {
private static final class DefaultOption extends Option {
private static final long serialVersionUID = 1L;
diff --git a/src/test/java/org/apache/commons/cli/OptionValidatorTest.java
b/src/test/java/org/apache/commons/cli/OptionValidatorTest.java
index 1f03840d..6ecbdb85 100644
--- a/src/test/java/org/apache/commons/cli/OptionValidatorTest.java
+++ b/src/test/java/org/apache/commons/cli/OptionValidatorTest.java
@@ -31,7 +31,7 @@ import org.junit.jupiter.params.ParameterizedTest;
import org.junit.jupiter.params.provider.Arguments;
import org.junit.jupiter.params.provider.MethodSource;
-public class OptionValidatorTest {
+class OptionValidatorTest {
/*
* Exemplars of various types of characters
diff --git a/src/test/java/org/apache/commons/cli/OptionsTest.java
b/src/test/java/org/apache/commons/cli/OptionsTest.java
index 9c570100..01ab716d 100644
--- a/src/test/java/org/apache/commons/cli/OptionsTest.java
+++ b/src/test/java/org/apache/commons/cli/OptionsTest.java
@@ -34,7 +34,7 @@ import java.util.Set;
import org.junit.jupiter.api.Test;
@SuppressWarnings("deprecation") // tests some deprecated classes
-public class OptionsTest {
+class OptionsTest {
private void assertToStrings(final Option option) {
// Should never throw.
diff --git a/src/test/java/org/apache/commons/cli/ParseExceptionTest.java
b/src/test/java/org/apache/commons/cli/ParseExceptionTest.java
index da89a268..1f7dfed1 100644
--- a/src/test/java/org/apache/commons/cli/ParseExceptionTest.java
+++ b/src/test/java/org/apache/commons/cli/ParseExceptionTest.java
@@ -26,7 +26,7 @@ import org.junit.jupiter.api.Test;
/**
* Tests {@link ParseException}.
*/
-public class ParseExceptionTest {
+class ParseExceptionTest {
@Test
void testConstructor() {
diff --git a/src/test/java/org/apache/commons/cli/PatternOptionBuilderTest.java
b/src/test/java/org/apache/commons/cli/PatternOptionBuilderTest.java
index de9139c0..dd1013ea 100644
--- a/src/test/java/org/apache/commons/cli/PatternOptionBuilderTest.java
+++ b/src/test/java/org/apache/commons/cli/PatternOptionBuilderTest.java
@@ -40,7 +40,7 @@ import org.junit.jupiter.api.Test;
* Test case for the PatternOptionBuilder class.
*/
@SuppressWarnings("deprecation") // tests some deprecated classes
-public class PatternOptionBuilderTest {
+class PatternOptionBuilderTest {
@Test
void testClassPattern() throws Exception {
diff --git a/src/test/java/org/apache/commons/cli/PosixParserTest.java
b/src/test/java/org/apache/commons/cli/PosixParserTest.java
index 581f09ff..4ce063e4 100644
--- a/src/test/java/org/apache/commons/cli/PosixParserTest.java
+++ b/src/test/java/org/apache/commons/cli/PosixParserTest.java
@@ -26,7 +26,7 @@ import org.junit.jupiter.api.Test;
*
* TODO Needs a rework using JUnit parameterized tests.
*/
-public class PosixParserTest extends AbstractParserTestCase {
+class PosixParserTest extends AbstractParserTestCase {
@Override
@SuppressWarnings("deprecation")
@BeforeEach
diff --git a/src/test/java/org/apache/commons/cli/SolrCreateToolTest.java
b/src/test/java/org/apache/commons/cli/SolrCreateToolTest.java
index cde29101..be2e6524 100644
--- a/src/test/java/org/apache/commons/cli/SolrCreateToolTest.java
+++ b/src/test/java/org/apache/commons/cli/SolrCreateToolTest.java
@@ -27,7 +27,7 @@ import java.util.List;
import org.junit.jupiter.api.Test;
-public class SolrCreateToolTest {
+class SolrCreateToolTest {
private List<Option> getOptions() {
// @formatter:off
diff --git a/src/test/java/org/apache/commons/cli/TypeHandlerTest.java
b/src/test/java/org/apache/commons/cli/TypeHandlerTest.java
index 7ef5d6c3..ad0a5d91 100644
--- a/src/test/java/org/apache/commons/cli/TypeHandlerTest.java
+++ b/src/test/java/org/apache/commons/cli/TypeHandlerTest.java
@@ -47,7 +47,7 @@ import org.junit.jupiter.params.ParameterizedTest;
import org.junit.jupiter.params.provider.Arguments;
import org.junit.jupiter.params.provider.MethodSource;
-public class TypeHandlerTest {
+class TypeHandlerTest {
/** Used for Class and Object creation tests. */
public static class Instantiable {
diff --git
a/src/test/java/org/apache/commons/cli/UnrecognizedOptionExceptionTest.java
b/src/test/java/org/apache/commons/cli/UnrecognizedOptionExceptionTest.java
index 56149e6f..e0555446 100644
--- a/src/test/java/org/apache/commons/cli/UnrecognizedOptionExceptionTest.java
+++ b/src/test/java/org/apache/commons/cli/UnrecognizedOptionExceptionTest.java
@@ -24,7 +24,7 @@ import org.junit.jupiter.api.Test;
/**
* Tests {@link UnrecognizedOptionException}.
*/
-public class UnrecognizedOptionExceptionTest {
+class UnrecognizedOptionExceptionTest {
@Test
void testConstructor() {
diff --git a/src/test/java/org/apache/commons/cli/UtilTest.java
b/src/test/java/org/apache/commons/cli/UtilTest.java
index f2a8ff66..b25ba62d 100644
--- a/src/test/java/org/apache/commons/cli/UtilTest.java
+++ b/src/test/java/org/apache/commons/cli/UtilTest.java
@@ -22,7 +22,7 @@ import static org.junit.jupiter.api.Assertions.assertNull;
import org.junit.jupiter.api.Test;
-public class UtilTest {
+class UtilTest {
@Test
void testStripLeadingAndTrailingQuotes() {
diff --git a/src/test/java/org/apache/commons/cli/ValueTest.java
b/src/test/java/org/apache/commons/cli/ValueTest.java
index 6e2d56b8..2d59913e 100644
--- a/src/test/java/org/apache/commons/cli/ValueTest.java
+++ b/src/test/java/org/apache/commons/cli/ValueTest.java
@@ -30,7 +30,7 @@ import org.junit.jupiter.params.ParameterizedTest;
import org.junit.jupiter.params.provider.MethodSource;
@SuppressWarnings("deprecation") // tests some deprecated classes
-public class ValueTest {
+class ValueTest {
private static final Option NULL_OPTION = null;
private static final String NULL_STRING = null;
diff --git a/src/test/java/org/apache/commons/cli/ValuesTest.java
b/src/test/java/org/apache/commons/cli/ValuesTest.java
index 638d8ae1..9a3b684d 100644
--- a/src/test/java/org/apache/commons/cli/ValuesTest.java
+++ b/src/test/java/org/apache/commons/cli/ValuesTest.java
@@ -26,7 +26,7 @@ import org.junit.jupiter.api.BeforeEach;
import org.junit.jupiter.api.Test;
@SuppressWarnings("deprecation") // tests some deprecated classes
-public class ValuesTest {
+class ValuesTest {
private CommandLine cmd;
@BeforeEach
diff --git a/src/test/java/org/apache/commons/cli/bug/BugCLI133Test.java
b/src/test/java/org/apache/commons/cli/bug/BugCLI133Test.java
index c6bed11a..3d224138 100644
--- a/src/test/java/org/apache/commons/cli/bug/BugCLI133Test.java
+++ b/src/test/java/org/apache/commons/cli/bug/BugCLI133Test.java
@@ -27,7 +27,7 @@ import org.apache.commons.cli.PosixParser;
import org.junit.jupiter.api.Test;
@SuppressWarnings("deprecation") // tests some deprecated classes
-public class BugCLI133Test {
+class BugCLI133Test {
@Test
void testOrder() throws ParseException {
final Option optionA = new Option("a", "first");
diff --git a/src/test/java/org/apache/commons/cli/bug/BugCLI13Test.java
b/src/test/java/org/apache/commons/cli/bug/BugCLI13Test.java
index efa3f290..d790e8ad 100644
--- a/src/test/java/org/apache/commons/cli/bug/BugCLI13Test.java
+++ b/src/test/java/org/apache/commons/cli/bug/BugCLI13Test.java
@@ -29,7 +29,7 @@ import org.apache.commons.cli.PosixParser;
import org.junit.jupiter.api.Test;
@SuppressWarnings("deprecation") // tests some deprecated classes
-public class BugCLI13Test {
+class BugCLI13Test {
@Test
void testCLI13() throws ParseException {
diff --git a/src/test/java/org/apache/commons/cli/bug/BugCLI148Test.java
b/src/test/java/org/apache/commons/cli/bug/BugCLI148Test.java
index 38d10199..17e36650 100644
--- a/src/test/java/org/apache/commons/cli/bug/BugCLI148Test.java
+++ b/src/test/java/org/apache/commons/cli/bug/BugCLI148Test.java
@@ -31,7 +31,7 @@ import org.junit.jupiter.api.Test;
* https://issues.apache.org/jira/browse/CLI-148
*/
@SuppressWarnings("deprecation") // tests some deprecated classes
-public class BugCLI148Test {
+class BugCLI148Test {
private Options options;
@BeforeEach
diff --git a/src/test/java/org/apache/commons/cli/bug/BugCLI162Test.java
b/src/test/java/org/apache/commons/cli/bug/BugCLI162Test.java
index ccba7b7d..a1a0e9f0 100644
--- a/src/test/java/org/apache/commons/cli/bug/BugCLI162Test.java
+++ b/src/test/java/org/apache/commons/cli/bug/BugCLI162Test.java
@@ -31,7 +31,7 @@ import org.apache.commons.cli.Options;
import org.junit.jupiter.api.BeforeEach;
import org.junit.jupiter.api.Test;
-public class BugCLI162Test {
+class BugCLI162Test {
/** Constant for the line separator. */
private static final String CR = System.lineSeparator();
diff --git a/src/test/java/org/apache/commons/cli/bug/BugCLI18Test.java
b/src/test/java/org/apache/commons/cli/bug/BugCLI18Test.java
index b1d6015e..94f0fe81 100644
--- a/src/test/java/org/apache/commons/cli/bug/BugCLI18Test.java
+++ b/src/test/java/org/apache/commons/cli/bug/BugCLI18Test.java
@@ -28,7 +28,7 @@ import org.junit.jupiter.api.Test;
/**
* https://issues.apache.org/jira/browse/CLI-18
*/
-public class BugCLI18Test {
+class BugCLI18Test {
@Test
void testCLI18() {
final Options options = new Options();
diff --git a/src/test/java/org/apache/commons/cli/bug/BugCLI252Test.java
b/src/test/java/org/apache/commons/cli/bug/BugCLI252Test.java
index 8d01338d..e5e07dee 100644
--- a/src/test/java/org/apache/commons/cli/bug/BugCLI252Test.java
+++ b/src/test/java/org/apache/commons/cli/bug/BugCLI252Test.java
@@ -26,7 +26,7 @@ import org.apache.commons.cli.Options;
import org.apache.commons.cli.ParseException;
import org.junit.jupiter.api.Test;
-public class BugCLI252Test {
+class BugCLI252Test {
private Options getOptions() {
final Options options = new Options();
diff --git a/src/test/java/org/apache/commons/cli/bug/BugCLI265Test.java
b/src/test/java/org/apache/commons/cli/bug/BugCLI265Test.java
index 9709232d..3baa3cca 100644
--- a/src/test/java/org/apache/commons/cli/bug/BugCLI265Test.java
+++ b/src/test/java/org/apache/commons/cli/bug/BugCLI265Test.java
@@ -35,7 +35,7 @@ import org.junit.jupiter.api.Test;
* <p>
* The issue is that a short option with an optional value will use whatever
comes next as value.
*/
-public class BugCLI265Test {
+class BugCLI265Test {
private DefaultParser parser;
private Options options;
diff --git a/src/test/java/org/apache/commons/cli/bug/BugCLI266Test.java
b/src/test/java/org/apache/commons/cli/bug/BugCLI266Test.java
index 7b2b55d5..a46cc1ad 100644
--- a/src/test/java/org/apache/commons/cli/bug/BugCLI266Test.java
+++ b/src/test/java/org/apache/commons/cli/bug/BugCLI266Test.java
@@ -31,7 +31,7 @@ import org.apache.commons.cli.OptionGroup;
import org.apache.commons.cli.Options;
import org.junit.jupiter.api.Test;
-public class BugCLI266Test {
+class BugCLI266Test {
private final List<String> insertedOrder = Arrays.asList("h", "d", "f",
"x", "s", "p", "t", "w", "o");
private final List<String> sortOrder = Arrays.asList("d", "f", "h", "o",
"p", "s", "t", "w", "x");
diff --git a/src/test/java/org/apache/commons/cli/bug/BugCLI312Test.java
b/src/test/java/org/apache/commons/cli/bug/BugCLI312Test.java
index aa1a0d0d..5d069647 100644
--- a/src/test/java/org/apache/commons/cli/bug/BugCLI312Test.java
+++ b/src/test/java/org/apache/commons/cli/bug/BugCLI312Test.java
@@ -35,7 +35,7 @@ import org.junit.jupiter.api.Test;
/**
* Demonstrates inconsistencies in parsing Java property-style options.
*/
-public class BugCLI312Test {
+class BugCLI312Test {
@Test
void testNoOptionValues() {
final Option o1 = Option.builder("A").build();
diff --git a/src/test/java/org/apache/commons/cli/bug/BugCLI325Test.java
b/src/test/java/org/apache/commons/cli/bug/BugCLI325Test.java
index 39131a30..35f53858 100644
--- a/src/test/java/org/apache/commons/cli/bug/BugCLI325Test.java
+++ b/src/test/java/org/apache/commons/cli/bug/BugCLI325Test.java
@@ -28,7 +28,7 @@ import org.apache.commons.cli.Options;
import org.apache.commons.cli.ParseException;
import org.junit.jupiter.api.Test;
-public class BugCLI325Test {
+class BugCLI325Test {
@Test
void testCli325() throws ParseException {
diff --git a/src/test/java/org/apache/commons/cli/bug/BugCLI71Test.java
b/src/test/java/org/apache/commons/cli/bug/BugCLI71Test.java
index 1b9300fc..d4edd48e 100644
--- a/src/test/java/org/apache/commons/cli/bug/BugCLI71Test.java
+++ b/src/test/java/org/apache/commons/cli/bug/BugCLI71Test.java
@@ -30,7 +30,7 @@ import org.junit.jupiter.api.BeforeEach;
import org.junit.jupiter.api.Test;
@SuppressWarnings("deprecation") // tests some deprecated classes
-public class BugCLI71Test {
+class BugCLI71Test {
private Options options;
private CommandLineParser parser;
diff --git a/src/test/java/org/apache/commons/cli/bug/BugsTest.java
b/src/test/java/org/apache/commons/cli/bug/BugsTest.java
index dc0e90bd..68a029ff 100644
--- a/src/test/java/org/apache/commons/cli/bug/BugsTest.java
+++ b/src/test/java/org/apache/commons/cli/bug/BugsTest.java
@@ -47,7 +47,7 @@ import org.apache.commons.lang3.ArrayUtils;
import org.junit.jupiter.api.Test;
@SuppressWarnings("deprecation") // tests some deprecated classes
-public class BugsTest {
+class BugsTest {
@Test
void test11456() throws Exception {
// POSIX
diff --git
a/src/test/java/org/apache/commons/cli/example/AptHelpAppendableTest.java
b/src/test/java/org/apache/commons/cli/example/AptHelpAppendableTest.java
index 3efddfb1..b4a9408e 100644
--- a/src/test/java/org/apache/commons/cli/example/AptHelpAppendableTest.java
+++ b/src/test/java/org/apache/commons/cli/example/AptHelpAppendableTest.java
@@ -35,7 +35,7 @@ import org.junit.jupiter.api.Test;
/**
* Tests {@link AptHelpAppendable}.
*/
-public class AptHelpAppendableTest {
+class AptHelpAppendableTest {
private StringBuilder sb;
private AptHelpAppendable underTest;
diff --git
a/src/test/java/org/apache/commons/cli/example/XhtmlHelpAppendableTest.java
b/src/test/java/org/apache/commons/cli/example/XhtmlHelpAppendableTest.java
index 643bead7..db78ee24 100644
--- a/src/test/java/org/apache/commons/cli/example/XhtmlHelpAppendableTest.java
+++ b/src/test/java/org/apache/commons/cli/example/XhtmlHelpAppendableTest.java
@@ -35,7 +35,7 @@ import org.junit.jupiter.api.Test;
/**
* Tests {@link XhtmlHelpAppendable}.
*/
-public class XhtmlHelpAppendableTest {
+class XhtmlHelpAppendableTest {
private StringBuilder sb;
private XhtmlHelpAppendable underTest;
diff --git a/src/test/java/org/apache/commons/cli/help/HelpFormatterTest.java
b/src/test/java/org/apache/commons/cli/help/HelpFormatterTest.java
index accd99fe..f808b477 100644
--- a/src/test/java/org/apache/commons/cli/help/HelpFormatterTest.java
+++ b/src/test/java/org/apache/commons/cli/help/HelpFormatterTest.java
@@ -36,7 +36,7 @@ import org.junit.jupiter.api.Test;
/**
* Tests {@link HelpFormatter}.
*/
-public class HelpFormatterTest {
+class HelpFormatterTest {
private Options getTestGroups() {
// @formatter:off
diff --git a/src/test/java/org/apache/commons/cli/help/OptionFormatterTest.java
b/src/test/java/org/apache/commons/cli/help/OptionFormatterTest.java
index feb13a29..e7262fb5 100644
--- a/src/test/java/org/apache/commons/cli/help/OptionFormatterTest.java
+++ b/src/test/java/org/apache/commons/cli/help/OptionFormatterTest.java
@@ -34,7 +34,7 @@ import org.junit.jupiter.params.provider.MethodSource;
/**
* Tests {@link OptionFormatter}.
*/
-public class OptionFormatterTest {
+class OptionFormatterTest {
public static Stream<Arguments> deprecatedAttributesData() {
final List<Arguments> lst = new ArrayList<>();
diff --git a/src/test/java/org/apache/commons/cli/help/UtilTest.java
b/src/test/java/org/apache/commons/cli/help/UtilTest.java
index e736b6fc..01914391 100644
--- a/src/test/java/org/apache/commons/cli/help/UtilTest.java
+++ b/src/test/java/org/apache/commons/cli/help/UtilTest.java
@@ -31,7 +31,7 @@ import org.junit.jupiter.params.ParameterizedTest;
import org.junit.jupiter.params.provider.Arguments;
import org.junit.jupiter.params.provider.MethodSource;
-public class UtilTest {
+class UtilTest {
public static Stream<Arguments> charArgs() {
final List<Arguments> lst = new ArrayList<>();
