Merge pull request #43 from aik099/40-auto-complete-for-build-in-commands

Auto-complete arguments of built-in "help" and "list" commands

Author: stecman

src/CompletionCommand.php

@@ -94,7 +94,7 @@ protected function runCompletion()
      */
     protected function configureCompletion(CompletionHandler $handler)
     {
-         // Override this method to configure custom value completions
+        // Override this method to configure custom value completions
     }
 
     /**

src/CompletionHandler.php

@@ -38,6 +38,24 @@ public function __construct(Application $application, CompletionContext $context
     {
         $this->application = $application;
         $this->context = $context;
+
+        $this->addHandler(
+            new Completion(
+                'help',
+                'command_name',
+                Completion::TYPE_ARGUMENT,
+                array_keys($application->all())
+            )
+        );
+
+        $this->addHandler(
+            new Completion(
+                'list',
+                'namespace',
+                Completion::TYPE_ARGUMENT,
+                $application->getNamespaces()
+            )
+        );
     }
 
     public function setContext(CompletionContext $context)

tests/Stecman/Component/Symfony/Console/BashCompletion/CompletionHandlerTest.php

@@ -143,4 +143,22 @@ public function testShortCommandNotMatched()
         $handler = $this->createHandler('app w --deploy');
         $this->assertEquals(array(), $this->getTerms($handler->runCompletion()));
     }
+
+    public function testHelpCommandCompletion()
+    {
+        $handler = $this->createHandler('app help ');
+        $this->assertEquals(
+            array('help', 'list', 'completion-aware', 'wave', 'walk:north'),
+            $this->getTerms($handler->runCompletion())
+        );
+    }
+
+    public function testListCommandCompletion()
+    {
+        $handler = $this->createHandler('app list ');
+        $this->assertEquals(
+            array('walk'),
+            $this->getTerms($handler->runCompletion())
+        );
+    }
 }