Use JUnit 5 convention for class visibility

Author: garydgregory

src/test/java/org/apache/commons/jxpath/BasicNodeSetTest.java

@@ -29,7 +29,7 @@
 /**
  * Test BasicNodeSet
  */
-public class BasicNodeSetTest extends AbstractJXPathTest {
+class BasicNodeSetTest extends AbstractJXPathTest {
 
     /** JXPathContext */
     protected JXPathContext context;

src/test/java/org/apache/commons/jxpath/issues/JXPath113Test.java

@@ -33,7 +33,7 @@
 import org.w3c.dom.Document;
 import org.xml.sax.InputSource;
 
-public class JXPath113Test extends AbstractJXPathTest {
+class JXPath113Test extends AbstractJXPathTest {
 
     static class JAXP {
 

src/test/java/org/apache/commons/jxpath/issues/JXPath118Test.java

@@ -28,7 +28,7 @@
 /**
  * Testcase proving JXPATH-118 issue with asPath() returning wrong names.
  */
-public class JXPath118Test {
+class JXPath118Test {
 
     public static class SomeChildClass {
 

src/test/java/org/apache/commons/jxpath/issues/JXPath149Test.java

@@ -21,7 +21,7 @@
 import org.apache.commons.jxpath.JXPathContext;
 import org.junit.jupiter.api.Test;
 
-public class JXPath149Test extends AbstractJXPathTest {
+class JXPath149Test extends AbstractJXPathTest {
 
     @Test
     void testComplexOperationWithVariables() {

src/test/java/org/apache/commons/jxpath/issues/JXPath172DynamicTest.java

@@ -29,7 +29,7 @@
 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.

src/test/java/org/apache/commons/jxpath/issues/JXPath172Test.java

@@ -30,7 +30,7 @@
 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 {
 

src/test/java/org/apache/commons/jxpath/issues/JXPath177Test.java

@@ -28,7 +28,7 @@
 import org.apache.commons.jxpath.Variables;
 import org.junit.jupiter.api.Test;
 
-public class JXPath177Test {
+class JXPath177Test {
 
     private static final class JXPathVariablesResolver implements Variables {
 

src/test/java/org/apache/commons/jxpath/ri/JXPathCompiledExpressionTest.java

@@ -48,7 +48,7 @@
 /**
  * Test compiler.
  */
-public class JXPathCompiledExpressionTest extends AbstractJXPathTest {
+class JXPathCompiledExpressionTest extends AbstractJXPathTest {
 
     private void assertXPathExpression(final String xpath, final Class expectedClass) {
         assertXPathExpression(xpath, expectedClass, xpath);

src/test/java/org/apache/commons/jxpath/ri/StressTest.java

@@ -26,7 +26,7 @@
 /**
  * Test thread safety.
  */
-public class StressTest {
+class StressTest {
 
     private static final class StressRunnable implements Runnable {
 

src/test/java/org/apache/commons/jxpath/ri/axes/RecursiveAxesTest.java

@@ -25,7 +25,7 @@
 /**
  * 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;