fix: add catch to any throwable in consumer manager

src/Connectors/Consumer/Manager.php

@@ -1,14 +1,14 @@
 <?php
 namespace Metamorphosis\Connectors\Consumer;
 
-use Exception;
 use Metamorphosis\Consumers\ConsumerInterface;
 use Metamorphosis\Exceptions\ResponseTimeoutException;
 use Metamorphosis\Exceptions\ResponseWarningException;
 use Metamorphosis\Middlewares\Handler\Dispatcher;
 use Metamorphosis\Record\ConsumerRecord;
 use Metamorphosis\TopicHandler\Consumer\Handler as ConsumerHandler;
 use RdKafka\Message;
+use Throwable;
 
 class Manager
 {
@@ -79,8 +79,8 @@ public function handleMessage(): void
         } catch (ResponseWarningException $exception) {
             $this->consumerHandler->warning($exception);
             return;
-        } catch (Exception $exception) {
-            $this->consumerHandler->failed($exception);
+        } catch (Throwable $throwable) {
+            $this->consumerHandler->failed($throwable);
             return;
         }
 

src/TopicHandler/Consumer/AbstractHandler.php

@@ -1,9 +1,9 @@
 <?php
 namespace Metamorphosis\TopicHandler\Consumer;
 
-use Exception;
 use Metamorphosis\Exceptions\ResponseWarningException;
 use Metamorphosis\TopicHandler\ConfigOptions\Consumer as ConsumerConfigOptions;
+use Throwable;
 
 abstract class AbstractHandler implements Handler
 {
@@ -23,7 +23,7 @@ public function warning(ResponseWarningException $exception): void
     {
     }
 
-    public function failed(Exception $exception): void
+    public function failed(Throwable $throwable): void
     {
     }
 

src/TopicHandler/Consumer/Handler.php

@@ -1,9 +1,9 @@
 <?php
 namespace Metamorphosis\TopicHandler\Consumer;
 
-use Exception;
 use Metamorphosis\Exceptions\ResponseWarningException;
 use Metamorphosis\Record\RecordInterface;
+use Throwable;
 
 interface Handler
 {
@@ -25,5 +25,5 @@ public function finished(): void;
     /**
      * Handle failure process.
      */
-    public function failed(Exception $exception): void;
+    public function failed(Throwable $throwable): void;
 }

tests/Integration/Dummies/MessageConsumer.php

@@ -1,11 +1,11 @@
 <?php
 namespace Tests\Integration\Dummies;
 
-use Exception;
 use Illuminate\Support\Facades\Log;
 use Metamorphosis\Exceptions\ResponseWarningException;
 use Metamorphosis\Record\RecordInterface;
 use Metamorphosis\TopicHandler\Consumer\AbstractHandler;
+use Throwable;
 
 class MessageConsumer extends AbstractHandler
 {
@@ -23,10 +23,10 @@ public function warning(ResponseWarningException $exception): void
         ]);
     }
 
-    public function failed(Exception $exception): void
+    public function failed(Throwable $throwable): void
     {
         Log::error('Failed to handle kafka record for sku.', [
-            'exception' => $exception,
+            'exception' => $throwable,
         ]);
     }
 }

tests/Unit/Connectors/Consumer/ManagerTest.php

@@ -1,7 +1,9 @@
 <?php
 namespace Tests\Unit\Connectors\Consumer;
 
+use Error;
 use Exception;
+use InvalidArgumentException;
 use Metamorphosis\Connectors\Consumer\Manager;
 use Metamorphosis\ConsumerConfigManager;
 use Metamorphosis\Consumers\ConsumerInterface;
@@ -14,6 +16,8 @@
 use RdKafka\Message as KafkaMessage;
 use Tests\LaravelTestCase;
 use Tests\Unit\Dummies\ConsumerHandlerDummy;
+use Throwable;
+use TypeError;
 
 class ManagerTest extends LaravelTestCase
 {
@@ -35,6 +39,30 @@ protected function setUp(): void
         ]);
     }
 
+    /**
+     * @dataProvider getThrowableScenarios
+     */
+    public function testShouldHandlerAnyThrowable(Throwable $throwable): void
+    {
+        // Set
+        $consumer = m::mock(ConsumerInterface::class);
+        $consumerHandler = m::mock(ConsumerHandler::class);
+        $dispatcher = m::mock(Dispatcher::class);
+
+        $runner = new Manager($consumer, $consumerHandler, $dispatcher, true, false);
+
+        // Expectations
+        $consumer->expects()
+            ->consume()
+            ->andThrow($throwable);
+
+        $consumerHandler->expects()
+            ->failed($throwable);
+
+        // Actions
+        $runner->handleMessage();
+    }
+
     public function testShouldHandleMultiplesMessages(): void
     {
         // Set
@@ -171,4 +199,22 @@ public function testShouldHandleAsyncCommit(): void
         $runner->handleMessage();
         $runner->handleMessage();
     }
+
+    public function getThrowableScenarios(): array
+    {
+        return [
+            'Exception' => [
+                'throwable' => new Exception(),
+            ],
+            'Error' => [
+                'throwable' => new Error(),
+            ],
+            'InvalidArgumentException' => [
+                'throwable' => new InvalidArgumentException(),
+            ],
+            'TypeError' => [
+                'throwable' => new TypeError(),
+            ],
+        ];
+    }
 }

tests/Unit/Dummies/ConsumerHandlerDummy.php

@@ -1,17 +1,17 @@
 <?php
 namespace Tests\Unit\Dummies;
 
-use Exception;
 use Metamorphosis\Record\RecordInterface;
 use Metamorphosis\TopicHandler\Consumer\AbstractHandler;
+use Throwable;
 
 class ConsumerHandlerDummy extends AbstractHandler
 {
     public function handle(RecordInterface $data): void
     {
     }
 
-    public function failed(Exception $exception): void
+    public function failed(Throwable $throwable): void
     {
     }
 }