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-pool.git
The following commit(s) were added to refs/heads/master by this push:
new 8e3a2983 Use JUnit 5 convention for class visibility
8e3a2983 is described below
commit 8e3a298394eddfd135c3bf9faf013b0de672e2b0
Author: Gary D. Gregory <[email protected]>
AuthorDate: Mon Jun 9 07:15:32 2025 -0400
Use JUnit 5 convention for class visibility
---
src/test/java/org/apache/commons/pool3/PoolTest.java | 2 +-
.../apache/commons/pool3/TestBaseKeyedPooledObjectFactory.java | 2 +-
.../org/apache/commons/pool3/TestBasePoolableObjectFactory.java | 2 +-
src/test/java/org/apache/commons/pool3/TestPoolUtils.java | 2 +-
src/test/java/org/apache/commons/pool3/TestTrackedUse.java | 2 +-
src/test/java/org/apache/commons/pool3/impl/CallStackTest.java | 2 +-
.../java/org/apache/commons/pool3/impl/NoOpCallStackTest.java | 2 +-
.../apache/commons/pool3/impl/TestAbandonedKeyedObjectPool.java | 2 +-
.../org/apache/commons/pool3/impl/TestAbandonedObjectPool.java | 2 +-
.../org/apache/commons/pool3/impl/TestBaseGenericObjectPool.java | 2 +-
.../java/org/apache/commons/pool3/impl/TestCallStackUtils.java | 2 +-
src/test/java/org/apache/commons/pool3/impl/TestConstants.java | 2 +-
.../org/apache/commons/pool3/impl/TestDefaultPooledObject.java | 2 +-
.../apache/commons/pool3/impl/TestDefaultPooledObjectInfo.java | 2 +-
.../java/org/apache/commons/pool3/impl/TestEvictionConfig.java | 2 +-
.../java/org/apache/commons/pool3/impl/TestEvictionTimer.java | 2 +-
.../org/apache/commons/pool3/impl/TestGenericKeyedObjectPool.java | 6 +++---
.../java/org/apache/commons/pool3/impl/TestGenericObjectPool.java | 8 ++++----
.../commons/pool3/impl/TestGenericObjectPoolClassLoaders.java | 2 +-
.../pool3/impl/TestGenericObjectPoolFactoryCreateFailure.java | 2 +-
.../org/apache/commons/pool3/impl/TestLinkedBlockingDeque.java | 2 +-
.../java/org/apache/commons/pool3/impl/TestPoolImplUtils.java | 2 +-
.../org/apache/commons/pool3/impl/TestPooledSoftReference.java | 2 +-
.../commons/pool3/impl/TestResilientPooledObjectFactory.java | 2 +-
.../org/apache/commons/pool3/impl/TestSoftRefOutOfMemory.java | 6 ++++--
.../apache/commons/pool3/impl/TestSoftReferenceObjectPool.java | 2 +-
.../org/apache/commons/pool3/performance/PerformanceTest.java | 2 +-
.../org/apache/commons/pool3/pool407/AbstractPool407Test.java | 2 +-
.../java/org/apache/commons/pool3/pool407/KeyedPool407Test.java | 2 +-
src/test/java/org/apache/commons/pool3/pool407/Pool407Test.java | 2 +-
.../pool3/proxy/TestProxiedKeyedObjectPoolWithCglibProxy.java | 2 +-
.../pool3/proxy/TestProxiedKeyedObjectPoolWithJdkProxy.java | 2 +-
.../commons/pool3/proxy/TestProxiedObjectPoolWithCglibProxy.java | 2 +-
.../commons/pool3/proxy/TestProxiedObjectPoolWithJdkProxy.java | 2 +-
34 files changed, 42 insertions(+), 40 deletions(-)
diff --git a/src/test/java/org/apache/commons/pool3/PoolTest.java
b/src/test/java/org/apache/commons/pool3/PoolTest.java
index 4029f814..5d87c883 100644
--- a/src/test/java/org/apache/commons/pool3/PoolTest.java
+++ b/src/test/java/org/apache/commons/pool3/PoolTest.java
@@ -29,7 +29,7 @@ import org.junit.jupiter.api.Disabled;
import org.junit.jupiter.api.Test;
@Disabled
-public class PoolTest {
+class PoolTest {
private static final class Foo {
}
diff --git
a/src/test/java/org/apache/commons/pool3/TestBaseKeyedPooledObjectFactory.java
b/src/test/java/org/apache/commons/pool3/TestBaseKeyedPooledObjectFactory.java
index e469930c..e97a415f 100644
---
a/src/test/java/org/apache/commons/pool3/TestBaseKeyedPooledObjectFactory.java
+++
b/src/test/java/org/apache/commons/pool3/TestBaseKeyedPooledObjectFactory.java
@@ -23,7 +23,7 @@ import org.junit.jupiter.api.Test;
/**
*/
-public class TestBaseKeyedPooledObjectFactory {
+class TestBaseKeyedPooledObjectFactory {
private static final class TestFactory extends
BaseKeyedPooledObjectFactory<Object, Object, RuntimeException> {
@Override
diff --git
a/src/test/java/org/apache/commons/pool3/TestBasePoolableObjectFactory.java
b/src/test/java/org/apache/commons/pool3/TestBasePoolableObjectFactory.java
index 2c6ddd85..7bbd6d00 100644
--- a/src/test/java/org/apache/commons/pool3/TestBasePoolableObjectFactory.java
+++ b/src/test/java/org/apache/commons/pool3/TestBasePoolableObjectFactory.java
@@ -26,7 +26,7 @@ import org.junit.jupiter.api.Test;
/**
*/
-public class TestBasePoolableObjectFactory {
+class TestBasePoolableObjectFactory {
private static final class TestFactory extends
BasePooledObjectFactory<AtomicInteger, RuntimeException> {
@Override
diff --git a/src/test/java/org/apache/commons/pool3/TestPoolUtils.java
b/src/test/java/org/apache/commons/pool3/TestPoolUtils.java
index 26398da8..8fdc8ca9 100644
--- a/src/test/java/org/apache/commons/pool3/TestPoolUtils.java
+++ b/src/test/java/org/apache/commons/pool3/TestPoolUtils.java
@@ -45,7 +45,7 @@ import org.opentest4j.AssertionFailedError;
*
* TODO Replace our own mocking with a mocking library like Mockito.
*/
-public class TestPoolUtils {
+class TestPoolUtils {
private static class MethodCallLogger implements InvocationHandler {
private final List<String> calledMethods;
diff --git a/src/test/java/org/apache/commons/pool3/TestTrackedUse.java
b/src/test/java/org/apache/commons/pool3/TestTrackedUse.java
index cbb940ce..b97bfb57 100644
--- a/src/test/java/org/apache/commons/pool3/TestTrackedUse.java
+++ b/src/test/java/org/apache/commons/pool3/TestTrackedUse.java
@@ -23,7 +23,7 @@ import java.time.Instant;
import org.junit.jupiter.api.Test;
-public class TestTrackedUse {
+class TestTrackedUse {
final class DefaultTrackedUse implements TrackedUse {
diff --git a/src/test/java/org/apache/commons/pool3/impl/CallStackTest.java
b/src/test/java/org/apache/commons/pool3/impl/CallStackTest.java
index 10be006c..b28067ed 100644
--- a/src/test/java/org/apache/commons/pool3/impl/CallStackTest.java
+++ b/src/test/java/org/apache/commons/pool3/impl/CallStackTest.java
@@ -27,7 +27,7 @@ import org.junit.jupiter.params.ParameterizedTest;
import org.junit.jupiter.params.provider.Arguments;
import org.junit.jupiter.params.provider.MethodSource;
-public class CallStackTest {
+class CallStackTest {
public static Stream<Arguments> data() {
// @formatter:off
diff --git a/src/test/java/org/apache/commons/pool3/impl/NoOpCallStackTest.java
b/src/test/java/org/apache/commons/pool3/impl/NoOpCallStackTest.java
index 704c683f..40f5ef0c 100644
--- a/src/test/java/org/apache/commons/pool3/impl/NoOpCallStackTest.java
+++ b/src/test/java/org/apache/commons/pool3/impl/NoOpCallStackTest.java
@@ -23,7 +23,7 @@ import java.io.StringWriter;
import org.junit.jupiter.api.Test;
-public class NoOpCallStackTest {
+class NoOpCallStackTest {
@Test
void testPrintStackTraceIsNoOp() {
final CallStack stack = NoOpCallStack.INSTANCE;
diff --git
a/src/test/java/org/apache/commons/pool3/impl/TestAbandonedKeyedObjectPool.java
b/src/test/java/org/apache/commons/pool3/impl/TestAbandonedKeyedObjectPool.java
index 1fbd77c2..3d7c414c 100644
---
a/src/test/java/org/apache/commons/pool3/impl/TestAbandonedKeyedObjectPool.java
+++
b/src/test/java/org/apache/commons/pool3/impl/TestAbandonedKeyedObjectPool.java
@@ -44,7 +44,7 @@ import org.junit.jupiter.api.Test;
/**
* Tests for {@link AbandonedConfig}.
*/
-public class TestAbandonedKeyedObjectPool {
+class TestAbandonedKeyedObjectPool {
private final class ConcurrentBorrower extends Thread {
private final ArrayList<PooledTestObject> borrowed;
diff --git
a/src/test/java/org/apache/commons/pool3/impl/TestAbandonedObjectPool.java
b/src/test/java/org/apache/commons/pool3/impl/TestAbandonedObjectPool.java
index f1390da6..5b7be2ec 100644
--- a/src/test/java/org/apache/commons/pool3/impl/TestAbandonedObjectPool.java
+++ b/src/test/java/org/apache/commons/pool3/impl/TestAbandonedObjectPool.java
@@ -113,7 +113,7 @@ final class PooledTestObject implements TrackedUse {
/**
* TestCase for AbandonedObjectPool
*/
-public class TestAbandonedObjectPool {
+class TestAbandonedObjectPool {
private final class ConcurrentBorrower extends Thread {
private final ArrayList<PooledTestObject> borrowed;
diff --git
a/src/test/java/org/apache/commons/pool3/impl/TestBaseGenericObjectPool.java
b/src/test/java/org/apache/commons/pool3/impl/TestBaseGenericObjectPool.java
index 87f7d797..d9fb5ec2 100644
--- a/src/test/java/org/apache/commons/pool3/impl/TestBaseGenericObjectPool.java
+++ b/src/test/java/org/apache/commons/pool3/impl/TestBaseGenericObjectPool.java
@@ -39,7 +39,7 @@ import org.junit.jupiter.api.Timeout;
/**
*/
-public class TestBaseGenericObjectPool {
+class TestBaseGenericObjectPool {
BaseGenericObjectPool<String, TestException> pool;
SimpleFactory factory;
diff --git
a/src/test/java/org/apache/commons/pool3/impl/TestCallStackUtils.java
b/src/test/java/org/apache/commons/pool3/impl/TestCallStackUtils.java
index 6df0d483..d484ef04 100644
--- a/src/test/java/org/apache/commons/pool3/impl/TestCallStackUtils.java
+++ b/src/test/java/org/apache/commons/pool3/impl/TestCallStackUtils.java
@@ -28,7 +28,7 @@ import org.junit.jupiter.api.Test;
/**
* Tests {@link CallStackUtils}.
*/
-public class TestCallStackUtils {
+class TestCallStackUtils {
private static final String MESSAGE_FORMAT = "'Timestamp:' yyyy-MM-dd
HH:mm:ss Z";
diff --git a/src/test/java/org/apache/commons/pool3/impl/TestConstants.java
b/src/test/java/org/apache/commons/pool3/impl/TestConstants.java
index 9e010291..04462da7 100644
--- a/src/test/java/org/apache/commons/pool3/impl/TestConstants.java
+++ b/src/test/java/org/apache/commons/pool3/impl/TestConstants.java
@@ -22,7 +22,7 @@ import java.time.Duration;
/**
* Constants used in tests.
*/
-public class TestConstants {
+class TestConstants {
/**
* A duration of one second.
diff --git
a/src/test/java/org/apache/commons/pool3/impl/TestDefaultPooledObject.java
b/src/test/java/org/apache/commons/pool3/impl/TestDefaultPooledObject.java
index 19d87bce..f4e9107c 100644
--- a/src/test/java/org/apache/commons/pool3/impl/TestDefaultPooledObject.java
+++ b/src/test/java/org/apache/commons/pool3/impl/TestDefaultPooledObject.java
@@ -35,7 +35,7 @@ import org.junit.jupiter.api.Test;
/**
* Tests {@link DefaultPooledObject}.
*/
-public class TestDefaultPooledObject {
+class TestDefaultPooledObject {
/**
* JIRA: POOL-279
diff --git
a/src/test/java/org/apache/commons/pool3/impl/TestDefaultPooledObjectInfo.java
b/src/test/java/org/apache/commons/pool3/impl/TestDefaultPooledObjectInfo.java
index 50005ae2..69df7c3d 100644
---
a/src/test/java/org/apache/commons/pool3/impl/TestDefaultPooledObjectInfo.java
+++
b/src/test/java/org/apache/commons/pool3/impl/TestDefaultPooledObjectInfo.java
@@ -26,7 +26,7 @@ import org.apache.commons.pool3.TestException;
import org.apache.commons.pool3.impl.TestGenericObjectPool.SimpleFactory;
import org.junit.jupiter.api.Test;
-public class TestDefaultPooledObjectInfo {
+class TestDefaultPooledObjectInfo {
@Test
void testGetLastBorrowTrace() throws Exception {
diff --git
a/src/test/java/org/apache/commons/pool3/impl/TestEvictionConfig.java
b/src/test/java/org/apache/commons/pool3/impl/TestEvictionConfig.java
index a20835c3..9cff06cc 100644
--- a/src/test/java/org/apache/commons/pool3/impl/TestEvictionConfig.java
+++ b/src/test/java/org/apache/commons/pool3/impl/TestEvictionConfig.java
@@ -26,7 +26,7 @@ import org.junit.jupiter.api.Test;
/**
* Tests for {@link EvictionConfig}.
*/
-public class TestEvictionConfig {
+class TestEvictionConfig {
@Test
void testConstructor1s() {
diff --git a/src/test/java/org/apache/commons/pool3/impl/TestEvictionTimer.java
b/src/test/java/org/apache/commons/pool3/impl/TestEvictionTimer.java
index 7ec66915..cd475b44 100644
--- a/src/test/java/org/apache/commons/pool3/impl/TestEvictionTimer.java
+++ b/src/test/java/org/apache/commons/pool3/impl/TestEvictionTimer.java
@@ -32,7 +32,7 @@ import org.junit.jupiter.api.Test;
/**
* Tests for {@link EvictionTimer}.
*/
-public class TestEvictionTimer {
+class TestEvictionTimer {
@Test
void testStartStopEvictionTimer() throws Exception {
diff --git
a/src/test/java/org/apache/commons/pool3/impl/TestGenericKeyedObjectPool.java
b/src/test/java/org/apache/commons/pool3/impl/TestGenericKeyedObjectPool.java
index 0a069d8d..72322a86 100644
---
a/src/test/java/org/apache/commons/pool3/impl/TestGenericKeyedObjectPool.java
+++
b/src/test/java/org/apache/commons/pool3/impl/TestGenericKeyedObjectPool.java
@@ -73,7 +73,7 @@ import org.junit.jupiter.params.provider.EnumSource;
/**
*/
-public class TestGenericKeyedObjectPool extends AbstractTestKeyedObjectPool {
+class TestGenericKeyedObjectPool extends AbstractTestKeyedObjectPool {
private static final class DaemonThreadFactory implements ThreadFactory {
@Override
@@ -179,11 +179,11 @@ public class TestGenericKeyedObjectPool extends
AbstractTestKeyedObjectPool {
boolean exceptionOnCreate;
- public SimpleFactory() {
+ SimpleFactory() {
this(true);
}
- public SimpleFactory(final boolean valid) {
+ SimpleFactory(final boolean valid) {
this.valid = valid;
}
diff --git
a/src/test/java/org/apache/commons/pool3/impl/TestGenericObjectPool.java
b/src/test/java/org/apache/commons/pool3/impl/TestGenericObjectPool.java
index 4a971695..74c10e4a 100644
--- a/src/test/java/org/apache/commons/pool3/impl/TestGenericObjectPool.java
+++ b/src/test/java/org/apache/commons/pool3/impl/TestGenericObjectPool.java
@@ -72,7 +72,7 @@ import org.junit.jupiter.api.Timeout;
/**
*/
-public class TestGenericObjectPool extends TestBaseObjectPool {
+class TestGenericObjectPool extends TestBaseObjectPool {
private final class ConcurrentBorrowAndEvictThread extends Thread {
private final boolean borrow;
@@ -276,15 +276,15 @@ public class TestGenericObjectPool extends
TestBaseObjectPool {
int maxTotal = Integer.MAX_VALUE;
- public SimpleFactory() {
+ SimpleFactory() {
this(true);
}
- public SimpleFactory(final boolean valid) {
+ SimpleFactory(final boolean valid) {
this(valid, valid);
}
- public SimpleFactory(final boolean evalid, final boolean ovalid) {
+ SimpleFactory(final boolean evalid, final boolean ovalid) {
evenValid = evalid;
oddValid = ovalid;
}
diff --git
a/src/test/java/org/apache/commons/pool3/impl/TestGenericObjectPoolClassLoaders.java
b/src/test/java/org/apache/commons/pool3/impl/TestGenericObjectPoolClassLoaders.java
index 8a216b29..da219a3a 100644
---
a/src/test/java/org/apache/commons/pool3/impl/TestGenericObjectPoolClassLoaders.java
+++
b/src/test/java/org/apache/commons/pool3/impl/TestGenericObjectPoolClassLoaders.java
@@ -26,7 +26,7 @@ import org.apache.commons.pool3.BasePooledObjectFactory;
import org.apache.commons.pool3.PooledObject;
import org.junit.jupiter.api.Test;
-public class TestGenericObjectPoolClassLoaders {
+class TestGenericObjectPoolClassLoaders {
private static final class CustomClassLoader extends URLClassLoader {
private final int n;
diff --git
a/src/test/java/org/apache/commons/pool3/impl/TestGenericObjectPoolFactoryCreateFailure.java
b/src/test/java/org/apache/commons/pool3/impl/TestGenericObjectPoolFactoryCreateFailure.java
index 40d6988a..0aaf9a9d 100644
---
a/src/test/java/org/apache/commons/pool3/impl/TestGenericObjectPoolFactoryCreateFailure.java
+++
b/src/test/java/org/apache/commons/pool3/impl/TestGenericObjectPoolFactoryCreateFailure.java
@@ -33,7 +33,7 @@ import org.junit.jupiter.api.Timeout;
/**
* Tests POOL-340.
*/
-public class TestGenericObjectPoolFactoryCreateFailure {
+class TestGenericObjectPoolFactoryCreateFailure {
private static final class SingleObjectFactory extends
BasePooledObjectFactory<Object, Exception> {
private final AtomicBoolean created = new AtomicBoolean();
diff --git
a/src/test/java/org/apache/commons/pool3/impl/TestLinkedBlockingDeque.java
b/src/test/java/org/apache/commons/pool3/impl/TestLinkedBlockingDeque.java
index 159cab8e..c2bef8de 100644
--- a/src/test/java/org/apache/commons/pool3/impl/TestLinkedBlockingDeque.java
+++ b/src/test/java/org/apache/commons/pool3/impl/TestLinkedBlockingDeque.java
@@ -37,7 +37,7 @@ import org.junit.jupiter.api.Timeout;
/**
* Tests for {@link LinkedBlockingDeque}.
*/
-public class TestLinkedBlockingDeque {
+class TestLinkedBlockingDeque {
private static final Duration TIMEOUT_50_MILLIS = Duration.ofMillis(50);
private static final Integer ONE = Integer.valueOf(1);
diff --git a/src/test/java/org/apache/commons/pool3/impl/TestPoolImplUtils.java
b/src/test/java/org/apache/commons/pool3/impl/TestPoolImplUtils.java
index a474f1ad..91b24608 100644
--- a/src/test/java/org/apache/commons/pool3/impl/TestPoolImplUtils.java
+++ b/src/test/java/org/apache/commons/pool3/impl/TestPoolImplUtils.java
@@ -27,7 +27,7 @@ import org.apache.commons.pool3.BasePooledObjectFactory;
import org.apache.commons.pool3.PooledObject;
import org.junit.jupiter.api.Test;
-public class TestPoolImplUtils {
+class TestPoolImplUtils {
@SuppressWarnings("unused")
private abstract static class FactoryAB<A, B> extends
BasePooledObjectFactory<B, RuntimeException> {
diff --git
a/src/test/java/org/apache/commons/pool3/impl/TestPooledSoftReference.java
b/src/test/java/org/apache/commons/pool3/impl/TestPooledSoftReference.java
index b616242a..1e26e145 100644
--- a/src/test/java/org/apache/commons/pool3/impl/TestPooledSoftReference.java
+++ b/src/test/java/org/apache/commons/pool3/impl/TestPooledSoftReference.java
@@ -26,7 +26,7 @@ import org.junit.jupiter.api.Test;
/**
* Tests for PooledSoftReference.
*/
-public class TestPooledSoftReference {
+class TestPooledSoftReference {
private static final String REFERENT = "test";
private static final String REFERENT2 = "test2";
diff --git
a/src/test/java/org/apache/commons/pool3/impl/TestResilientPooledObjectFactory.java
b/src/test/java/org/apache/commons/pool3/impl/TestResilientPooledObjectFactory.java
index 003fd177..8d1ac26e 100644
---
a/src/test/java/org/apache/commons/pool3/impl/TestResilientPooledObjectFactory.java
+++
b/src/test/java/org/apache/commons/pool3/impl/TestResilientPooledObjectFactory.java
@@ -27,7 +27,7 @@ import org.apache.commons.pool3.PooledObject;
import org.apache.commons.pool3.PooledObjectFactory;
import org.junit.jupiter.api.Test;
-public class TestResilientPooledObjectFactory {
+class TestResilientPooledObjectFactory {
/**
* Factory that suffers outages and fails in configurable ways when it is
down.
*/
diff --git
a/src/test/java/org/apache/commons/pool3/impl/TestSoftRefOutOfMemory.java
b/src/test/java/org/apache/commons/pool3/impl/TestSoftRefOutOfMemory.java
index d891a572..a0818813 100644
--- a/src/test/java/org/apache/commons/pool3/impl/TestSoftRefOutOfMemory.java
+++ b/src/test/java/org/apache/commons/pool3/impl/TestSoftRefOutOfMemory.java
@@ -32,12 +32,14 @@ import org.junit.jupiter.api.Test;
/**
*/
-public class TestSoftRefOutOfMemory {
+class TestSoftRefOutOfMemory {
+
public static class LargePoolableObjectFactory extends
BasePooledObjectFactory<String, RuntimeException> {
+
private final String buffer;
private int counter;
- public LargePoolableObjectFactory(final int size) {
+ LargePoolableObjectFactory(final int size) {
final char[] data = new char[size];
Arrays.fill(data, '.');
buffer = new String(data);
diff --git
a/src/test/java/org/apache/commons/pool3/impl/TestSoftReferenceObjectPool.java
b/src/test/java/org/apache/commons/pool3/impl/TestSoftReferenceObjectPool.java
index c2167eaa..1c31590d 100644
---
a/src/test/java/org/apache/commons/pool3/impl/TestSoftReferenceObjectPool.java
+++
b/src/test/java/org/apache/commons/pool3/impl/TestSoftReferenceObjectPool.java
@@ -24,7 +24,7 @@ import org.apache.commons.pool3.TestBaseObjectPool;
/**
*/
-public class TestSoftReferenceObjectPool extends TestBaseObjectPool {
+class TestSoftReferenceObjectPool extends TestBaseObjectPool {
private static final class SimpleFactory extends
BasePooledObjectFactory<String, RuntimeException> {
diff --git
a/src/test/java/org/apache/commons/pool3/performance/PerformanceTest.java
b/src/test/java/org/apache/commons/pool3/performance/PerformanceTest.java
index b44c3643..55c92d8b 100644
--- a/src/test/java/org/apache/commons/pool3/performance/PerformanceTest.java
+++ b/src/test/java/org/apache/commons/pool3/performance/PerformanceTest.java
@@ -31,7 +31,7 @@ import org.apache.commons.pool3.impl.GenericObjectPool;
/**
* Multi-thread performance test
*/
-public class PerformanceTest {
+class PerformanceTest {
final class PerfTask implements Callable<TaskStats> {
final TaskStats taskStats = new TaskStats();
diff --git
a/src/test/java/org/apache/commons/pool3/pool407/AbstractPool407Test.java
b/src/test/java/org/apache/commons/pool3/pool407/AbstractPool407Test.java
index 6267b622..4935e89f 100644
--- a/src/test/java/org/apache/commons/pool3/pool407/AbstractPool407Test.java
+++ b/src/test/java/org/apache/commons/pool3/pool407/AbstractPool407Test.java
@@ -27,7 +27,7 @@ import org.apache.commons.pool3.PooledObject;
/**
* Tests POOL-407.
*/
-public class AbstractPool407Test {
+class AbstractPool407Test {
protected <T> void assertShutdown(final boolean termination, final
Duration poolConfigMaxWait, final T obj, final PooledObject<T> pooledObject) {
if (pooledObject != null) {
diff --git
a/src/test/java/org/apache/commons/pool3/pool407/KeyedPool407Test.java
b/src/test/java/org/apache/commons/pool3/pool407/KeyedPool407Test.java
index 41cc635c..a43258a5 100644
--- a/src/test/java/org/apache/commons/pool3/pool407/KeyedPool407Test.java
+++ b/src/test/java/org/apache/commons/pool3/pool407/KeyedPool407Test.java
@@ -29,7 +29,7 @@ import org.junit.jupiter.api.Test;
/**
* Tests POOL-407.
*/
-public class KeyedPool407Test extends AbstractPool407Test {
+class KeyedPool407Test extends AbstractPool407Test {
/**
* Borrows from a pool and then immediately returns to that a pool.
diff --git a/src/test/java/org/apache/commons/pool3/pool407/Pool407Test.java
b/src/test/java/org/apache/commons/pool3/pool407/Pool407Test.java
index 65f3b4d1..ccba1a9f 100644
--- a/src/test/java/org/apache/commons/pool3/pool407/Pool407Test.java
+++ b/src/test/java/org/apache/commons/pool3/pool407/Pool407Test.java
@@ -29,7 +29,7 @@ import org.junit.jupiter.api.Test;
/**
* Tests POOL-407.
*/
-public class Pool407Test extends AbstractPool407Test {
+class Pool407Test extends AbstractPool407Test {
/**
* Borrows from a pool and then immediately returns to that a pool.
diff --git
a/src/test/java/org/apache/commons/pool3/proxy/TestProxiedKeyedObjectPoolWithCglibProxy.java
b/src/test/java/org/apache/commons/pool3/proxy/TestProxiedKeyedObjectPoolWithCglibProxy.java
index a1b7ad6a..d7b3ca55 100644
---
a/src/test/java/org/apache/commons/pool3/proxy/TestProxiedKeyedObjectPoolWithCglibProxy.java
+++
b/src/test/java/org/apache/commons/pool3/proxy/TestProxiedKeyedObjectPoolWithCglibProxy.java
@@ -16,7 +16,7 @@
*/
package org.apache.commons.pool3.proxy;
-public class TestProxiedKeyedObjectPoolWithCglibProxy extends
+class TestProxiedKeyedObjectPoolWithCglibProxy extends
AbstractTestProxiedKeyedObjectPool {
@Override
diff --git
a/src/test/java/org/apache/commons/pool3/proxy/TestProxiedKeyedObjectPoolWithJdkProxy.java
b/src/test/java/org/apache/commons/pool3/proxy/TestProxiedKeyedObjectPoolWithJdkProxy.java
index 5da2e45c..a0804d7f 100644
---
a/src/test/java/org/apache/commons/pool3/proxy/TestProxiedKeyedObjectPoolWithJdkProxy.java
+++
b/src/test/java/org/apache/commons/pool3/proxy/TestProxiedKeyedObjectPoolWithJdkProxy.java
@@ -16,7 +16,7 @@
*/
package org.apache.commons.pool3.proxy;
-public class TestProxiedKeyedObjectPoolWithJdkProxy
+class TestProxiedKeyedObjectPoolWithJdkProxy
extends AbstractTestProxiedKeyedObjectPool {
@Override
diff --git
a/src/test/java/org/apache/commons/pool3/proxy/TestProxiedObjectPoolWithCglibProxy.java
b/src/test/java/org/apache/commons/pool3/proxy/TestProxiedObjectPoolWithCglibProxy.java
index c7914918..ef4930af 100644
---
a/src/test/java/org/apache/commons/pool3/proxy/TestProxiedObjectPoolWithCglibProxy.java
+++
b/src/test/java/org/apache/commons/pool3/proxy/TestProxiedObjectPoolWithCglibProxy.java
@@ -19,7 +19,7 @@ package org.apache.commons.pool3.proxy;
import java.lang.reflect.InvocationTargetException;
-public class TestProxiedObjectPoolWithCglibProxy extends
AbstractTestProxiedObjectPool {
+class TestProxiedObjectPoolWithCglibProxy extends
AbstractTestProxiedObjectPool {
@Override
protected ProxySource<TestObject> getProxySource(boolean
unwrapInvocationTargetException) {
diff --git
a/src/test/java/org/apache/commons/pool3/proxy/TestProxiedObjectPoolWithJdkProxy.java
b/src/test/java/org/apache/commons/pool3/proxy/TestProxiedObjectPoolWithJdkProxy.java
index becd2418..a3e88243 100644
---
a/src/test/java/org/apache/commons/pool3/proxy/TestProxiedObjectPoolWithJdkProxy.java
+++
b/src/test/java/org/apache/commons/pool3/proxy/TestProxiedObjectPoolWithJdkProxy.java
@@ -19,7 +19,7 @@ package org.apache.commons.pool3.proxy;
import java.lang.reflect.UndeclaredThrowableException;
-public class TestProxiedObjectPoolWithJdkProxy extends
AbstractTestProxiedObjectPool {
+class TestProxiedObjectPoolWithJdkProxy extends AbstractTestProxiedObjectPool {
@Override
protected ProxySource<TestObject> getProxySource(boolean
unwrapInvocationTargetException) {
