Skip to content

Conversation

simstern
Copy link
Contributor

Tiny change to comply with the current phpunit version used in Flow 7.
Please tag a new version when merging :-)

@riea
Copy link

riea commented Feb 28, 2023

Without the changes in this PR to the setUp function declaration, i get an error complaining about the compatibility of the method declaration on Flow 8.2 (on PHP 8.1) as well:

Exception #1355480641 in line 392 of /var/www/html/Packages/Framework/Neos.Flow/Classes/Core/Booting/Scripts.php: PHP Fatal error: Declaration of Flowpack\JobQueue\Redis\Tests\Functional\Queue\RedisAuthenticationTest::setUp() must be compatible with Neos\Flow\Tests\FunctionalTestCase::setUp(): void in /var/www/html/Packages/Application/Flowpack.JobQueue.Redis/Tests/Functional/Queue/RedisAuthenticationTest.php on line 43

@bwaidelich
Copy link
Contributor

Thanks!

@bwaidelich bwaidelich merged commit 0e38be9 into Flowpack:master Mar 1, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants