WebformViewsTestBase::setup() must be compatible with KernelTestBase::setup: void

Created on 18 January 2024, 5 months ago

Updated 1 March 2024, 4 months ago

Problem/Motivation

When attempting to run phpunit in Drupal 10.2, I get

PHP Fatal error: Declaration of Drupal\Tests\webform_views\Kernel\WebformViewsTestBase::setUp() must be compatible with Drupal\KernelTests\KernelTestBase::setUp(): void in /app/web/modules/contrib/webform_views/tests/src/Kernel/WebformViewsTestBase.php on line 80

Steps to reproduce

Install phpunit as described at https://www.drupal.org/docs/automated-testing/phpunit-in-drupal/running-... → . Run it.

Proposed resolution

Define the setUp function as void so it is compatible with the base class.

Remaining tasks

make a fork

🐛 Bug report

Status

Needs review

Version

5.2

Component

Code

Created by

🇺🇸United States BenStallings

Live updates comments and jobs are added and updated live.

Merge Requests

!18WebformViewsTestBase::setup() must be compatible with KernelTestBase::setup: void
Open
🇺🇸United States BenStallings
updated 5 months ago

Comments & Activities

Issue created by @BenStallings
Merge request !18define test setUp() functions as void → (Open) created by BenStallings
Status changed to Needs review 5 months ago6:15pm 18 January 2024
Comment 5 months ago →
🇺🇸United States BenStallings
Comment 4 months ago →
🇨🇦Canada ryanrobinson_wlu
Came here to report the same one. I believe the change required is as simple as the change in the branch here, so I would love this merged into a release soon.
Comment 4 months ago →
🇨🇦Canada ryanrobinson_wlu
I can confirm that I added this change as a patch to my composer.json and I no longer receive that fatal error running tests (I get a different one from a different module instead).

contrib.social Blog FAQ Discussions

Production build 0.69.0 2024