As same as the MockResponse class has getRequestOptions() when doing a request; I've added:

• getRequestUrl() - returns the URL used when doing the request
• getRequestMethod() - returns the HTTP method used when doing the request

With these two getters, we would be able to assert that the method and URL passed to the HttpClient were well generated. I've tried to assert the URL generated in a unit test of a class with a SymfonyHttpClient injected and it wasn't possible. Calling $mock->getInfo('url') returns null.

Example, if we have a class with HttpClientInterface injected like this:

final class SymfonyHttpUserClient
{
    private HttpClientInterface $httpClient;
    private string $baseUri;

    public function __construct(
        HttpClientInterface $httpClient,
        string $baseUri
    ) {
        $this->httpClient = $httpClient;
        $this->baseUri = $baseUri;
    }

    public function getById(string $userId): void
    {
        $this->httpClient->request(
            'GET',
            $this->baseUri . $customerId
        );
    }
}

And if we want to do a unit test of SymfonyHttpUserClient right now we are not able to check if the URL of the request was well-formed passing a MockResponse. Also, we weren't able to assert the HTTP method (maybe this piece is not so critical to assert in the unit test).

class SymfonyHttpUserClientTests extends TestCase
{
    public function testGetById(): void
    {
        $baseUri = 'https://user-api.test/api/v1/users/';
        $mockResponse = new MockResponse();
        $symfonyHttpUserClient = new SymfonyHttpUserClient(
            new MockHttpClient(
                [$mockResponse],
                $baseUri
            ),
            $baseUri
        );    

        // test get by id:
        $symfonyHttpUserClient->getById('some-user-id');

        // cannot be asserted right now:
        $this->assertSame($mockResponse->getInfo('url'), $baseUri . 'some-user-id'); // fail
        $this->assertSame($mockResponse->getInfo('http_method'), 'GET'); // fail

        // it could be with the new getters:
        $this->assertSame($mockResponse->getRequestUrl(), $baseUri . 'some-user-id');
        $this->assertSame($mockResponse->getRequestMethod(), 'GET');
    }
}

This only happens if the class has injected the HttpClient and if it is used inside void methods with no being able to return the response. If the class returns the response, url and http_method are available with $response->getInfo() call. But this response object is a new one, is not the mock passed by argument when you instance the MockHttpClient.

Var dumps of getInfo array:

$response->getInfo() from MockClient:

..array(10) {
  ["start_time"]=>
  float(1587109014.7985)
  ["user_data"]=>
  NULL
  ["http_code"]=>
  int(200)
  ["response_headers"]=>
  array(0) {
  }
  ["error"]=>
  NULL
  ["canceled"]=>
  bool(false)
  ["redirect_count"]=>
  int(0)
  ["redirect_url"]=>
  NULL
  ["http_method"]=>
  string(3) "GET"
  ["url"]=>
  string(47) "https://user-api.test/api/v1/users/some-user-id"
}

$mock->getInfo()

array(4) {
  ["http_code"]=>
  int(200)
  ["response_headers"]=>
  array(0) {
  }
  ["error"]=>
  NULL
  ["canceled"]=>
  bool(false)
}

This is a minor change, I opened the PR with 4.4 as base branch; but not sure if it should be opened with 5.0 as base branch or even master taking account is a feature (add two new getters in MockResponse class). Let me know if I didn't follow right the instructions (first PR on Symfony project 😃)

Thanks!