Add LoggingPostRequestSender decorator

src/LoggingPostRequestSender.php

@@ -0,0 +1,34 @@
+<?php
+
+declare( strict_types = 1 );
+
+namespace Jeroen\PostRequestSender;
+
+use Psr\Http\Message\ResponseInterface;
+use Psr\Log\LoggerInterface;
+use Psr\Log\LogLevel;
+
+class LoggingPostRequestSender implements PostRequestSender {
+
+	/**
+	 * @param string[] $fieldsToOmit
+	 */
+	public function __construct(
+		private PostRequestSender $requestSender,
+		private LoggerInterface $logger,
+		private readonly string $logLevel = LogLevel::INFO,
+		private readonly array $fieldsToOmit = []
+	) {
+	}
+
+	public function post( string $url, array $fields ): ResponseInterface {
+		$this->logger->log(
+			$this->logLevel,
+			'Post request to ' . $url,
+			array_diff_key( $fields, array_flip( $this->fieldsToOmit ) )
+		);
+
+		return $this->requestSender->post( $url, $fields );
+	}
+
+}

src/PostRequestSender.php

@@ -5,12 +5,15 @@
 namespace Jeroen\PostRequestSender;
 
 use Psr\Http\Message\ResponseInterface;
+use Throwable;
 
 interface PostRequestSender {
 
 	/**
 	 * @param string $url
 	 * @param array<string, mixed> $fields
+	 *
+	 * @throws Throwable
 	 */
 	public function post( string $url, array $fields ): ResponseInterface;
 

tests/LoggingPostRequestSenderTest.php

@@ -0,0 +1,64 @@
+<?php
+
+declare( strict_types = 1 );
+
+namespace Jeroen\PostRequestSender\Tests;
+
+use Jeroen\PostRequestSender\LoggingPostRequestSender;
+use Jeroen\PostRequestSender\StubPostRequestSender;
+use PHPUnit\Framework\TestCase;
+use Psr\Log\LogLevel;
+use WMDE\PsrLogTestDoubles\LoggerSpy;
+
+/**
+ * @covers \Jeroen\PostRequestSender\LoggingPostRequestSender
+ * @covers \Jeroen\PostRequestSender\StubPostRequestSender
+ * @covers \Jeroen\PostRequestSender\PostRequest
+ */
+class LoggingPostRequestSenderTest extends TestCase {
+
+	public function testLogsRequests(): void {
+		$loggerSpy = new LoggerSpy();
+
+		$sender = new LoggingPostRequestSender(
+			new StubPostRequestSender(),
+			$loggerSpy
+		);
+
+		$sender->post( 'https://first.url', [ 'foo' => 'bar' ] );
+		$sender->post( 'https://second.url', [ 'baz' => 'bah', 'pew' => 1337 ] );
+
+		$this->assertSame(
+			[ 'Post request to https://first.url', 'Post request to https://second.url' ],
+			$loggerSpy->getLogCalls()->getMessages()
+		);
+
+		$this->assertSame(
+			[ 'baz' => 'bah', 'pew' => 1337 ],
+			$loggerSpy->getLogCalls()->getLastCall()?->getContext()
+		);
+
+		$this->assertSame(
+			LogLevel::INFO,
+			$loggerSpy->getLogCalls()->getLastCall()?->getLevel()
+		);
+	}
+
+	public function testOmitsExcludedFields(): void {
+		$loggerSpy = new LoggerSpy();
+
+		$sender = new LoggingPostRequestSender(
+			new StubPostRequestSender(),
+			$loggerSpy,
+			fieldsToOmit: [ 'secret' ]
+		);
+
+		$sender->post( 'https://example.url', [ 'foo' => 'bar', 'secret' => 1337, 'baz' => 'bah' ] );
+
+		$this->assertSame(
+			[ 'foo' => 'bar', 'baz' => 'bah' ],
+			$loggerSpy->getLogCalls()->getLastCall()?->getContext()
+		);
+	}
+
+}