Merge pull request #32 from pbories/MultipleSources

Allow multiple sources as first parameter of convert() method

Author: Pierstoval

Tests/CommandTest.php

@@ -44,6 +44,40 @@ public function provideWrongConvertDirs(): ?\Generator
         yield ['./', 'The specified path (.) is not a file.', MagickBinaryNotFoundException::class];
     }
 
+    /**
+     * @param $fileSources
+     *
+     * @dataProvider convertDataProvider
+     */
+    public function testConvert($fileSources, string $fileOutput): void
+    {
+
+        $command = new Command(IMAGEMAGICK_DIR);
+
+        $response = $command
+            ->convert($fileSources)
+            ->output($fileOutput)
+            ->run();
+
+        static::assertFileExists($fileOutput);
+
+        static::assertFalse($response->hasFailed());
+    }
+
+    public function convertDataProvider(): array
+    {
+        $singleSource = $this->resourcesDir.'/moon_180.jpg';
+        $multipleSources = [
+            $this->resourcesDir.'/moon_180.jpg',
+            $this->resourcesDir.'/dabug.png',
+        ];
+
+        return [
+            [$singleSource, $this->resourcesDir.'/outputs/output1.pdf'],
+            [$multipleSources, $this->resourcesDir.'/outputs/output2.pdf'],
+        ];
+    }
+
     public function testResizeImage(): void
     {
         $command = new Command(IMAGEMAGICK_DIR);

Tests/Resources/dabug.png

@@ -177,9 +177,18 @@ public function newCommand(?string $binary = null): self
     /**
      * @see https://imagemagick.org/script/convert.php
      */
-    public function convert(string $sourceFile, bool $checkIfFileExists = true): self
+    public function convert($sourceFiles, bool $checkIfFileExists = true): self
     {
-        return $this->newCommand('convert')->file($sourceFile, $checkIfFileExists);
+        if (!is_array($sourceFiles)) {
+            $sourceFiles = [$sourceFiles];
+        }
+
+        $this->newCommand('convert');
+        foreach ($sourceFiles as $sourceFile) {
+            $this->file($sourceFile, $checkIfFileExists);
+        }
+
+        return $this;
     }
 
     /**