feature/PHPGL-10 Add form_type tag to compiler pass collector

composer.json

@@ -21,7 +21,8 @@
         "matthiasnoback/symfony-console-form": "^1.1",
         "symfony/finder": "~2.5",
         "symfony/validator": "~2.5",
-        "symfony/var-dumper": "~2.5"
+        "symfony/var-dumper": "~2.5",
+        "matthiasnoback/symfony-dependency-injection-test": "0.*"
     },
     "autoload": {
         "psr-4" : { "Clearcode\\CommandBusConsole\\": "src" }

src/Bundle/DependencyInjection/Compiler/CommandHandlersCompilerPass.php

@@ -12,14 +12,24 @@ class CommandHandlersCompilerPass implements CompilerPassInterface
      */
     public function process(ContainerBuilder $container)
     {
+        if (!$container->hasDefinition('command_bus_console.command_collector')) {
+            return;
+        }
+
         $commandHandlersTags = $container->findTaggedServiceIds('command_handler');
         $commandCollector = $container->getDefinition('command_bus_console.command_collector');
 
         $commands = [];
 
         foreach ($commandHandlersTags as $service) {
             foreach ($service as $tags) {
-                $commands[] = $tags['handles'];
+                $arguments = ['handles'  => $tags['handles']];
+
+                if (array_key_exists('form_type', $tags)) {
+                    $arguments['form_type'] = $tags['form_type'];
+                }
+
+                $commands[] = $arguments;
             }
         }
 

src/CommandCollector.php

@@ -15,7 +15,7 @@ class CommandCollector
     public function processCommandServices(array $services)
     {
         foreach ($services as $service) {
-            $this->commands[] = CommandReflection::fromClass($service);
+            $this->commands[] = CommandReflection::fromClass($service['handles']);
         }
     }
 

tests/Bundle/DependencyInjection/Compiler/CommandHandlersCompilerPassTest.php

@@ -0,0 +1,67 @@
+<?php
+
+namespace tests\Clearcode\CommandBusConsole\Bundle\DependencyInjection\Compiler;
+
+use Clearcode\CommandBusConsole\Bundle\DependencyInjection\Compiler\CommandHandlersCompilerPass;
+use Matthias\SymfonyDependencyInjectionTest\PhpUnit\AbstractCompilerPassTestCase;
+use Symfony\Component\DependencyInjection\ContainerBuilder;
+use Symfony\Component\DependencyInjection\Definition;
+
+class CommandHandlersCompilerPassTest extends AbstractCompilerPassTestCase
+{
+    /**
+     * @test
+     */
+    public function it_should_collects_classes_names_by_adding_method_when_there_is_handles_argument()
+    {
+        $collectingService = new Definition();
+        $this->setDefinition('command_bus_console.command_collector', $collectingService);
+
+        $collectedService = new Definition();
+        $collectedService->addTag('command_handler', ['handles' => 'test_class']);
+        $this->setDefinition('colected_service_id', $collectedService);
+
+        $this->compile();
+
+        $this->assertContainerBuilderHasServiceDefinitionWithMethodCall(
+            'command_bus_console.command_collector',
+            'processCommandServices',
+            [[['handles' => 'test_class']]]
+        );
+    }
+
+    /**
+     * @test
+     */
+    public function it_should_collects_classes_names_and_form_types_by_adding_method_when_there_is_form_type_argument()
+    {
+        $collectingService = new Definition();
+        $this->setDefinition('command_bus_console.command_collector', $collectingService);
+
+        $collectedService = new Definition();
+        $collectedService->addTag('command_handler', ['handles' => 'test_class', 'form_type' => 'form_type_name']);
+        $this->setDefinition('colected_service_id', $collectedService);
+
+        $this->compile();
+
+        $this->assertContainerBuilderHasServiceDefinitionWithMethodCall(
+            'command_bus_console.command_collector',
+            'processCommandServices',
+            [[[
+                'handles'   => 'test_class',
+                'form_type' => 'form_type_name'
+            ]]]
+        );
+    }
+
+    /**
+     * Register the compiler pass under test, just like you would do inside a bundle's load()
+     * method:
+     *
+     *   $container->addCompilerPass(new MyCompilerPass());
+     */
+    protected function registerCompilerPass(ContainerBuilder $container)
+    {
+        $container->addCompilerPass(new CommandHandlersCompilerPass());
+    }
+}

tests/CommandCollectorTest.php

@@ -59,7 +59,7 @@ public function setUp()
         $this->sut = new CommandCollector();
 
         $this->commands = [
-            DummyCommand::class,
+            ['handles' => DummyCommand::class],
         ];
     }
 }