update some for flag helper

Author: inhere

Diff for: src/Helper/FlagHelper.php

@@ -112,7 +112,7 @@ public static function isOptionValue($val): bool
      */
     public static function isValidName(string $name): bool
     {
-        return preg_match('#^\w+$#', $name) === 1;
+        return preg_match('#^\w[\w-]{0,36}$#', $name) === 1;
     }
 
     /**

Diff for: test/ColorTagTest.php

@@ -106,7 +106,7 @@ public function testParseNestTag(): void
 
         // nested Tags
         $text = ColorTag::parse('<info>INFO <cyan>CYAN mess</cyan>age</info>', true);
-        echo $text, PHP_EOL;
+        echo 'nested: ' . $text, PHP_EOL;
         $this->assertSame("\033[0;32mINFO \033[0;36mCYAN mess\033[0mage\033[0m", $text);
 
         Color::resetConfig();

Diff for: test/Helper/FlagHelperTest.php

@@ -0,0 +1,35 @@
+<?php declare(strict_types=1);
+
+namespace Toolkit\CliTest\Helper;
+
+use PHPUnit\Framework\TestCase;
+use Toolkit\Cli\Helper\FlagHelper;
+
+/**
+ * class FlagHelperTest
+ */
+class FlagHelperTest extends TestCase
+{
+    public function testIsValidName(): void
+    {
+        $tests = [
+            'name'      => true,
+            'name1'     => true,
+            'some-name' => true,
+            'some_name' => true,
+            'someName'  => true,
+            'SomeName'  => true,
+            '_2'        => true,
+            '_someName' => true,
+            '_SomeName' => true,
+            '_name-'    => true,
+            '-name'     => false,
+            ' name'     => false,
+            '+name'     => false,
+        ];
+
+        foreach ($tests as $name => $ok) {
+            $this->assertSame($ok, FlagHelper::isValidName($name));
+        }
+    }
+}