Isolate remoteStorage Redis specs from concurrent runs #261

2026-11-03T05:11:25Z

raucao commented

2026-11-03 05:11:25 +00:00

The specs deleted a global authorizations:* pattern from the shared test Redis DB, so concurrent test processes could wipe each other's keys mid-example, causing intermittent empty/mismatched set failures.

Drop all global pattern deletes and the now-unused redis_rs_delete_keys helper. Each spec now reads or asserts only the exact key derived from its own random token, so runs can no longer interfere regardless of ordering or concurrency.

Also fixes a vacuous assertion in the #destroy spec, which checked authorizations:<user.address>:* (never matching the keys, which use user.cn), to assert the exact key is gone.

The specs deleted a global authorizations:* pattern from the shared test Redis DB, so concurrent test processes could wipe each other's keys mid-example, causing intermittent empty/mismatched set failures. Drop all global pattern deletes and the now-unused redis_rs_delete_keys helper. Each spec now reads or asserts only the exact key derived from its own random token, so runs can no longer interfere regardless of ordering or concurrency. Also fixes a vacuous assertion in the #destroy spec, which checked authorizations:<user.address>:* (never matching the keys, which use user.cn), to assert the exact key is gone.

raucao added 1 commit 2026-11-03 05:11:26 +00:00

Isolate remoteStorage Redis specs from concurrent runs

CI / Test (pull_request) Successful in 42s

Details

Release Drafter / Update release notes draft (pull_request) Successful in 3s

Details

1ac5c7725a

The specs deleted a global authorizations:* pattern from the shared
test Redis DB, so concurrent test processes could wipe each other's
keys mid-example, causing intermittent empty/mismatched set failures.

Drop all global pattern deletes and the now-unused
redis_rs_delete_keys helper. Each spec now reads or asserts only the
exact key derived from its own random token, so runs can no longer
interfere regardless of ordering or concurrency.

Also fixes a vacuous assertion in the #destroy spec, which checked
authorizations:<user.address>:* (never matching the keys, which use
user.cn), to assert the exact key is gone.

raucao added the bug dev environment kredits-1 labels 2026-11-03 05:12:26 +00:00

raucao merged commit bf23931822 into master

2026-11-03 05:13:13 +00:00

raucao deleted branch fix/rs-redis-flake

2026-11-03 05:13:13 +00:00

Sign in to join this conversation.

1 Participants

Notifications

Due Date

No due date set.

Dependencies

No dependencies set.

Reference: kosmos/akkounts#261