Skip to content

Commit 4de656f

Browse files
abnegateclaude
andcommitted
fix: use docker stop instead of docker kill for Redis-Destructive tests
docker kill sends SIGKILL which prevents Redis from clean shutdown, causing ~18 minute delays in test recovery. docker stop sends SIGTERM for graceful shutdown, matching the main branch behavior. Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
1 parent f558e40 commit 4de656f

1 file changed

Lines changed: 2 additions & 2 deletions

File tree

tests/e2e/Adapter/Scopes/GeneralTests.php

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -373,7 +373,7 @@ public function testCacheFallback(): void
373373
// Bring down Redis
374374
$stdout = '';
375375
$stderr = '';
376-
Console::execute('docker ps -a --filter "name=utopia-redis" --format "{{.Names}}" | xargs -r docker kill', '', $stdout, $stderr);
376+
Console::execute('docker ps -a --filter "name=utopia-redis" --format "{{.Names}}" | xargs -r docker stop', '', $stdout, $stderr);
377377

378378
// Check we can read data still
379379
$this->assertCount(1, $database->find('testRedisFallback', [Query::equal('string', ['text📝'])]));
@@ -443,7 +443,7 @@ public function testCacheReconnect(): void
443443
// Bring down Redis
444444
$stdout = '';
445445
$stderr = '';
446-
Console::execute('docker ps -a --filter "name=utopia-redis" --format "{{.Names}}" | xargs -r docker kill', '', $stdout, $stderr);
446+
Console::execute('docker ps -a --filter "name=utopia-redis" --format "{{.Names}}" | xargs -r docker stop', '', $stdout, $stderr);
447447
sleep(1);
448448

449449
// Restart Redis containers

0 commit comments

Comments
 (0)