Browse Source

Merge pull request #10935 from cakephp/client-response

Add failing test for Http/Client/Response
Mark Story 8 years ago
parent
commit
ff1ac8eff4
2 changed files with 5 additions and 2 deletions
  1. 1 0
      src/Http/Client/Response.php
  2. 4 2
      tests/TestCase/Http/Client/ResponseTest.php

+ 1 - 0
src/Http/Client/Response.php

@@ -151,6 +151,7 @@ class Response extends Message implements ResponseInterface
         }
         $stream = new Stream('php://memory', 'wb+');
         $stream->write($body);
+        $stream->rewind();
         $this->stream = $stream;
     }
 

+ 4 - 2
tests/TestCase/Http/Client/ResponseTest.php

@@ -102,10 +102,12 @@ class ResponseTest extends TestCase
         $encoded = json_encode($data);
 
         $response = new Response([], $encoded);
-        $result = $response->body('json_decode');
-        $this->assertEquals($data['property'], $result->property);
+
+        $this->assertEquals($encoded, $response->getBody()->getContents());
         $this->assertEquals($encoded, $response->body());
 
+        $result = $response->body('json_decode');
+        $this->assertEquals($data['property'], $result->property);
         $this->assertEquals($encoded, $response->body);
         $this->assertTrue(isset($response->body));
     }