MockHttpClient::getConfig must be compatible with GuzzleHttp\ClientInterface::getConfig

Created on 25 January 2023, almost 2 years ago
Updated 27 February 2023, almost 2 years ago

Problem/Motivation

We are getting an error running kernel tests with video_embed_field enabled:

PHP Fatal error:  Declaration of Drupal\Tests\video_embed_field\Kernel\MockHttpClient::getConfig(?string $option = null) must be compatible with GuzzleHttp\ClientInterface::getConfig($option = null) in /modules/contrib/video_embed_field/tests/src/Kernel/MockHttpClient.php on line 51

Proposed resolution

Change MockHttpClient::getConfig to respect ClientInterface::getConfig

🐛 Bug report
Status

Needs review

Version

2.0

Component

Code

Created by

🇳🇱Netherlands seanB Netherlands

Live updates comments and jobs are added and updated live.
Sign in to follow issues

Comments & Activities

Production build 0.71.5 2024