Use JUnit 5 convention for class visibility
diff --git a/src/test/java/org/apache/commons/crypto/CryptoTest.java b/src/test/java/org/apache/commons/crypto/CryptoTest.java
index 812b6da..44f0bbb 100644
--- a/src/test/java/org/apache/commons/crypto/CryptoTest.java
+++ b/src/test/java/org/apache/commons/crypto/CryptoTest.java
@@ -22,7 +22,7 @@
import org.junit.jupiter.api.Test;
-public class CryptoTest {
+class CryptoTest {
// If defined, then fail if the version does not match major/minor bits
private static final String EXPECTED_VERSION_PROPERTY = "CryptoTest.expectedVersion";
diff --git a/src/test/java/org/apache/commons/crypto/NativeCodeLoaderTest.java b/src/test/java/org/apache/commons/crypto/NativeCodeLoaderTest.java
index ff2abca..313d0ae 100644
--- a/src/test/java/org/apache/commons/crypto/NativeCodeLoaderTest.java
+++ b/src/test/java/org/apache/commons/crypto/NativeCodeLoaderTest.java
@@ -30,7 +30,7 @@
import org.junit.jupiter.api.Disabled;
import org.junit.jupiter.api.Test;
-public class NativeCodeLoaderTest {
+class NativeCodeLoaderTest {
@Test
void test() {
diff --git a/src/test/java/org/apache/commons/crypto/OsInfoTest.java b/src/test/java/org/apache/commons/crypto/OsInfoTest.java
index 4a5da73..5f6b83b 100644
--- a/src/test/java/org/apache/commons/crypto/OsInfoTest.java
+++ b/src/test/java/org/apache/commons/crypto/OsInfoTest.java
@@ -21,7 +21,7 @@
import org.junit.jupiter.api.Test;
-public class OsInfoTest {
+class OsInfoTest {
private static final String EXPECTED_PATH_PROPERTY = "OsInfoTest.expectedPath";
@Test
diff --git a/src/test/java/org/apache/commons/crypto/cipher/CryptoCipherFactoryTest.java b/src/test/java/org/apache/commons/crypto/cipher/CryptoCipherFactoryTest.java
index 2a23248..2224c44 100644
--- a/src/test/java/org/apache/commons/crypto/cipher/CryptoCipherFactoryTest.java
+++ b/src/test/java/org/apache/commons/crypto/cipher/CryptoCipherFactoryTest.java
@@ -27,7 +27,7 @@
import org.apache.commons.crypto.utils.AES;
import org.junit.jupiter.api.Test;
-public class CryptoCipherFactoryTest {
+class CryptoCipherFactoryTest {
@Test
void testDefaultCipher() throws GeneralSecurityException, IOException {
diff --git a/src/test/java/org/apache/commons/crypto/cipher/CryptoCipherTest.java b/src/test/java/org/apache/commons/crypto/cipher/CryptoCipherTest.java
index 8d53f8e..c853091 100644
--- a/src/test/java/org/apache/commons/crypto/cipher/CryptoCipherTest.java
+++ b/src/test/java/org/apache/commons/crypto/cipher/CryptoCipherTest.java
@@ -27,7 +27,7 @@
/**
* Tests default methods.
*/
-public class CryptoCipherTest {
+class CryptoCipherTest {
@Test
void testUpdateAADByteArray() {
diff --git a/src/test/java/org/apache/commons/crypto/cipher/GcmCipherTest.java b/src/test/java/org/apache/commons/crypto/cipher/GcmCipherTest.java
index 1ad0f03..9602cfe 100644
--- a/src/test/java/org/apache/commons/crypto/cipher/GcmCipherTest.java
+++ b/src/test/java/org/apache/commons/crypto/cipher/GcmCipherTest.java
@@ -38,7 +38,7 @@
import jakarta.xml.bind.DatatypeConverter;
-public class GcmCipherTest {
+class GcmCipherTest {
private static final String GCM_NO_PADDING = "AES/GCM/NoPadding";
Properties props;
diff --git a/src/test/java/org/apache/commons/crypto/cipher/JceCipherTest.java b/src/test/java/org/apache/commons/crypto/cipher/JceCipherTest.java
index 67f6c10..2d8c61e 100644
--- a/src/test/java/org/apache/commons/crypto/cipher/JceCipherTest.java
+++ b/src/test/java/org/apache/commons/crypto/cipher/JceCipherTest.java
@@ -27,7 +27,7 @@
import org.apache.commons.crypto.utils.AES;
import org.junit.jupiter.api.BeforeAll;
-public class JceCipherTest extends AbstractCipherTest {
+class JceCipherTest extends AbstractCipherTest {
private static final int MAX_KEY_LEN_LOWER_BOUND = 256;
diff --git a/src/test/java/org/apache/commons/crypto/cipher/OpenSslCipherTest.java b/src/test/java/org/apache/commons/crypto/cipher/OpenSslCipherTest.java
index fb5f416..5799a26 100644
--- a/src/test/java/org/apache/commons/crypto/cipher/OpenSslCipherTest.java
+++ b/src/test/java/org/apache/commons/crypto/cipher/OpenSslCipherTest.java
@@ -39,7 +39,7 @@
import org.junit.jupiter.api.Test;
import org.junit.jupiter.api.Timeout;
-public class OpenSslCipherTest extends AbstractCipherTest {
+class OpenSslCipherTest extends AbstractCipherTest {
private ByteBuffer dummyBuffer() {
return ByteBuffer.allocateDirect(8);
diff --git a/src/test/java/org/apache/commons/crypto/cipher/OpenSslCommonModeTest.java b/src/test/java/org/apache/commons/crypto/cipher/OpenSslCommonModeTest.java
index 7d943b1..6daf8f2 100644
--- a/src/test/java/org/apache/commons/crypto/cipher/OpenSslCommonModeTest.java
+++ b/src/test/java/org/apache/commons/crypto/cipher/OpenSslCommonModeTest.java
@@ -22,7 +22,7 @@
import org.junit.jupiter.api.Test;
-public class OpenSslCommonModeTest {
+class OpenSslCommonModeTest {
@Test
void testUpdateAAD() {
diff --git a/src/test/java/org/apache/commons/crypto/jna/CbcNoPaddingCipherJnaStreamTest.java b/src/test/java/org/apache/commons/crypto/jna/CbcNoPaddingCipherJnaStreamTest.java
index 69439e3..0250518 100644
--- a/src/test/java/org/apache/commons/crypto/jna/CbcNoPaddingCipherJnaStreamTest.java
+++ b/src/test/java/org/apache/commons/crypto/jna/CbcNoPaddingCipherJnaStreamTest.java
@@ -19,7 +19,7 @@
import org.apache.commons.crypto.utils.AES;
-public class CbcNoPaddingCipherJnaStreamTest extends AbstractCipherJnaStreamTest {
+class CbcNoPaddingCipherJnaStreamTest extends AbstractCipherJnaStreamTest {
@Override
public void setUp() {
diff --git a/src/test/java/org/apache/commons/crypto/jna/CbcPkcs5PaddingCipherJnaStreamTest.java b/src/test/java/org/apache/commons/crypto/jna/CbcPkcs5PaddingCipherJnaStreamTest.java
index 2d78e2a..58c690b 100644
--- a/src/test/java/org/apache/commons/crypto/jna/CbcPkcs5PaddingCipherJnaStreamTest.java
+++ b/src/test/java/org/apache/commons/crypto/jna/CbcPkcs5PaddingCipherJnaStreamTest.java
@@ -19,7 +19,7 @@
import org.apache.commons.crypto.utils.AES;
-public class CbcPkcs5PaddingCipherJnaStreamTest extends AbstractCipherJnaStreamTest {
+class CbcPkcs5PaddingCipherJnaStreamTest extends AbstractCipherJnaStreamTest {
@Override
public void setUp() {
diff --git a/src/test/java/org/apache/commons/crypto/jna/CtrCryptoJnaStreamTest.java b/src/test/java/org/apache/commons/crypto/jna/CtrCryptoJnaStreamTest.java
index 917552c..52a48a6 100644
--- a/src/test/java/org/apache/commons/crypto/jna/CtrCryptoJnaStreamTest.java
+++ b/src/test/java/org/apache/commons/crypto/jna/CtrCryptoJnaStreamTest.java
@@ -19,7 +19,7 @@
import org.apache.commons.crypto.utils.AES;
-public class CtrCryptoJnaStreamTest extends AbstractCipherJnaStreamTest {
+class CtrCryptoJnaStreamTest extends AbstractCipherJnaStreamTest {
@Override
public void setUp() {
diff --git a/src/test/java/org/apache/commons/crypto/jna/CtrNoPaddingCipherJnaStreamTest.java b/src/test/java/org/apache/commons/crypto/jna/CtrNoPaddingCipherJnaStreamTest.java
index 123cd7c..54ca8e2 100644
--- a/src/test/java/org/apache/commons/crypto/jna/CtrNoPaddingCipherJnaStreamTest.java
+++ b/src/test/java/org/apache/commons/crypto/jna/CtrNoPaddingCipherJnaStreamTest.java
@@ -19,7 +19,7 @@
import org.apache.commons.crypto.utils.AES;
-public class CtrNoPaddingCipherJnaStreamTest extends AbstractCipherJnaStreamTest {
+class CtrNoPaddingCipherJnaStreamTest extends AbstractCipherJnaStreamTest {
@Override
public void setUp() {
diff --git a/src/test/java/org/apache/commons/crypto/jna/OpenSslJnaCipherTest.java b/src/test/java/org/apache/commons/crypto/jna/OpenSslJnaCipherTest.java
index 9b1380c..5e6c103 100644
--- a/src/test/java/org/apache/commons/crypto/jna/OpenSslJnaCipherTest.java
+++ b/src/test/java/org/apache/commons/crypto/jna/OpenSslJnaCipherTest.java
@@ -22,7 +22,7 @@
import org.apache.commons.crypto.cipher.AbstractCipherTest;
-public class OpenSslJnaCipherTest extends AbstractCipherTest {
+class OpenSslJnaCipherTest extends AbstractCipherTest {
@Override
public void init() {
diff --git a/src/test/java/org/apache/commons/crypto/jna/OpenSslJnaCryptoRandomTest.java b/src/test/java/org/apache/commons/crypto/jna/OpenSslJnaCryptoRandomTest.java
index 96ef918..242c2f5 100644
--- a/src/test/java/org/apache/commons/crypto/jna/OpenSslJnaCryptoRandomTest.java
+++ b/src/test/java/org/apache/commons/crypto/jna/OpenSslJnaCryptoRandomTest.java
@@ -28,7 +28,7 @@
import org.junit.jupiter.api.Assumptions;
import org.junit.jupiter.api.BeforeEach;
-public class OpenSslJnaCryptoRandomTest extends AbstractRandomTest {
+class OpenSslJnaCryptoRandomTest extends AbstractRandomTest {
@Override
public CryptoRandom getCryptoRandom() throws GeneralSecurityException {
diff --git a/src/test/java/org/apache/commons/crypto/jna/OpenSslJnaTest.java b/src/test/java/org/apache/commons/crypto/jna/OpenSslJnaTest.java
index 46ba739..590a0d0 100644
--- a/src/test/java/org/apache/commons/crypto/jna/OpenSslJnaTest.java
+++ b/src/test/java/org/apache/commons/crypto/jna/OpenSslJnaTest.java
@@ -21,7 +21,7 @@
import org.junit.jupiter.api.Test;
-public class OpenSslJnaTest {
+class OpenSslJnaTest {
// If defined, then fail if the version does not match major/minor bits
private static final String EXPECTED_VERSION_PROPERTY = "OpenSslJnaTest.expectedVersion";
diff --git a/src/test/java/org/apache/commons/crypto/jna/OpenSslNativeJnaTest.java b/src/test/java/org/apache/commons/crypto/jna/OpenSslNativeJnaTest.java
index c6dc028..bf334f5 100644
--- a/src/test/java/org/apache/commons/crypto/jna/OpenSslNativeJnaTest.java
+++ b/src/test/java/org/apache/commons/crypto/jna/OpenSslNativeJnaTest.java
@@ -23,7 +23,7 @@
import org.junit.jupiter.api.Test;
import org.junit.jupiter.api.TestReporter;
-public class OpenSslNativeJnaTest {
+class OpenSslNativeJnaTest {
@Test
void test(final TestReporter reporter) {
diff --git a/src/test/java/org/apache/commons/crypto/jna/PositionedCryptoInputStreamJnaTest.java b/src/test/java/org/apache/commons/crypto/jna/PositionedCryptoInputStreamJnaTest.java
index 66c4f84..cba5ebd 100644
--- a/src/test/java/org/apache/commons/crypto/jna/PositionedCryptoInputStreamJnaTest.java
+++ b/src/test/java/org/apache/commons/crypto/jna/PositionedCryptoInputStreamJnaTest.java
@@ -24,7 +24,7 @@
/**
*/
-public class PositionedCryptoInputStreamJnaTest extends PositionedCryptoInputStreamTest {
+class PositionedCryptoInputStreamJnaTest extends PositionedCryptoInputStreamTest {
@BeforeEach
public void init() {
diff --git a/src/test/java/org/apache/commons/crypto/random/CryptoRandomFactoryTest.java b/src/test/java/org/apache/commons/crypto/random/CryptoRandomFactoryTest.java
index e09159c..81e4bbc 100644
--- a/src/test/java/org/apache/commons/crypto/random/CryptoRandomFactoryTest.java
+++ b/src/test/java/org/apache/commons/crypto/random/CryptoRandomFactoryTest.java
@@ -30,7 +30,7 @@
import org.apache.commons.lang3.SystemProperties;
import org.junit.jupiter.api.Test;
-public class CryptoRandomFactoryTest {
+class CryptoRandomFactoryTest {
@Test
void testAbstractRandom() {
diff --git a/src/test/java/org/apache/commons/crypto/random/JavaCryptoRandomTest.java b/src/test/java/org/apache/commons/crypto/random/JavaCryptoRandomTest.java
index 359d669..0a26b3a 100644
--- a/src/test/java/org/apache/commons/crypto/random/JavaCryptoRandomTest.java
+++ b/src/test/java/org/apache/commons/crypto/random/JavaCryptoRandomTest.java
@@ -22,7 +22,7 @@
import java.security.GeneralSecurityException;
import java.util.Properties;
-public class JavaCryptoRandomTest extends AbstractRandomTest {
+class JavaCryptoRandomTest extends AbstractRandomTest {
@Override
public CryptoRandom getCryptoRandom() throws GeneralSecurityException {
diff --git a/src/test/java/org/apache/commons/crypto/random/OpenSslCryptoRandomTest.java b/src/test/java/org/apache/commons/crypto/random/OpenSslCryptoRandomTest.java
index 09dac85..558ac2c 100644
--- a/src/test/java/org/apache/commons/crypto/random/OpenSslCryptoRandomTest.java
+++ b/src/test/java/org/apache/commons/crypto/random/OpenSslCryptoRandomTest.java
@@ -26,7 +26,7 @@
import org.apache.commons.crypto.Crypto;
import org.junit.jupiter.api.BeforeAll;
-public class OpenSslCryptoRandomTest extends AbstractRandomTest {
+class OpenSslCryptoRandomTest extends AbstractRandomTest {
@BeforeAll
public static void beforeAll() {
diff --git a/src/test/java/org/apache/commons/crypto/random/OsCryptoRandomTest.java b/src/test/java/org/apache/commons/crypto/random/OsCryptoRandomTest.java
index d6a09f3..6d3d3d9 100644
--- a/src/test/java/org/apache/commons/crypto/random/OsCryptoRandomTest.java
+++ b/src/test/java/org/apache/commons/crypto/random/OsCryptoRandomTest.java
@@ -30,7 +30,7 @@
import org.apache.commons.lang3.SystemProperties;
import org.junit.jupiter.api.Test;
-public class OsCryptoRandomTest extends AbstractRandomTest {
+class OsCryptoRandomTest extends AbstractRandomTest {
@Override
public CryptoRandom getCryptoRandom() throws GeneralSecurityException {
diff --git a/src/test/java/org/apache/commons/crypto/stream/CbcNoPaddingCipherStreamTest.java b/src/test/java/org/apache/commons/crypto/stream/CbcNoPaddingCipherStreamTest.java
index 3aa6dee..df21206 100644
--- a/src/test/java/org/apache/commons/crypto/stream/CbcNoPaddingCipherStreamTest.java
+++ b/src/test/java/org/apache/commons/crypto/stream/CbcNoPaddingCipherStreamTest.java
@@ -19,7 +19,7 @@
import org.apache.commons.crypto.utils.AES;
-public class CbcNoPaddingCipherStreamTest extends AbstractCipherStreamTest {
+class CbcNoPaddingCipherStreamTest extends AbstractCipherStreamTest {
@Override
public void setUp() {
diff --git a/src/test/java/org/apache/commons/crypto/stream/CbcPkcs5PaddingCipherStreamTest.java b/src/test/java/org/apache/commons/crypto/stream/CbcPkcs5PaddingCipherStreamTest.java
index 57a0445..fd0104c 100644
--- a/src/test/java/org/apache/commons/crypto/stream/CbcPkcs5PaddingCipherStreamTest.java
+++ b/src/test/java/org/apache/commons/crypto/stream/CbcPkcs5PaddingCipherStreamTest.java
@@ -19,7 +19,7 @@
import org.apache.commons.crypto.utils.AES;
-public class CbcPkcs5PaddingCipherStreamTest extends AbstractCipherStreamTest {
+class CbcPkcs5PaddingCipherStreamTest extends AbstractCipherStreamTest {
@Override
public void setUp() {
diff --git a/src/test/java/org/apache/commons/crypto/stream/CtrCryptoStreamTest.java b/src/test/java/org/apache/commons/crypto/stream/CtrCryptoStreamTest.java
index 83409c9..e2c8976 100644
--- a/src/test/java/org/apache/commons/crypto/stream/CtrCryptoStreamTest.java
+++ b/src/test/java/org/apache/commons/crypto/stream/CtrCryptoStreamTest.java
@@ -43,7 +43,7 @@
import org.junit.jupiter.api.Test;
import org.junit.jupiter.api.Timeout;
-public class CtrCryptoStreamTest extends AbstractCipherStreamTest {
+class CtrCryptoStreamTest extends AbstractCipherStreamTest {
protected void doDecryptTest(final String cipherClass, final boolean withChannel)
throws IOException {
diff --git a/src/test/java/org/apache/commons/crypto/stream/CtrNoPaddingCipherStreamTest.java b/src/test/java/org/apache/commons/crypto/stream/CtrNoPaddingCipherStreamTest.java
index 0c0c7f9..09c8aa8 100644
--- a/src/test/java/org/apache/commons/crypto/stream/CtrNoPaddingCipherStreamTest.java
+++ b/src/test/java/org/apache/commons/crypto/stream/CtrNoPaddingCipherStreamTest.java
@@ -19,7 +19,7 @@
import org.apache.commons.crypto.utils.AES;
-public class CtrNoPaddingCipherStreamTest extends AbstractCipherStreamTest {
+class CtrNoPaddingCipherStreamTest extends AbstractCipherStreamTest {
@Override
public void setUp() {
diff --git a/src/test/java/org/apache/commons/crypto/stream/input/ChannelInputTest.java b/src/test/java/org/apache/commons/crypto/stream/input/ChannelInputTest.java
index ebdd25a..b44bf37 100644
--- a/src/test/java/org/apache/commons/crypto/stream/input/ChannelInputTest.java
+++ b/src/test/java/org/apache/commons/crypto/stream/input/ChannelInputTest.java
@@ -29,7 +29,7 @@
/**
* Tests {@link ChannelInput}.
*/
-public class ChannelInputTest {
+class ChannelInputTest {
@Test
void testSkipWithSkipBuffer() throws IOException {
diff --git a/src/test/java/org/apache/commons/crypto/stream/output/StreamOutputTest.java b/src/test/java/org/apache/commons/crypto/stream/output/StreamOutputTest.java
index 9973b1b..6181e7b 100644
--- a/src/test/java/org/apache/commons/crypto/stream/output/StreamOutputTest.java
+++ b/src/test/java/org/apache/commons/crypto/stream/output/StreamOutputTest.java
@@ -28,7 +28,7 @@
/**
* Tests {@link StreamOutput}.
*/
-public class StreamOutputTest {
+class StreamOutputTest {
@Test
void testGetOut() throws IOException {
diff --git a/src/test/java/org/apache/commons/crypto/utils/EnumTest.java b/src/test/java/org/apache/commons/crypto/utils/EnumTest.java
index bc6d58a..ffcb408 100644
--- a/src/test/java/org/apache/commons/crypto/utils/EnumTest.java
+++ b/src/test/java/org/apache/commons/crypto/utils/EnumTest.java
@@ -31,7 +31,7 @@
/**
* Test the enums used to define the internal implementation classes
*/
-public class EnumTest {
+class EnumTest {
private void checkImplClass(final CipherProvider value) {
final Class<? extends CryptoCipher> implClass = value.getImplClass();
diff --git a/src/test/java/org/apache/commons/crypto/utils/UtilsTest.java b/src/test/java/org/apache/commons/crypto/utils/UtilsTest.java
index 0ec7f91..9f952dc 100644
--- a/src/test/java/org/apache/commons/crypto/utils/UtilsTest.java
+++ b/src/test/java/org/apache/commons/crypto/utils/UtilsTest.java
@@ -26,7 +26,7 @@
import org.junit.jupiter.api.Test;
-public class UtilsTest {
+class UtilsTest {
@Test
void testGetProperties() {
final Properties props = new Properties();
