Make command bus method configurable

Now the command bus doesn't need a method named handle but can use
anything it likes.

README.md

@@ -71,16 +71,19 @@ And you're good to go!
 
 It's very common to have a bridge interface from your application to any external packages, including Tactician. In that case, you'll have a different command bus class and Tactician-PHPStan won't catch errors because it's looking for usages of the `League\Tactician\CommandBus`.
 
-Instead, you can configure the command bus class to scan for:
+Instead, you can configure the command bus class to scan for, as well as (optionally) the method to use:
 
 ~~~
 # phpstan.neon
 parameters:
     tactician:
         bootstrap: handler-mapping-loader.php
         bus: My\App\CommandBus
+        method: execute
 ~~~
 
+If neither is specified, the default class is `League\Tactician\CommandBus` and a method named `handle`.
+
 ## Testing
 To run all unit tests, use the locally installed PHPUnit:
 

extension.neon

@@ -1,19 +1,22 @@
 parameters:
     tactician:
         bus: League\Tactician\CommandBus
+        method: handle
 
 services:
     -
         class: League\Tactician\PHPStan\TacticianRuleSet
         arguments:
             commandBusClass: %tactician.bus%
+            commandBusMethod: %tactician.method%
         tags:
             - phpstan.rules.rule
 
     -
         class: League\Tactician\PHPStan\HandlerReturnTypeExtension
         arguments:
             commandBusClass: %tactician.bus%
+            commandBusMethod: %tactician.method%
         tags:
             - phpstan.broker.dynamicMethodReturnTypeExtension
 

src/HandlerReturnTypeExtension.php

@@ -31,11 +31,16 @@ final class HandlerReturnTypeExtension implements DynamicMethodReturnTypeExtensi
      * @var string
      */
     private $commandBusClass;
+    /**
+     * @var string
+     */
+    private $commandBusMethod;
 
-    public function __construct(CommandToHandlerMapping $mapping, string $commandBusClass)
+    public function __construct(CommandToHandlerMapping $mapping, string $commandBusClass, string $commandBusMethod)
     {
         $this->mapping = $mapping;
         $this->commandBusClass = $commandBusClass;
+        $this->commandBusMethod = $commandBusMethod;
     }
 
     public function setBroker(Broker $broker): void
@@ -50,7 +55,7 @@ public function getClass(): string
 
     public function isMethodSupported(MethodReflection $methodReflection): bool
     {
-        return $methodReflection->getName() === 'handle';
+        return $methodReflection->getName() === $this->commandBusMethod;
     }
 
     public function getTypeFromMethodCall(

src/TacticianRuleSet.php

@@ -33,12 +33,17 @@ final class TacticianRuleSet implements Rule
      * @var string
      */
     private $commandBusClass;
+    /**
+     * @var string
+     */
+    private $commandBusMethod;
 
-    public function __construct(CommandToHandlerMapping $mapping, Broker $broker, string $commandBusClass)
+    public function __construct(CommandToHandlerMapping $mapping, Broker $broker, string $commandBusClass, string $commandBusMethod)
     {
         $this->mapping = $mapping;
         $this->broker = $broker;
         $this->commandBusClass = $commandBusClass;
+        $this->commandBusMethod = $commandBusMethod;
     }
 
     public function getNodeType(): string
@@ -50,7 +55,7 @@ public function processNode(Node $methodCall, Scope $scope): array
     {
         if (! $methodCall instanceof MethodCall
             || ! $methodCall->name instanceof Node\Identifier
-            || $methodCall->name->name !== 'handle') {
+            || $methodCall->name->name !== $this->commandBusMethod) {
             return [];
         }
 

tests/TacticianRuleSetTest.php

@@ -21,7 +21,8 @@ protected function getRule(): Rule
                 new Handle()
             ),
             $this->createBroker(),
-            CommandBus::class
+            CommandBus::class,
+            'handle'
         );
     }
 

tests/data/MissingHandlerClassForAlternateBus.php

@@ -10,11 +10,11 @@ class SomeCommand
 
 class DerpBus
 {
-    public function handle(object $command): void
+    public function execute(object $command): void
     {
 
     }
 }
 
 $commandBus = new DerpBus();
-$commandBus->handle(new SomeCommand());
+$commandBus->execute(new SomeCommand());

tests/data/UnionAndIntersectionClassesBehaveProperly.php

@@ -27,9 +27,9 @@ class SomeTest extends TestCase
 
 class DerpBus
 {
-    public function handle(object $command): void
+    public function execute(object $command): void
     {
 
     }
 }
-$commandBus->expects(TestCase::once())->method('handle') ;
+$commandBus->expects(TestCase::once())->method('execute') ;

tests/phpstan-alternate-class.neon

@@ -5,3 +5,4 @@ parameters:
     tactician:
         bootstrap: tests/handler-mapping-loader.php
         bus: MissingHandlerClassForAlternateBus\DerpBus
+        method: execute