systemhaus / guzzlehttpmock Goto Github PK

The original functionality makes one feel it should be possible to set up multiple expectations in one test scenario, e.g.:

• Expect a GET request /test
• Expect a POST request /test
• Expect a GET request /somewhere

In fact, this tests does not run through. The counter contained output shows the encountered three requests and maps them to one expected request only: the first of the tree which were set up.

I suspect the expected_count is only for replaying the same request multiple times, not for separate requests in one tests. But sometimes it is necessary for the integrity. We should extend the framework to expect multiple requests in the order in which they were set up, distinguishing between an individual expected_count and a total one.

A slightly modified code from the readme (after fixing the namespace with \Aeris\GuzzleHttpMock\ instead of \Aeris\GuzzleHttp\):

$httpMock = new \Aeris\GuzzleHttpMock\Mock();

// Create a guzzle http client and attach the mock handler
$guzzleClient = new \GuzzleHttp\Client([
	'base_url' => 'http://www.example.com',
	'handler' => $httpMock->getHandlerStackWithMiddleware()
]);

// Setup a request expectation
$httpMock
	->shouldReceiveRequest()
    ->withUrl('http://www.example.com/foo')
    ->withMethod('GET')
    ->withBodyParams([ 'foo' => 'bar' ])
    ->andRespondWithJson([ 'faz', 'baz' ], 403);

// Make a matching request
$response = $guzzleClient->get('/foo', ['json' => ['foo' => 'bar'] ]);
echo $response->getStatusCode(); //200, even though it should return 403

What am I doing wrong?

It would be great if the library had also the ability to validate the request headers. Some APIs require certain headers being set and currently there's no way to test it using this library. You have revert to the built-in mocking capabilities of Guzzle, which are much lower-level and not so easy to use.

Currently, the library only works with Guzzle 5.