Merge pull request #38 from gabbanaesteban/add-restriction-to-map-mul…

…tiple

Add validation to mapMultiple operation

src/AutoMapper.php

@@ -6,6 +6,7 @@
 use AutoMapperPlus\Configuration\AutoMapperConfigInterface;
 use AutoMapperPlus\Configuration\MappingInterface;
 use AutoMapperPlus\Exception\AutoMapperPlusException;
+use AutoMapperPlus\Exception\InvalidArgumentException;
 use AutoMapperPlus\Exception\UnregisteredMappingException;
 use AutoMapperPlus\MappingOperation\ContextAwareOperation;
 use AutoMapperPlus\MappingOperation\MapperAwareOperation;
@@ -96,6 +97,13 @@ public function mapMultiple(
         string $destinationClass,
         array $context = []
     ): array {
+
+        if(!is_iterable($sourceCollection)){
+            throw new InvalidArgumentException(
+                'The collection provided should be iterable.'
+            );
+        }
+
         $mappedResults = [];
         foreach ($sourceCollection as $source) {
             $mappedResults[] = $this->map($source, $destinationClass, $context);

src/Exception/InvalidArgumentException.php

@@ -0,0 +1,12 @@
+<?php
+
+namespace AutoMapperPlus\Exception;
+
+/**
+ * Class InvalidArgumentException
+ *
+ * @package AutoMapperPlus\Exception
+ */
+class InvalidArgumentException extends AutoMapperPlusException
+{
+}

test/AutoMapperTest.php

@@ -43,6 +43,7 @@
 use AutoMapperPlus\Test\Models\Visibility\Visibility;
 use AutoMapperPlus\Test\Models\SimilarPropertyNames\Source as SimilarSource;
 use AutoMapperPlus\Test\Models\SimilarPropertyNames\Destination as SimilarDestination;
+use AutoMapperPlus\Exception\InvalidArgumentException;
 
 /**
  * Class AutoMapperTest
@@ -721,4 +722,16 @@ public function testMappingsCanBeGeneratedOnTheFlyIfOptionIsSet()
 
         $this->assertEquals('a name', $result->name);
     }
+
+    public function testAnExceptionIsThrownForNoIterableSourceInMultpleMappings()
+    {
+        $this->expectException(InvalidArgumentException::class);
+
+        $this->config->registerMapping(Source::class, Destination::class);
+        $mapper = new AutoMapper($this->config);
+
+        $sourceCollection = new \stdClass();
+
+        $mapper->mapMultiple($sourceCollection, Destination::class);
+    }
 }