Add missing type hints and parameter types to FuncProxy (#135)

* Add missing type hints and parameter types to FuncProxy

* Adapt test to fit for array type hint

Author: SenseException

src/AspectMock/Proxy/FuncProxy.php

@@ -25,15 +25,30 @@
  */
 class FuncProxy
 {
+    /**
+     * @var string
+     */
     protected $func;
+    
+    /**
+     * @var string
+     */
     protected $ns;
+    
+    /**
+     * @var string
+     */
     protected $fullFuncName;
 
     /**
      * @var FuncVerifier
      */
     protected $funcVerifier;
 
+    /**
+     * @param string
+     * @param string
+     */
     public function __construct($namespace, $func)
     {
         $this->func = $func;
@@ -43,34 +58,34 @@ public function __construct($namespace, $func)
     }
 
     /**
-     * @param null $params
+     * @param null|array $params
      */
-    public function verifyInvoked($params = null)
+    public function verifyInvoked(array $params = null)
     {
         $this->funcVerifier->verifyInvoked($this->func, $params);
     }
 
     /**
-     * @param null $params
+     * @param null|array $params
      */
-    public function verifyInvokedOnce($params = null)
+    public function verifyInvokedOnce(array $params = null)
     {
         $this->funcVerifier->verifyInvokedMultipleTimes($this->func, 1, $params);
     }
 
     /**
-     * @param null $params
+     * @param null|array $params
      */
-    public function verifyNeverInvoked($params = null)
+    public function verifyNeverInvoked(array $params = null)
     {
         $this->funcVerifier->verifyNeverInvoked($this->func, $params);
     }
 
     /**
-     * @param $times
-     * @param null $params
+     * @param int        $times
+     * @param null|array $params
      */
-    public function verifyInvokedMultipleTimes($times, $params = null)
+    public function verifyInvokedMultipleTimes($times, array $params = null)
     {
         $this->funcVerifier->verifyInvokedMultipleTimes($this->func, $times, $params);
     }
@@ -84,10 +99,13 @@ public function __invoke()
         return call_user_func_array($this->ns .'\\'.$this->func, func_get_args());
     }
 
-
+    /**
+     * @param string
+     * @return array
+     */
     public function getCallsForMethod($func)
     {
         $calls = Registry::getFuncCallsFor($this->ns . '\\' . $func);
         return $calls;
     }
-}
+}

tests/unit/FunctionInjectorTest.php

@@ -3,18 +3,19 @@
 
 use AspectMock\Intercept\FunctionInjector;
 use AspectMock\Test as test;
-use Codeception\TestCase;
 
 class FunctionInjectorTest extends \Codeception\TestCase\Test
 {
     /**
      * @var FunctionInjector
      */
     protected $funcInjector;
+
     /**
      * @var FunctionInjector
      */
     protected $funcOptionalParameterInjector;
+
     /**
      * @var FunctionInjector
      */
@@ -101,7 +102,7 @@ public function testFailedVerification()
     {
         $func = test::func('demo', 'strlen', function() { return 10; });
         expect(strlen('hello'))->equals(10);
-        $func->verifyNeverInvoked('strlen');
+        $func->verifyNeverInvoked();
     }
 
     public function testReferencedParameter()