From a8818e01664b0ee6d5bce6bf486a95a7e0f73b28 Mon Sep 17 00:00:00 2001
From: kayjaydee <killian.dalcin@gmail.com>
Date: Fri, 24 Apr 2026 17:02:26 +0200
Subject: [PATCH] fix(04-bugfix): reject names with spaces in define subcommand

- Add explicit regression tests for "nom avec espaces", leading/trailing
  spaces, and tab characters in DefineValidationTest.
- Validation regex ^[a-zA-Z0-9_-]{1,32}$ already rejects these; tests
  seal B4 UAT observation.
---
 .../mythlane/gravityflip/command/DefineValidationTest.java   | 5 +++++
 1 file changed, 5 insertions(+)

diff --git a/src/test/java/com/mythlane/gravityflip/command/DefineValidationTest.java b/src/test/java/com/mythlane/gravityflip/command/DefineValidationTest.java
index d39128b..5ebeffd 100644
--- a/src/test/java/com/mythlane/gravityflip/command/DefineValidationTest.java
+++ b/src/test/java/com/mythlane/gravityflip/command/DefineValidationTest.java
@@ -36,6 +36,11 @@ class DefineValidationTest {
         assertFalse(DefineValidation.isValidName(""));
         assertFalse(DefineValidation.isValidName(" "));
         assertFalse(DefineValidation.isValidName("my zone"));
+        // B4 UAT regression — quoted arg with spaces must be rejected
+        assertFalse(DefineValidation.isValidName("nom avec espaces"));
+        assertFalse(DefineValidation.isValidName("a\tb"));
+        assertFalse(DefineValidation.isValidName(" leading"));
+        assertFalse(DefineValidation.isValidName("trailing "));
         assertFalse(DefineValidation.isValidName("a".repeat(33)));
         assertFalse(DefineValidation.isValidName("name!"));
         assertFalse(DefineValidation.isValidName("名前"));