Modified: poi/trunk/src/testcases/org/apache/poi/util/TestIntList.java
URL: 
http://svn.apache.org/viewvc/poi/trunk/src/testcases/org/apache/poi/util/TestIntList.java?rev=1885281&r1=1885280&r2=1885281&view=diff
==============================================================================
--- poi/trunk/src/testcases/org/apache/poi/util/TestIntList.java (original)
+++ poi/trunk/src/testcases/org/apache/poi/util/TestIntList.java Fri Jan  8 
23:50:02 2021
@@ -34,7 +34,7 @@ import org.junit.jupiter.api.Test;
  */
 public final class TestIntList {
     @Test
-    public void testConstructors() {
+    void testConstructors() {
         IntList list = new IntList();
 
         assertTrue(list.isEmpty());
@@ -49,7 +49,7 @@ public final class TestIntList {
     }
 
     @Test
-    public void testAdd() {
+    void testAdd() {
         IntList list = new IntList();
         int[] testArray =
             {
@@ -111,7 +111,7 @@ public final class TestIntList {
     }
 
     @Test
-    public void testAddAll() {
+    void testAddAll() {
         IntList list = new IntList();
 
         for (int j = 0; j < 5; j++) {
@@ -175,7 +175,7 @@ public final class TestIntList {
     }
 
     @Test
-    public void testAddAllGrow() {
+    void testAddAllGrow() {
         IntList list = new IntList(0);
         IntList addList = new IntList(0);
         addList.add(1);
@@ -185,7 +185,7 @@ public final class TestIntList {
     }
 
     @Test
-    public void testClear() {
+    void testClear() {
         IntList list = new IntList();
 
         for (int j = 0; j < 500; j++) {
@@ -204,7 +204,7 @@ public final class TestIntList {
     }
 
     @Test
-    public void testContains() {
+    void testContains() {
         IntList list = new IntList();
 
         for (int j = 0; j < 1000; j += 2) {
@@ -220,7 +220,7 @@ public final class TestIntList {
     }
 
     @Test
-    public void testContainsAll() {
+    void testContainsAll() {
         IntList list = new IntList();
 
         assertTrue(list.containsAll(list));
@@ -240,7 +240,7 @@ public final class TestIntList {
     }
 
     @Test
-    public void testEquals() {
+    void testEquals() {
         IntList list = new IntList();
 
         assertEquals(list, list);
@@ -267,7 +267,7 @@ public final class TestIntList {
     }
 
     @Test
-    public void testGet() {
+    void testGet() {
         IntList list = new IntList();
 
         for (int j = 0; j < 1000; j++) {
@@ -281,7 +281,7 @@ public final class TestIntList {
     }
 
     @Test
-    public void testIndexOf() {
+    void testIndexOf() {
         IntList list = new IntList();
 
         for (int j = 0; j < 1000; j++) {
@@ -297,7 +297,7 @@ public final class TestIntList {
     }
 
     @Test
-    public void testIsEmpty() {
+    void testIsEmpty() {
         IntList list1 = new IntList();
         IntList list2 = new IntList(1000);
         IntList list3 = new IntList(list1);
@@ -320,7 +320,7 @@ public final class TestIntList {
     }
 
     @Test
-    public void testLastIndexOf() {
+    void testLastIndexOf() {
         IntList list = new IntList();
 
         for (int j = 0; j < 1000; j++) {
@@ -336,7 +336,7 @@ public final class TestIntList {
     }
 
     @Test
-    public void testRemove() {
+    void testRemove() {
         IntList list = new IntList();
 
         for (int j = 0; j < 1000; j++) {
@@ -358,7 +358,7 @@ public final class TestIntList {
     }
 
     @Test
-    public void testRemoveValue() {
+    void testRemoveValue() {
         IntList list = new IntList();
 
         for (int j = 0; j < 1000; j++) {
@@ -374,7 +374,7 @@ public final class TestIntList {
     }
 
     @Test
-    public void testRemoveAll() {
+    void testRemoveAll() {
         IntList list = new IntList();
 
         for (int j = 0; j < 1000; j++) {
@@ -411,7 +411,7 @@ public final class TestIntList {
     }
 
     @Test
-    public void testRetainAll() {
+    void testRetainAll() {
         IntList list = new IntList();
 
         for (int j = 0; j < 1000; j++) {
@@ -447,7 +447,7 @@ public final class TestIntList {
     }
 
     @Test
-    public void testSet() {
+    void testSet() {
         IntList list = new IntList();
 
         for (int j = 0; j < 1000; j++) {
@@ -461,7 +461,7 @@ public final class TestIntList {
     }
 
     @Test
-    public void testSize() {
+    void testSize() {
         IntList list = new IntList();
 
         for (int j = 0; j < 1000; j++) {
@@ -477,7 +477,7 @@ public final class TestIntList {
     }
 
     @Test
-    public void testToArray() {
+    void testToArray() {
         IntList list = new IntList();
 
         for (int j = 0; j < 1000; j++) {

Modified: poi/trunk/src/testcases/org/apache/poi/util/TestIntegerField.java
URL: 
http://svn.apache.org/viewvc/poi/trunk/src/testcases/org/apache/poi/util/TestIntegerField.java?rev=1885281&r1=1885280&r2=1885281&view=diff
==============================================================================
--- poi/trunk/src/testcases/org/apache/poi/util/TestIntegerField.java (original)
+++ poi/trunk/src/testcases/org/apache/poi/util/TestIntegerField.java Fri Jan  
8 23:50:02 2021
@@ -33,7 +33,7 @@ public final class TestIntegerField {
     private static final int[] _test_array = {Integer.MIN_VALUE, -1, 0, 1, 
Integer.MAX_VALUE};
 
     @Test
-    public void testConstructors() {
+    void testConstructors() {
         assertThrows(ArrayIndexOutOfBoundsException.class, () -> new 
IntegerField(-1));
         IntegerField field = new IntegerField(2);
 
@@ -62,7 +62,7 @@ public final class TestIntegerField {
     }
 
     @Test
-    public void testSet() {
+    void testSet() {
         IntegerField field = new IntegerField(0);
         byte[]       array = new byte[ 4 ];
 
@@ -80,7 +80,7 @@ public final class TestIntegerField {
     }
 
     @Test
-    public void testReadFromBytes() {
+    void testReadFromBytes() {
         IntegerField field1 = new IntegerField(1);
         byte[]       array = new byte[ 4 ];
 
@@ -97,7 +97,7 @@ public final class TestIntegerField {
     }
 
     @Test
-    public void testReadFromStream() throws IOException {
+    void testReadFromStream() throws IOException {
         IntegerField field  = new IntegerField(0);
         byte[]       buffer = new byte[ _test_array.length * 4 ];
 
@@ -116,7 +116,7 @@ public final class TestIntegerField {
     }
 
     @Test
-    public void testWriteToBytes() {
+    void testWriteToBytes() {
         IntegerField field = new IntegerField(0);
         byte[]       array = new byte[ 4 ];
 

Modified: poi/trunk/src/testcases/org/apache/poi/util/TestLittleEndian.java
URL: 
http://svn.apache.org/viewvc/poi/trunk/src/testcases/org/apache/poi/util/TestLittleEndian.java?rev=1885281&r1=1885280&r2=1885281&view=diff
==============================================================================
--- poi/trunk/src/testcases/org/apache/poi/util/TestLittleEndian.java (original)
+++ poi/trunk/src/testcases/org/apache/poi/util/TestLittleEndian.java Fri Jan  
8 23:50:02 2021
@@ -37,7 +37,7 @@ public final class TestLittleEndian {
      * test the getShort() method
      */
     @Test
-    public void testGetShort() {
+    void testGetShort() {
         byte[] testdata = new byte[ LittleEndianConsts.SHORT_SIZE + 1 ];
 
         testdata[0] = 0x01;
@@ -52,7 +52,7 @@ public final class TestLittleEndian {
     }
 
     @Test
-    public void testGetUShort() {
+    void testGetUShort() {
         byte[] testdata = {
             (byte) 0x01,
             (byte) 0xFF,
@@ -99,7 +99,7 @@ public final class TestLittleEndian {
      * test the getDouble() method
      */
     @Test
-    public void testGetDouble() {
+    void testGetDouble() {
         assertEquals(_doubles[0], LittleEndian.getDouble(_double_array, 0), 
0.000001 );
         assertEquals(_doubles[1], LittleEndian.getDouble( _double_array, 
LittleEndianConsts.DOUBLE_SIZE), 0.000001);
         assertTrue(Double.isNaN(LittleEndian.getDouble(_nan_double_array, 0)));
@@ -116,7 +116,7 @@ public final class TestLittleEndian {
      * test the getInt() method
      */
     @Test
-    public void testGetInt() {
+    void testGetInt() {
         // reading 4 byte data from a 5 byte buffer
         byte[] testdata = {
                 (byte) 0x01,
@@ -134,7 +134,7 @@ public final class TestLittleEndian {
      * test the getLong method
      */
     @Test
-    public void testGetLong() {
+    void testGetLong() {
 
         // reading 8 byte values from a 9 byte buffer
         byte[] testdata = {
@@ -157,7 +157,7 @@ public final class TestLittleEndian {
      * test the PutShort method
      */
     @Test
-    public void testPutShort() {
+    void testPutShort() {
         byte[] expected = new byte[ LittleEndianConsts.SHORT_SIZE + 1 ];
 
         expected[0] = 0x01;
@@ -178,7 +178,7 @@ public final class TestLittleEndian {
      * test the putInt method
      */
     @Test
-    public void testPutInt() {
+    void testPutInt() {
         // writing 4 byte data to a 5 byte buffer
         byte[] expected = {
                 (byte) 0x01,
@@ -199,7 +199,7 @@ public final class TestLittleEndian {
      * test the putDouble methods
      */
     @Test
-    public void testPutDouble() {
+    void testPutDouble() {
         byte[] received = new byte[ LittleEndianConsts.DOUBLE_SIZE + 1 ];
 
         LittleEndian.putDouble(received, 0, _doubles[0]);
@@ -216,7 +216,7 @@ public final class TestLittleEndian {
      * test the putLong method
      */
     @Test
-    public void testPutLong() {
+    void testPutLong() {
         // writing 8 byte values to a 9 byte buffer
         byte[] expected = {
             (byte) 0x01,
@@ -253,7 +253,7 @@ public final class TestLittleEndian {
      * test the readShort method
      */
     @Test
-    public void testReadShort() throws IOException {
+    void testReadShort() throws IOException {
         short       expected_value = 0x0201;
         InputStream stream         = new ByteArrayInputStream(_good_array);
         int         count          = 0;
@@ -277,7 +277,7 @@ public final class TestLittleEndian {
      * test the readInt method
      */
     @Test
-    public void testReadInt() throws IOException {
+    void testReadInt() throws IOException {
         int         expected_value = 0x02010201;
         InputStream stream         = new ByteArrayInputStream(_good_array);
         int         count          = 0;
@@ -300,7 +300,7 @@ public final class TestLittleEndian {
      * test the readLong method
      */
     @Test
-    public void testReadLong() throws IOException {
+    void testReadLong() throws IOException {
         long        expected_value = 0x0201020102010201L;
         InputStream stream         = new ByteArrayInputStream(_good_array);
         int         count          = 0;
@@ -320,7 +320,7 @@ public final class TestLittleEndian {
     }
 
     @Test
-    public void testReadFromStream() throws IOException {
+    void testReadFromStream() throws IOException {
         int actual;
         actual = LittleEndian.readUShort(new ByteArrayInputStream(new byte[] { 
5, -128, }));
         assertEquals(32773, actual);
@@ -333,7 +333,7 @@ public final class TestLittleEndian {
     }
 
     @Test
-    public void testUnsignedByteToInt() {
+    void testUnsignedByteToInt() {
         assertEquals(255, LittleEndian.ubyteToInt((byte)255));
     }
 
@@ -350,7 +350,7 @@ public final class TestLittleEndian {
     }
 
     @Test
-    public void testUnsignedShort() {
+    void testUnsignedShort() {
         assertEquals(0xffff, LittleEndian.getUShort(new byte[] { (byte)0xff, 
(byte)0xff }, 0));
     }
 }

Modified: 
poi/trunk/src/testcases/org/apache/poi/util/TestLittleEndianStreams.java
URL: 
http://svn.apache.org/viewvc/poi/trunk/src/testcases/org/apache/poi/util/TestLittleEndianStreams.java?rev=1885281&r1=1885280&r2=1885281&view=diff
==============================================================================
--- poi/trunk/src/testcases/org/apache/poi/util/TestLittleEndianStreams.java 
(original)
+++ poi/trunk/src/testcases/org/apache/poi/util/TestLittleEndianStreams.java 
Fri Jan  8 23:50:02 2021
@@ -35,7 +35,7 @@ import org.junit.jupiter.api.Test;
 public final class TestLittleEndianStreams {
 
        @Test
-       public void testRead() throws IOException {
+       void testRead() throws IOException {
                ByteArrayOutputStream baos = new ByteArrayOutputStream();
                try (LittleEndianOutputStream leo = new 
LittleEndianOutputStream(baos)) {
                        leo.writeInt(12345678);
@@ -64,7 +64,7 @@ public final class TestLittleEndianStrea
         * array.
         */
        @Test
-       public void testReadFully() {
+       void testReadFully() {
                byte[] srcBuf = HexRead.readFromString("99 88 77 66 55 44 33");
                LittleEndianInput lei = new 
LittleEndianByteArrayInputStream(srcBuf);
 
@@ -84,7 +84,7 @@ public final class TestLittleEndianStrea
        }
 
        @Test
-       public void testBufferOverrun() {
+       void testBufferOverrun() {
                byte[] srcBuf = HexRead.readFromString("99 88 77");
                LittleEndianInput lei = new 
LittleEndianByteArrayInputStream(srcBuf);
 
@@ -97,7 +97,7 @@ public final class TestLittleEndianStrea
        }
 
        @Test
-       public void testBufferOverrunStartOffset() {
+       void testBufferOverrunStartOffset() {
                byte[] srcBuf = HexRead.readFromString("99 88 77 88 99");
                LittleEndianInput lei = new 
LittleEndianByteArrayInputStream(srcBuf, 2);
 
@@ -107,7 +107,7 @@ public final class TestLittleEndianStrea
        }
 
        @Test
-       public void testBufferOverrunStartOffset2() {
+       void testBufferOverrunStartOffset2() {
                byte[] srcBuf = HexRead.readFromString("99 88 77 88 99");
                LittleEndianInput lei = new 
LittleEndianByteArrayInputStream(srcBuf, 2, 2);
 

Modified: poi/trunk/src/testcases/org/apache/poi/util/TestLongField.java
URL: 
http://svn.apache.org/viewvc/poi/trunk/src/testcases/org/apache/poi/util/TestLongField.java?rev=1885281&r1=1885280&r2=1885281&view=diff
==============================================================================
--- poi/trunk/src/testcases/org/apache/poi/util/TestLongField.java (original)
+++ poi/trunk/src/testcases/org/apache/poi/util/TestLongField.java Fri Jan  8 
23:50:02 2021
@@ -37,7 +37,7 @@ public final class TestLongField {
     };
 
     @Test
-    public void testConstructors() {
+    void testConstructors() {
         assertThrows(ArrayIndexOutOfBoundsException.class, () -> new 
LongField(-1));
 
         LongField field1 = new LongField(2);
@@ -73,7 +73,7 @@ public final class TestLongField {
     }
 
     @Test
-    public void testSet() {
+    void testSet() {
         LongField field = new LongField(0);
         byte[]    array = new byte[ 8 ];
 
@@ -95,7 +95,7 @@ public final class TestLongField {
     }
 
     @Test
-    public void testReadFromBytes() {
+    void testReadFromBytes() {
         LongField field = new LongField(1);
         byte[]    array = new byte[ 8 ];
 
@@ -121,7 +121,7 @@ public final class TestLongField {
     }
 
     @Test
-    public void testReadFromStream() throws IOException {
+    void testReadFromStream() throws IOException {
         LongField field  = new LongField(0);
         byte[]    buffer = new byte[ _test_array.length * 8 ];
 
@@ -144,7 +144,7 @@ public final class TestLongField {
     }
 
     @Test
-    public void testWriteToBytes() {
+    void testWriteToBytes() {
         LongField field = new LongField(0);
         byte[]    array = new byte[ 8 ];
 

Modified: poi/trunk/src/testcases/org/apache/poi/util/TestPOILogFactory.java
URL: 
http://svn.apache.org/viewvc/poi/trunk/src/testcases/org/apache/poi/util/TestPOILogFactory.java?rev=1885281&r1=1885280&r2=1885281&view=diff
==============================================================================
--- poi/trunk/src/testcases/org/apache/poi/util/TestPOILogFactory.java 
(original)
+++ poi/trunk/src/testcases/org/apache/poi/util/TestPOILogFactory.java Fri Jan  
8 23:50:02 2021
@@ -26,7 +26,7 @@ public final class TestPOILogFactory {
      * test log creation
      */
     @Test
-    public void testLog() {
+    void testLog() {
         //NKB Testing only that logging classes use gives no exception
         //    Since logging can be disabled, no checking of logging
         //    output is done.

Modified: poi/trunk/src/testcases/org/apache/poi/util/TestPOILogger.java
URL: 
http://svn.apache.org/viewvc/poi/trunk/src/testcases/org/apache/poi/util/TestPOILogger.java?rev=1885281&r1=1885280&r2=1885281&view=diff
==============================================================================
--- poi/trunk/src/testcases/org/apache/poi/util/TestPOILogger.java (original)
+++ poi/trunk/src/testcases/org/apache/poi/util/TestPOILogger.java Fri Jan  8 
23:50:02 2021
@@ -35,7 +35,7 @@ public final class TestPOILogger impleme
      * Test different types of log output.
      */
     @Test
-    public void testVariousLogTypes() throws Exception {
+    void testVariousLogTypes() throws Exception {
         String oldLCN = POILogFactory._loggerClassName;
         try {
             POILogFactory._loggerClassName = TestPOILogger.class.getName();

Modified: poi/trunk/src/testcases/org/apache/poi/util/TestShortField.java
URL: 
http://svn.apache.org/viewvc/poi/trunk/src/testcases/org/apache/poi/util/TestShortField.java?rev=1885281&r1=1885280&r2=1885281&view=diff
==============================================================================
--- poi/trunk/src/testcases/org/apache/poi/util/TestShortField.java (original)
+++ poi/trunk/src/testcases/org/apache/poi/util/TestShortField.java Fri Jan  8 
23:50:02 2021
@@ -34,7 +34,7 @@ public final class TestShortField {
     private static final short[] _test_array = {Short.MIN_VALUE, -1, 0, 1, 
Short.MAX_VALUE};
 
     @Test
-    public void testConstructors() {
+    void testConstructors() {
         assertThrows(ArrayIndexOutOfBoundsException.class, () -> new 
ShortField(-1));
         ShortField field = new ShortField(2);
 
@@ -61,7 +61,7 @@ public final class TestShortField {
     }
 
     @Test
-    public void testSet() {
+    void testSet() {
         ShortField field = new ShortField(0);
         byte[]     array = new byte[ 2 ];
 
@@ -77,7 +77,7 @@ public final class TestShortField {
     }
 
     @Test
-    public void testReadFromBytes() {
+    void testReadFromBytes() {
         ShortField field1 = new ShortField(1);
         byte[]     array = new byte[ 2 ];
 
@@ -94,7 +94,7 @@ public final class TestShortField {
     }
 
     @Test
-    public void testReadFromStream() throws IOException {
+    void testReadFromStream() throws IOException {
         ShortField field  = new ShortField(0);
         byte[]     buffer = new byte[ _test_array.length * 2 ];
 
@@ -113,7 +113,7 @@ public final class TestShortField {
     }
 
     @Test
-    public void testWriteToBytes() {
+    void testWriteToBytes() {
         ShortField field = new ShortField(0);
         byte[]     array = new byte[ 2 ];
 

Modified: 
poi/trunk/src/testcases/org/apache/poi/util/TestStringCodepointsIterable.java
URL: 
http://svn.apache.org/viewvc/poi/trunk/src/testcases/org/apache/poi/util/TestStringCodepointsIterable.java?rev=1885281&r1=1885280&r2=1885281&view=diff
==============================================================================
--- 
poi/trunk/src/testcases/org/apache/poi/util/TestStringCodepointsIterable.java 
(original)
+++ 
poi/trunk/src/testcases/org/apache/poi/util/TestStringCodepointsIterable.java 
Fri Jan  8 23:50:02 2021
@@ -30,7 +30,7 @@ import org.junit.jupiter.api.Test;
 public class TestStringCodepointsIterable {
 
     @Test
-    public void testIterable() {
+    void testIterable() {
         final String unicodeSurrogates = 
"\uD835\uDF4A\uD835\uDF4B\uD835\uDF4C\uD835\uDF4D\uD835\uDF4E"
                 + "abcdef123456";
         StringCodepointsIterable sci = new 
StringCodepointsIterable(unicodeSurrogates);

Modified: poi/trunk/src/testcases/org/apache/poi/util/TestStringUtil.java
URL: 
http://svn.apache.org/viewvc/poi/trunk/src/testcases/org/apache/poi/util/TestStringUtil.java?rev=1885281&r1=1885280&r2=1885281&view=diff
==============================================================================
--- poi/trunk/src/testcases/org/apache/poi/util/TestStringUtil.java (original)
+++ poi/trunk/src/testcases/org/apache/poi/util/TestStringUtil.java Fri Jan  8 
23:50:02 2021
@@ -36,7 +36,7 @@ public class TestStringUtil {
      * test getFromUnicodeHigh for symbols with code below and more 127
      */
     @Test
-    public void testGetFromUnicodeHighSymbolsWithCodesMoreThan127() {
+    void testGetFromUnicodeHighSymbolsWithCodesMoreThan127() {
         byte[] test_data = new byte[]{0x22, 0x04,
                                       0x35, 0x04,
                                       0x41, 0x04,
@@ -54,7 +54,7 @@ public class TestStringUtil {
     }
 
     @Test
-    public void testPutCompressedUnicode() {
+    void testPutCompressedUnicode() {
         byte[] output = new byte[100];
         byte[] expected_output =
                 {
@@ -81,7 +81,7 @@ public class TestStringUtil {
     }
 
     @Test
-    public void testPutUncompressedUnicode() {
+    void testPutUncompressedUnicode() {
         byte[] output = new byte[100];
         String input = "Hello World";
         byte[] expected_output = {

Modified: poi/trunk/src/testcases/org/apache/poi/util/TestTempFile.java
URL: 
http://svn.apache.org/viewvc/poi/trunk/src/testcases/org/apache/poi/util/TestTempFile.java?rev=1885281&r1=1885280&r2=1885281&view=diff
==============================================================================
--- poi/trunk/src/testcases/org/apache/poi/util/TestTempFile.java (original)
+++ poi/trunk/src/testcases/org/apache/poi/util/TestTempFile.java Fri Jan  8 
23:50:02 2021
@@ -86,7 +86,7 @@ public class TestTempFile {
     }
 
     @Test
-    public void testCreateTempFile() throws IOException
+    void testCreateTempFile() throws IOException
     {
         File tempFile = TempFile.createTempFile("test", ".txt");
         FileOutputStream fos = new FileOutputStream(tempFile);
@@ -110,7 +110,7 @@ public class TestTempFile {
     }
 
     @Test
-    public void testCreateTempDirectory() throws IOException
+    void testCreateTempDirectory() throws IOException
     {
         File tempDir = TempFile.createTempDirectory("testDir");
         assertTrue(tempDir.exists());
@@ -124,7 +124,7 @@ public class TestTempFile {
     }
 
     @Test
-    public void testSetTempFileCreationStrategy() throws IOException {
+    void testSetTempFileCreationStrategy() throws IOException {
         TempFile.setTempFileCreationStrategy(new 
DefaultTempFileCreationStrategy());
 
         // Should be able to create two tempfiles with same prefix and suffix

Modified: poi/trunk/src/testcases/org/apache/poi/util/TestXMLHelper.java
URL: 
http://svn.apache.org/viewvc/poi/trunk/src/testcases/org/apache/poi/util/TestXMLHelper.java?rev=1885281&r1=1885280&r2=1885281&view=diff
==============================================================================
--- poi/trunk/src/testcases/org/apache/poi/util/TestXMLHelper.java (original)
+++ poi/trunk/src/testcases/org/apache/poi/util/TestXMLHelper.java Fri Jan  8 
23:50:02 2021
@@ -43,7 +43,7 @@ import org.xml.sax.XMLReader;
 
 public class TestXMLHelper {
     @Test
-    public void testDocumentBuilder() throws Exception {
+    void testDocumentBuilder() throws Exception {
         DocumentBuilder documentBuilder = XMLHelper.newDocumentBuilder();
         assertNotSame(documentBuilder, XMLHelper.newDocumentBuilder());
         assertTrue(documentBuilder.isNamespaceAware());
@@ -52,7 +52,7 @@ public class TestXMLHelper {
     }
 
     @Test
-    public void testCreatingManyDocumentBuilders() throws Exception {
+    void testCreatingManyDocumentBuilders() throws Exception {
         int limit = 1000;
         ArrayList<CompletableFuture<DocumentBuilder>> futures = new 
ArrayList<>();
         for (int i = 0; i < limit; i++) {
@@ -68,7 +68,7 @@ public class TestXMLHelper {
     }
 
     @Test
-    public void testDocumentBuilderFactory() throws Exception {
+    void testDocumentBuilderFactory() throws Exception {
         try {
             DocumentBuilderFactory dbf = XMLHelper.getDocumentBuilderFactory();
             assertTrue(dbf.getFeature(XMLConstants.FEATURE_SECURE_PROCESSING));
@@ -81,7 +81,7 @@ public class TestXMLHelper {
     }
 
     @Test
-    public void testXMLReader() throws Exception {
+    void testXMLReader() throws Exception {
         XMLReader reader = XMLHelper.newXMLReader();
         assertNotSame(reader, XMLHelper.newXMLReader());
         try {
@@ -100,7 +100,7 @@ public class TestXMLHelper {
     }
 
     @Test
-    public void testCreatingManyXMLReaders() throws Exception {
+    void testCreatingManyXMLReaders() throws Exception {
         int limit = 1000;
         ArrayList<CompletableFuture<XMLReader>> futures = new ArrayList<>();
         for (int i = 0; i < limit; i++) {
@@ -134,7 +134,7 @@ public class TestXMLHelper {
      * test that newXMLInputFactory returns a factory with sensible defaults
      */
     @Test
-    public void testNewXMLInputFactory() {
+    void testNewXMLInputFactory() {
         XMLInputFactory factory = XMLHelper.newXMLInputFactory();
         
assertTrue((boolean)factory.getProperty(XMLInputFactory.IS_NAMESPACE_AWARE));
         
assertFalse((boolean)factory.getProperty(XMLInputFactory.IS_VALIDATING));
@@ -146,7 +146,7 @@ public class TestXMLHelper {
      * test that newXMLOutputFactory returns a factory with sensible defaults
      */
     @Test
-    public void testNewXMLOutputFactory() {
+    void testNewXMLOutputFactory() {
         XMLOutputFactory factory = XMLHelper.newXMLOutputFactory();
         
assertTrue((boolean)factory.getProperty(XMLOutputFactory.IS_REPAIRING_NAMESPACES));
     }
@@ -155,7 +155,7 @@ public class TestXMLHelper {
      * test that newXMLEventFactory returns a factory
      */
     @Test
-    public void testNewXMLEventFactory() {
+    void testNewXMLEventFactory() {
         assertNotNull(XMLHelper.newXMLEventFactory());
     }
 }



---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscr...@poi.apache.org
For additional commands, e-mail: commits-h...@poi.apache.org

Reply via email to