Merge pull request #211 from bogkonstantin/fix-func-map-check

When you expect value null to be returned, fake functions was not invoked
Codeception · Nov 16, 2024 · 6ce94f8 · 6ce94f8
2 parents 7247230 + dd3a8d9
commit 6ce94f8
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 1 deletion.
diff --git a/src/AspectMock/Core/Mocker.php b/src/AspectMock/Core/Mocker.php
@@ -71,7 +71,7 @@ public function fakeFunctionAndRegisterCalls($namespace, $function, $args)
         $fullFuncName = sprintf('%s\%s', $namespace, $function);
         Registry::registerFunctionCall($fullFuncName, $args);
 
-        if (isset($this->funcMap[$fullFuncName])) {
+        if (array_key_exists($fullFuncName, $this->funcMap)) {
             $func = $this->funcMap[$fullFuncName];
             $result = is_callable($func) ? call_user_func_array($func, $args) : $func;
         }

diff --git a/tests/unit/FunctionInjectorTest.php b/tests/unit/FunctionInjectorTest.php
@@ -65,6 +65,12 @@ public function testReimplementFunc()
         verify(strlen('hello'))->equals(10);
     }
 
+    public function testFuncReturnsNull()
+    {
+        test::func('demo', 'strlen', null);
+        verify(strlen('hello'))->equals(null);
+    }
+
     public function testVerifier()
     {
         $func = test::func('demo', 'strlen', 10);