Skip to content

test: fix several test cases for cluster mode#1336

Merged
byroot merged 1 commit intoredis:masterfrom
supercaracal:fix-cluster-test
Feb 2, 2026
Merged

test: fix several test cases for cluster mode#1336
byroot merged 1 commit intoredis:masterfrom
supercaracal:fix-cluster-test

Conversation

@supercaracal
Copy link
Copy Markdown
Contributor

@supercaracal supercaracal commented Feb 2, 2026

Several cluster test cases fail after redis-cluster-client was adapted to multiple databases support for Valkey cluster. This pull request fixes them.

  • SELECT
    • This command should send to all nodes. So the error type is changed.
  • MOVE, COPY
    • The common test cases in lint file are using SELECT command in it. So cluster test should arrange them.

Related issues:

@jit-ci
Copy link
Copy Markdown

jit-ci bot commented Feb 2, 2026

Hi, I’m Jit, a friendly security platform designed to help developers build secure applications from day zero with an MVS (Minimal viable security) mindset.

In case there are security findings, they will be communicated to you as a comment inside the PR.

Hope you’ll enjoy using Jit.

Questions? Comments? Want to learn more? Get in touch with us.

@supercaracal supercaracal marked this pull request as ready for review February 2, 2026 12:33
@byroot byroot merged commit 59bb945 into redis:master Feb 2, 2026
29 checks passed
@supercaracal supercaracal deleted the fix-cluster-test branch February 2, 2026 21:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants