fix(http-client-mock): add content-length to body

Author: daFish

src/HttpClientMock/MockRequestBuilderFactory.php

@@ -57,13 +57,14 @@ private function processBody(
         array $headers,
     ): void {
         $contentType = (string) $mockRequestBuilder->getHeader('Content-Type');
+        $contentLength = $mockRequestBuilder->getHeader('Content-Length') ?? 0;
 
         // application/json; charset=utf-8
         if (strpos($contentType, 'application/json') === 0) {
             if (is_string($body)) {
                 $mockRequestBuilder->json(json_decode($body, true));
             } elseif (is_callable($body)) {
-                $mockRequestBuilder->json(json_decode((string) $body(), true));
+                $mockRequestBuilder->json(json_decode((string) $body((int) $contentLength), true));
             } elseif (is_array($body)) {
                 $mockRequestBuilder->json($body);
             } else {

tests/HttpClientMock/MockRequestBuilderFactoryTest.php

@@ -0,0 +1,28 @@
+<?php
+
+declare(strict_types=1);
+
+namespace Brainbits\FunctionalTestHelpers\Tests\HttpClientMock;
+
+use Brainbits\FunctionalTestHelpers\HttpClientMock\MockRequestBuilderFactory;
+use PHPUnit\Framework\Attributes\CoversClass;
+use PHPUnit\Framework\TestCase;
+
+#[CoversClass(MockRequestBuilderFactory::class)]
+final class MockRequestBuilderFactoryTest extends TestCase
+{
+    public function testBuildsRequestWithoutBody(): void
+    {
+        $mockRequestBuilderFactory = new MockRequestBuilderFactory();
+
+        $options = [
+            'headers' => ['Content-Type: application/json'],
+            'json' => ['foo' => 'bar'],
+        ];
+        $request = ($mockRequestBuilderFactory)('POST', 'https://service.com', $options);
+
+        self::assertSame('POST', $request->getMethod());
+        self::assertSame('https://service.com', $request->getUri());
+        self::assertSame(['foo' => 'bar'], $request->getJson());
+    }
+}