8000 minor #35980 [Messenger] Add ext-redis min version req to tests (ogiz… · symfony/symfony@ff30b43 · GitHub
[go: up one dir, main page]

Skip to content

Commit ff30b43

Browse files
committed
minor #35980 [Messenger] Add ext-redis min version req to tests (ogizanagi)
This PR was merged into the 4.4 branch. Discussion ---------- [Messenger] Add ext-redis min version req to tests | Q | A | ------------- | --- | Branch? | 4.4 <!-- see below --> | Bug fix? | no | New feature? | no <!-- please update src/**/CHANGELOG.md files --> | Deprecations? | no <!-- please update UPGRADE-*.md and src/**/CHANGELOG.md files --> | Tickets | N/A <!-- prefix each issue number with "Fix #", if any --> | License | MIT | Doc PR | N/A Properly skip the test instead of: ``` Testing src/Symfony/Component/Messenger E 1 / 1 (100%) Time: 597 ms, Memory: 16.00 MB There was 1 error: 1) Symfony\Component\Messenger\Tests\Transport\RedisExt\RedisTransportFactoryTest::testCreateTransport Symfony\Component\Messenger\Exception\LogicException: The redis transport requires php-redis 4.3.0 or higher. ``` Commits ------- 31f8990 [Messenger] Add ext-redis min version req to tests
2 parents 3801ddc + 31f8990 commit ff30b43

File tree

1 file changed

+1
-1
lines changed

1 file changed

+1
-1
lines changed

src/Symfony/Component/Messenger/Tests/Transport/RedisExt/RedisTransportFactoryTest.php

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -18,7 +18,7 @@
1818
use Symfony\Component\Messenger\Transport\Serialization\SerializerInterface;
1919

2020
/**
21-
* @requires extension redis
21+
* @requires extension redis >= 4.3.0
2222
*/
2323
class RedisTransportFactoryTest extends TestCase
2424
{

0 commit comments

Comments
 (0)
0