✅ Increase coverage

Author: tobi-laa

src/main/kotlin/io/github/tobi/laa/spring/boot/embedded/redis/conf/RedisConfParser.kt

@@ -81,7 +81,7 @@ internal object RedisConfParser {
         }
     }
 
-    private enum class ArgsParseState {
+    internal enum class ArgsParseState {
         UNESCAPED, ESCAPED_SINGLE, ESCAPED_DOUBLE
     }
 }

src/test/kotlin/io/github/tobi/laa/spring/boot/embedded/redis/RedisFlushAllTest.kt

@@ -0,0 +1,25 @@
+package io.github.tobi.laa.spring.boot.embedded.redis
+
+import org.assertj.core.api.Assertions.assertThat
+import org.junit.jupiter.api.DisplayName
+import org.junit.jupiter.api.Nested
+import org.junit.jupiter.api.Test
+
+@DisplayName("RedisFlushAll tests")
+internal class RedisFlushAllTest {
+
+    @Nested
+    @DisplayName("RedisFlushAll.Mode tests")
+    internal inner class ModeTest {
+
+        @Test
+        @DisplayName("Static field entries returns all enum values")
+        fun getEntries_returnsAllValues() {
+            assertThat(RedisFlushAll.Mode.entries).containsExactlyInAnyOrder(
+                RedisFlushAll.Mode.AFTER_METHOD,
+                RedisFlushAll.Mode.AFTER_CLASS,
+                RedisFlushAll.Mode.NEVER
+            )
+        }
+    }
+}

src/test/kotlin/io/github/tobi/laa/spring/boot/embedded/redis/conf/RedisConfLocatorTest.kt

@@ -1,19 +1,39 @@
 package io.github.tobi.laa.spring.boot.embedded.redis.conf
 
+import io.mockk.junit5.MockKExtension
 import org.assertj.core.api.Assertions.assertThatThrownBy
 import org.junit.jupiter.api.DisplayName
 import org.junit.jupiter.api.Test
-import redis.embedded.RedisServer
+import org.junit.jupiter.api.extension.ExtendWith
+import redis.embedded.RedisInstance
 
 @DisplayName("Tests for RedisConfLocator")
+@ExtendWith(MockKExtension::class)
 internal class RedisConfLocatorTest {
 
     @Test
-    @DisplayName("RedisConfLocator should throw exception for missing file")
+    @DisplayName("RedisConfLocator should throw exception for empty args")
+    fun emptyArgs_shouldThrowException() {
+        val redis = TestRedis(emptyList())
+        assertThatThrownBy { RedisConfLocator.locate(redis) }
+            .isExactlyInstanceOf(IllegalStateException::class.java)
+            .hasMessage("No config file found for embedded Redis server: $redis")
+    }
+
+    @Test
+    @DisplayName("RedisConfLocator should throw exception for missing file with .conf extension")
     fun missingFile_shouldThrowException() {
-        val redis = RedisServer(6379, emptyList(), false)
+        val redis = TestRedis(listOf("redis.properties"))
         assertThatThrownBy { RedisConfLocator.locate(redis) }
             .isExactlyInstanceOf(IllegalStateException::class.java)
             .hasMessage("No config file found for embedded Redis server: $redis")
     }
+
+    private class TestRedis(args: List<String>) : RedisInstance(0, args, null, false, null, null) {
+
+        override fun start() {}
+        override fun stop() {}
+        override fun ports(): MutableList<Int> = mutableListOf()
+        override fun isActive(): Boolean = false
+    }
 }

src/test/kotlin/io/github/tobi/laa/spring/boot/embedded/redis/conf/RedisConfParserTest.kt

@@ -5,6 +5,8 @@ import org.assertj.core.api.Assertions.*
 import org.assertj.core.api.ThrowableAssert.ThrowingCallable
 import org.junit.jupiter.api.DisplayName
 import org.junit.jupiter.api.Named.named
+import org.junit.jupiter.api.Nested
+import org.junit.jupiter.api.Test
 import org.junit.jupiter.api.extension.ExtensionContext
 import org.junit.jupiter.params.ParameterizedTest
 import org.junit.jupiter.params.provider.Arguments
@@ -201,4 +203,19 @@ internal class RedisConfParserTest {
             )
         }
     }
+
+    @Nested
+    @DisplayName("RedisConfParser.ArgsParseState tests")
+    internal inner class ArgsParseStateTest {
+
+        @Test
+        @DisplayName("Static field entries returns all enum values")
+        fun getEntries_returnsAllValues() {
+            assertThat(RedisConfParser.ArgsParseState.entries).containsExactlyInAnyOrder(
+                RedisConfParser.ArgsParseState.UNESCAPED,
+                RedisConfParser.ArgsParseState.ESCAPED_SINGLE,
+                RedisConfParser.ArgsParseState.ESCAPED_DOUBLE
+            )
+        }
+    }
 }