diff --git a/src/main/java/org/springframework/data/redis/core/RedisCommand.java b/src/main/java/org/springframework/data/redis/core/RedisCommand.java index 16f373393..12da22aba 100644 --- a/src/main/java/org/springframework/data/redis/core/RedisCommand.java +++ b/src/main/java/org/springframework/data/redis/core/RedisCommand.java @@ -257,8 +257,8 @@ public enum RedisCommand { private final boolean read; private final boolean write; - final int minArgs; - final int maxArgs; + private final int minArgs; + private final int maxArgs; private final @Nullable String alias; diff --git a/src/test/java/org/springframework/data/redis/core/RedisCommandUnitTests.java b/src/test/java/org/springframework/data/redis/core/RedisCommandUnitTests.java index 6f995df86..60638ae0f 100644 --- a/src/test/java/org/springframework/data/redis/core/RedisCommandUnitTests.java +++ b/src/test/java/org/springframework/data/redis/core/RedisCommandUnitTests.java @@ -20,6 +20,7 @@ import static org.assertj.core.api.Assertions.*; import java.util.Arrays; import org.junit.jupiter.api.Test; +import org.springframework.test.util.ReflectionTestUtils; /** * Unit tests for {@link RedisCommand}. @@ -123,19 +124,19 @@ class RedisCommandUnitTests { @Test // GH-2646 void commandRequiresArgumentsIsCorrect() { - Arrays.stream(RedisCommand.values()).forEach(command -> - assertThat(command.requiresArguments()) + Arrays.stream(RedisCommand.values()) + .forEach(command -> assertThat(command.requiresArguments()) .describedAs("Redis command [%s] failed required arguments check", command) - .isEqualTo(command.minArgs > 0)); + .isEqualTo((int) ReflectionTestUtils.getField(command, "minArgs") > 0)); } @Test // GH-2646 void commandRequiresExactNumberOfArgumentsIsCorrect() { - Arrays.stream(RedisCommand.values()).forEach(command -> - assertThat(command.requiresExactNumberOfArguments()) - .describedAs("Redis command [%s] failed requires exact arguments check") - .isEqualTo(command.minArgs == command.maxArgs)); + Arrays.stream(RedisCommand.values()) + .forEach(command -> assertThat(command.requiresExactNumberOfArguments()) + .describedAs("Redis command [%s] failed requires exact arguments check").isEqualTo( + ReflectionTestUtils.getField(command, "minArgs") == ReflectionTestUtils.getField(command, "maxArgs"))); } }