refactor: move normal unit test below container tests; cleanup import

Author: wwisser

modules/valkey/src/test/java/org/testcontainers/valkey/ValkeyContainerTest.java

@@ -6,7 +6,6 @@
 import io.valkey.Jedis;
 import io.valkey.JedisPool;
 import java.nio.file.Paths;
-import lombok.val;
 import org.junit.jupiter.api.Test;
 import org.junit.jupiter.api.io.TempDir;
 
@@ -80,18 +79,6 @@ void shouldPersistData() {
         }
     }
 
-    @Test
-    void shouldValidateSnapshottingConfiguration() {
-        ValkeyContainer container = new ValkeyContainer();
-        assertThatThrownBy(() -> container.withSnapshotting(0, 10))
-            .isInstanceOf(IllegalArgumentException.class)
-            .hasMessageContaining("seconds must be greater than 0");
-
-        assertThatThrownBy(() -> container.withSnapshotting(10, 0))
-            .isInstanceOf(IllegalArgumentException.class)
-            .hasMessageContaining("changedKeys must be non-negative");
-    }
-
     @Test
     void shouldInitializeDatabaseWithPayload() throws Exception {
         Path importFile = Paths.get(getClass().getResource("/initData.valkey").toURI());
@@ -137,4 +124,16 @@ void shouldMountValkeyConfigToContainer() throws Exception {
             }
         }
     }
+
+    @Test
+    void shouldValidateSnapshottingConfiguration() {
+        ValkeyContainer container = new ValkeyContainer();
+        assertThatThrownBy(() -> container.withSnapshotting(0, 10))
+            .isInstanceOf(IllegalArgumentException.class)
+            .hasMessageContaining("seconds must be greater than 0");
+
+        assertThatThrownBy(() -> container.withSnapshotting(10, 0))
+            .isInstanceOf(IllegalArgumentException.class)
+            .hasMessageContaining("changedKeys must be non-negative");
+    }
 }