Skip to content
This repository has been archived by the owner on Mar 20, 2024. It is now read-only.

Fix setClient issue for new guzzle version #381

Closed
wants to merge 3 commits into from
Closed

Fix setClient issue for new guzzle version #381

Changes from all commits
Commits
Show all changes
3 commits
Select commit Hold shift + click to select a range
9b90348
Easy fix for when Reflection parameter throws an exception in new ver…
joshlopes Mar 24, 2022
a16699c
Fix usage of the new Goutte client
joshlopes Mar 24, 2022
6b71d19
Remove the guzzle client as is not supported by the new fabpot. Symfo…
joshlopes Mar 24, 2022
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
8 changes: 6 additions & 2 deletions src/Behat/MinkExtension/ServiceContainer/Driver/GoutteFactory.php
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -68,10 +68,14 @@ public function buildDriver(array $config)
);
}

if ($this->isGuzzle6()) {
$clientDefinition = new Definition('Behat\Mink\Driver\Goutte\Client');

return new Definition('Behat\Mink\Driver\GoutteDriver', array($clientDefinition));
}

if ($this->isGoutte1()) {
$guzzleClient = $this->buildGuzzle3Client($config['guzzle_parameters']);
} elseif ($this->isGuzzle6()) {
$guzzleClient = $this->buildGuzzle6Client($config['guzzle_parameters']);
} else {
$guzzleClient = $this->buildGuzzle4Client($config['guzzle_parameters']);
}
Expand Down