Allow BotMan to register the same callback to multiple events. (botman#849)

Author: Lloople

src/BotMan.php

@@ -274,15 +274,23 @@ public function hears($pattern, $callback, $in = null)
     /**
      * Listen for messaging service events.
      *
-     * @param string $name
+     * @param array|string $names
      * @param Closure|string $callback
      */
-    public function on($name, $callback)
+    public function on($names, $callback)
     {
-        $this->events[] = [
-            'name' => $name,
-            'callback' => $this->getCallable($callback),
-        ];
+        if (! is_array($names)) {
+            $names = [$names];
+        }
+
+        $callable = $this->getCallable($callback);
+
+        foreach ($names as $name) {
+            $this->events[] = [
+                'name' => $name,
+                'callback' => $callable,
+            ];
+        }
     }
 
     /**

tests/BotManDriverEventTest.php

@@ -52,6 +52,26 @@ public function it_calls_driver_events()
         $this->assertTrue($called);
     }
 
+    /** @test */
+    public function it_can_assign_same_callback_for_events()
+    {
+        $called = 0;
+        $this->fakeDriver->hasMatchingEvent = new TestEvent([]);
+        $this->botman->on(['test_event', 'test_event_two'], function ($data, BotMan $bot) use (&$called) {
+            $called++;
+        });
+
+        $this->botman->listen();
+
+        $this->assertEquals(1, $called);
+
+        $this->fakeDriver->hasMatchingEvent = new TestEventTwo([]);
+
+        $this->botman->listen();
+
+        $this->assertEquals(2, $called);
+    }
+
     /** @test */
     public function it_calls_driver_events_without_closure()
     {
@@ -114,6 +134,19 @@ public function getPayload()
     }
 }
 
+class TestEventTwo extends TestEvent
+{
+    /**
+     * Return the event name to match.
+     *
+     * @return string
+     */
+    public function getName()
+    {
+         return 'test_event_two';
+    }
+}
+
 class TestEventClass
 {
     public function event($payload, $bot)