Apply suggestions from code review

drop and/or replace maxsize where we can

Co-authored-by: M Sazzadul Hoque <[email protected]>

Author: atakavci

Diff for: src/test/java/redis/clients/jedis/csc/ClientSideCacheFunctionalityTest.java

@@ -43,7 +43,7 @@ public void flushAllTest() {
       control.set("k" + i, "v" + i);
     }
 
-    try (JedisPooled jedis = new JedisPooled(hnp, clientConfig.get(), CacheConfig.builder().maxSize(1000).build())) {
+    try (JedisPooled jedis = new JedisPooled(hnp, clientConfig.get(), CacheConfig.builder().build())) {
       Cache cache = jedis.getCache();
       for (int i = 0; i < count; i++) {
         jedis.get("k" + i);
@@ -92,7 +92,7 @@ public void lruEvictionTest() {
 
   @Test // T.5.2
   public void deleteByKeyUsingMGetTest() {
-    try (JedisPooled jedis = new JedisPooled(hnp, clientConfig.get(), CacheConfig.builder().maxSize(1000).build())) {
+    try (JedisPooled jedis = new JedisPooled(hnp, clientConfig.get(), CacheConfig.builder().build())) {
       Cache clientSideCache = jedis.getCache();
 
       jedis.set("1", "one");
@@ -243,8 +243,7 @@ public void maximumSizeExact() {
   public void testInvalidationWithUnifiedJedis() {
     Cache cache = new TestCache();
     Cache mock = Mockito.spy(cache);
-    UnifiedJedis client = new UnifiedJedis(hnp, clientConfig.get(), mock) {
-    };
+    UnifiedJedis client = new UnifiedJedis(hnp, clientConfig.get(), mock);
     UnifiedJedis controlClient = new UnifiedJedis(hnp, clientConfig.get());
 
     try {
@@ -272,7 +271,7 @@ public void testInvalidationWithUnifiedJedis() {
   public void differentInstanceOnEachCacheHit() {
 
     // fill the cache for maxSize
-    try (JedisPooled jedis = new JedisPooled(hnp, clientConfig.get(), CacheConfig.builder().maxSize(1000).build())) {
+    try (JedisPooled jedis = new JedisPooled(hnp, clientConfig.get(), CacheConfig.builder().build())) {
       Cache cache = jedis.getCache();
       jedis.sadd("foo", "a");
       jedis.sadd("foo", "b");
@@ -348,7 +347,7 @@ public void testConcurrentAccessWithStats() throws InterruptedException {
     ExecutorService executorService = Executors.newFixedThreadPool(threadCount);
 
     // Create the shared mock instance of cache
-    try (JedisPooled jedis = new JedisPooled(endpoint.getHostAndPort(), clientConfig.get(), CacheConfig.builder().maxSize(1000).build())) {
+    try (JedisPooled jedis = new JedisPooled(endpoint.getHostAndPort(), clientConfig.get(), CacheConfig.builder().build())) {
       Cache cache = jedis.getCache();
       // Submit multiple threads to perform concurrent operations
       CountDownLatch latch = new CountDownLatch(threadCount);
@@ -385,7 +384,7 @@ public void testMaxSize() throws InterruptedException {
 
     ExecutorService executorService = Executors.newFixedThreadPool(threadCount);
 
-    try (JedisPooled jedis = new JedisPooled(endpoint.getHostAndPort(), clientConfig.get(), CacheConfig.builder().maxSize(1000).build())) {
+    try (JedisPooled jedis = new JedisPooled(endpoint.getHostAndPort(), clientConfig.get(), CacheConfig.builder().maxSize(maxSize).build())) {
       Cache testCache = jedis.getCache();
       // Submit multiple threads to perform concurrent operations
       CountDownLatch latch = new CountDownLatch(threadCount);