Skip to content

[HttpClient][DI] Add an option to use the MockClient in functional tests #35893

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Merged
fabpot merged 1 commit into from
Aug 27, 2020
Merged

[HttpClient][DI] Add an option to use the MockClient in functional tests #35893

fabpot merged 1 commit into from
Aug 27, 2020

Conversation

@GaryPEGEOT
Copy link
Contributor

@GaryPEGEOT GaryPEGEOT commented Feb 28, 2020

Q A
Branch? master
Bug fix? no
New feature? yes
Deprecations? no
Tickets
License MIT
Doc PR TODO

Let's say you want to mock HTTP responses in your test or dev environment:

First, you create an invokable class responsible of generating the response:

namespace App\Tests;

use Symfony\Contracts\HttpClient\ResponseInterface;

class MyAwesomeCallback
{
    public function __invoke(string $method, string $url, array $options = []): ResponseInterface
    {
        // load a fixture file or generate dynamic data
    }
}

Then configure it:

# config/services_test.yaml
services:
   # ...
   App\Tests\MyAwesomeCallback: ~

# config/packages/test/framework.yaml
framework:
    http_client:
        mock_response_factory: App\Tests\MyAwesomeCallback

The HttpClient will now be using MockHttpClient in your functional test:

$client = static::createClient();

// No live HTTP connection made
$client->request('GET', '/path-with-http-call');

@nicolas-grekas
Copy link
Member

nicolas-grekas commented Feb 29, 2020

Very related to #35651 and #35677

@nicolas-grekas nicolas-grekas added this to the next milestone Feb 29, 2020
@GaryPEGEOT GaryPEGEOT mentioned this pull request Aug 11, 2020
Closed
2 tasks
@fabpot
Copy link
Member

fabpot commented Aug 11, 2020

@nicolas-grekas Can you have a look at this one as it would unblock some other PRs as well?

nicolas-grekas
nicolas-grekas approved these changes Aug 26, 2020
View reviewed changes
Copy link
Member

@nicolas-grekas nicolas-grekas left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, I just have minor notes.

@fabpot
Copy link
Member

fabpot commented Aug 27, 2020

Thank you @GaryPEGEOT.

@fabpot fabpot mentioned this pull request Aug 27, 2020
Closed
@fabpot fabpot merged commit fe4d928 into symfony:master Aug 27, 2020
@GaryPEGEOT GaryPEGEOT deleted the feature/response-factory-di branch August 27, 2020 14:54
@nicolas-grekas nicolas-grekas modified the milestones: next, 5.2 Oct 5, 2020
@fabpot fabpot mentioned this pull request Oct 5, 2020
Merged
@Toflar Toflar mentioned this pull request Dec 1, 2020
Closed
@welcoMattic welcoMattic mentioned this pull request Apr 11, 2023
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@fabpot fabpot fabpot approved these changes

@nicolas-grekas nicolas-grekas nicolas-grekas approved these changes

Assignees

No one assigned

Labels

DependencyInjection Feature HttpClient Status: Reviewed

Projects

None yet

Milestone

5.2

Development

Successfully merging this pull request may close these issues.

4 participants

@GaryPEGEOT @nicolas-grekas @fabpot @carsonbot