Add test to make sure all event handler exist

Author: stayallive

src/Sentry/Laravel/EventHandler.php

@@ -150,12 +150,14 @@ public function subscribeQueueEvents()
      */
     public function __call($method, $arguments)
     {
-        if (!method_exists($this, $method . 'handler')) {
-            throw new RuntimeException('Missing event handler:' . $method . 'handler');
+        $handlerMethod = $handlerMethod = "{$method}Handler";
+
+        if (!method_exists($this, $handlerMethod)) {
+            throw new RuntimeException("Missing event handler: {$handlerMethod}");
         }
 
         try {
-            call_user_func_array([$this, $method . 'handler'], $arguments);
+            call_user_func_array([$this, $handlerMethod], $arguments);
         } catch (Exception $exception) {
             // Ignore
         }

test/Sentry/EventHandlerTest.php

@@ -2,15 +2,62 @@
 
 namespace Sentry\Laravel\Tests;
 
-class EventHandlerTest extends \Orchestra\Testbench\TestCase
+use ReflectionClass;
+use Sentry\Laravel\EventHandler;
+use Orchestra\Testbench\TestCase;
+
+class EventHandlerTest extends TestCase
 {
     /**
      * @expectedException \RuntimeException
      */
     public function test_missing_event_handler_throws_exception()
     {
-        $handler = new \Sentry\Laravel\EventHandler($this->app->events, []);
+        $handler = new EventHandler($this->app->events, []);
 
         $handler->thisIsNotAHandlerAndShouldThrowAnException();
     }
+
+    public function test_all_mapped_event_handlers_exist()
+    {
+        $this->tryAllEventHandlerMethods(
+            $this->getStaticPropertyValueFromClass(EventHandler::class, 'eventHandlerMap')
+        );
+    }
+
+    public function test_all_mapped_auth_event_handlers_exist()
+    {
+        $this->tryAllEventHandlerMethods(
+            $this->getStaticPropertyValueFromClass(EventHandler::class, 'authEventHandlerMap')
+        );
+    }
+
+    public function test_all_mapped_queue_event_handlers_exist()
+    {
+        $this->tryAllEventHandlerMethods(
+            $this->getStaticPropertyValueFromClass(EventHandler::class, 'queueEventHandlerMap')
+        );
+    }
+
+    private function tryAllEventHandlerMethods(array $methods): void
+    {
+        $handler = new EventHandler($this->app->events, []);
+
+        $methods = array_map(static function ($method) {
+            return "{$method}Handler";
+        }, array_unique(array_values($methods)));
+
+        foreach ($methods as $handlerMethod) {
+            $this->assertTrue(method_exists($handler, $handlerMethod));
+        }
+    }
+
+    private function getStaticPropertyValueFromClass($className, $attributeName)
+    {
+        $class = new ReflectionClass($className);
+
+        $attributes = $class->getStaticProperties();
+
+        return $attributes[$attributeName];
+    }
 }