Merge pull request #1 from clue-labs/process

ProcessLauncher now also accepts a Process instance

Author: clue

src/ProcessLauncher.php

@@ -25,12 +25,15 @@ public function __construct(LoopInterface $loop)
      * If the command prints output to STDERR, make sure to redirect it to
      * STDOUT by appending " 2>&1".
      *
-     * @param string $command
+     * @param string|Process $process accepts either a command string to execute or a Process instance
      * @return DeferredShell
      */
-    public function createDeferredShell($command)
+    public function createDeferredShell($process)
     {
-        $process = new Process($command);
+        if (!($process instanceof Process)) {
+            $process = new Process($process);
+        }
+
         $process->start($this->loop);
 
         $stream = new CompositeStream($process->stdout, $process->stdin);

tests/ProcessLauncherTest.php

@@ -0,0 +1,28 @@
+<?php
+
+use Clue\React\Shell\ProcessLauncher;
+
+class ProcessLauncherTest extends TestCase
+{
+    private $loop;
+    private $processLauncher;
+
+    public function setUp()
+    {
+        $this->loop = $this->getMock('React\EventLoop\LoopInterface');
+        $this->processLauncher = new ProcessLauncher($this->loop);
+    }
+
+    public function testProcessWillBeStarted()
+    {
+        $process = $this->getMockBuilder('React\ChildProcess\Process')->disableOriginalConstructor()->getMock();
+        $process->stdout = $this->getMock('React\Stream\ReadableStreamInterface');
+        $process->stdin = $this->getMock('React\Stream\WritableStreamInterface');
+
+        $process->expects($this->once())->method('start');
+
+        $shell = $this->processLauncher->createDeferredShell($process);
+
+        $this->assertInstanceOf('Clue\React\Shell\DeferredShell', $shell);
+    }
+}