Merge pull request #6402 from kenjis/fix-command-usage

samsonasik · web-flow · commit aed7114a5790 · 2022-08-21T20:07:36.000+07:00
fix: command usages
diff --git a/system/Commands/Cache/ClearCache.php b/system/Commands/Cache/ClearCache.php
@@ -46,7 +46,7 @@ class ClearCache extends BaseCommand
      *
      * @var string
      */
-    protected $usage = 'cache:clear [driver]';
+    protected $usage = 'cache:clear [<driver>]';
 
     /**
      * the Command's Arguments
diff --git a/system/Commands/Help.php b/system/Commands/Help.php
@@ -48,7 +48,7 @@ class Help extends BaseCommand
      *
      * @var string
      */
-    protected $usage = 'help command_name';
+    protected $usage = 'help [<command_name>]';
 
     /**
      * the Command's Arguments
diff --git a/tests/system/CLI/CommandRunnerTest.php b/tests/system/CLI/CommandRunnerTest.php
@@ -79,7 +79,7 @@ public function testHelpCommand()
 
         // make sure the result looks like basic help
         $this->assertStringContainsString('Displays basic usage information.', $result);
-        $this->assertStringContainsString('help command_name', $result);
+        $this->assertStringContainsString('help [<command_name>]', $result);
     }
 
     public function testHelpCommandDetails()

Original file line number	Diff line number	Diff line change
`@@ -46,7 +46,7 @@ class ClearCache extends BaseCommand`
`46`	`46`	`*`
`47`	`47`	`* @var string`
`48`	`48`	`*/`
`49`		`- protected $usage = 'cache:clear [driver]';`
	`49`	`+ protected $usage = 'cache:clear [<driver>]';`
`50`	`50`
`51`	`51`	`/**`
`52`	`52`	`* the Command's Arguments`
Original file line number	Diff line number	Diff line change
`@@ -48,7 +48,7 @@ class Help extends BaseCommand`
`48`	`48`	`*`
`49`	`49`	`* @var string`
`50`	`50`	`*/`
`51`		`- protected $usage = 'help command_name';`
	`51`	`+ protected $usage = 'help [<command_name>]';`
`52`	`52`
`53`	`53`	`/**`
`54`	`54`	`* the Command's Arguments`
Original file line number	Diff line number	Diff line change
`@@ -79,7 +79,7 @@ public function testHelpCommand()`
`79`	`79`
`80`	`80`	`// make sure the result looks like basic help`
`81`	`81`	`$this->assertStringContainsString('Displays basic usage information.', $result);`
`82`		`- $this->assertStringContainsString('help command_name', $result);`
	`82`	`+ $this->assertStringContainsString('help [<command_name>]', $result);`
`83`	`83`	`}`
`84`	`84`
`85`	`85`	`public function testHelpCommandDetails()`