From aae6d359aab1b316451fcd123e4dc34fbe7acf13 Mon Sep 17 00:00:00 2001
From: Oliver Vogel <oliver@olivervogel.com>
Date: Sat, 4 Nov 2023 08:57:28 +0100
Subject: [PATCH] Add InputHandlerInterface

---
 src/Drivers/Abstract/AbstractInputHandler.php | 28 +++++++++----------
 src/Interfaces/InputHandlerInterface.php      | 21 ++++++++++++++
 .../Abstract/AbstractInputHandlerTest.php     |  7 +++--
 3 files changed, 39 insertions(+), 17 deletions(-)
 create mode 100644 src/Interfaces/InputHandlerInterface.php

diff --git a/src/Drivers/Abstract/AbstractInputHandler.php b/src/Drivers/Abstract/AbstractInputHandler.php
index faf90ad5..4fdbe951 100644
--- a/src/Drivers/Abstract/AbstractInputHandler.php
+++ b/src/Drivers/Abstract/AbstractInputHandler.php
@@ -6,8 +6,9 @@ use Intervention\Image\Drivers\Abstract\Decoders\AbstractDecoder;
 use Intervention\Image\Exceptions\DecoderException;
 use Intervention\Image\Interfaces\ColorInterface;
 use Intervention\Image\Interfaces\ImageInterface;
+use Intervention\Image\Interfaces\InputHandlerInterface;
 
-abstract class AbstractInputHandler
+abstract class AbstractInputHandler implements InputHandlerInterface
 {
     /**
      * Array of decoders which will be stacked into to the input handler chain
@@ -15,15 +16,25 @@ abstract class AbstractInputHandler
     protected array $decoders = [];
 
     /**
-     * Create new instance
+     * {@inheritdoc}
      *
-     * @param array $decoders
+     * @see InputHandlerInterface::__construct()
      */
     public function __construct(array $decoders = [])
     {
         $this->decoders = count($decoders) ? $decoders : $this->decoders;
     }
 
+    /**
+     * {@inheritdoc}
+     *
+     * @see InputHandlerInterface::handle()
+     */
+    public function handle($input): ImageInterface|ColorInterface
+    {
+        return $this->chain()->handle($input);
+    }
+
     /**
      * Stack the decoder array into a nested decoder object
      *
@@ -46,15 +57,4 @@ abstract class AbstractInputHandler
 
         return $chain;
     }
-
-    /**
-     * Try to decode the given input with each decoder of the the handler chain
-     *
-     * @param  mixed $input
-     * @return ImageInterface|ColorInterface
-     */
-    public function handle($input): ImageInterface|ColorInterface
-    {
-        return $this->chain()->handle($input);
-    }
 }
diff --git a/src/Interfaces/InputHandlerInterface.php b/src/Interfaces/InputHandlerInterface.php
new file mode 100644
index 00000000..9112021f
--- /dev/null
+++ b/src/Interfaces/InputHandlerInterface.php
@@ -0,0 +1,21 @@
+<?php
+
+namespace Intervention\Image\Interfaces;
+
+interface InputHandlerInterface
+{
+    /**
+     * Create new instance with an array of decoders
+     *
+     * @param array $decoders
+     */
+    public function __construct(array $decoders = []);
+
+    /**
+     * Try to decode the given input with each decoder of the the handler chain
+     *
+     * @param  mixed $input
+     * @return ImageInterface|ColorInterface
+     */
+    public function handle($input): ImageInterface|ColorInterface;
+}
diff --git a/tests/Drivers/Abstract/AbstractInputHandlerTest.php b/tests/Drivers/Abstract/AbstractInputHandlerTest.php
index ee40bb97..3f2ea472 100644
--- a/tests/Drivers/Abstract/AbstractInputHandlerTest.php
+++ b/tests/Drivers/Abstract/AbstractInputHandlerTest.php
@@ -29,15 +29,16 @@ final class AbstractInputHandlerTest extends TestCase
 
     private function getModifier(AbstractDecoder $chain): AbstractInputHandler
     {
-        return new class ($chain) extends AbstractInputHandler {
-            public function __construct(private AbstractDecoder $chain)
+        return new class ([$chain]) extends AbstractInputHandler
+        {
+            public function __construct(protected array $decoders = [])
             {
                 //
             }
 
             protected function chain(): AbstractDecoder
             {
-                return $this->chain;
+                return $this->decoders[0];
             }
         };
     }