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-digester.git


The following commit(s) were added to refs/heads/master by this push:
     new f835c33f Use JUnit 5 convention for test method visibility
f835c33f is described below

commit f835c33f627fd75a8be5360569f1663cc6635a94
Author: Gary D. Gregory <garydgreg...@gmail.com>
AuthorDate: Sat Jun 7 15:05:44 2025 -0400

    Use JUnit 5 convention for test method visibility
---
 .../commons/digester3/AsyncReaderTestCase.java     | 12 +++----
 .../digester3/BeanPropertySetterRuleTestCase.java  | 12 +++----
 .../commons/digester3/CallMethodRuleTestCase.java  | 30 ++++++++--------
 .../commons/digester3/DTDValidationTestCase.java   |  8 ++---
 .../commons/digester3/Digester133TestCase.java     |  2 +-
 .../commons/digester3/Digester153TestCase.java     | 14 ++++----
 .../commons/digester3/Digester162TestCase.java     |  2 +-
 .../commons/digester3/Digester171TestCase.java     |  4 +--
 .../apache/commons/digester3/DigesterTestCase.java | 40 +++++++++++-----------
 .../apache/commons/digester3/ErrorHandlerTest.java |  6 ++--
 .../digester3/ExtendedBaseRulesTestCase.java       | 16 ++++-----
 .../commons/digester3/LocationTrackerTestCase.java |  2 +-
 .../digester3/NamespaceSnapshotTestCase.java       |  2 +-
 .../commons/digester3/NodeCreateRuleTestCase.java  | 38 ++++++++++----------
 .../commons/digester3/ObjectParamRuleTestCase.java |  2 +-
 .../OverlappingCallMethodRuleTestCase.java         | 12 +++----
 .../commons/digester3/RegexRulesTestCase.java      | 10 +++---
 .../org/apache/commons/digester3/RuleTestCase.java | 30 ++++++++--------
 .../commons/digester3/RulesBaseTestCase.java       | 10 +++---
 .../digester3/SetNestedPropertiesRuleTestCase.java | 18 +++++-----
 .../digester3/SetPropertiesRuleTestCase.java       |  8 ++---
 .../commons/digester3/SetPropertyRuleTestCase.java |  6 ++--
 .../commons/digester3/TestEntityResolution.java    |  4 +--
 .../commons/digester3/TestFactoryCreate.java       |  4 +--
 .../org/apache/commons/digester3/URLTestCase.java  |  4 +--
 .../WithDefaultsRulesWrapperTestCase.java          |  6 ++--
 .../apache/commons/digester3/XIncludeTestCase.java |  2 +-
 .../commons/digester3/XMLSchemaTestCase.java       |  4 +--
 .../addressbook/AddressBookTestCase.java           |  2 +-
 .../annotations/catalog/CatalogTestCase.java       |  2 +-
 .../annotations/employee/EmployeeTestCase.java     |  2 +-
 .../annotations/person/PersonTestCase.java         |  2 +-
 .../digester3/annotations/rss/RssTestCase.java     |  2 +-
 .../servletbean/ServletBeanTestCase.java           |  2 +-
 .../binder/BinderClassLoaderTestCase.java          | 22 ++++++------
 .../digester3/binder/Digester163TestCase.java      |  4 +--
 .../digester3/binder/DigesterLoaderTestCase.java   |  6 ++--
 .../plugins/TestConfigurablePluginAttributes.java  |  6 ++--
 .../commons/digester3/plugins/TestDeclaration.java |  2 +-
 .../digester3/plugins/TestDefaultPlugin.java       |  6 ++--
 .../commons/digester3/plugins/TestDelegate.java    |  2 +-
 .../commons/digester3/plugins/TestInline.java      |  4 +--
 .../commons/digester3/plugins/TestLocalRules.java  |  4 +--
 .../commons/digester3/plugins/TestRecursion.java   |  2 +-
 .../commons/digester3/plugins/TestRuleInfo.java    |  6 ++--
 .../commons/digester3/plugins/TestXmlRuleInfo.java |  6 ++--
 .../substitution/CompoundSubstitutorTestCase.java  |  4 +--
 .../substitution/VariableExpansionTestCase.java    | 14 ++++----
 .../xmlrules/BeanPropertySetterRuleTestCase.java   |  2 +-
 .../xmlrules/DigesterPatternStackTest.java         | 14 ++++----
 .../digester3/xmlrules/FromXmlRuleSetTest.java     | 20 +++++------
 .../commons/digester3/xmlrules/IncludeTest.java    |  6 ++--
 .../xmlrules/SetNamespaceURITestCase.java          |  2 +-
 53 files changed, 226 insertions(+), 226 deletions(-)

diff --git 
a/commons-digester3-core/src/test/java/org/apache/commons/digester3/AsyncReaderTestCase.java
 
b/commons-digester3-core/src/test/java/org/apache/commons/digester3/AsyncReaderTestCase.java
index abe25b7a..6812c43e 100644
--- 
a/commons-digester3-core/src/test/java/org/apache/commons/digester3/AsyncReaderTestCase.java
+++ 
b/commons-digester3-core/src/test/java/org/apache/commons/digester3/AsyncReaderTestCase.java
@@ -64,7 +64,7 @@ public final class AsyncReaderTestCase
     }
 
     @Test
-    public void testParseFromClasspathURL()
+    void testParseFromClasspathURL()
         throws Exception
     {
         final Future<Employee> future = digester.asyncParse( 
getClass().getResource( "Test9.xml" ) );
@@ -72,7 +72,7 @@ public final class AsyncReaderTestCase
     }
 
     @Test
-    public void testParseFromFile()
+    void testParseFromFile()
         throws Exception
     {
         final Future<Employee> future = digester.asyncParse( new File( 
getProperty( "user.dir" ),
@@ -81,7 +81,7 @@ public final class AsyncReaderTestCase
     }
 
     @Test
-    public void testParseFromInputSource()
+    void testParseFromInputSource()
         throws Exception
     {
         final Future<Employee> future =
@@ -90,7 +90,7 @@ public final class AsyncReaderTestCase
     }
 
     @Test
-    public void testParseFromInputStream()
+    void testParseFromInputStream()
         throws Exception
     {
         final Future<Employee> future = digester.asyncParse( 
getClass().getResource( "Test9.xml" ).openStream() );
@@ -98,7 +98,7 @@ public final class AsyncReaderTestCase
     }
 
     @Test
-    public void testParseFromReader()
+    void testParseFromReader()
         throws Exception
     {
         final Future<Employee> future =
@@ -107,7 +107,7 @@ public final class AsyncReaderTestCase
     }
 
     @Test
-    public void testParseFromUri()
+    void testParseFromUri()
         throws Exception
     {
         final Future<Employee> future = digester.asyncParse( 
getClass().getResource( "Test9.xml" ).toExternalForm() );
diff --git 
a/commons-digester3-core/src/test/java/org/apache/commons/digester3/BeanPropertySetterRuleTestCase.java
 
b/commons-digester3-core/src/test/java/org/apache/commons/digester3/BeanPropertySetterRuleTestCase.java
index cb103c74..fa03ec1c 100644
--- 
a/commons-digester3-core/src/test/java/org/apache/commons/digester3/BeanPropertySetterRuleTestCase.java
+++ 
b/commons-digester3-core/src/test/java/org/apache/commons/digester3/BeanPropertySetterRuleTestCase.java
@@ -53,7 +53,7 @@ public class BeanPropertySetterRuleTestCase
      * Test that you can successfully automatically set properties.
      */
     @Test
-    public void testAutomaticallySetProperties()
+    void testAutomaticallySetProperties()
         throws SAXException, IOException
     {
         final Digester digester = newLoader(new AbstractRulesModule()
@@ -84,7 +84,7 @@ public class BeanPropertySetterRuleTestCase
      * functioning correctly.
      */
     @Test
-    public void testDigesterBodyTextStack()
+    void testDigesterBodyTextStack()
         throws SAXException, IOException
     {
         final List<Rule> callOrder = new ArrayList<>();
@@ -120,7 +120,7 @@ public class BeanPropertySetterRuleTestCase
      * properly enforced.
      */
     @Test
-    public void testDigesterRuleCallOrder()
+    void testDigesterRuleCallOrder()
         throws SAXException, IOException
     {
 
@@ -172,7 +172,7 @@ public class BeanPropertySetterRuleTestCase
     }
 
     @Test
-    public void testExtractPropertyNameFromAttribute() throws Exception
+    void testExtractPropertyNameFromAttribute() throws Exception
     {
         final Employee expected = new Employee( "John", "Doe" );
 
@@ -198,7 +198,7 @@ public class BeanPropertySetterRuleTestCase
      * Test that you can successfully set a given property
      */
     @Test
-    public void testSetGivenProperty()
+    void testSetGivenProperty()
         throws SAXException, IOException
     {
         final Digester digester = newLoader(new AbstractRulesModule()
@@ -237,7 +237,7 @@ public class BeanPropertySetterRuleTestCase
      * Test that trying to set an unknown property throws an exception.
      */
     @Test
-    public void testSetUnknownProperty()
+    void testSetUnknownProperty()
     {
         final Digester digester = newLoader(new AbstractRulesModule()
         {
diff --git 
a/commons-digester3-core/src/test/java/org/apache/commons/digester3/CallMethodRuleTestCase.java
 
b/commons-digester3-core/src/test/java/org/apache/commons/digester3/CallMethodRuleTestCase.java
index 0a01a5ce..31d0eef9 100644
--- 
a/commons-digester3-core/src/test/java/org/apache/commons/digester3/CallMethodRuleTestCase.java
+++ 
b/commons-digester3-core/src/test/java/org/apache/commons/digester3/CallMethodRuleTestCase.java
@@ -64,7 +64,7 @@ public class CallMethodRuleTestCase
      * several rule syntaxes.
      */
     @Test
-    public void testBasic() throws Exception
+    void testBasic() throws Exception
     {
         final Digester digester = newLoader( new AbstractRulesModule()
         {
@@ -97,7 +97,7 @@ public class CallMethodRuleTestCase
      * Test invoking an object which does not exist on the stack.
      */
     @Test
-    public void testCallInvalidTarget()
+    void testCallInvalidTarget()
     {
 
         final Digester digester = new Digester();
@@ -116,7 +116,7 @@ public class CallMethodRuleTestCase
      * Test method calls with the CallMethodRule reading from the element 
body, with no CallParamMethod rules added.
      */
     @Test
-    public void testCallMethodOnly()
+    void testCallMethodOnly()
         throws Exception
     {
         final Digester digester = newLoader( new AbstractRulesModule()
@@ -145,7 +145,7 @@ public class CallMethodRuleTestCase
      * Test invoking an object which is at top-1 on the stack, like 
SetNextRule does...
      */
     @Test
-    public void testCallNext()
+    void testCallNext()
         throws Exception
     {
 
@@ -176,7 +176,7 @@ public class CallMethodRuleTestCase
      * Test invoking an object which is at the root of the stack, like SetRoot 
does...
      */
     @Test
-    public void testCallRoot()
+    void testCallRoot()
         throws Exception
     {
 
@@ -204,7 +204,7 @@ public class CallMethodRuleTestCase
     }
 
     @Test
-    public void testFromStack()
+    void testFromStack()
         throws Exception
     {
 
@@ -266,7 +266,7 @@ public class CallMethodRuleTestCase
     }
 
     @Test
-    public void testNestedBody()
+    void testNestedBody()
         throws Exception
     {
 
@@ -325,7 +325,7 @@ public class CallMethodRuleTestCase
      * CallMethodRule triggered on.
      */
     @Test
-    public void testOrderNestedPartA()
+    void testOrderNestedPartA()
         throws Exception
     {
         final Digester digester = newLoader( new AbstractRulesModule()
@@ -370,7 +370,7 @@ public class CallMethodRuleTestCase
      * on this call order so this test is present to ensure that no-one 
changes this behavior.
      */
     @Test
-    public void testOrderNestedPartB()
+    void testOrderNestedPartB()
         throws Exception
     {
         final Digester digester = newLoader( new AbstractRulesModule()
@@ -401,7 +401,7 @@ public class CallMethodRuleTestCase
      * This tests the call methods params enhancement that provides for more 
complex stack-based calls.
      */
     @Test
-    public void testParamsFromStack() throws Exception
+    void testParamsFromStack() throws Exception
     {
         final Digester digester = newLoader( new AbstractRulesModule()
         {
@@ -439,7 +439,7 @@ public class CallMethodRuleTestCase
 
     /** Test for the PathCallParamRule */
     @Test
-    public void testPathCallParam()
+    void testPathCallParam()
         throws Exception
     {
         final String xml =
@@ -465,7 +465,7 @@ public class CallMethodRuleTestCase
     }
 
     @Test
-    public void testPrimitiveReading()
+    void testPrimitiveReading()
         throws Exception
     {
         final StringReader reader =
@@ -518,7 +518,7 @@ public class CallMethodRuleTestCase
     }
 
     @Test
-    public void testProcessingHook()
+    void testProcessingHook()
         throws Exception
     {
 
@@ -568,7 +568,7 @@ public class CallMethodRuleTestCase
      * float should all be acceptable as parameter types.
      */
     @Test
-    public void testSettingProperties() throws Exception
+    void testSettingProperties() throws Exception
     {
         Digester digester = newLoader( new AbstractRulesModule()
         {
@@ -649,7 +649,7 @@ public class CallMethodRuleTestCase
     }
 
     @Test
-    public void testTwoCalls()
+    void testTwoCalls()
         throws Exception
     {
 
diff --git 
a/commons-digester3-core/src/test/java/org/apache/commons/digester3/DTDValidationTestCase.java
 
b/commons-digester3-core/src/test/java/org/apache/commons/digester3/DTDValidationTestCase.java
index 4f7196ac..66bfec20 100644
--- 
a/commons-digester3-core/src/test/java/org/apache/commons/digester3/DTDValidationTestCase.java
+++ 
b/commons-digester3-core/src/test/java/org/apache/commons/digester3/DTDValidationTestCase.java
@@ -34,7 +34,7 @@ public class DTDValidationTestCase
 {
 
     @Test
-    public void testDigesterDTDError()
+    void testDigesterDTDError()
     {
         Digester digester = newLoader( new AbstractRulesModule() {
 
@@ -76,7 +76,7 @@ public class DTDValidationTestCase
     }
 
     @Test
-    public void testDigesterLoaderFeatureDisabled()
+    void testDigesterLoaderFeatureDisabled()
         throws Exception
     {
        newLoader( new AbstractRulesModule()
@@ -98,7 +98,7 @@ public class DTDValidationTestCase
     }
 
     @Test
-    public void testDigesterNoDTDValidation()
+    void testDigesterNoDTDValidation()
         throws Exception
     {
         newLoader( new AbstractRulesModule()
@@ -117,7 +117,7 @@ public class DTDValidationTestCase
     }
 
     @Test
-    public void testDigesterValidation()
+    void testDigesterValidation()
         throws Exception
     {
         newLoader( new AbstractRulesModule()
diff --git 
a/commons-digester3-core/src/test/java/org/apache/commons/digester3/Digester133TestCase.java
 
b/commons-digester3-core/src/test/java/org/apache/commons/digester3/Digester133TestCase.java
index 7eadc919..c9a3c3ba 100644
--- 
a/commons-digester3-core/src/test/java/org/apache/commons/digester3/Digester133TestCase.java
+++ 
b/commons-digester3-core/src/test/java/org/apache/commons/digester3/Digester133TestCase.java
@@ -92,7 +92,7 @@ public final class Digester133TestCase
     }
 
     @Test
-    public void testDigester()
+    void testDigester()
         throws IOException, SAXException
     {
         final PropertyUtilsBean propertyUtils = new MyPropertyUtilsBean();
diff --git 
a/commons-digester3-core/src/test/java/org/apache/commons/digester3/Digester153TestCase.java
 
b/commons-digester3-core/src/test/java/org/apache/commons/digester3/Digester153TestCase.java
index 14108edf..293a9f8d 100644
--- 
a/commons-digester3-core/src/test/java/org/apache/commons/digester3/Digester153TestCase.java
+++ 
b/commons-digester3-core/src/test/java/org/apache/commons/digester3/Digester153TestCase.java
@@ -62,7 +62,7 @@ public final class Digester153TestCase
     }
 
     @Test
-    public void testBasicConstructor()
+    void testBasicConstructor()
         throws Exception
     {
         final ObjectCreateRule createRule = new ObjectCreateRule( 
TestBean.class );
@@ -86,7 +86,7 @@ public final class Digester153TestCase
     }
 
     @Test
-    public void testBasicConstructorViaAnnotations()
+    void testBasicConstructorViaAnnotations()
         throws Exception
     {
         successfulConstructor( new FromAnnotationsRuleModule()
@@ -102,7 +102,7 @@ public final class Digester153TestCase
     }
 
     @Test
-    public void testBasicConstructorViaBinder()
+    void testBasicConstructorViaBinder()
         throws Exception
     {
         successfulConstructor( new AbstractRulesModule()
@@ -123,7 +123,7 @@ public final class Digester153TestCase
     }
 
     @Test
-    public void testBasicConstructorViaXML()
+    void testBasicConstructorViaXML()
         throws Exception
     {
         successfulConstructor( new FromXmlRulesModule()
@@ -139,7 +139,7 @@ public final class Digester153TestCase
     }
 
     @Test
-    public void testBasicConstructorWithValuesNotFound()
+    void testBasicConstructorWithValuesNotFound()
         throws Exception
     {
         final ObjectCreateRule createRule = new ObjectCreateRule( 
TestBean.class );
@@ -157,7 +157,7 @@ public final class Digester153TestCase
     }
 
     @Test
-    public void testConstructorWithAttributeAndElement()
+    void testConstructorWithAttributeAndElement()
         throws Exception
     {
         final ObjectCreateRule createRule = new ObjectCreateRule( 
TestBean.class );
@@ -184,7 +184,7 @@ public final class Digester153TestCase
     }
 
     @Test
-    public void testConstructorWithClassDefinedInAttribute()
+    void testConstructorWithClassDefinedInAttribute()
         throws Exception
     {
         final ObjectCreateRule createRule = new ObjectCreateRule( null, "type" 
);
diff --git 
a/commons-digester3-core/src/test/java/org/apache/commons/digester3/Digester162TestCase.java
 
b/commons-digester3-core/src/test/java/org/apache/commons/digester3/Digester162TestCase.java
index 07fa0126..785e3149 100644
--- 
a/commons-digester3-core/src/test/java/org/apache/commons/digester3/Digester162TestCase.java
+++ 
b/commons-digester3-core/src/test/java/org/apache/commons/digester3/Digester162TestCase.java
@@ -30,7 +30,7 @@ public final class Digester162TestCase
 {
 
     @Test
-    public void 
testAllowCreateObjectsWhichTypesAreSpecifiedInAttributeNameOnly()
+    void testAllowCreateObjectsWhichTypesAreSpecifiedInAttributeNameOnly()
         throws Exception
     {
         final Digester digester = newLoader( new AbstractRulesModule()
diff --git 
a/commons-digester3-core/src/test/java/org/apache/commons/digester3/Digester171TestCase.java
 
b/commons-digester3-core/src/test/java/org/apache/commons/digester3/Digester171TestCase.java
index 63727fa3..f1c2071b 100644
--- 
a/commons-digester3-core/src/test/java/org/apache/commons/digester3/Digester171TestCase.java
+++ 
b/commons-digester3-core/src/test/java/org/apache/commons/digester3/Digester171TestCase.java
@@ -33,7 +33,7 @@ public class Digester171TestCase
 {
 
     @Test
-    public void testDefaultThrowingErrorHandler()
+    void testDefaultThrowingErrorHandler()
         throws Exception
     {
         final ErrorHandler customErrorHandler = new 
DefaultThrowingErrorHandler();
@@ -55,7 +55,7 @@ public class Digester171TestCase
     }
 
     @Test
-    public void testNoErrorHandler()
+    void testNoErrorHandler()
         throws Exception
     {
         newLoader( new AbstractRulesModule()
diff --git 
a/commons-digester3-core/src/test/java/org/apache/commons/digester3/DigesterTestCase.java
 
b/commons-digester3-core/src/test/java/org/apache/commons/digester3/DigesterTestCase.java
index 2ed55658..bbec7f38 100644
--- 
a/commons-digester3-core/src/test/java/org/apache/commons/digester3/DigesterTestCase.java
+++ 
b/commons-digester3-core/src/test/java/org/apache/commons/digester3/DigesterTestCase.java
@@ -125,7 +125,7 @@ public class DigesterTestCase
     }
 
     @Test
-    public void testBasicSubstitution()
+    void testBasicSubstitution()
         throws Exception
     {
         final class TestSubRule
@@ -194,7 +194,7 @@ public class DigesterTestCase
      * Test the Digester.getRoot method.
      */
     @Test
-    public void testGetRoot()
+    void testGetRoot()
         throws Exception
     {
         final Digester digester = new Digester();
@@ -212,7 +212,7 @@ public class DigesterTestCase
 
     /** Tests that values are stored independently */
     @Test
-    public void testNamedIndependence()
+    void testNamedIndependence()
     {
         final String testStackOneName = 
"org.apache.commons.digester3.tests.testNamedIndependenceOne";
         final String testStackTwoName = 
"org.apache.commons.digester3.tests.testNamedIndependenceTwo";
@@ -225,7 +225,7 @@ public class DigesterTestCase
 
     /** Tests for isEmpty */
     @Test
-    public void testNamedStackIsEmpty()
+    void testNamedStackIsEmpty()
     {
         final String testStackName = 
"org.apache.commons.digester3.tests.testNamedStackIsEmpty";
         final Digester digester = new Digester();
@@ -243,7 +243,7 @@ public class DigesterTestCase
 
     /** Tests the push-peek-pop cycle for a named stack */
     @Test
-    public void testNamedStackPushPeekPop()
+    void testNamedStackPushPeekPop()
     {
         final BigDecimal archimedesAveragePi = new BigDecimal( "3.1418" );
         final String testStackName = 
"org.apache.commons.digester3.tests.testNamedStackPushPeekPop";
@@ -274,43 +274,43 @@ public class DigesterTestCase
      * Test {@code null} parsing. (should lead to {@code 
IllegalArgumentException}s)
      */
     @Test
-    public void testNullFileParse()
+    void testNullFileParse()
     {
         assertThrows( IllegalArgumentException.class, () -> digester.parse( ( 
File ) null ) );
     }
 
     @Test
-    public void testNullInputSourceParse()
+    void testNullInputSourceParse()
     {
         assertThrows( IllegalArgumentException.class, () -> digester.parse( ( 
InputSource ) null ) );
     }
 
     @Test
-    public void testNullInputStreamParse()
+    void testNullInputStreamParse()
     {
         assertThrows( IllegalArgumentException.class, () -> digester.parse( ( 
InputStream ) null ) );
     }
 
     @Test
-    public void testNullReaderParse()
+    void testNullReaderParse()
     {
         assertThrows( IllegalArgumentException.class, () -> digester.parse( ( 
Reader ) null ) );
     }
 
     @Test
-    public void testNullStringParse()
+    void testNullStringParse()
     {
         assertThrows( IllegalArgumentException.class, () -> digester.parse( ( 
String ) null ) );
     }
 
     @Test
-    public void testNullURLParse()
+    void testNullURLParse()
     {
         assertThrows( IllegalArgumentException.class, () -> digester.parse( ( 
URL ) null ) );
     }
 
     @Test
-    public void testOnceAndOnceOnly()
+    void testOnceAndOnceOnly()
         throws Exception
     {
 
@@ -340,7 +340,7 @@ public class DigesterTestCase
 
     /** Tests popping named stack not yet pushed */
     @Test
-    public void testPopNamedStackNotPushed()
+    void testPopNamedStackNotPushed()
     {
         String testStackName = 
"org.apache.commons.digester3.tests.testPopNamedStackNotPushed";
         Digester digester = new Digester();
@@ -349,7 +349,7 @@ public class DigesterTestCase
 
     /** Tests peeking named stack not yet pushed */
     @Test
-    public void testPeekNamedStackNotPushed()
+    void testPeekNamedStackNotPushed()
     {
         String testStackName = 
"org.apache.commons.digester3.tests.testPopNamedStackNotPushed";
         Digester digester = new Digester();
@@ -360,7 +360,7 @@ public class DigesterTestCase
      * Test the basic property getters and setters.
      */
     @Test
-    public void testProperties()
+    void testProperties()
     {
 
         assertNull( digester.getErrorHandler(), "Initial error handler is 
null" );
@@ -387,7 +387,7 @@ public class DigesterTestCase
      * Test registration of URLs for specified public identifiers.
      */
     @Test
-    public void testRegistrations()
+    void testRegistrations()
     {
 
         Map<String, URL> map = digester.getRegistrations();
@@ -430,7 +430,7 @@ public class DigesterTestCase
      * Basic test for rule creation and matching.
      */
     @Test
-    public void testRules()
+    void testRules()
     {
 
         assertEquals( 0, digester.getRules().match( null, "a", null, null 
).size(), "Initial rules list is empty" );
@@ -459,7 +459,7 @@ public class DigesterTestCase
      * </ul>
      */
     @Test
-    public void testRulesBase()
+    void testRulesBase()
     {
 
         assertEquals( 0, digester.getRules().rules().size(), "Initial rules 
list is empty" );
@@ -487,7 +487,7 @@ public class DigesterTestCase
      * Test custom StackAction subclasses.
      */
     @Test
-    public void testStackAction()
+    void testStackAction()
     {
         final TrackingStackAction action = new TrackingStackAction();
 
@@ -557,7 +557,7 @@ public class DigesterTestCase
      * Test the basic stack mechanisms.
      */
     @Test
-    public void testStackMethods()
+    void testStackMethods()
     {
 
         Object value;
diff --git 
a/commons-digester3-core/src/test/java/org/apache/commons/digester3/ErrorHandlerTest.java
 
b/commons-digester3-core/src/test/java/org/apache/commons/digester3/ErrorHandlerTest.java
index 1056d5ce..c261f894 100644
--- 
a/commons-digester3-core/src/test/java/org/apache/commons/digester3/ErrorHandlerTest.java
+++ 
b/commons-digester3-core/src/test/java/org/apache/commons/digester3/ErrorHandlerTest.java
@@ -34,7 +34,7 @@ public class ErrorHandlerTest
 {
 
     @Test
-    public void testCustomErrorHandler()
+    void testCustomErrorHandler()
     {
 
         final ErrorHandler customErrorHandler = new ErrorHandler()
@@ -74,7 +74,7 @@ public class ErrorHandlerTest
     }
 
     @Test
-    public void testCustomErrorHandlerWithStack()
+    void testCustomErrorHandlerWithStack()
     {
 
         final ErrorHandler customErrorHandler = new ErrorHandler()
@@ -114,7 +114,7 @@ public class ErrorHandlerTest
     }
 
     @Test
-    public void testNoCustomErrorHandler()
+    void testNoCustomErrorHandler()
     {
         Digester digester = newLoader( new AbstractRulesModule()
         {
diff --git 
a/commons-digester3-core/src/test/java/org/apache/commons/digester3/ExtendedBaseRulesTestCase.java
 
b/commons-digester3-core/src/test/java/org/apache/commons/digester3/ExtendedBaseRulesTestCase.java
index 8c54e8e4..62e68295 100644
--- 
a/commons-digester3-core/src/test/java/org/apache/commons/digester3/ExtendedBaseRulesTestCase.java
+++ 
b/commons-digester3-core/src/test/java/org/apache/commons/digester3/ExtendedBaseRulesTestCase.java
@@ -47,7 +47,7 @@ public class ExtendedBaseRulesTestCase
     }
 
     @Test
-    public void testAncestorMatch()
+    void testAncestorMatch()
     {
         // test fixed root ancestor
         digester.getRules().clear();
@@ -110,7 +110,7 @@ public class ExtendedBaseRulesTestCase
      * parent has a wildcard prefix. This method tests non-universal wildcards.
      */
     @Test
-    public void testBasicParentMatch()
+    void testBasicParentMatch()
     {
 
         // clear any existing rules
@@ -171,7 +171,7 @@ public class ExtendedBaseRulesTestCase
      * Basic test of universal matching rules. Universal rules act independent.
      */
     @Test
-    public void testBasicUniversal()
+    void testBasicUniversal()
     {
 
         // clear any existing rules
@@ -245,7 +245,7 @@ public class ExtendedBaseRulesTestCase
     }
 
     @Test
-    public void testInstructors()
+    void testInstructors()
     {
         digester.getRules().clear();
 
@@ -259,7 +259,7 @@ public class ExtendedBaseRulesTestCase
     }
 
     @Test
-    public void testLongMatch()
+    void testLongMatch()
     {
 
         digester.getRules().clear();
@@ -283,7 +283,7 @@ public class ExtendedBaseRulesTestCase
     }
 
     @Test
-    public void testMiddleInstructors()
+    void testMiddleInstructors()
     {
         digester.getRules().clear();
 
@@ -299,7 +299,7 @@ public class ExtendedBaseRulesTestCase
      * not matched by something else. This method tests non-universal and 
universal wildcard matches.
      */
     @Test
-    public void testRootTailMatch()
+    void testRootTailMatch()
     {
 
         // clear any existing rules
@@ -345,7 +345,7 @@ public class ExtendedBaseRulesTestCase
      * not matched by something else. This method tests non-universal and 
universal wildcard matches.
      */
     @Test
-    public void testWildCardMatch()
+    void testWildCardMatch()
     {
 
         // clear any existing rules
diff --git 
a/commons-digester3-core/src/test/java/org/apache/commons/digester3/LocationTrackerTestCase.java
 
b/commons-digester3-core/src/test/java/org/apache/commons/digester3/LocationTrackerTestCase.java
index 01f905e8..d9086145 100644
--- 
a/commons-digester3-core/src/test/java/org/apache/commons/digester3/LocationTrackerTestCase.java
+++ 
b/commons-digester3-core/src/test/java/org/apache/commons/digester3/LocationTrackerTestCase.java
@@ -70,7 +70,7 @@ public class LocationTrackerTestCase
     }
 
     @Test
-    public void testAll()
+    void testAll()
         throws Exception
     {
         final String TEST_XML =
diff --git 
a/commons-digester3-core/src/test/java/org/apache/commons/digester3/NamespaceSnapshotTestCase.java
 
b/commons-digester3-core/src/test/java/org/apache/commons/digester3/NamespaceSnapshotTestCase.java
index 0db35c3a..0a7be606 100644
--- 
a/commons-digester3-core/src/test/java/org/apache/commons/digester3/NamespaceSnapshotTestCase.java
+++ 
b/commons-digester3-core/src/test/java/org/apache/commons/digester3/NamespaceSnapshotTestCase.java
@@ -85,7 +85,7 @@ public class NamespaceSnapshotTestCase
      * Namespace snapshot test case.
      */
     @Test
-    public void testNamespaceSnapshots()
+    void testNamespaceSnapshots()
         throws Exception
     {
 
diff --git 
a/commons-digester3-core/src/test/java/org/apache/commons/digester3/NodeCreateRuleTestCase.java
 
b/commons-digester3-core/src/test/java/org/apache/commons/digester3/NodeCreateRuleTestCase.java
index 06dca7b9..37d86a3a 100644
--- 
a/commons-digester3-core/src/test/java/org/apache/commons/digester3/NodeCreateRuleTestCase.java
+++ 
b/commons-digester3-core/src/test/java/org/apache/commons/digester3/NodeCreateRuleTestCase.java
@@ -74,7 +74,7 @@ public class NodeCreateRuleTestCase
      * Tests whether attributes are correctly imported into the fragment, 
using the example in the Test1 XML file.
      */
     @Test
-    public void testAttributes()
+    void testAttributes()
         throws SAXException, IOException
     {
         final Digester digester = newLoader( new AbstractRulesModule()
@@ -119,7 +119,7 @@ public class NodeCreateRuleTestCase
      * Tests simple fragment construction, using the {@link #TEST_XML} XML 
input data.
      */
     @Test
-    public void testDocumentFragment()
+    void testDocumentFragment()
         throws SAXException, IOException
     {
         final Digester digester = newLoader( new AbstractRulesModule()
@@ -172,7 +172,7 @@ public class NodeCreateRuleTestCase
      * Tests simple element construction, using the {@link #TEST_XML} XML 
input data.
      */
     @Test
-    public void testElement()
+    void testElement()
         throws SAXException, IOException
     {
         final Digester digester = newLoader( new AbstractRulesModule()
@@ -203,7 +203,7 @@ public class NodeCreateRuleTestCase
      * one.
      */
     @Test
-    public void testEntityText()
+    void testEntityText()
         throws Exception
     {
         final String TEST_XML2 = "<?xml version='1.0'?><root><alpha>&#65; 
&#65;</alpha></root>";
@@ -234,7 +234,7 @@ public class NodeCreateRuleTestCase
      * Tests whether the created fragment can be imported into an existing 
document.
      */
     @Test
-    public void testImport()
+    void testImport()
         throws SAXException, ParserConfigurationException, IOException
     {
         final Digester digester = newLoader( new AbstractRulesModule()
@@ -264,61 +264,61 @@ public class NodeCreateRuleTestCase
      * Tests simple element construction, using the {@link #TEST_XML} XML 
input data.
      */
     @Test
-    public void testInvalidAttributeNode()
+    void testInvalidAttributeNode()
     {
         assertThrows( IllegalArgumentException.class, () -> new 
NodeCreateRule( Node.ATTRIBUTE_NODE ) );
     }
 
     @Test
-    public void testInvalidCDATANode()
+    void testInvalidCDATANode()
     {
         assertThrows( IllegalArgumentException.class, () -> new 
NodeCreateRule( Node.CDATA_SECTION_NODE ) );
     }
 
     @Test
-    public void testInvalidCommentNode()
+    void testInvalidCommentNode()
     {
         assertThrows( IllegalArgumentException.class, () -> new 
NodeCreateRule( Node.COMMENT_NODE ) );
     }
 
     @Test
-    public void testInvalidDocumentNode()
+    void testInvalidDocumentNode()
     {
         assertThrows( IllegalArgumentException.class, () -> new 
NodeCreateRule( Node.DOCUMENT_NODE ) );
     }
 
     @Test
-    public void testInvalidTypeNode()
+    void testInvalidTypeNode()
     {
         assertThrows( IllegalArgumentException.class, () -> new 
NodeCreateRule( Node.DOCUMENT_TYPE_NODE ) );
     }
 
     @Test
-    public void testInvalidEntityNode()
+    void testInvalidEntityNode()
     {
         assertThrows( IllegalArgumentException.class, () -> new 
NodeCreateRule( Node.ENTITY_NODE ) );
     }
 
     @Test
-    public void testInvalidReferenceNode()
+    void testInvalidReferenceNode()
     {
         assertThrows( IllegalArgumentException.class, () -> new 
NodeCreateRule( Node.ENTITY_REFERENCE_NODE ) );
     }
 
     @Test
-    public void testInvalidNotationNode()
+    void testInvalidNotationNode()
     {
         assertThrows( IllegalArgumentException.class, () -> new 
NodeCreateRule( Node.NOTATION_NODE ) );
     }
 
     @Test
-    public void testInvalidProcessingInstructionNode()
+    void testInvalidProcessingInstructionNode()
     {
         assertThrows( IllegalArgumentException.class, () -> new 
NodeCreateRule( Node.PROCESSING_INSTRUCTION_NODE ) );
     }
 
     @Test
-    public void testInvalidTextNode()
+    void testInvalidTextNode()
     {
         assertThrows( IllegalArgumentException.class, () -> new 
NodeCreateRule( Node.TEXT_NODE ) );
     }
@@ -327,7 +327,7 @@ public class NodeCreateRuleTestCase
      * Tests whether namespaced attributes are handled correctly, using the 
example from the file Test10 XML file.
      */
     @Test
-    public void testNamespacedAttribute()
+    void testNamespacedAttribute()
         throws SAXException, IOException
     {
         final Digester digester = newLoader( new AbstractRulesModule()
@@ -359,7 +359,7 @@ public class NodeCreateRuleTestCase
      * Tests whether namespaces are handled correctly, using the example from 
the file Test3 XML file.
      */
     @Test
-    public void testNamespaces()
+    void testNamespaces()
         throws SAXException, IOException
     {
         final Digester digester = newLoader( new AbstractRulesModule()
@@ -410,7 +410,7 @@ public class NodeCreateRuleTestCase
      * Tests whether control is returned to digester after fragment 
construction.
      */
     @Test
-    public void testNested()
+    void testNested()
         throws SAXException, IOException
     {
         final Digester digester = newLoader( new AbstractRulesModule()
@@ -452,7 +452,7 @@ public class NodeCreateRuleTestCase
      * Tests whether non-namespaced attributes are handled correctly, using 
the example from the file Test11 XML file.
      */
     @Test
-    public void testNonNamespacedAttribute()
+    void testNonNamespacedAttribute()
         throws SAXException, IOException
     {
         final Digester digester = newLoader( new AbstractRulesModule()
diff --git 
a/commons-digester3-core/src/test/java/org/apache/commons/digester3/ObjectParamRuleTestCase.java
 
b/commons-digester3-core/src/test/java/org/apache/commons/digester3/ObjectParamRuleTestCase.java
index 430b3f7f..0a94b0ba 100644
--- 
a/commons-digester3-core/src/test/java/org/apache/commons/digester3/ObjectParamRuleTestCase.java
+++ 
b/commons-digester3-core/src/test/java/org/apache/commons/digester3/ObjectParamRuleTestCase.java
@@ -47,7 +47,7 @@ public class ObjectParamRuleTestCase
      * parameter, provided that either the element or the element + attribute 
has been matched.
      */
     @Test
-    public void testBasic()
+    void testBasic()
         throws SAXException, IOException
     {
         final Digester digester = newLoader( new AbstractRulesModule()
diff --git 
a/commons-digester3-core/src/test/java/org/apache/commons/digester3/OverlappingCallMethodRuleTestCase.java
 
b/commons-digester3-core/src/test/java/org/apache/commons/digester3/OverlappingCallMethodRuleTestCase.java
index b3a9bf02..cc7cdde5 100644
--- 
a/commons-digester3-core/src/test/java/org/apache/commons/digester3/OverlappingCallMethodRuleTestCase.java
+++ 
b/commons-digester3-core/src/test/java/org/apache/commons/digester3/OverlappingCallMethodRuleTestCase.java
@@ -50,7 +50,7 @@ public class OverlappingCallMethodRuleTestCase
     }
 
     @Test
-    public void testItem1() throws Exception
+    void testItem1() throws Exception
     {
         final StringBuilder input = new StringBuilder();
         input.append( "<root>" );
@@ -84,7 +84,7 @@ public class OverlappingCallMethodRuleTestCase
     }
 
     @Test
-    public void testItem2() throws Exception
+    void testItem2() throws Exception
     {
         final StringBuilder input = new StringBuilder();
         input.append( "<root>" );
@@ -118,7 +118,7 @@ public class OverlappingCallMethodRuleTestCase
     }
 
     @Test
-    public void testItem3() throws Exception
+    void testItem3() throws Exception
     {
         final StringBuilder input = new StringBuilder();
         input.append( "<root>" );
@@ -175,7 +175,7 @@ public class OverlappingCallMethodRuleTestCase
      * design.
      */
     @Test
-    public void testItem4() throws Exception
+    void testItem4() throws Exception
     {
         final StringBuilder input = new StringBuilder();
         input.append( "<root>" );
@@ -220,7 +220,7 @@ public class OverlappingCallMethodRuleTestCase
      * begin[1]/begin[2]/body[2]/end[2]/body[1]/end[1]
      */
     @Test
-    public void testWildcard1() throws Exception
+    void testWildcard1() throws Exception
     {
         final StringBuilder input = new StringBuilder();
         input.append( "<box id='A1'>" );
@@ -263,7 +263,7 @@ public class OverlappingCallMethodRuleTestCase
      * begin[1]/begin[2]/body[2]/end[2]/body[1]/end[1]
      */
     @Test
-    public void testWildcard2() throws Exception
+    void testWildcard2() throws Exception
     {
         final StringBuilder input = new StringBuilder();
         input.append( "<box>A1" );
diff --git 
a/commons-digester3-core/src/test/java/org/apache/commons/digester3/RegexRulesTestCase.java
 
b/commons-digester3-core/src/test/java/org/apache/commons/digester3/RegexRulesTestCase.java
index f1227c6f..9c58c5be 100644
--- 
a/commons-digester3-core/src/test/java/org/apache/commons/digester3/RegexRulesTestCase.java
+++ 
b/commons-digester3-core/src/test/java/org/apache/commons/digester3/RegexRulesTestCase.java
@@ -34,7 +34,7 @@ public class RegexRulesTestCase
 
     /** Test rules and clear methods */
     @Test
-    public void testClear()
+    void testClear()
     {
         // set up which should match every rule
         final RegexRules rules = new RegexRules( new RegexMatcher()
@@ -73,7 +73,7 @@ public class RegexRulesTestCase
 
     /** Test regex that matches everything */
     @Test
-    public void testMatchAll()
+    void testMatchAll()
     {
         // set up which should match every rule
         final RegexRules rules = new RegexRules( new RegexMatcher()
@@ -106,7 +106,7 @@ public class RegexRulesTestCase
 
     /** Test a mixed regex - in other words, one that sometimes returns true 
and sometimes false */
     @Test
-    public void testMatchMixed()
+    void testMatchMixed()
     {
         // set up which should match every rule
         final RegexRules rules = new RegexRules( new RegexMatcher()
@@ -134,7 +134,7 @@ public class RegexRulesTestCase
 
     /** Test regex matcher that matches nothing */
     @Test
-    public void testMatchNothing()
+    void testMatchNothing()
     {
         // set up which should match every rule
         final RegexRules rules = new RegexRules( new RegexMatcher()
@@ -160,7 +160,7 @@ public class RegexRulesTestCase
     }
 
     @Test
-    public void testSimpleRegexMatch()
+    void testSimpleRegexMatch()
     {
 
         final SimpleRegexMatcher matcher = new SimpleRegexMatcher();
diff --git 
a/commons-digester3-core/src/test/java/org/apache/commons/digester3/RuleTestCase.java
 
b/commons-digester3-core/src/test/java/org/apache/commons/digester3/RuleTestCase.java
index 9d3417dd..9240da1a 100644
--- 
a/commons-digester3-core/src/test/java/org/apache/commons/digester3/RuleTestCase.java
+++ 
b/commons-digester3-core/src/test/java/org/apache/commons/digester3/RuleTestCase.java
@@ -84,7 +84,7 @@ public class RuleTestCase
      * Test rule addition - this boils down to making sure that digester is 
set properly on rule addition.
      */
     @Test
-    public void testAddRule()
+    void testAddRule()
     {
         final Digester digester = new Digester();
         final TestRule rule = new TestRule( "Test" );
@@ -99,7 +99,7 @@ public class RuleTestCase
      * appropriate Employee object to be returned.
      */
     @Test
-    public void testObjectCreate1() throws Exception
+    void testObjectCreate1() throws Exception
     {
 
         // Configure the digester as required
@@ -121,7 +121,7 @@ public class RuleTestCase
      * well, but will not attempt to add them to the Employee.
      */
     @Test
-    public void testObjectCreate2() throws Exception
+    void testObjectCreate2() throws Exception
     {
 
         // Configure the digester as required
@@ -145,7 +145,7 @@ public class RuleTestCase
      * well, and will add them to the owning Employee.
      */
     @Test
-    public void testObjectCreate3() throws Exception
+    void testObjectCreate3() throws Exception
     {
 
         // Configure the digester as required
@@ -171,7 +171,7 @@ public class RuleTestCase
      * Same as testObjectCreate1(), except use individual call method rules to 
set the properties of the Employee.
      */
     @Test
-    public void testObjectCreate4() throws Exception
+    void testObjectCreate4() throws Exception
     {
 
         // Configure the digester as required
@@ -196,7 +196,7 @@ public class RuleTestCase
      * the body of the element is the argument of the method).
      */
     @Test
-    public void testObjectCreate5() throws Exception
+    void testObjectCreate5() throws Exception
     {
 
         // Configure the digester as required
@@ -218,7 +218,7 @@ public class RuleTestCase
      * identical object instances.
      */
     @Test
-    public void testRepeatedParse() throws Exception
+    void testRepeatedParse() throws Exception
     {
 
         // Configure the digester as required
@@ -251,7 +251,7 @@ public class RuleTestCase
      * well, but will not attempt to add them to the Employee.
      */
     @Test
-    public void testRuleSet1() throws Exception
+    void testRuleSet1() throws Exception
     {
 
         // Configure the digester as required
@@ -273,7 +273,7 @@ public class RuleTestCase
      * Same as {@code testRuleSet1} except using a single namespace.
      */
     @Test
-    public void testRuleSet2() throws Exception
+    void testRuleSet2() throws Exception
     {
 
         // Configure the digester as required
@@ -297,7 +297,7 @@ public class RuleTestCase
      * for address that we should skip.
      */
     @Test
-    public void testRuleSet3() throws Exception
+    void testRuleSet3() throws Exception
     {
 
         // Configure the digester as required
@@ -319,7 +319,7 @@ public class RuleTestCase
     /**
      */
     @Test
-    public void testSetCustomProperties() throws Exception
+    void testSetCustomProperties() throws Exception
     {
 
         final Digester digester = new Digester();
@@ -378,7 +378,7 @@ public class RuleTestCase
     }
 
     @Test
-    public void testSetNext() throws Exception
+    void testSetNext() throws Exception
     {
         final Digester digester = new Digester();
         digester.setRules( new ExtendedBaseRules() );
@@ -410,7 +410,7 @@ public class RuleTestCase
     }
 
     @Test
-    public void testSetTop() throws Exception
+    void testSetTop() throws Exception
     {
         final Digester digester = new Digester();
         digester.setRules( new ExtendedBaseRules() );
@@ -461,7 +461,7 @@ public class RuleTestCase
      * argument form is tested in {@code testSetTopRule2}.
      */
     @Test
-    public void testSetTopRule1() throws Exception
+    void testSetTopRule1() throws Exception
     {
 
         // Configure the digester as required
@@ -482,7 +482,7 @@ public class RuleTestCase
      * Same as {@code testSetTopRule1} except using the three argument form of 
the SetTopRule rule.
      */
     @Test
-    public void testSetTopRule2() throws Exception
+    void testSetTopRule2() throws Exception
     {
 
         // Configure the digester as required
diff --git 
a/commons-digester3-core/src/test/java/org/apache/commons/digester3/RulesBaseTestCase.java
 
b/commons-digester3-core/src/test/java/org/apache/commons/digester3/RulesBaseTestCase.java
index d73bfd12..e55c4ecf 100644
--- 
a/commons-digester3-core/src/test/java/org/apache/commons/digester3/RulesBaseTestCase.java
+++ 
b/commons-digester3-core/src/test/java/org/apache/commons/digester3/RulesBaseTestCase.java
@@ -79,7 +79,7 @@ public class RulesBaseTestCase
      * Test basic matching involving namespaces.
      */
     @Test
-    public void testBasicNamespaceMatching()
+    void testBasicNamespaceMatching()
     {
 
         // clear any existing rules
@@ -128,7 +128,7 @@ public class RulesBaseTestCase
      * Rules must always be returned in the correct order.
      */
     @Test
-    public void testOrdering()
+    void testOrdering()
     {
 
         // clear any existing rules
@@ -160,7 +160,7 @@ public class RulesBaseTestCase
      * Basic test for rule creation and matching.
      */
     @Test
-    public void testRules()
+    void testRules()
     {
 
         // clear any existing rules
@@ -197,7 +197,7 @@ public class RulesBaseTestCase
      * </ul>
      */
     @Test
-    public void testRulesBase()
+    void testRulesBase()
     {
 
         // clear any existing rules
@@ -235,7 +235,7 @@ public class RulesBaseTestCase
 
     /** Tests the behavior when a rule is added with a trailing slash */
     @Test
-    public void testTrailingSlash()
+    void testTrailingSlash()
     {
         // clear any existing rules
         digester.getRules().clear();
diff --git 
a/commons-digester3-core/src/test/java/org/apache/commons/digester3/SetNestedPropertiesRuleTestCase.java
 
b/commons-digester3-core/src/test/java/org/apache/commons/digester3/SetNestedPropertiesRuleTestCase.java
index 4fd827d3..7c4b3c0c 100644
--- 
a/commons-digester3-core/src/test/java/org/apache/commons/digester3/SetNestedPropertiesRuleTestCase.java
+++ 
b/commons-digester3-core/src/test/java/org/apache/commons/digester3/SetNestedPropertiesRuleTestCase.java
@@ -52,7 +52,7 @@ public class SetNestedPropertiesRuleTestCase
      * Test that you can successfully automatically set properties.
      */
     @Test
-    public void testAutomaticallySetProperties() throws Exception
+    void testAutomaticallySetProperties() throws Exception
     {
         final Digester digester = newLoader( new AbstractRulesModule()
         {
@@ -83,7 +83,7 @@ public class SetNestedPropertiesRuleTestCase
      * Test that you can customize the property mappings using the constructor 
which takes arrays-of-strings.
      */
     @Test
-    public void testCustomizedProperties1() throws Exception
+    void testCustomizedProperties1() throws Exception
     {
         final Digester digester = newLoader( new AbstractRulesModule()
         {
@@ -124,7 +124,7 @@ public class SetNestedPropertiesRuleTestCase
      * Test that you can ignore a single input XML element using the 
constructor which takes a single remapping.
      */
     @Test
-    public void testCustomizedProperties2a() throws Exception
+    void testCustomizedProperties2a() throws Exception
     {
         final Digester digester = newLoader( new AbstractRulesModule()
         {
@@ -163,7 +163,7 @@ public class SetNestedPropertiesRuleTestCase
      * Test that you can customize the property mappings using the constructor 
which takes a single remapping.
      */
     @Test
-    public void testCustomizedProperties2b() throws Exception
+    void testCustomizedProperties2b() throws Exception
     {
         final Digester digester = newLoader( new AbstractRulesModule()
         {
@@ -202,7 +202,7 @@ public class SetNestedPropertiesRuleTestCase
      * Test that it is an error when a child element exists but no 
corresponding Java property exists.
      */
     @Test
-    public void testMandatoryProperties()
+    void testMandatoryProperties()
     {
         final Digester digester = newLoader( new AbstractRulesModule()
         {
@@ -232,7 +232,7 @@ public class SetNestedPropertiesRuleTestCase
      * </ul>
      */
     @Test
-    public void testMultiRuleMatch() throws Exception
+    void testMultiRuleMatch() throws Exception
     {
         final Digester digester = newLoader( new AbstractRulesModule()
         {
@@ -281,7 +281,7 @@ public class SetNestedPropertiesRuleTestCase
      * See bugzilla entry 31393.
      */
     @Test
-    public void testRecursiveNestedProperties() throws Exception
+    void testRecursiveNestedProperties() throws Exception
     {
         final Digester digester = newLoader( new AbstractRulesModule()
         {
@@ -310,7 +310,7 @@ public class SetNestedPropertiesRuleTestCase
      * Test that unknown child elements trigger an exception.
      */
     @Test
-    public void testUnknownChildrenCausesException()
+    void testUnknownChildrenCausesException()
     {
         final Digester digester = newLoader( new AbstractRulesModule()
         {
@@ -339,7 +339,7 @@ public class SetNestedPropertiesRuleTestCase
      * Test that unknown child elements are allowed if the appropriate flag is 
set.
      */
     @Test
-    public void testUnknownChildrenExceptionOverride() throws Exception
+    void testUnknownChildrenExceptionOverride() throws Exception
     {
         final Digester digester = newLoader( new AbstractRulesModule()
         {
diff --git 
a/commons-digester3-core/src/test/java/org/apache/commons/digester3/SetPropertiesRuleTestCase.java
 
b/commons-digester3-core/src/test/java/org/apache/commons/digester3/SetPropertiesRuleTestCase.java
index c2d7213b..8c696ad2 100644
--- 
a/commons-digester3-core/src/test/java/org/apache/commons/digester3/SetPropertiesRuleTestCase.java
+++ 
b/commons-digester3-core/src/test/java/org/apache/commons/digester3/SetPropertiesRuleTestCase.java
@@ -61,7 +61,7 @@ public class SetPropertiesRuleTestCase
      * Positive test for SetPropertyRule ignoring missing properties.
      */
     @Test
-    public void testIgnoreMissing()
+    void testIgnoreMissing()
         throws Exception
     {
         final Digester digester = newLoader( new AbstractRulesModule()
@@ -92,7 +92,7 @@ public class SetPropertiesRuleTestCase
      * Negative test for SetPropertyRule ignoring missing properties.
      */
     @Test
-    public void testNegativeNotIgnoreMissing()
+    void testNegativeNotIgnoreMissing()
     {
         final Digester digester = newLoader( new AbstractRulesModule()
         {
@@ -116,7 +116,7 @@ public class SetPropertiesRuleTestCase
      * Positive test for SetPropertiesRule.
      */
     @Test
-    public void testPositive()
+    void testPositive()
         throws Exception
     {
         final Digester digester = newLoader( new AbstractRulesModule()
@@ -147,7 +147,7 @@ public class SetPropertiesRuleTestCase
      * Negative test for SetPropertyRule ignoring missing properties.
      */
     @Test
-    public void testPositiveNotIgnoreMissingWithIgnoreAttributes()
+    void testPositiveNotIgnoreMissingWithIgnoreAttributes()
         throws Exception
     {
         final Digester digester = newLoader( new AbstractRulesModule()
diff --git 
a/commons-digester3-core/src/test/java/org/apache/commons/digester3/SetPropertyRuleTestCase.java
 
b/commons-digester3-core/src/test/java/org/apache/commons/digester3/SetPropertyRuleTestCase.java
index 1496c40c..d7c1e2a4 100644
--- 
a/commons-digester3-core/src/test/java/org/apache/commons/digester3/SetPropertyRuleTestCase.java
+++ 
b/commons-digester3-core/src/test/java/org/apache/commons/digester3/SetPropertyRuleTestCase.java
@@ -98,7 +98,7 @@ public class SetPropertyRuleTestCase
      * Test SetPropertyRule when matched XML element has no attributes. See: 
DIGESTER-114
      */
     @Test
-    public void testElementWithNoAttributes()
+    void testElementWithNoAttributes()
         throws Exception
     {
         final String TEST_XML_3 = "<?xml version='1.0'?><root><set/></root>";
@@ -113,7 +113,7 @@ public class SetPropertyRuleTestCase
      * Negative test for SetPropertyRule.
      */
     @Test
-    public void testNegative()
+    void testNegative()
     {
         // Parse the input (should fail)
         SAXException e = assertThrows( SAXException.class, () -> 
digester.parse( xmlTestReader( TEST_XML_2 ) ) );
@@ -124,7 +124,7 @@ public class SetPropertyRuleTestCase
      * Positive test for SetPropertyRule.
      */
     @Test
-    public void testPositive()
+    void testPositive()
         throws Exception
     {
         // Parse the input
diff --git 
a/commons-digester3-core/src/test/java/org/apache/commons/digester3/TestEntityResolution.java
 
b/commons-digester3-core/src/test/java/org/apache/commons/digester3/TestEntityResolution.java
index f10ced20..37383146 100644
--- 
a/commons-digester3-core/src/test/java/org/apache/commons/digester3/TestEntityResolution.java
+++ 
b/commons-digester3-core/src/test/java/org/apache/commons/digester3/TestEntityResolution.java
@@ -32,7 +32,7 @@ public class TestEntityResolution
 {
 
     @Test
-    public void testDigesterResolveRelative()
+    void testDigesterResolveRelative()
         throws Exception
     {
         final Digester digester = new Digester();
@@ -41,7 +41,7 @@ public class TestEntityResolution
     }
 
     @Test
-    public void testParserResolveRelative()
+    void testParserResolveRelative()
         throws Exception
     {
         final SAXParserFactory factory = SAXParserFactory.newInstance();
diff --git 
a/commons-digester3-core/src/test/java/org/apache/commons/digester3/TestFactoryCreate.java
 
b/commons-digester3-core/src/test/java/org/apache/commons/digester3/TestFactoryCreate.java
index 39cbcf12..2410a6ea 100644
--- 
a/commons-digester3-core/src/test/java/org/apache/commons/digester3/TestFactoryCreate.java
+++ 
b/commons-digester3-core/src/test/java/org/apache/commons/digester3/TestFactoryCreate.java
@@ -38,7 +38,7 @@ public class TestFactoryCreate
 {
     @ParameterizedTest
     @ValueSource( booleans = { true, false } )
-    public void testFactoryCreateRule( boolean ignoreCreateExceptions )
+    void testFactoryCreateRule( boolean ignoreCreateExceptions )
         throws Exception
     {
 
@@ -141,7 +141,7 @@ public class TestFactoryCreate
 
     @ParameterizedTest
     @ValueSource( booleans = { true, false } )
-    public void testPropagateException( boolean ignoreCreateExceptions )
+    void testPropagateException( boolean ignoreCreateExceptions )
     {
 
         // only used with this method
diff --git 
a/commons-digester3-core/src/test/java/org/apache/commons/digester3/URLTestCase.java
 
b/commons-digester3-core/src/test/java/org/apache/commons/digester3/URLTestCase.java
index 73c865ae..61b19a3a 100644
--- 
a/commons-digester3-core/src/test/java/org/apache/commons/digester3/URLTestCase.java
+++ 
b/commons-digester3-core/src/test/java/org/apache/commons/digester3/URLTestCase.java
@@ -87,7 +87,7 @@ public class URLTestCase
 
     // Test a pristine instance
     @Test
-    public void testPristine()
+    void testPristine()
     {
 
         assertNotNull( digester );
@@ -96,7 +96,7 @@ public class URLTestCase
 
     // Test parsing a resource, using a registered DTD, both passed with URLs
     @Test
-    public void testResource()
+    void testResource()
         throws Exception
     {
 
diff --git 
a/commons-digester3-core/src/test/java/org/apache/commons/digester3/WithDefaultsRulesWrapperTestCase.java
 
b/commons-digester3-core/src/test/java/org/apache/commons/digester3/WithDefaultsRulesWrapperTestCase.java
index 1ebd6550..e8698d16 100644
--- 
a/commons-digester3-core/src/test/java/org/apache/commons/digester3/WithDefaultsRulesWrapperTestCase.java
+++ 
b/commons-digester3-core/src/test/java/org/apache/commons/digester3/WithDefaultsRulesWrapperTestCase.java
@@ -32,7 +32,7 @@ public class WithDefaultsRulesWrapperTestCase
 {
 
     @Test
-    public void testClear()
+    void testClear()
     {
         // test clear wrapped
         final WithDefaultsRulesWrapper rules = new WithDefaultsRulesWrapper( 
new RulesBase() );
@@ -61,7 +61,7 @@ public class WithDefaultsRulesWrapperTestCase
     }
 
     @Test
-    public void testMatch()
+    void testMatch()
     {
         // test no defaults
         final WithDefaultsRulesWrapper rules = new WithDefaultsRulesWrapper( 
new RulesBase() );
@@ -84,7 +84,7 @@ public class WithDefaultsRulesWrapperTestCase
     }
 
     @Test
-    public void testRules()
+    void testRules()
     {
         // test rules
         final WithDefaultsRulesWrapper rules = new WithDefaultsRulesWrapper( 
new RulesBase() );
diff --git 
a/commons-digester3-core/src/test/java/org/apache/commons/digester3/XIncludeTestCase.java
 
b/commons-digester3-core/src/test/java/org/apache/commons/digester3/XIncludeTestCase.java
index e5cd5505..a5fcd09e 100644
--- 
a/commons-digester3-core/src/test/java/org/apache/commons/digester3/XIncludeTestCase.java
+++ 
b/commons-digester3-core/src/test/java/org/apache/commons/digester3/XIncludeTestCase.java
@@ -53,7 +53,7 @@ public class XIncludeTestCase
      * Test XInclude.
      */
     @Test
-    public void testXInclude() throws Exception
+    void testXInclude() throws Exception
     {
         final Digester digester = newLoader( new AbstractRulesModule()
         {
diff --git 
a/commons-digester3-core/src/test/java/org/apache/commons/digester3/XMLSchemaTestCase.java
 
b/commons-digester3-core/src/test/java/org/apache/commons/digester3/XMLSchemaTestCase.java
index 1de8eac7..11d49e31 100644
--- 
a/commons-digester3-core/src/test/java/org/apache/commons/digester3/XMLSchemaTestCase.java
+++ 
b/commons-digester3-core/src/test/java/org/apache/commons/digester3/XMLSchemaTestCase.java
@@ -130,7 +130,7 @@ public class XMLSchemaTestCase
     }
 
     @Test
-    public void testBadDocument() throws Exception
+    void testBadDocument() throws Exception
     {
 
         // Listen to validation errors
@@ -147,7 +147,7 @@ public class XMLSchemaTestCase
      * Test XML Schema validation.
      */
     @Test
-    public void testGoodDocument() throws Exception
+    void testGoodDocument() throws Exception
     {
 
         // Listen to validation errors
diff --git 
a/commons-digester3-core/src/test/java/org/apache/commons/digester3/annotations/addressbook/AddressBookTestCase.java
 
b/commons-digester3-core/src/test/java/org/apache/commons/digester3/annotations/addressbook/AddressBookTestCase.java
index ebadfd62..5e6f4e42 100644
--- 
a/commons-digester3-core/src/test/java/org/apache/commons/digester3/annotations/addressbook/AddressBookTestCase.java
+++ 
b/commons-digester3-core/src/test/java/org/apache/commons/digester3/annotations/addressbook/AddressBookTestCase.java
@@ -27,7 +27,7 @@ public final class AddressBookTestCase
 {
 
     @Test
-    public void testAddressBook()
+    void testAddressBook()
         throws Exception
     {
         final AddressBook addressBook = new AddressBook();
diff --git 
a/commons-digester3-core/src/test/java/org/apache/commons/digester3/annotations/catalog/CatalogTestCase.java
 
b/commons-digester3-core/src/test/java/org/apache/commons/digester3/annotations/catalog/CatalogTestCase.java
index 9dd3ec3c..e2bc0853 100644
--- 
a/commons-digester3-core/src/test/java/org/apache/commons/digester3/annotations/catalog/CatalogTestCase.java
+++ 
b/commons-digester3-core/src/test/java/org/apache/commons/digester3/annotations/catalog/CatalogTestCase.java
@@ -49,7 +49,7 @@ public final class CatalogTestCase
     }
 
     @Test
-    public void testCatalog()
+    void testCatalog()
         throws Exception
     {
         final Catalog catalog = new Catalog();
diff --git 
a/commons-digester3-core/src/test/java/org/apache/commons/digester3/annotations/employee/EmployeeTestCase.java
 
b/commons-digester3-core/src/test/java/org/apache/commons/digester3/annotations/employee/EmployeeTestCase.java
index 31b85840..c38b95a0 100644
--- 
a/commons-digester3-core/src/test/java/org/apache/commons/digester3/annotations/employee/EmployeeTestCase.java
+++ 
b/commons-digester3-core/src/test/java/org/apache/commons/digester3/annotations/employee/EmployeeTestCase.java
@@ -49,7 +49,7 @@ public final class EmployeeTestCase
     }
 
     @Test
-    public void testEmployee()
+    void testEmployee()
         throws Exception
     {
         final Employee employee = new Employee();
diff --git 
a/commons-digester3-core/src/test/java/org/apache/commons/digester3/annotations/person/PersonTestCase.java
 
b/commons-digester3-core/src/test/java/org/apache/commons/digester3/annotations/person/PersonTestCase.java
index 173a1270..66c5b733 100644
--- 
a/commons-digester3-core/src/test/java/org/apache/commons/digester3/annotations/person/PersonTestCase.java
+++ 
b/commons-digester3-core/src/test/java/org/apache/commons/digester3/annotations/person/PersonTestCase.java
@@ -27,7 +27,7 @@ public final class PersonTestCase
 {
 
     @Test
-    public void testPerson()
+    void testPerson()
         throws Exception
     {
         final Person person = new Person();
diff --git 
a/commons-digester3-core/src/test/java/org/apache/commons/digester3/annotations/rss/RssTestCase.java
 
b/commons-digester3-core/src/test/java/org/apache/commons/digester3/annotations/rss/RssTestCase.java
index a3d7b2ef..26b3a9f7 100644
--- 
a/commons-digester3-core/src/test/java/org/apache/commons/digester3/annotations/rss/RssTestCase.java
+++ 
b/commons-digester3-core/src/test/java/org/apache/commons/digester3/annotations/rss/RssTestCase.java
@@ -27,7 +27,7 @@ public final class RssTestCase
 {
 
     @Test
-    public void testRss()
+    void testRss()
         throws Exception
     {
         final Channel channel = new Channel();
diff --git 
a/commons-digester3-core/src/test/java/org/apache/commons/digester3/annotations/servletbean/ServletBeanTestCase.java
 
b/commons-digester3-core/src/test/java/org/apache/commons/digester3/annotations/servletbean/ServletBeanTestCase.java
index c545ac84..315ae102 100644
--- 
a/commons-digester3-core/src/test/java/org/apache/commons/digester3/annotations/servletbean/ServletBeanTestCase.java
+++ 
b/commons-digester3-core/src/test/java/org/apache/commons/digester3/annotations/servletbean/ServletBeanTestCase.java
@@ -27,7 +27,7 @@ public final class ServletBeanTestCase
 {
 
     @Test
-    public void testServletBean()
+    void testServletBean()
         throws Exception
     {
         final ServletBean servletBean = new ServletBean();
diff --git 
a/commons-digester3-core/src/test/java/org/apache/commons/digester3/binder/BinderClassLoaderTestCase.java
 
b/commons-digester3-core/src/test/java/org/apache/commons/digester3/binder/BinderClassLoaderTestCase.java
index 5ebf7093..b8afbd9a 100644
--- 
a/commons-digester3-core/src/test/java/org/apache/commons/digester3/binder/BinderClassLoaderTestCase.java
+++ 
b/commons-digester3-core/src/test/java/org/apache/commons/digester3/binder/BinderClassLoaderTestCase.java
@@ -177,7 +177,7 @@ public final class BinderClassLoaderTestCase
     private final BinderClassLoader classLoader = createBinderClassLoader( new 
ExtendedClassLoader() );
 
     @Test
-    public void testGetPrefixedResource()
+    void testGetPrefixedResource()
         throws Exception
     {
         final URL resource = classLoader.getResource( "inmemory:dummyResource" 
);
@@ -191,7 +191,7 @@ public final class BinderClassLoaderTestCase
     }
 
     @Test
-    public void testGetResource()
+    void testGetResource()
     {
         final ClassLoader clToAdapt = new ClassLoader()
         {
@@ -212,28 +212,28 @@ public final class BinderClassLoaderTestCase
     }
 
     @Test
-    public void testLoadBoolean()
+    void testLoadBoolean()
         throws Exception
     {
         typeFound( "boolean", boolean.class );
     }
 
     @Test
-    public void testLoadByte()
+    void testLoadByte()
         throws Exception
     {
         typeFound( "byte", byte.class );
     }
 
     @Test
-    public void testLoadChar()
+    void testLoadChar()
         throws Exception
     {
         typeFound( "char", char.class );
     }
 
     @Test
-    public void testLoadClass()
+    void testLoadClass()
         throws Exception
     {
         final Class<?> dummyClass1 = Dummy.class;
@@ -247,35 +247,35 @@ public final class BinderClassLoaderTestCase
     }
 
     @Test
-    public void testLoadDouble()
+    void testLoadDouble()
         throws Exception
     {
         typeFound( "double", double.class );
     }
 
     @Test
-    public void testLoadFloat()
+    void testLoadFloat()
         throws Exception
     {
         typeFound( "float", float.class );
     }
 
     @Test
-    public void testLoadInt()
+    void testLoadInt()
         throws Exception
     {
         typeFound( "int", int.class );
     }
 
     @Test
-    public void testLoadLong()
+    void testLoadLong()
         throws Exception
     {
         typeFound( "long", long.class );
     }
 
     @Test
-    public void testLoadShort()
+    void testLoadShort()
         throws Exception
     {
         typeFound( "short", short.class );
diff --git 
a/commons-digester3-core/src/test/java/org/apache/commons/digester3/binder/Digester163TestCase.java
 
b/commons-digester3-core/src/test/java/org/apache/commons/digester3/binder/Digester163TestCase.java
index 03b324fd..a9ad33c5 100644
--- 
a/commons-digester3-core/src/test/java/org/apache/commons/digester3/binder/Digester163TestCase.java
+++ 
b/commons-digester3-core/src/test/java/org/apache/commons/digester3/binder/Digester163TestCase.java
@@ -61,7 +61,7 @@ public class Digester163TestCase
     }
 
     @Test
-    public void test()
+    void test()
         throws InterruptedException
     {
         final ThreadPoolExecutor executor = new ThreadPoolExecutor( 
MAX_THREADS,
@@ -116,7 +116,7 @@ public class Digester163TestCase
     }
 
     @Test
-    public void testSingle()
+    void testSingle()
         throws IOException, SAXException
     {
         final Digester dig = loader.newDigester();
diff --git 
a/commons-digester3-core/src/test/java/org/apache/commons/digester3/binder/DigesterLoaderTestCase.java
 
b/commons-digester3-core/src/test/java/org/apache/commons/digester3/binder/DigesterLoaderTestCase.java
index 13a5450e..9000ca3f 100644
--- 
a/commons-digester3-core/src/test/java/org/apache/commons/digester3/binder/DigesterLoaderTestCase.java
+++ 
b/commons-digester3-core/src/test/java/org/apache/commons/digester3/binder/DigesterLoaderTestCase.java
@@ -32,7 +32,7 @@ public final class DigesterLoaderTestCase
 {
 
     @Test
-    public void testDigester152()
+    void testDigester152()
     {
         final Locator expected = new Locator()
         {
@@ -86,7 +86,7 @@ public final class DigesterLoaderTestCase
      * DIGESTER-151
      */
     @Test
-    public void testDigester151()
+    void testDigester151()
     {
         final ErrorHandler expected = new ErrorHandler()
         {
@@ -128,7 +128,7 @@ public final class DigesterLoaderTestCase
     }
 
     @Test
-    public void testDigester155()
+    void testDigester155()
     {
         final ClassLoader expected = getClass().getClassLoader();
 
diff --git 
a/commons-digester3-core/src/test/java/org/apache/commons/digester3/plugins/TestConfigurablePluginAttributes.java
 
b/commons-digester3-core/src/test/java/org/apache/commons/digester3/plugins/TestConfigurablePluginAttributes.java
index a2fa4ffa..46ec46ff 100644
--- 
a/commons-digester3-core/src/test/java/org/apache/commons/digester3/plugins/TestConfigurablePluginAttributes.java
+++ 
b/commons-digester3-core/src/test/java/org/apache/commons/digester3/plugins/TestConfigurablePluginAttributes.java
@@ -65,7 +65,7 @@ public class TestConfigurablePluginAttributes
     }
 
     @Test
-    public void testDefaultBehaviour()
+    void testDefaultBehaviour()
         throws Exception
     {
         // tests that by default the attributes used are
@@ -112,7 +112,7 @@ public class TestConfigurablePluginAttributes
     }
 
     @Test
-    public void testGlobalOverride()
+    void testGlobalOverride()
         throws Exception
     {
         // Tests that using setDefaultPluginXXXX overrides behavior for all
@@ -168,7 +168,7 @@ public class TestConfigurablePluginAttributes
     // inner classes used for testing
 
     @Test
-    public void testInstanceOverride()
+    void testInstanceOverride()
         throws Exception
     {
         // Tests that using setPluginXXXX overrides behavior for only
diff --git 
a/commons-digester3-core/src/test/java/org/apache/commons/digester3/plugins/TestDeclaration.java
 
b/commons-digester3-core/src/test/java/org/apache/commons/digester3/plugins/TestDeclaration.java
index 2f506289..aba38daa 100644
--- 
a/commons-digester3-core/src/test/java/org/apache/commons/digester3/plugins/TestDeclaration.java
+++ 
b/commons-digester3-core/src/test/java/org/apache/commons/digester3/plugins/TestDeclaration.java
@@ -33,7 +33,7 @@ public class TestDeclaration
 {
 
     @Test
-    public void testPredeclaration()
+    void testPredeclaration()
         throws Exception
     {
         // * tests that rules can be declared via a PluginDeclarationRule
diff --git 
a/commons-digester3-core/src/test/java/org/apache/commons/digester3/plugins/TestDefaultPlugin.java
 
b/commons-digester3-core/src/test/java/org/apache/commons/digester3/plugins/TestDefaultPlugin.java
index 85b69e32..94513732 100644
--- 
a/commons-digester3-core/src/test/java/org/apache/commons/digester3/plugins/TestDefaultPlugin.java
+++ 
b/commons-digester3-core/src/test/java/org/apache/commons/digester3/plugins/TestDefaultPlugin.java
@@ -32,7 +32,7 @@ public class TestDefaultPlugin
 {
 
     @Test
-    public void testDefaultPlugins1()
+    void testDefaultPlugins1()
         throws Exception
     {
         // * tests that when a PluginCreateRule is defined with a default
@@ -76,7 +76,7 @@ public class TestDefaultPlugin
     }
 
     @Test
-    public void testDefaultPlugins2()
+    void testDefaultPlugins2()
     {
         // * tests that when there is no default plugin, it is an error
         // not to have one of plugin-class or plugin-id specified
@@ -106,7 +106,7 @@ public class TestDefaultPlugin
     }
 
     @Test
-    public void testDefaultPlugins3()
+    void testDefaultPlugins3()
     {
         // * tests that the default plugin must implement or extend the
         // plugin base class.
diff --git 
a/commons-digester3-core/src/test/java/org/apache/commons/digester3/plugins/TestDelegate.java
 
b/commons-digester3-core/src/test/java/org/apache/commons/digester3/plugins/TestDelegate.java
index 1d334d79..c3f2fa4c 100644
--- 
a/commons-digester3-core/src/test/java/org/apache/commons/digester3/plugins/TestDelegate.java
+++ 
b/commons-digester3-core/src/test/java/org/apache/commons/digester3/plugins/TestDelegate.java
@@ -30,7 +30,7 @@ public class TestDelegate
 {
     @Test
     @Disabled
-    public void testDelegate() throws Exception
+    void testDelegate() throws Exception
     {
         // this method tests the Delegate functionality by capturing all
         // data below the specified pattern, and printing it to stdout.
diff --git 
a/commons-digester3-core/src/test/java/org/apache/commons/digester3/plugins/TestInline.java
 
b/commons-digester3-core/src/test/java/org/apache/commons/digester3/plugins/TestInline.java
index 104ad133..37818912 100644
--- 
a/commons-digester3-core/src/test/java/org/apache/commons/digester3/plugins/TestInline.java
+++ 
b/commons-digester3-core/src/test/java/org/apache/commons/digester3/plugins/TestInline.java
@@ -33,7 +33,7 @@ public class TestInline
 {
 
     @Test
-    public void testInlineDeclaration()
+    void testInlineDeclaration()
         throws Exception
     {
         // * tests that plugins can be specified by class, and that the
@@ -75,7 +75,7 @@ public class TestInline
     }
 
     @Test
-    public void testLeadingSlash()
+    void testLeadingSlash()
         throws Exception
     {
         // Tests that PluginRules handles patterns with a leading slash.
diff --git 
a/commons-digester3-core/src/test/java/org/apache/commons/digester3/plugins/TestLocalRules.java
 
b/commons-digester3-core/src/test/java/org/apache/commons/digester3/plugins/TestLocalRules.java
index d4c23811..5d01b8a7 100644
--- 
a/commons-digester3-core/src/test/java/org/apache/commons/digester3/plugins/TestLocalRules.java
+++ 
b/commons-digester3-core/src/test/java/org/apache/commons/digester3/plugins/TestLocalRules.java
@@ -33,7 +33,7 @@ public class TestLocalRules
 {
 
     @Test
-    public void testLocalRules()
+    void testLocalRules()
         throws Exception
     {
         // * tests that the plugin class can define an addRules method,
@@ -87,7 +87,7 @@ public class TestLocalRules
     }
 
     @Test
-    public void testNonStandardLocalRules()
+    void testNonStandardLocalRules()
         throws Exception
     {
         // * tests that using PluginDeclarationRule to declare an alternate
diff --git 
a/commons-digester3-core/src/test/java/org/apache/commons/digester3/plugins/TestRecursion.java
 
b/commons-digester3-core/src/test/java/org/apache/commons/digester3/plugins/TestRecursion.java
index c1132ef7..e92931a1 100644
--- 
a/commons-digester3-core/src/test/java/org/apache/commons/digester3/plugins/TestRecursion.java
+++ 
b/commons-digester3-core/src/test/java/org/apache/commons/digester3/plugins/TestRecursion.java
@@ -49,7 +49,7 @@ public class TestRecursion
     }
 
     @Test
-    public void testRecursiveRules()
+    void testRecursiveRules()
         throws Exception
     {
         // * tests that a rule can declare custom PluginCreateRules
diff --git 
a/commons-digester3-core/src/test/java/org/apache/commons/digester3/plugins/TestRuleInfo.java
 
b/commons-digester3-core/src/test/java/org/apache/commons/digester3/plugins/TestRuleInfo.java
index 8bd6cc8d..62895e42 100644
--- 
a/commons-digester3-core/src/test/java/org/apache/commons/digester3/plugins/TestRuleInfo.java
+++ 
b/commons-digester3-core/src/test/java/org/apache/commons/digester3/plugins/TestRuleInfo.java
@@ -33,7 +33,7 @@ public class TestRuleInfo
 {
 
     @Test
-    public void testRuleInfoAutoDetect()
+    void testRuleInfoAutoDetect()
         throws Exception
     {
         // * tests that custom rules can be declared on a
@@ -72,7 +72,7 @@ public class TestRuleInfo
     }
 
     @Test
-    public void testRuleInfoExplicitClass()
+    void testRuleInfoExplicitClass()
         throws Exception
     {
         // * tests that custom rules can be declared on a
@@ -110,7 +110,7 @@ public class TestRuleInfo
     }
 
     @Test
-    public void testRuleInfoExplicitMethod()
+    void testRuleInfoExplicitMethod()
         throws Exception
     {
         // * tests that custom rules can be declared on a
diff --git 
a/commons-digester3-core/src/test/java/org/apache/commons/digester3/plugins/TestXmlRuleInfo.java
 
b/commons-digester3-core/src/test/java/org/apache/commons/digester3/plugins/TestXmlRuleInfo.java
index 6e886538..3066983d 100644
--- 
a/commons-digester3-core/src/test/java/org/apache/commons/digester3/plugins/TestXmlRuleInfo.java
+++ 
b/commons-digester3-core/src/test/java/org/apache/commons/digester3/plugins/TestXmlRuleInfo.java
@@ -33,7 +33,7 @@ public class TestXmlRuleInfo
 {
 
     @Test
-    public void testXmlRuleImplicitResource()
+    void testXmlRuleImplicitResource()
         throws Exception
     {
         // * tests that custom rules can be declared on a
@@ -69,7 +69,7 @@ public class TestXmlRuleInfo
     }
 
     @Test
-    public void testXmlRuleInfoExplicitFile()
+    void testXmlRuleInfoExplicitFile()
         throws Exception
     {
         // * tests that custom rules can be declared on a
@@ -105,7 +105,7 @@ public class TestXmlRuleInfo
     }
 
     @Test
-    public void testXmlRuleInfoExplicitResource()
+    void testXmlRuleInfoExplicitResource()
         throws Exception
     {
         // * tests that custom rules can be declared on a
diff --git 
a/commons-digester3-core/src/test/java/org/apache/commons/digester3/substitution/CompoundSubstitutorTestCase.java
 
b/commons-digester3-core/src/test/java/org/apache/commons/digester3/substitution/CompoundSubstitutorTestCase.java
index 5f8f8213..ce1c1b18 100644
--- 
a/commons-digester3-core/src/test/java/org/apache/commons/digester3/substitution/CompoundSubstitutorTestCase.java
+++ 
b/commons-digester3-core/src/test/java/org/apache/commons/digester3/substitution/CompoundSubstitutorTestCase.java
@@ -114,7 +114,7 @@ public final class CompoundSubstitutorTestCase
     }
 
     @Test
-    public void testChaining()
+    void testChaining()
     {
         final Substitutor a = new SubstitutorStub( "XYZ", "", "a", "", "abc" );
         final Substitutor b = new SubstitutorStub( "STU", "", "b", "", "bcd" );
@@ -130,7 +130,7 @@ public final class CompoundSubstitutorTestCase
     }
 
     @Test
-    public void testConstructors()
+    void testConstructors()
     {
         assertThrows( IllegalArgumentException.class, () -> new 
CompoundSubstitutor( null, null ) );
 
diff --git 
a/commons-digester3-core/src/test/java/org/apache/commons/digester3/substitution/VariableExpansionTestCase.java
 
b/commons-digester3-core/src/test/java/org/apache/commons/digester3/substitution/VariableExpansionTestCase.java
index 11cec928..49fe6877 100644
--- 
a/commons-digester3-core/src/test/java/org/apache/commons/digester3/substitution/VariableExpansionTestCase.java
+++ 
b/commons-digester3-core/src/test/java/org/apache/commons/digester3/substitution/VariableExpansionTestCase.java
@@ -89,7 +89,7 @@ public class VariableExpansionTestCase
      * Test expansion of text in element bodies.
      */
     @Test
-    public void testBodyExpansion()
+    void testBodyExpansion()
         throws SAXException, IOException
     {
 
@@ -128,7 +128,7 @@ public class VariableExpansionTestCase
      * Test that an unknown variable causes a RuntimeException.
      */
     @Test
-    public void testExpansionException()
+    void testExpansionException()
     {
 
         final String xml = "<root alpha='${attr1}'/>";
@@ -158,7 +158,7 @@ public class VariableExpansionTestCase
      * @throws SAXException
      */
     @Test
-    public void testExpansionOfPropertyInProperty()
+    void testExpansionOfPropertyInProperty()
         throws SAXException, IOException
     {
         final String xml =
@@ -181,7 +181,7 @@ public class VariableExpansionTestCase
      * multiple elements exist.
      */
     @Test
-    public void testExpansionWithMultipleSources()
+    void testExpansionWithMultipleSources()
         throws SAXException, IOException
     {
 
@@ -241,7 +241,7 @@ public class VariableExpansionTestCase
      * @throws SAXException
      */
     @Test
-    public void testExpansionWithMutableSource()
+    void testExpansionWithMutableSource()
         throws SAXException, IOException
     {
         final String xml = "<root>" + "<property name='attr' 
value='prop.value'/>" + "<bean alpha='${attr}'/>" + "</root>";
@@ -261,7 +261,7 @@ public class VariableExpansionTestCase
      * Test that a MultiVariableExpander with no sources does no expansion.
      */
     @Test
-    public void testExpansionWithNoSource()
+    void testExpansionWithNoSource()
         throws SAXException, IOException
     {
 
@@ -288,7 +288,7 @@ public class VariableExpansionTestCase
      * Test that by default no expansion occurs.
      */
     @Test
-    public void testNoExpansion()
+    void testNoExpansion()
         throws SAXException, IOException
     {
 
diff --git 
a/commons-digester3-core/src/test/java/org/apache/commons/digester3/xmlrules/BeanPropertySetterRuleTestCase.java
 
b/commons-digester3-core/src/test/java/org/apache/commons/digester3/xmlrules/BeanPropertySetterRuleTestCase.java
index 8fb28c22..bf46f274 100644
--- 
a/commons-digester3-core/src/test/java/org/apache/commons/digester3/xmlrules/BeanPropertySetterRuleTestCase.java
+++ 
b/commons-digester3-core/src/test/java/org/apache/commons/digester3/xmlrules/BeanPropertySetterRuleTestCase.java
@@ -29,7 +29,7 @@ public final class BeanPropertySetterRuleTestCase
 {
 
     @Test
-    public void testExtractPropertyNameFromAttribute() throws Exception
+    void testExtractPropertyNameFromAttribute() throws Exception
     {
         final Employee expected = new Employee( "John", "Doe" );
 
diff --git 
a/commons-digester3-core/src/test/java/org/apache/commons/digester3/xmlrules/DigesterPatternStackTest.java
 
b/commons-digester3-core/src/test/java/org/apache/commons/digester3/xmlrules/DigesterPatternStackTest.java
index 850dd407..33af6bf7 100644
--- 
a/commons-digester3-core/src/test/java/org/apache/commons/digester3/xmlrules/DigesterPatternStackTest.java
+++ 
b/commons-digester3-core/src/test/java/org/apache/commons/digester3/xmlrules/DigesterPatternStackTest.java
@@ -40,13 +40,13 @@ public class DigesterPatternStackTest
     }
 
     @Test
-    public void test1()
+    void test1()
     {
         assertEquals( "", patternStack.toString() );
     }
 
     @Test
-    public void test2()
+    void test2()
     {
         patternStack.push( "A" );
         assertEquals( "A", patternStack.toString() );
@@ -55,7 +55,7 @@ public class DigesterPatternStackTest
     }
 
     @Test
-    public void test3()
+    void test3()
     {
         patternStack.push( "A" );
         patternStack.push( "B" );
@@ -66,7 +66,7 @@ public class DigesterPatternStackTest
     }
 
     @Test
-    public void test4()
+    void test4()
     {
         patternStack.push( "" );
         assertEquals( "", patternStack.toString() );
@@ -76,7 +76,7 @@ public class DigesterPatternStackTest
     }
 
     @Test
-    public void test5()
+    void test5()
     {
         patternStack.push( "A" );
         assertEquals( "A", patternStack.toString() );
@@ -88,7 +88,7 @@ public class DigesterPatternStackTest
     }
 
     @Test
-    public void test6()
+    void test6()
     {
         patternStack.push( "A" );
         patternStack.push( "B" );
@@ -97,7 +97,7 @@ public class DigesterPatternStackTest
     }
 
     @Test
-    public void test7()
+    void test7()
     {
         patternStack.push( "///" );
         assertEquals( "///", patternStack.toString() );
diff --git 
a/commons-digester3-core/src/test/java/org/apache/commons/digester3/xmlrules/FromXmlRuleSetTest.java
 
b/commons-digester3-core/src/test/java/org/apache/commons/digester3/xmlrules/FromXmlRuleSetTest.java
index 854bdf85..136c0572 100644
--- 
a/commons-digester3-core/src/test/java/org/apache/commons/digester3/xmlrules/FromXmlRuleSetTest.java
+++ 
b/commons-digester3-core/src/test/java/org/apache/commons/digester3/xmlrules/FromXmlRuleSetTest.java
@@ -79,7 +79,7 @@ public class FromXmlRuleSetTest
      * test loading rules at a base position other than the root.
      */
     @Test
-    public void testBasePath()
+    void testBasePath()
         throws Exception
     {
         final String xmlRules =
@@ -106,7 +106,7 @@ public class FromXmlRuleSetTest
     }
 
     @Test
-    public void testCallParamRule()
+    void testCallParamRule()
         throws Exception
     {
         final URL rules = getClass().getResource( "test-call-param-rules.xml" 
);
@@ -132,7 +132,7 @@ public class FromXmlRuleSetTest
      * file, and also includes programmatically created rules.
      */
     @Test
-    public void testCreateDigester()
+    void testCreateDigester()
         throws Exception
     {
         final URL rules = getClass().getResource( "testrules.xml" );
@@ -145,7 +145,7 @@ public class FromXmlRuleSetTest
     }
 
     @Test
-    public void testFactoryCreateRule()
+    void testFactoryCreateRule()
         throws Exception
     {
         final URL rules = getClass().getResource( "testfactory.xml" );
@@ -171,7 +171,7 @@ public class FromXmlRuleSetTest
     }
 
     @Test
-    public void testFactoryIgnoreCreateRule()
+    void testFactoryIgnoreCreateRule()
     {
         final URL rules = getClass().getResource( "testfactoryignore.xml" );
 
@@ -184,7 +184,7 @@ public class FromXmlRuleSetTest
     }
 
     @Test
-    public void testFactoryNotIgnoreCreateRule()
+    void testFactoryNotIgnoreCreateRule()
     {
         final URL rules = getClass().getResource( "testfactorynoignore.xml" );
 
@@ -194,7 +194,7 @@ public class FromXmlRuleSetTest
     }
 
     @Test
-    public void testInputSourceLoader() throws Exception {
+    void testInputSourceLoader() throws Exception {
         final String rulesXml = "<?xml version='1.0'?>"
                 + "<digester-rules>"
                 + " <pattern value='root'>"
@@ -235,7 +235,7 @@ public class FromXmlRuleSetTest
      * also includes programmatically created rules.
      */
     @Test
-    public void testLoad1()
+    void testLoad1()
         throws Exception
     {
         final ClassLoader classLoader = getClass().getClassLoader();
@@ -266,7 +266,7 @@ public class FromXmlRuleSetTest
      * DigesterLoader as an InputStream instead of a URL.
      */
     @Test
-    public void testLoad2()
+    void testLoad2()
         throws Exception
     {
         final URL rules = getClass().getResource( "testrules.xml" );
@@ -287,7 +287,7 @@ public class FromXmlRuleSetTest
     /**
      */
     @Test
-    public void testSetCustomProperties()
+    void testSetCustomProperties()
         throws Exception
     {
         final URL rules = this.getClass().getResource( 
"testPropertyAliasRules.xml" );
diff --git 
a/commons-digester3-core/src/test/java/org/apache/commons/digester3/xmlrules/IncludeTest.java
 
b/commons-digester3-core/src/test/java/org/apache/commons/digester3/xmlrules/IncludeTest.java
index c2929280..ddfeecad 100644
--- 
a/commons-digester3-core/src/test/java/org/apache/commons/digester3/xmlrules/IncludeTest.java
+++ 
b/commons-digester3-core/src/test/java/org/apache/commons/digester3/xmlrules/IncludeTest.java
@@ -62,7 +62,7 @@ public class IncludeTest
     }
 
     @Test
-    public void testBasicInclude()
+    void testBasicInclude()
         throws Exception
     {
         final String rulesXml = "<?xml version='1.0'?>"
@@ -98,7 +98,7 @@ public class IncludeTest
      * Validates that circular includes are detected and result in an exception
      */
     @Test
-    public void testCircularInclude()
+    void testCircularInclude()
     {
         final URL url = ClassLoader.getSystemResource( 
"org/apache/commons/digester3/xmlrules/testCircularRules.xml" );
         assertThrows( DigesterLoadingException.class, () -> newLoader( new 
FromXmlRulesModule()
@@ -114,7 +114,7 @@ public class IncludeTest
     }
 
     @Test
-    public void testUrlInclude()
+    void testUrlInclude()
         throws Exception
     {
         final String rulesXml = "<?xml version='1.0'?>"
diff --git 
a/commons-digester3-core/src/test/java/org/apache/commons/digester3/xmlrules/SetNamespaceURITestCase.java
 
b/commons-digester3-core/src/test/java/org/apache/commons/digester3/xmlrules/SetNamespaceURITestCase.java
index 339580b5..28f36da6 100644
--- 
a/commons-digester3-core/src/test/java/org/apache/commons/digester3/xmlrules/SetNamespaceURITestCase.java
+++ 
b/commons-digester3-core/src/test/java/org/apache/commons/digester3/xmlrules/SetNamespaceURITestCase.java
@@ -34,7 +34,7 @@ public final class SetNamespaceURITestCase
 {
 
     @Test
-    public void testAtomWithNamespaceParse()
+    void testAtomWithNamespaceParse()
         throws Exception
     {
         // Drive commons-beanutils how to convert dates

Reply via email to