diff --git a/src/main/java/org/springframework/data/redis/core/RedisAccessor.java b/src/main/java/org/springframework/data/redis/core/RedisAccessor.java index 93b4fef34..903cef1f3 100644 --- a/src/main/java/org/springframework/data/redis/core/RedisAccessor.java +++ b/src/main/java/org/springframework/data/redis/core/RedisAccessor.java @@ -29,7 +29,7 @@ import org.springframework.lang.Nullable; * @author John Blum * @see org.springframework.beans.factory.InitializingBean */ -public class RedisAccessor implements InitializingBean { +public abstract class RedisAccessor implements InitializingBean { /** Logger available to subclasses */ protected final Log logger = LogFactory.getLog(getClass()); diff --git a/src/test/java/org/springframework/data/redis/core/RedisAccessorUnitTests.java b/src/test/java/org/springframework/data/redis/core/RedisAccessorUnitTests.java index ccbfb2e51..c427030e9 100644 --- a/src/test/java/org/springframework/data/redis/core/RedisAccessorUnitTests.java +++ b/src/test/java/org/springframework/data/redis/core/RedisAccessorUnitTests.java @@ -41,7 +41,7 @@ class RedisAccessorUnitTests { RedisConnectionFactory mockConnectionFactory = mock(RedisConnectionFactory.class); - RedisAccessor redisAccessor = new RedisAccessor(); + RedisAccessor redisAccessor = new TestRedisAccessor(); assertThat(redisAccessor.getConnectionFactory()).isNull(); @@ -61,7 +61,7 @@ class RedisAccessorUnitTests { void getRequiredConnectionFactoryWhenNull() { assertThatIllegalStateException() - .isThrownBy(() -> new RedisAccessor().getRequiredConnectionFactory()) + .isThrownBy(() -> new TestRedisAccessor().getRequiredConnectionFactory()) .withMessage("RedisConnectionFactory is required") .withNoCause(); } @@ -71,7 +71,7 @@ class RedisAccessorUnitTests { RedisConnectionFactory mockConnectionFactory = mock(RedisConnectionFactory.class); - RedisAccessor redisAccessor = spy(new RedisAccessor()); + RedisAccessor redisAccessor = spy(new TestRedisAccessor()); doReturn(mockConnectionFactory).when(redisAccessor).getRequiredConnectionFactory(); @@ -81,4 +81,7 @@ class RedisAccessorUnitTests { verify(redisAccessor, times(1)).getRequiredConnectionFactory(); verifyNoMoreInteractions(redisAccessor); } + + static class TestRedisAccessor extends RedisAccessor { } + }