Skip to content

Commit adeca4d

Browse files
committed
fix test
Do not yield Redis Sentinel test data if the required environment variables are not configured.
1 parent a59ff05 commit adeca4d

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)