Skip to content

Use flushdb, not flushall, in specs

Nick Thomas requested to merge use-redis-flushdb-in-specs into master

What does this MR do?

Some specs need a clean Redis state to run successfully. We want to add per-environment isolation using redis databases, but to do that, we need to use flushdb instead of flushall.

This combines with gitlab-development-kit!2047 (merged) to give development/test isolation in GDK. It's safe to merge the two MRs in any order.

Without both these MRs, running certain specs in the test suite logs you out of your GDK session, and cache entries set in development can be loaded by a test, which is all sorts of fun.

Does this MR meet the acceptance criteria?

Conformity

Availability and Testing

Related to gitlab-development-kit#818 (closed)

Edited by Nick Thomas

Merge request reports

Loading