Make ResponseByMethod respect sub MultiResponseInterfaces

Adds `next()` to ResponseByMethod which itself invokes next on the most recently called method.

Author: donatj

src/ResponseByMethod.php

@@ -5,7 +5,7 @@
 /**
  * ResponseByMethod is used to vary the response to a request by the called HTTP Method.
  */
-class ResponseByMethod implements ResponseInterface {
+class ResponseByMethod implements MultiResponseInterface {
 
 	public const METHOD_GET     = 'GET';
 	public const METHOD_POST    = 'POST';
@@ -22,13 +22,16 @@ class ResponseByMethod implements ResponseInterface {
 	/** @var ResponseInterface */
 	private $defaultResponse;
 
+	/** @var string|null */
+	private $latestMethod;
+
 	/**
 	 * MethodResponse constructor.
 	 *
 	 * @param array<string, ResponseInterface> $responses       A map of responses keyed by their method.
-	 * @param ResponseInterface|null           $defaultResponse The fallthrough response to return if a response for a given
-	 *                                                          method is not found. If this is not defined the server will
-	 *                                                          return an HTTP 501 error.
+	 * @param ResponseInterface|null           $defaultResponse The fallthrough response to return if a response for a
+	 *                                                          given method is not found. If this is not defined the
+	 *                                                          server will return an HTTP 501 error.
 	 */
 	public function __construct( array $responses = [], ?ResponseInterface $defaultResponse = null ) {
 		foreach( $responses as $method => $response ) {
@@ -64,7 +67,8 @@ public function getStatus( RequestInfo $request ) : int {
 	}
 
 	private function getMethodResponse( RequestInfo $request ) : ResponseInterface {
-		$method = $request->getRequestMethod();
+		$method             = $request->getRequestMethod();
+		$this->latestMethod = $method;
 
 		return $this->responses[$method] ?? $this->defaultResponse;
 	}
@@ -76,4 +80,21 @@ public function setMethodResponse( string $method, ResponseInterface $response )
 		$this->responses[$method] = $response;
 	}
 
+	public function next() : bool {
+		$method = $this->latestMethod;
+		if( !$method ) {
+			return false;
+		}
+
+		if( !isset($this->responses[$method]) ) {
+			return false;
+		}
+
+		if( !$this->responses[$method] instanceof MultiResponseInterface ) {
+			return false;
+		}
+
+		return $this->responses[$method]->next();
+	}
+
 }

test/Regression/ResponseByMethod_RegressionTest.php

@@ -0,0 +1,49 @@
+<?php
+
+namespace Test\Regression;
+
+use donatj\MockWebServer\MockWebServer;
+use donatj\MockWebServer\Response;
+use donatj\MockWebServer\ResponseByMethod;
+use donatj\MockWebServer\ResponseStack;
+use PHPUnit\Framework\TestCase;
+
+class ResponseByMethod_RegressionTest extends TestCase {
+
+	public function test_forwardNextAsExpected() : void {
+		$server = new MockWebServer;
+		$path   = $server->setResponseOfPath(
+			'/interest-categories/cat-xyz/interests',
+			new ResponseByMethod([
+				ResponseByMethod::METHOD_GET    => new ResponseStack(
+					new Response('get-a'),
+					new Response('get-b'),
+					new Response('get-c')
+				),
+				ResponseByMethod::METHOD_DELETE => new ResponseStack(
+					new Response('delete-a'),
+					new Response('delete-b'),
+					new Response('delete-c')
+				),
+			])
+		);
+
+		$server->start();
+
+		$method = function ( string $method ) {
+			return stream_context_create([ 'http' => [ 'method' => $method ] ]);
+		};
+
+		$this->assertSame('get-a', file_get_contents($path));
+		$this->assertSame('delete-a', file_get_contents($path, false, $method('DELETE')));
+		$this->assertSame('get-b', file_get_contents($path));
+		$this->assertSame('delete-b', file_get_contents($path, false, $method('DELETE')));
+		$this->assertSame('delete-c', file_get_contents($path, false, $method('DELETE')));
+		$this->assertSame(false, @file_get_contents($path, false, $method('DELETE')));
+		$this->assertSame('get-c', file_get_contents($path));
+		$this->assertSame(false, @file_get_contents($path));
+
+		$server->stop();
+	}
+
+}