Merge pull request #257 from sasezaki/fix255

fix #255 - Don't evaluate '0' as empty

Author: weierophinney

src/PhpInputStream.php

@@ -59,7 +59,7 @@ public function isWritable()
     public function read($length)
     {
         $content = parent::read($length);
-        if ($content && ! $this->reachedEof) {
+        if (! $this->reachedEof) {
             $this->cache .= $content;
         }
 

test/PhpInputStreamTest.php

@@ -64,6 +64,17 @@ public function testGetContentsReturnsRemainingContentsOfStream()
         $this->assertEquals(substr($contents, 128), $remainder);
     }
 
+    public function testGetContentsReturnCacheWhenReachedEof()
+    {
+        $this->stream->getContents();
+        $this->assertStreamContents($this->stream->getContents());
+
+        $stream = new PhpInputStream('data://,0');
+        $stream->read(1);
+        $stream->read(1);
+        $this->assertSame('0', $stream->getContents(), 'Don\'t evaluate 0 as empty');
+    }
+
     public function testCastingToStringReturnsFullContentsRegardlesOfPriorReads()
     {
         $start = $this->stream->read(128);