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-jxpath.git
The following commit(s) were added to refs/heads/master by this push:
new 8977652 Use JUnit 5 convention for class visibility
8977652 is described below
commit 8977652cb2016e1a7aec15191055c894f08f3b28
Author: Gary D. Gregory <garydgreg...@gmail.com>
AuthorDate: Sun Jun 8 15:15:03 2025 -0400
Use JUnit 5 convention for class visibility
---
src/test/java/org/apache/commons/jxpath/BasicNodeSetTest.java | 2 +-
src/test/java/org/apache/commons/jxpath/issues/JXPath113Test.java | 2 +-
src/test/java/org/apache/commons/jxpath/issues/JXPath118Test.java | 2 +-
src/test/java/org/apache/commons/jxpath/issues/JXPath149Test.java | 2 +-
.../java/org/apache/commons/jxpath/issues/JXPath172DynamicTest.java | 2 +-
src/test/java/org/apache/commons/jxpath/issues/JXPath172Test.java | 2 +-
src/test/java/org/apache/commons/jxpath/issues/JXPath177Test.java | 2 +-
.../java/org/apache/commons/jxpath/ri/JXPathCompiledExpressionTest.java | 2 +-
src/test/java/org/apache/commons/jxpath/ri/StressTest.java | 2 +-
src/test/java/org/apache/commons/jxpath/ri/axes/RecursiveAxesTest.java | 2 +-
.../org/apache/commons/jxpath/ri/axes/SimplePathInterpreterTest.java | 2 +-
.../org/apache/commons/jxpath/ri/compiler/ContextDependencyTest.java | 2 +-
.../java/org/apache/commons/jxpath/ri/compiler/CoreFunctionTest.java | 2 +-
.../java/org/apache/commons/jxpath/ri/compiler/CoreOperationTest.java | 2 +-
.../org/apache/commons/jxpath/ri/compiler/ExtensionFunctionTest.java | 2 +-
src/test/java/org/apache/commons/jxpath/ri/compiler/VariableTest.java | 2 +-
.../apache/commons/jxpath/ri/model/AliasedNamespaceIterationTest.java | 2 +-
.../org/apache/commons/jxpath/ri/model/EmbeddedColonMapKeysTest.java | 2 +-
.../java/org/apache/commons/jxpath/ri/model/EmptyCollectionTest.java | 2 +-
.../org/apache/commons/jxpath/ri/model/ExternalXMLNamespaceTest.java | 2 +-
src/test/java/org/apache/commons/jxpath/ri/model/JXPath151Test.java | 2 +-
src/test/java/org/apache/commons/jxpath/ri/model/JXPath154Test.java | 2 +-
src/test/java/org/apache/commons/jxpath/ri/model/MixedModelTest.java | 2 +-
.../java/org/apache/commons/jxpath/ri/model/XMLPreserveSpaceTest.java | 2 +-
src/test/java/org/apache/commons/jxpath/ri/model/XMLSpaceTest.java | 2 +-
.../org/apache/commons/jxpath/ri/model/XMLUpperCaseElementsTest.java | 2 +-
.../commons/jxpath/ri/model/beans/BadlyImplementedFactoryTest.java | 2 +-
.../java/org/apache/commons/jxpath/ri/model/beans/BeanModelTest.java | 2 +-
.../apache/commons/jxpath/ri/model/container/ContainerModelTest.java | 2 +-
src/test/java/org/apache/commons/jxpath/ri/model/dom/DOMModelTest.java | 2 +-
.../org/apache/commons/jxpath/ri/model/dynabeans/LazyDynaBeanTest.java | 2 +-
.../commons/jxpath/ri/model/dynamic/DynamicPropertiesModelTest.java | 2 +-
.../java/org/apache/commons/jxpath/ri/model/jdom/JDOMModelTest.java | 2 +-
.../org/apache/commons/jxpath/servlet/JXPathServletContextTest.java | 2 +-
.../java/org/apache/commons/jxpath/util/BasicTypeConverterTest.java | 2 +-
src/test/java/org/apache/commons/jxpath/util/ValueUtilsTest.java | 2 +-
36 files changed, 36 insertions(+), 36 deletions(-)
diff --git a/src/test/java/org/apache/commons/jxpath/BasicNodeSetTest.java
b/src/test/java/org/apache/commons/jxpath/BasicNodeSetTest.java
index c01792d..88ec65f 100644
--- a/src/test/java/org/apache/commons/jxpath/BasicNodeSetTest.java
+++ b/src/test/java/org/apache/commons/jxpath/BasicNodeSetTest.java
@@ -29,7 +29,7 @@ import org.w3c.dom.Element;
/**
* Test BasicNodeSet
*/
-public class BasicNodeSetTest extends AbstractJXPathTest {
+class BasicNodeSetTest extends AbstractJXPathTest {
/** JXPathContext */
protected JXPathContext context;
diff --git a/src/test/java/org/apache/commons/jxpath/issues/JXPath113Test.java
b/src/test/java/org/apache/commons/jxpath/issues/JXPath113Test.java
index dbe6a16..04c1895 100644
--- a/src/test/java/org/apache/commons/jxpath/issues/JXPath113Test.java
+++ b/src/test/java/org/apache/commons/jxpath/issues/JXPath113Test.java
@@ -33,7 +33,7 @@ import org.junit.jupiter.api.Test;
import org.w3c.dom.Document;
import org.xml.sax.InputSource;
-public class JXPath113Test extends AbstractJXPathTest {
+class JXPath113Test extends AbstractJXPathTest {
static class JAXP {
diff --git a/src/test/java/org/apache/commons/jxpath/issues/JXPath118Test.java
b/src/test/java/org/apache/commons/jxpath/issues/JXPath118Test.java
index d0e6033..86f608c 100644
--- a/src/test/java/org/apache/commons/jxpath/issues/JXPath118Test.java
+++ b/src/test/java/org/apache/commons/jxpath/issues/JXPath118Test.java
@@ -28,7 +28,7 @@ import org.junit.jupiter.api.Test;
/**
* Testcase proving JXPATH-118 issue with asPath() returning wrong names.
*/
-public class JXPath118Test {
+class JXPath118Test {
public static class SomeChildClass {
diff --git a/src/test/java/org/apache/commons/jxpath/issues/JXPath149Test.java
b/src/test/java/org/apache/commons/jxpath/issues/JXPath149Test.java
index 9cc7fb8..037c3d9 100644
--- a/src/test/java/org/apache/commons/jxpath/issues/JXPath149Test.java
+++ b/src/test/java/org/apache/commons/jxpath/issues/JXPath149Test.java
@@ -21,7 +21,7 @@ import org.apache.commons.jxpath.AbstractJXPathTest;
import org.apache.commons.jxpath.JXPathContext;
import org.junit.jupiter.api.Test;
-public class JXPath149Test extends AbstractJXPathTest {
+class JXPath149Test extends AbstractJXPathTest {
@Test
void testComplexOperationWithVariables() {
diff --git
a/src/test/java/org/apache/commons/jxpath/issues/JXPath172DynamicTest.java
b/src/test/java/org/apache/commons/jxpath/issues/JXPath172DynamicTest.java
index 825a7f8..5a12ac9 100644
--- a/src/test/java/org/apache/commons/jxpath/issues/JXPath172DynamicTest.java
+++ b/src/test/java/org/apache/commons/jxpath/issues/JXPath172DynamicTest.java
@@ -29,7 +29,7 @@ import org.apache.commons.jxpath.Pointer;
import org.apache.commons.jxpath.ri.model.dynamic.DynamicPropertyPointer;
import org.junit.jupiter.api.Test;
-public class JXPath172DynamicTest extends AbstractJXPathTest {
+class JXPath172DynamicTest extends AbstractJXPathTest {
/**
* Helper, returns a {@link JXPathContext} filled with a Map whose "value"
key is associated to the passed {@code val} value.
diff --git a/src/test/java/org/apache/commons/jxpath/issues/JXPath172Test.java
b/src/test/java/org/apache/commons/jxpath/issues/JXPath172Test.java
index 372e3c7..c015faf 100644
--- a/src/test/java/org/apache/commons/jxpath/issues/JXPath172Test.java
+++ b/src/test/java/org/apache/commons/jxpath/issues/JXPath172Test.java
@@ -30,7 +30,7 @@ import
org.apache.commons.jxpath.ri.model.beans.BeanPropertyPointer;
import org.apache.commons.jxpath.ri.model.beans.NullPropertyPointer;
import org.junit.jupiter.api.Test;
-public class JXPath172Test extends AbstractJXPathTest {
+class JXPath172Test extends AbstractJXPathTest {
public static class TestBean172 {
diff --git a/src/test/java/org/apache/commons/jxpath/issues/JXPath177Test.java
b/src/test/java/org/apache/commons/jxpath/issues/JXPath177Test.java
index bd56972..38c5fc2 100644
--- a/src/test/java/org/apache/commons/jxpath/issues/JXPath177Test.java
+++ b/src/test/java/org/apache/commons/jxpath/issues/JXPath177Test.java
@@ -28,7 +28,7 @@ import org.apache.commons.jxpath.Pointer;
import org.apache.commons.jxpath.Variables;
import org.junit.jupiter.api.Test;
-public class JXPath177Test {
+class JXPath177Test {
private static final class JXPathVariablesResolver implements Variables {
diff --git
a/src/test/java/org/apache/commons/jxpath/ri/JXPathCompiledExpressionTest.java
b/src/test/java/org/apache/commons/jxpath/ri/JXPathCompiledExpressionTest.java
index d38fafe..ea2cd1f 100644
---
a/src/test/java/org/apache/commons/jxpath/ri/JXPathCompiledExpressionTest.java
+++
b/src/test/java/org/apache/commons/jxpath/ri/JXPathCompiledExpressionTest.java
@@ -48,7 +48,7 @@ import org.junit.jupiter.api.Test;
/**
* Test compiler.
*/
-public class JXPathCompiledExpressionTest extends AbstractJXPathTest {
+class JXPathCompiledExpressionTest extends AbstractJXPathTest {
private void assertXPathExpression(final String xpath, final Class
expectedClass) {
assertXPathExpression(xpath, expectedClass, xpath);
diff --git a/src/test/java/org/apache/commons/jxpath/ri/StressTest.java
b/src/test/java/org/apache/commons/jxpath/ri/StressTest.java
index ca822ea..feb2d06 100644
--- a/src/test/java/org/apache/commons/jxpath/ri/StressTest.java
+++ b/src/test/java/org/apache/commons/jxpath/ri/StressTest.java
@@ -26,7 +26,7 @@ import org.junit.jupiter.api.Test;
/**
* Test thread safety.
*/
-public class StressTest {
+class StressTest {
private static final class StressRunnable implements Runnable {
diff --git
a/src/test/java/org/apache/commons/jxpath/ri/axes/RecursiveAxesTest.java
b/src/test/java/org/apache/commons/jxpath/ri/axes/RecursiveAxesTest.java
index 7463d8c..2185d5d 100644
--- a/src/test/java/org/apache/commons/jxpath/ri/axes/RecursiveAxesTest.java
+++ b/src/test/java/org/apache/commons/jxpath/ri/axes/RecursiveAxesTest.java
@@ -25,7 +25,7 @@ import org.junit.jupiter.api.Test;
/**
* Test for the protection mechanism that stops infinite recursion in descent
down a recursive graph.
*/
-public class RecursiveAxesTest extends AbstractJXPathTest {
+class RecursiveAxesTest extends AbstractJXPathTest {
private RecursiveBean bean;
private JXPathContext context;
diff --git
a/src/test/java/org/apache/commons/jxpath/ri/axes/SimplePathInterpreterTest.java
b/src/test/java/org/apache/commons/jxpath/ri/axes/SimplePathInterpreterTest.java
index 0206b87..9f8dcf0 100644
---
a/src/test/java/org/apache/commons/jxpath/ri/axes/SimplePathInterpreterTest.java
+++
b/src/test/java/org/apache/commons/jxpath/ri/axes/SimplePathInterpreterTest.java
@@ -42,7 +42,7 @@ import
org.apache.commons.jxpath.ri.model.dynamic.DynamicPropertyPointer;
import org.junit.jupiter.api.BeforeEach;
import org.junit.jupiter.api.Test;
-public class SimplePathInterpreterTest {
+class SimplePathInterpreterTest {
private TestBeanWithNode bean;
private JXPathContext context;
diff --git
a/src/test/java/org/apache/commons/jxpath/ri/compiler/ContextDependencyTest.java
b/src/test/java/org/apache/commons/jxpath/ri/compiler/ContextDependencyTest.java
index 926104c..4daeeee 100644
---
a/src/test/java/org/apache/commons/jxpath/ri/compiler/ContextDependencyTest.java
+++
b/src/test/java/org/apache/commons/jxpath/ri/compiler/ContextDependencyTest.java
@@ -26,7 +26,7 @@ import org.junit.jupiter.api.Test;
/**
* Tests the determination of whether an expression is context dependent.
*/
-public class ContextDependencyTest extends AbstractJXPathTest {
+class ContextDependencyTest extends AbstractJXPathTest {
@Test
void testContextDependency() {
diff --git
a/src/test/java/org/apache/commons/jxpath/ri/compiler/CoreFunctionTest.java
b/src/test/java/org/apache/commons/jxpath/ri/compiler/CoreFunctionTest.java
index 13afd26..251a0a4 100644
--- a/src/test/java/org/apache/commons/jxpath/ri/compiler/CoreFunctionTest.java
+++ b/src/test/java/org/apache/commons/jxpath/ri/compiler/CoreFunctionTest.java
@@ -35,7 +35,7 @@ import org.junit.jupiter.api.Test;
/**
* Test basic functionality of JXPath - core functions.
*/
-public class CoreFunctionTest extends AbstractJXPathTest {
+class CoreFunctionTest extends AbstractJXPathTest {
private JXPathContext context;
diff --git
a/src/test/java/org/apache/commons/jxpath/ri/compiler/CoreOperationTest.java
b/src/test/java/org/apache/commons/jxpath/ri/compiler/CoreOperationTest.java
index f007588..7c47790 100644
--- a/src/test/java/org/apache/commons/jxpath/ri/compiler/CoreOperationTest.java
+++ b/src/test/java/org/apache/commons/jxpath/ri/compiler/CoreOperationTest.java
@@ -26,7 +26,7 @@ import org.junit.jupiter.api.Test;
/**
* Test basic functionality of JXPath - infoset types, operations.
*/
-public class CoreOperationTest extends AbstractJXPathTest {
+class CoreOperationTest extends AbstractJXPathTest {
private JXPathContext context;
diff --git
a/src/test/java/org/apache/commons/jxpath/ri/compiler/ExtensionFunctionTest.java
b/src/test/java/org/apache/commons/jxpath/ri/compiler/ExtensionFunctionTest.java
index ed3b756..eb76e76 100644
---
a/src/test/java/org/apache/commons/jxpath/ri/compiler/ExtensionFunctionTest.java
+++
b/src/test/java/org/apache/commons/jxpath/ri/compiler/ExtensionFunctionTest.java
@@ -48,7 +48,7 @@ import org.junit.jupiter.api.Test;
/**
* Test extension functions.
*/
-public class ExtensionFunctionTest extends AbstractJXPathTest {
+class ExtensionFunctionTest extends AbstractJXPathTest {
private static final class Context implements ExpressionContext {
diff --git
a/src/test/java/org/apache/commons/jxpath/ri/compiler/VariableTest.java
b/src/test/java/org/apache/commons/jxpath/ri/compiler/VariableTest.java
index ccb3efc..a86a542 100644
--- a/src/test/java/org/apache/commons/jxpath/ri/compiler/VariableTest.java
+++ b/src/test/java/org/apache/commons/jxpath/ri/compiler/VariableTest.java
@@ -31,7 +31,7 @@ import org.junit.jupiter.api.Test;
/**
* Test basic functionality of JXPath - infoset types, operations.
*/
-public class VariableTest extends AbstractJXPathTest {
+class VariableTest extends AbstractJXPathTest {
private JXPathContext context;
diff --git
a/src/test/java/org/apache/commons/jxpath/ri/model/AliasedNamespaceIterationTest.java
b/src/test/java/org/apache/commons/jxpath/ri/model/AliasedNamespaceIterationTest.java
index 5204ad6..eafa752 100644
---
a/src/test/java/org/apache/commons/jxpath/ri/model/AliasedNamespaceIterationTest.java
+++
b/src/test/java/org/apache/commons/jxpath/ri/model/AliasedNamespaceIterationTest.java
@@ -27,7 +27,7 @@ import org.junit.jupiter.api.Test;
/**
* Test aliased/doubled XML namespace iteration; JXPATH-125.
*/
-public class AliasedNamespaceIterationTest extends AbstractJXPathTest {
+class AliasedNamespaceIterationTest extends AbstractJXPathTest {
protected JXPathContext context;
diff --git
a/src/test/java/org/apache/commons/jxpath/ri/model/EmbeddedColonMapKeysTest.java
b/src/test/java/org/apache/commons/jxpath/ri/model/EmbeddedColonMapKeysTest.java
index b0f0c1d..91f74c1 100644
---
a/src/test/java/org/apache/commons/jxpath/ri/model/EmbeddedColonMapKeysTest.java
+++
b/src/test/java/org/apache/commons/jxpath/ri/model/EmbeddedColonMapKeysTest.java
@@ -27,7 +27,7 @@ import org.junit.jupiter.api.Test;
/**
* JXPATH-104 test.
*/
-public class EmbeddedColonMapKeysTest extends AbstractJXPathTest {
+class EmbeddedColonMapKeysTest extends AbstractJXPathTest {
private JXPathContext context;
diff --git
a/src/test/java/org/apache/commons/jxpath/ri/model/EmptyCollectionTest.java
b/src/test/java/org/apache/commons/jxpath/ri/model/EmptyCollectionTest.java
index 331a619..b6a0f48 100644
--- a/src/test/java/org/apache/commons/jxpath/ri/model/EmptyCollectionTest.java
+++ b/src/test/java/org/apache/commons/jxpath/ri/model/EmptyCollectionTest.java
@@ -26,7 +26,7 @@ import org.junit.jupiter.api.Test;
/**
* Be sure empty lists/sets/arrays work.
*/
-public class EmptyCollectionTest extends AbstractJXPathTest {
+class EmptyCollectionTest extends AbstractJXPathTest {
public static class HasChild {
diff --git
a/src/test/java/org/apache/commons/jxpath/ri/model/ExternalXMLNamespaceTest.java
b/src/test/java/org/apache/commons/jxpath/ri/model/ExternalXMLNamespaceTest.java
index e73bb03..40c9388 100644
---
a/src/test/java/org/apache/commons/jxpath/ri/model/ExternalXMLNamespaceTest.java
+++
b/src/test/java/org/apache/commons/jxpath/ri/model/ExternalXMLNamespaceTest.java
@@ -25,7 +25,7 @@ import org.junit.jupiter.api.Test;
/**
* Test externally registered XML namespaces; JXPATH-97.
*/
-public class ExternalXMLNamespaceTest extends AbstractJXPathTest {
+class ExternalXMLNamespaceTest extends AbstractJXPathTest {
protected JXPathContext context;
diff --git
a/src/test/java/org/apache/commons/jxpath/ri/model/JXPath151Test.java
b/src/test/java/org/apache/commons/jxpath/ri/model/JXPath151Test.java
index 7c9a3d3..89bdee3 100644
--- a/src/test/java/org/apache/commons/jxpath/ri/model/JXPath151Test.java
+++ b/src/test/java/org/apache/commons/jxpath/ri/model/JXPath151Test.java
@@ -26,7 +26,7 @@ import org.apache.commons.jxpath.TestBean;
import org.junit.jupiter.api.BeforeEach;
import org.junit.jupiter.api.Test;
-public class JXPath151Test extends AbstractJXPathTest {
+class JXPath151Test extends AbstractJXPathTest {
private JXPathContext context;
diff --git
a/src/test/java/org/apache/commons/jxpath/ri/model/JXPath154Test.java
b/src/test/java/org/apache/commons/jxpath/ri/model/JXPath154Test.java
index e0659ea..7043473 100644
--- a/src/test/java/org/apache/commons/jxpath/ri/model/JXPath154Test.java
+++ b/src/test/java/org/apache/commons/jxpath/ri/model/JXPath154Test.java
@@ -24,7 +24,7 @@ import org.apache.commons.jxpath.JXPathContext;
import org.apache.commons.jxpath.xml.DocumentContainer;
import org.junit.jupiter.api.Test;
-public class JXPath154Test extends AbstractJXPathTest {
+class JXPath154Test extends AbstractJXPathTest {
protected JXPathContext context;
diff --git
a/src/test/java/org/apache/commons/jxpath/ri/model/MixedModelTest.java
b/src/test/java/org/apache/commons/jxpath/ri/model/MixedModelTest.java
index b7f04af..a9a6d7a 100644
--- a/src/test/java/org/apache/commons/jxpath/ri/model/MixedModelTest.java
+++ b/src/test/java/org/apache/commons/jxpath/ri/model/MixedModelTest.java
@@ -42,7 +42,7 @@ import org.junit.jupiter.api.Test;
/**
* Tests JXPath with mixed model: beans, maps, DOM etc.
*/
-public class MixedModelTest extends AbstractJXPathTest {
+class MixedModelTest extends AbstractJXPathTest {
private JXPathContext context;
diff --git
a/src/test/java/org/apache/commons/jxpath/ri/model/XMLPreserveSpaceTest.java
b/src/test/java/org/apache/commons/jxpath/ri/model/XMLPreserveSpaceTest.java
index adaa8b4..178e13d 100644
--- a/src/test/java/org/apache/commons/jxpath/ri/model/XMLPreserveSpaceTest.java
+++ b/src/test/java/org/apache/commons/jxpath/ri/model/XMLPreserveSpaceTest.java
@@ -27,7 +27,7 @@ import org.junit.jupiter.api.Test;
/**
* Test for text trimming from JXPATH-83.
*/
-public class XMLPreserveSpaceTest extends AbstractJXPathTest {
+class XMLPreserveSpaceTest extends AbstractJXPathTest {
protected JXPathContext context;
diff --git a/src/test/java/org/apache/commons/jxpath/ri/model/XMLSpaceTest.java
b/src/test/java/org/apache/commons/jxpath/ri/model/XMLSpaceTest.java
index f525a9a..5e131ee 100644
--- a/src/test/java/org/apache/commons/jxpath/ri/model/XMLSpaceTest.java
+++ b/src/test/java/org/apache/commons/jxpath/ri/model/XMLSpaceTest.java
@@ -27,7 +27,7 @@ import org.junit.jupiter.api.Test;
/**
* Test for text trimming from JXPATH-83.
*/
-public class XMLSpaceTest extends AbstractJXPathTest {
+class XMLSpaceTest extends AbstractJXPathTest {
protected JXPathContext context;
diff --git
a/src/test/java/org/apache/commons/jxpath/ri/model/XMLUpperCaseElementsTest.java
b/src/test/java/org/apache/commons/jxpath/ri/model/XMLUpperCaseElementsTest.java
index 84f8cca..6ca9fee 100644
---
a/src/test/java/org/apache/commons/jxpath/ri/model/XMLUpperCaseElementsTest.java
+++
b/src/test/java/org/apache/commons/jxpath/ri/model/XMLUpperCaseElementsTest.java
@@ -27,7 +27,7 @@ import org.junit.jupiter.api.Test;
/**
* Test for uppercase element matching, etc. showing JXPATH-136 is not
reproducible.
*/
-public class XMLUpperCaseElementsTest extends AbstractJXPathTest {
+class XMLUpperCaseElementsTest extends AbstractJXPathTest {
protected JXPathContext context;
diff --git
a/src/test/java/org/apache/commons/jxpath/ri/model/beans/BadlyImplementedFactoryTest.java
b/src/test/java/org/apache/commons/jxpath/ri/model/beans/BadlyImplementedFactoryTest.java
index ea388e0..e53c98d 100644
---
a/src/test/java/org/apache/commons/jxpath/ri/model/beans/BadlyImplementedFactoryTest.java
+++
b/src/test/java/org/apache/commons/jxpath/ri/model/beans/BadlyImplementedFactoryTest.java
@@ -34,7 +34,7 @@ import org.junit.jupiter.api.Test;
/**
* Badly-implemented Factory test. From JIRA JXPATH-68.
*/
-public class BadlyImplementedFactoryTest {
+class BadlyImplementedFactoryTest {
private JXPathContext context;
diff --git
a/src/test/java/org/apache/commons/jxpath/ri/model/beans/BeanModelTest.java
b/src/test/java/org/apache/commons/jxpath/ri/model/beans/BeanModelTest.java
index f48e9f0..1e8cc0d 100644
--- a/src/test/java/org/apache/commons/jxpath/ri/model/beans/BeanModelTest.java
+++ b/src/test/java/org/apache/commons/jxpath/ri/model/beans/BeanModelTest.java
@@ -27,7 +27,7 @@ import org.junit.jupiter.api.Test;
* Tests JXPath with JavaBeans
*
*/
-public class BeanModelTest extends AbstractBeanModelTest {
+class BeanModelTest extends AbstractBeanModelTest {
@Override
protected Object createContextBean() {
diff --git
a/src/test/java/org/apache/commons/jxpath/ri/model/container/ContainerModelTest.java
b/src/test/java/org/apache/commons/jxpath/ri/model/container/ContainerModelTest.java
index 7315240..a57395d 100644
---
a/src/test/java/org/apache/commons/jxpath/ri/model/container/ContainerModelTest.java
+++
b/src/test/java/org/apache/commons/jxpath/ri/model/container/ContainerModelTest.java
@@ -32,7 +32,7 @@ import org.junit.jupiter.api.Test;
/**
* Tests JXPath with containers as root or value of a variable, property, etc.
*/
-public class ContainerModelTest extends AbstractJXPathTest {
+class ContainerModelTest extends AbstractJXPathTest {
private static final class ArrayContainer implements Container {
diff --git
a/src/test/java/org/apache/commons/jxpath/ri/model/dom/DOMModelTest.java
b/src/test/java/org/apache/commons/jxpath/ri/model/dom/DOMModelTest.java
index bef5e7f..b87a05f 100644
--- a/src/test/java/org/apache/commons/jxpath/ri/model/dom/DOMModelTest.java
+++ b/src/test/java/org/apache/commons/jxpath/ri/model/dom/DOMModelTest.java
@@ -33,7 +33,7 @@ import org.w3c.dom.NodeList;
/**
* Tests JXPath with DOM
*/
-public class DOMModelTest extends AbstractXMLModelTest {
+class DOMModelTest extends AbstractXMLModelTest {
private void appendXMLSignature(final StringBuilder buffer, final NodeList
children, final boolean elements, final boolean attributes, final boolean text,
final boolean pi) {
diff --git
a/src/test/java/org/apache/commons/jxpath/ri/model/dynabeans/LazyDynaBeanTest.java
b/src/test/java/org/apache/commons/jxpath/ri/model/dynabeans/LazyDynaBeanTest.java
index b9a6eff..3f553e3 100644
---
a/src/test/java/org/apache/commons/jxpath/ri/model/dynabeans/LazyDynaBeanTest.java
+++
b/src/test/java/org/apache/commons/jxpath/ri/model/dynabeans/LazyDynaBeanTest.java
@@ -28,7 +28,7 @@ import org.junit.jupiter.api.Test;
/**
*/
-public class LazyDynaBeanTest extends AbstractJXPathTest {
+class LazyDynaBeanTest extends AbstractJXPathTest {
@Test
void testLazyProperty() throws JXPathNotFoundException {
diff --git
a/src/test/java/org/apache/commons/jxpath/ri/model/dynamic/DynamicPropertiesModelTest.java
b/src/test/java/org/apache/commons/jxpath/ri/model/dynamic/DynamicPropertiesModelTest.java
index 3bc3743..8be0fc2 100644
---
a/src/test/java/org/apache/commons/jxpath/ri/model/dynamic/DynamicPropertiesModelTest.java
+++
b/src/test/java/org/apache/commons/jxpath/ri/model/dynamic/DynamicPropertiesModelTest.java
@@ -34,7 +34,7 @@ import org.junit.jupiter.api.Test;
/**
* TODO more iterator testing with maps
*/
-public class DynamicPropertiesModelTest extends AbstractJXPathTest {
+class DynamicPropertiesModelTest extends AbstractJXPathTest {
private JXPathContext context;
diff --git
a/src/test/java/org/apache/commons/jxpath/ri/model/jdom/JDOMModelTest.java
b/src/test/java/org/apache/commons/jxpath/ri/model/jdom/JDOMModelTest.java
index 5f6ffb5..a2a9ac7 100644
--- a/src/test/java/org/apache/commons/jxpath/ri/model/jdom/JDOMModelTest.java
+++ b/src/test/java/org/apache/commons/jxpath/ri/model/jdom/JDOMModelTest.java
@@ -36,7 +36,7 @@ import org.junit.jupiter.api.Test;
/**
* Tests JXPath with JDOM
*/
-public class JDOMModelTest extends AbstractXMLModelTest {
+class JDOMModelTest extends AbstractXMLModelTest {
private void appendXMLSignature(final StringBuilder buffer, final List
children, final boolean elements, final boolean attributes, final boolean text,
final boolean pi) {
diff --git
a/src/test/java/org/apache/commons/jxpath/servlet/JXPathServletContextTest.java
b/src/test/java/org/apache/commons/jxpath/servlet/JXPathServletContextTest.java
index 3cd0e82..71af6cd 100644
---
a/src/test/java/org/apache/commons/jxpath/servlet/JXPathServletContextTest.java
+++
b/src/test/java/org/apache/commons/jxpath/servlet/JXPathServletContextTest.java
@@ -41,7 +41,7 @@ import com.mockrunner.mock.web.MockServletContext;
/**
*/
-public class JXPathServletContextTest {
+class JXPathServletContextTest {
private void checkPointerIterator(final JXPathContext context) {
final Iterator<Pointer> it = context.iteratePointers("/*");
diff --git
a/src/test/java/org/apache/commons/jxpath/util/BasicTypeConverterTest.java
b/src/test/java/org/apache/commons/jxpath/util/BasicTypeConverterTest.java
index c7cbb26..9ddf402 100644
--- a/src/test/java/org/apache/commons/jxpath/util/BasicTypeConverterTest.java
+++ b/src/test/java/org/apache/commons/jxpath/util/BasicTypeConverterTest.java
@@ -36,7 +36,7 @@ import org.junit.jupiter.api.Test;
/**
* Tests BasicTypeConverter
*/
-public class BasicTypeConverterTest {
+class BasicTypeConverterTest {
public void assertConversion(final Object from, final Class toType, final
Object expected) {
final boolean can = TypeUtils.canConvert(from, toType);
diff --git a/src/test/java/org/apache/commons/jxpath/util/ValueUtilsTest.java
b/src/test/java/org/apache/commons/jxpath/util/ValueUtilsTest.java
index be4923b..bd86bb9 100644
--- a/src/test/java/org/apache/commons/jxpath/util/ValueUtilsTest.java
+++ b/src/test/java/org/apache/commons/jxpath/util/ValueUtilsTest.java
@@ -27,7 +27,7 @@ import java.util.Set;
import org.junit.jupiter.api.Test;
-public class ValueUtilsTest {
+class ValueUtilsTest {
@Test
void testGetValueFromArray() {
