Merge branch '5.4' into 6.0

* 5.4:
  fix SQLSRV throws for method_exists()
  [HttpKernel] Add basic support for language negotiation
  [Messenger] Add a middleware to log when transaction has been left open
  [HttpClient] Add method to set response factory in mock client
  Move array_merge calls out of loops to improve performance
  Remove references to DBALException
  [VarDumper] Fix handling of "new" in initializers on PHP 8.1

Author: derrabus

CHANGELOG.md

@@ -1,6 +1,11 @@
 CHANGELOG
 =========
 
+5.4
+---
+
+ * Add `MockHttpClient::setResponseFactory()` method to be able to set response factory after client creating
+
 5.3
 ---
 

MockHttpClient.php

@@ -35,6 +35,15 @@ class MockHttpClient implements HttpClientInterface
      * @param callable|callable[]|ResponseInterface|ResponseInterface[]|iterable|null $responseFactory
      */
     public function __construct(callable|iterable|ResponseInterface $responseFactory = null, ?string $baseUri = 'https://example.com')
+    {
+        $this->setResponseFactory($responseFactory);
+        $this->defaultOptions['base_uri'] = $baseUri;
+    }
+
+    /**
+     * @param callable|callable[]|ResponseInterface|ResponseInterface[]|iterable|null $responseFactory
+     */
+    public function setResponseFactory($responseFactory): void
     {
         if ($responseFactory instanceof ResponseInterface) {
             $responseFactory = [$responseFactory];
@@ -47,7 +56,6 @@ public function __construct(callable|iterable|ResponseInterface $responseFactory
         }
 
         $this->responseFactory = !\is_callable($responseFactory) || $responseFactory instanceof \Closure ? $responseFactory : \Closure::fromCallable($responseFactory);
-        $this->defaultOptions['base_uri'] = $baseUri;
     }
 
     /**

Tests/MockHttpClientTest.php

@@ -291,6 +291,7 @@ protected function getHttpClient(string $testCase): HttpClientInterface
             case 'testReentrantBufferCallback':
             case 'testThrowingBufferCallback':
             case 'testInfoOnCanceledResponse':
+            case 'testChangeResponseFactory':
                 $responses[] = new MockResponse($body, ['response_headers' => $headers]);
                 break;
 
@@ -387,4 +388,16 @@ public function testHttp2PushVulcainWithUnusedResponse()
     {
         $this->markTestSkipped('MockHttpClient doesn\'t support HTTP/2 PUSH.');
     }
+
+    public function testChangeResponseFactory()
+    {
+        /* @var MockHttpClient $client */
+        $client = $this->getHttpClient(__METHOD__);
+        $expectedBody = '{"foo": "bar"}';
+        $client->setResponseFactory(new MockResponse($expectedBody));
+
+        $response = $client->request('GET', 'http://localhost:8057');
+
+        $this->assertSame($expectedBody, $response->getContent());
+    }
 }