Fix TestResponse::getBody

README.md

@@ -47,6 +47,10 @@ Test doubles
 
 ## Release notes
 
+### 1.0.1 (2022-02-02)
+
+* Fixed behavior of `TestResponse::getBody`
+
 ### 1.0.0 (2022-01-30)
 
 Initial release with

src/TestResponse.php

@@ -10,8 +10,6 @@
 
 class TestResponse implements ResponseInterface {
 
-	private bool $gotBody = false;
-
 	public function __construct(
 		private readonly string $body = '',
 		private readonly int $statusCode = 200
@@ -91,12 +89,14 @@ public function withoutHeader( $name ): self {
 	}
 
 	public function getBody(): StreamInterface {
-		return new PumpStream( function() {
-			if ( $this->gotBody ) {
+		$gotBody = false;
+
+		return new PumpStream( function() use ( &$gotBody ) {
+			if ( $gotBody ) {
 				return null;
 			}
 
-			$this->gotBody = true;
+			$gotBody = true;
 			return $this->body;
 		} );
 	}

tests/TestResponseTest.php

@@ -0,0 +1,22 @@
+<?php
+
+declare( strict_types = 1 );
+
+namespace Jeroen\PostRequestSender\Tests;
+
+use Jeroen\PostRequestSender\TestResponse;
+use PHPUnit\Framework\TestCase;
+
+/**
+ * @covers \Jeroen\PostRequestSender\TestResponse
+ */
+class TestResponseTest extends TestCase {
+
+	public function testGetBodyCanBeCalledMultipleTimes(): void {
+		$response = new TestResponse( 'foo' );
+
+		$this->assertSame( 'foo', $response->getBody()->getContents() );
+		$this->assertSame( 'foo', $response->getBody()->getContents() );
+	}
+
+}