Skip to content

[12.x] Add tests for PhpRedisClusterConnection flushdb method#58917

Merged
taylorotwell merged 1 commit into
laravel:12.xfrom
miladev95:test/flushdb-master-nodes-coverage
Feb 19, 2026
Merged

[12.x] Add tests for PhpRedisClusterConnection flushdb method#58917
taylorotwell merged 1 commit into
laravel:12.xfrom
miladev95:test/flushdb-master-nodes-coverage

Conversation

@miladev95

Copy link
Copy Markdown
Contributor

This PR adds test coverage for PhpRedisClusterConnection methods.

Added tests:

  • testItFlushesAllMasterNodes - verifies sync flush calls flushdb on all master nodes
  • testItFlushesAllMasterNodesAsync - verifies async flush calls rawCommand with flushdb async on all master nodes
  • testItReturnsFalseWhenCursorIsZeroAndResultIsEmpty - verifies scan returns false when cursor is 0 and Redis returns no results

@taylorotwell taylorotwell merged commit 30fa36e into laravel:12.x Feb 19, 2026
72 checks passed
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