Increase test coverage to 100%

Author: cweiske

src/JsonMapper.php

@@ -822,7 +822,7 @@ protected function isArrayOfType($strType)
 
     /**
      * Returns true if accessor is a method and has only one parameter
-     * which is variadic.
+     * which is variadic ("...$args").
      *
      * @param ReflectionMethod|ReflectionProperty|null $accessor accessor
      *                                                           to set value

tests/ArrayTest.php

@@ -527,6 +527,20 @@ public function testMapArrayFromVariadicFunctionWithObjectType()
         $this->assertSame('2014-01-02', $variadicArray[0]->format('Y-m-d'));
         $this->assertSame('2014-05-07', $variadicArray[1]->format('Y-m-d'));
     }
+
+    /**
+     * Test the "if (count($parameters) !== 1) {" condition in "hasVariadicArrayType()"
+     */
+    public function testMapArrayVariadicMethodWithMultipleParams()
+    {
+        $jm = new JsonMapper();
+        $sn = $jm->map(
+            json_decode('{"multipleParams":[23]}'),
+            new JsonMapperTest_VariadicArray()
+        );
+
+        $this->assertSame([23], $sn->multipleParamsVal);
+    }
 }
 
 ?>

tests/support/JsonMapperTest/VariadicArray.php

@@ -30,6 +30,8 @@ class JsonMapperTest_VariadicArray
      */
     private $variadicInt;
 
+    public $multipleParamsVal;
+
     /**
      * @param DateTime[] $items
      *
@@ -69,5 +71,10 @@ public function setVariadicInt(int ...$items): self
 
         return $this;
     }
+
+    public function setMultipleParams(array $param, int ...$dummy)
+    {
+        $this->multipleParamsVal = $param;
+    }
 }
 ?>