Merge pull request consolidation#158 from fabfuel/master

PhpServer: Wrap `php` command with `exec`
fajar-khairil · Apr 14, 2015 · b529585 · b529585
2 parents 0516822 + 615d4b4
commit b529585
Show file tree

Hide file tree

Showing 2 changed files with 13 additions and 2 deletions.
diff --git a/src/Task/Development/PhpServer.php b/src/Task/Development/PhpServer.php
@@ -34,6 +34,10 @@ class PhpServer extends Exec
     public function __construct($port)
     {
         $this->port = $port;
+
+        if (strtolower(PHP_OS) === 'linux') {
+            $this->command = 'exec php -S %s:%d ';
+        }
     }
 
     public function host($host)

diff --git a/tests/unit/PHPServerTest.php b/tests/unit/PHPServerTest.php
@@ -34,11 +34,18 @@ public function testServerRun()
 
     public function testServerCommand()
     {
+        if (strtolower(PHP_OS) === 'linux') {
+            $expectedCommand = 'exec php -S 127.0.0.1:8000 -t web';
+        } else {
+            $expectedCommand = 'php -S 127.0.0.1:8000 -t web';
+        }
+
         verify(
             $this->taskServer('8000')
+                ->host('127.0.0.1')
                 ->dir('web')
                 ->getCommand()
-        )->equals('php -S 127.0.0.1:8000 -t web');
+        )->equals($expectedCommand);
     }
 
-}
+}