Fix missing return value of Coroutine\go

swoole · Jun 25, 2021 · 1ed49db · 1ed49db
1 parent 4287a48
commit 1ed49db
Show file tree

Hide file tree

Showing 2 changed files with 11 additions and 1 deletion.
diff --git a/src/core/Coroutine/functions.php b/src/core/Coroutine/functions.php
@@ -26,7 +26,7 @@ function run(callable $fn, ...$args)
 
 function go(callable $fn, ...$args)
 {
-    Coroutine::create($fn, ...$args);
+    return Coroutine::create($fn, ...$args);
 }
 
 function defer(callable $fn)

diff --git a/tests/unit/Coroutine/FunctionTest.php b/tests/unit/Coroutine/FunctionTest.php
@@ -88,6 +88,16 @@ public function testBatch()
         });
     }
 
+    public function testGo()
+    {
+        run(function () {
+            $cid = go(function() {
+                System::sleep(0.001);
+            });
+            $this->assertTrue(is_int($cid) and $cid > 0);
+        });
+    }
+
     public function testParallel()
     {
         run(function () {