Merge pull request #32 from php-fn/pr-variadic-parameters

Support functions with (non typed) variadic parameters

Author: mnapoli

src/Invoker.php

@@ -62,9 +62,8 @@ public function call($callable, array $parameters = [])
 
         // Check all parameters are resolved
         $diff = array_diff_key($callableReflection->getParameters(), $args);
-        if (! empty($diff)) {
-            $parameter = reset($diff);
-            \assert($parameter instanceof ReflectionParameter);
+        $parameter = reset($diff);
+        if ($parameter && \assert($parameter instanceof ReflectionParameter) && ! $parameter->isVariadic()) {
             throw new NotEnoughParametersException(sprintf(
                 'Unable to invoke the callable because no value was given for parameter %d ($%s)',
                 $parameter->getPosition() + 1,

tests/InvokerTest.php

@@ -451,11 +451,11 @@ public function should_throw_if_no_value_for_optional_parameter_2()
      */
     public function should_invoke_callable_with_variadic_parameter()
     {
-        $result = $this->invoker->call(function (...$param) {
+        $callable = function (...$param) {
             return $param;
-        }, [1, 2, 3]);
-
-        $this->assertEquals([1, 2, 3], $result);
+        };
+        $this->assertEquals([1, 2, 3], $this->invoker->call($callable, [1, 2, 3]), 'non-empty variadic');
+        $this->assertEquals([], $this->invoker->call($callable, []), 'empty variadic');
     }
 
     private function assertWasCalled(CallableSpy $callableSpy)