Skip to content

Commit

Permalink
minor #20474 [HttpClient] Update http_client.rst Test Examples (ricko…
Browse files Browse the repository at this point in the history
…gden)

This PR was submitted for the 7.2 branch but it was merged into the 6.4 branch instead.

Discussion
----------

[HttpClient] Update http_client.rst Test Examples

Reordered assertSame arguments to use the conventional and consistent order to `$this->assertSame($expected, $actual);`.

<!--

If your pull request fixes a BUG, use the oldest maintained branch that contains
the bug (see https://symfony.com/releases for the list of maintained branches).

If your pull request documents a NEW FEATURE, use the same Symfony branch where
the feature was introduced (and `7.x` for features of unreleased versions).

-->

Commits
-------

b487d45 Update http_client.rst Test Examples
  • Loading branch information
javiereguiluz committed Dec 17, 2024
2 parents 0a53599 + b487d45 commit 48aadc5
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions http_client.rst
Original file line number Diff line number Diff line change
Expand Up @@ -2233,7 +2233,7 @@ test it in a real application::
);
$this->assertSame($expectedRequestData, $mockResponse->getRequestOptions()['body']);

$this->assertSame($responseData, $expectedResponseData);
$this->assertSame($expectedResponseData, $responseData);
}
}

Expand Down Expand Up @@ -2266,7 +2266,7 @@ test. Then, save that information as a ``.har`` file somewhere in your applicati
$responseData = $service->createArticle($requestData);

// Assert
$this->assertSame($responseData, 'the expected response');
$this->assertSame('the expected response', $responseData);
}
}

Expand Down

0 comments on commit 48aadc5

Please sign in to comment.