run pipes immediately

laravel · Apr 11, 2023 · e34ab39 · e34ab39
1 parent a1b2196
commit e34ab39
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 4 deletions.
diff --git a/src/Illuminate/Process/Factory.php b/src/Illuminate/Process/Factory.php
@@ -277,9 +277,9 @@ public function pool(callable $callback)
      * @param  callable  $callback
      * @return \Illuminate\Process\Pipe
      */
-    public function pipe(callable $callback)
+    public function pipe(callable $callback, ?callable $output = null)
     {
-        return new Pipe($this, $callback);
+        return (new Pipe($this, $callback))->run(output: $output);
     }
 
     /**

diff --git a/tests/Process/ProcessTest.php b/tests/Process/ProcessTest.php
@@ -512,7 +512,7 @@ public function testProcessPipe()
             $pipe->command('grep -i "foo"');
         });
 
-        $this->assertSame("foo\n", $pipe->run()->output());
+        $this->assertSame("foo\n", $pipe->output());
     }
 
     public function testProcessPipeFailed()
@@ -531,7 +531,7 @@ public function testProcessPipeFailed()
             $pipe->command('grep -i "foo"');
         });
 
-        $this->assertTrue($pipe->run()->failed());
+        $this->assertTrue($pipe->failed());
     }
 
     public function testFakeInvokedProcessOutputWithLatestOutput()