Tests

Author: tatevikg1

config/services/services.yml

@@ -106,4 +106,4 @@ services:
 
   PhpList\Core\Domain\Messaging\Service\BounceActionResolver:
     arguments:
-      - !tagged_iterator { tag: 'phplist.bounce_action_handler', default_index_method: 'supports' }
+      - !tagged_iterator { tag: 'phplist.bounce_action_handler' }

src/Domain/Messaging/Service/BounceActionResolver.php

@@ -55,10 +55,11 @@ public function handle(string $action, array $context): void
     private function find(string $action): ?BounceActionHandlerInterface
     {
         foreach ($this->handlers as $handler) {
-            if ($handler->supports($action)) {
+            if ($handler::supports($action)) {
                 return $handler;
             }
         }
+
         return null;
     }
 }

src/Domain/Messaging/Service/Handler/BlacklistEmailAndDeleteBounceHandler.php

@@ -24,7 +24,7 @@ public function __construct(
         $this->bounceManager = $bounceManager;
     }
 
-    public function supports(string $action): bool
+    public static function supports(string $action): bool
     {
         return $action === 'blacklistemailanddeletebounce';
     }

src/Domain/Messaging/Service/Handler/BlacklistEmailHandler.php

@@ -20,7 +20,7 @@ public function __construct(
         $this->subscriberManager = $subscriberManager;
     }
 
-    public function supports(string $action): bool
+    public static function supports(string $action): bool
     {
         return $action === 'blacklistemail';
     }

src/Domain/Messaging/Service/Handler/BlacklistUserAndDeleteBounceHandler.php

@@ -24,7 +24,7 @@ public function __construct(
         $this->bounceManager = $bounceManager;
     }
 
-    public function supports(string $action): bool
+    public static function supports(string $action): bool
     {
         return $action === 'blacklistuseranddeletebounce';
     }

src/Domain/Messaging/Service/Handler/BlacklistUserHandler.php

@@ -20,7 +20,7 @@ public function __construct(
         $this->subscriberManager = $subscriberManager;
     }
 
-    public function supports(string $action): bool
+    public static function supports(string $action): bool
     {
         return $action === 'blacklistuser';
     }

src/Domain/Messaging/Service/Handler/BounceActionHandlerInterface.php

@@ -6,6 +6,6 @@
 
 interface BounceActionHandlerInterface
 {
-    public function supports(string $action): bool;
+    public static function supports(string $action): bool;
     public function handle(array $closureData): void;
 }

src/Domain/Messaging/Service/Handler/DecreaseCountConfirmUserAndDeleteBounceHandler.php

@@ -28,7 +28,7 @@ public function __construct(
         $this->subscriberRepository = $subscriberRepository;
     }
 
-    public function supports(string $action): bool
+    public static function supports(string $action): bool
     {
         return $action === 'decreasecountconfirmuseranddeletebounce';
     }

src/Domain/Messaging/Service/Handler/DeleteBounceHandler.php

@@ -15,7 +15,7 @@ public function __construct(BounceManager $bounceManager)
         $this->bounceManager = $bounceManager;
     }
 
-    public function supports(string $action): bool
+    public static function supports(string $action): bool
     {
         return $action === 'deletebounce';
     }

src/Domain/Messaging/Service/Handler/DeleteUserAndBounceHandler.php

@@ -18,7 +18,7 @@ public function __construct(BounceManager $bounceManager, SubscriberManager $sub
         $this->subscriberManager = $subscriberManager;
     }
 
-    public function supports(string $action): bool
+    public static function supports(string $action): bool
     {
         return $action === 'deleteuserandbounce';
     }