Complete Tests, change protected to public in addHandlers method on laravel locator

Author: joselfonseca

src/Locator/LaravelLocator.php

@@ -35,7 +35,7 @@ public function addHandler($handler, $commandClassName)
      *
      * @param array $commandClassToHandlerMap
      */
-    protected function addHandlers(array $commandClassToHandlerMap)
+    public function addHandlers(array $commandClassToHandlerMap)
     {
         foreach ($commandClassToHandlerMap as $commandClass => $handler) {
             $this->addHandler($handler, $commandClass);

tests/Bus/TestBus.php

@@ -35,4 +35,36 @@ public function test_it_applies_a_middleware()
         $this->assertEquals('Handled', $commandHandled->addedPropertyInMiddleware);
     }
 
+    /**
+     * Test if the bus is able to map a property in the command
+     */
+    public function test_it_maps_input_to_command()
+    {
+        $bus = app('Joselfonseca\LaravelTactician\CommandBusInterface');
+        $bus->addHandler('Joselfonseca\LaravelTactician\Tests\Stubs\TestCommand',
+            'Joselfonseca\LaravelTactician\Tests\Stubs\TestCommandHandler');
+        $commandHandled = $bus->dispatch('Joselfonseca\LaravelTactician\Tests\Stubs\TestCommand', [
+            'property' => 'Jhon Doe'
+        ], [
+            'Joselfonseca\LaravelTactician\Tests\Stubs\Middleware'
+        ]);
+        $this->assertEquals('Jhon Doe', $commandHandled->property);
+    }
+
+    /**
+     * Test the InvalidArgumentException
+     * @expectedException InvalidArgumentException
+     */
+    public function test_it_trows_exception_if_input_can_not_be_mapped_to_the_command()
+    {
+        $bus = app('Joselfonseca\LaravelTactician\CommandBusInterface');
+        $bus->addHandler('Joselfonseca\LaravelTactician\Tests\Stubs\TestCommandInput',
+            'Joselfonseca\LaravelTactician\Tests\Stubs\TestCommandHandler');
+        $commandHandled = $bus->dispatch('Joselfonseca\LaravelTactician\Tests\Stubs\TestCommandInput', [
+
+        ], [
+            'Joselfonseca\LaravelTactician\Tests\Stubs\Middleware'
+        ]);
+    }
+
 }

tests/Locator/TestLaravelLocator.php

@@ -52,4 +52,20 @@ public function test_it_is_able_to_resolve_handler_from_laravel_container()
         $this->assertInstanceOf('Joselfonseca\LaravelTactician\Tests\Stubs\TestCommandHandler', $handler);
     }
 
+    /**
+     * Add more than one command => handler to the bus
+     */
+    public function test_it_maps_array_commands()
+    {
+        $locator = app('League\Tactician\Handler\Locator\HandlerLocator');
+        $locator->addHandlers([
+            'Joselfonseca\LaravelTactician\Tests\Stubs\TestCommand' => 'Joselfonseca\LaravelTactician\Tests\Stubs\TestCommandHandler',
+            'Joselfonseca\LaravelTactician\Tests\Stubs\TestCommandInput' => 'Joselfonseca\LaravelTactician\Tests\Stubs\TestCommandSeccondHandler'
+        ]);
+        $handler = $locator->getHandlerForCommand('Joselfonseca\LaravelTactician\Tests\Stubs\TestCommand');
+        $handler2 = $locator->getHandlerForCommand('Joselfonseca\LaravelTactician\Tests\Stubs\TestCommandInput');
+        $this->assertInstanceOf('Joselfonseca\LaravelTactician\Tests\Stubs\TestCommandHandler', $handler);
+        $this->assertInstanceOf('Joselfonseca\LaravelTactician\Tests\Stubs\TestCommandSeccondHandler', $handler2);
+    }
+
 }

tests/Stubs/TestCommand.php

@@ -5,9 +5,14 @@
 
 class TestCommand {
 
-    public function __construct()
-    {
+    public $property;
+
+    public $propertyTwo;
 
+    public function __construct($property = null, $propertyTwo = "First Name")
+    {
+        $this->property = $property;
+        $this->propertyTwo = $propertyTwo;
     }
 
 }

tests/Stubs/TestCommandInput.php

@@ -0,0 +1,14 @@
+<?php
+
+namespace Joselfonseca\LaravelTactician\Tests\Stubs;
+
+
+class TestCommandInput
+{
+    public $property;
+
+    public function __construct($property)
+    {
+        $this->property = $property;
+    }
+}

tests/Stubs/TestCommandSeccondHandler.php

@@ -0,0 +1,12 @@
+<?php
+
+namespace Joselfonseca\LaravelTactician\Tests\Stubs;
+
+
+class TestCommandSeccondHandler
+{
+    public function handle($command)
+    {
+        return $command;
+    }
+}