[Process] Use correct test for empty string in UnixPipes

whs · romainneutron · commit 9e1ea4aa4b21 · 2014-07-12T16:25:05.000+02:00
diff --git a/src/Symfony/Component/Process/ProcessPipes.php b/src/Symfony/Component/Process/ProcessPipes.php
@@ -313,11 +313,11 @@ private function readStreams($blocking, $close = false)
             $type = array_search($pipe, $this->pipes);
 
             $data = '';
-            while ($dataread = fread($pipe, self::CHUNK_SIZE)) {
+            while ('' !== $dataread = (string) fread($pipe, self::CHUNK_SIZE)) {
                 $data .= $dataread;
             }
 
-            if ($data) {
+            if ('' !== $data) {
                 $read[$type] = $data;
             }
 
diff --git a/src/Symfony/Component/Process/Tests/AbstractProcessTest.php b/src/Symfony/Component/Process/Tests/AbstractProcessTest.php
@@ -287,6 +287,12 @@ public function testGetIncrementalOutput()
         }
     }
 
+    public function testZeroAsOutput(){
+        $p = $this->getProcess('printf 0');
+        $p->run();
+        $this->assertSame('0', $p->getOutput());
+    }
+
     public function testExitCodeCommandFailed()
     {
         if (defined('PHP_WINDOWS_VERSION_BUILD')) {

Original file line number	Diff line number	Diff line change
`@@ -287,6 +287,12 @@ public function testGetIncrementalOutput()`
`287`	`287`	`}`
`288`	`288`	`}`
`289`	`289`
	`290`	`+ public function testZeroAsOutput(){`
	`291`	`+ $p = $this->getProcess('printf 0');`
	`292`	`+ $p->run();`
	`293`	`+ $this->assertSame('0', $p->getOutput());`
	`294`	`+ }`
	`295`	`+`
`290`	`296`	`public function testExitCodeCommandFailed()`
`291`	`297`	`{`
`292`	`298`	`if (defined('PHP_WINDOWS_VERSION_BUILD')) {`