Skip to content

Commit 403a9c3

Browse files
committed
minor symfony#58990 [Cache] fix test (xabbuh)
This PR was merged into the 6.4 branch. Discussion ---------- [Cache] fix test | Q | A | ------------- | --- | Branch? | 6.4 | Bug fix? | no | New feature? | no | Deprecations? | no | Issues | | License | MIT Do not yield Redis Sentinel test data if the required environment variables are not configured. see https://github.com/symfony/symfony/actions/runs/12025721914/job/33523351272#step:9:615 Commits ------- adeca4d fix test
2 parents a59ff05 + adeca4d commit 403a9c3

File tree

1 file changed

+6
-4
lines changed

1 file changed

+6
-4
lines changed

src/Symfony/Component/Messenger/Bridge/Redis/Tests/Transport/RedisTransportFactoryTest.php

Lines changed: 6 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -66,10 +66,12 @@ public static function createTransportProvider(): iterable
6666
['stream' => 'bar', 'delete_after_ack' => true],
6767
];
6868

69-
yield 'redis_sentinel' => [
70-
'redis:?host['.str_replace(' ', ']&host[', getenv('REDIS_SENTINEL_HOSTS')).']',
71-
['sentinel_master' => getenv('REDIS_SENTINEL_SERVICE')],
72-
];
69+
if (false !== getenv('REDIS_SENTINEL_HOSTS') && false !== getenv('REDIS_SENTINEL_SERVICE')) {
70+
yield 'redis_sentinel' => [
71+
'redis:?host['.str_replace(' ', ']&host[', getenv('REDIS_SENTINEL_HOSTS')).']',
72+
['sentinel_master' => getenv('REDIS_SENTINEL_SERVICE')],
73+
];
74+
}
7375
}
7476

7577
private function skipIfRedisUnavailable()

0 commit comments

Comments
 (0)