Skip to content

Add sharded-pubsub tcl test to test_helper all test set#267

Merged
madolson merged 1 commit into
valkey-io:unstablefrom
hpatro:fix_test_helper
Apr 9, 2024
Merged

Add sharded-pubsub tcl test to test_helper all test set#267
madolson merged 1 commit into
valkey-io:unstablefrom
hpatro:fix_test_helper

Conversation

@hpatro

@hpatro hpatro commented Apr 8, 2024

Copy link
Copy Markdown
Contributor

Maybe we could support listing all files via glob/regex pattern and we could just pass unit/* and avoid this issue. Adding the test to the all_tests set manually for now.

Signed-off-by: Harkrishn Patro <harkrisp@amazon.com>
@hpatro hpatro requested a review from madolson April 8, 2024 20:46
@madolson

madolson commented Apr 9, 2024

Copy link
Copy Markdown
Member

Maybe we could support listing all files via glob/regex pattern and we could just pass unit/* and avoid this issue. Adding the test to the all_tests set manually for now.

@hpatro Can you create an issue for this? Also attach the help wanted tag.

@madolson madolson merged commit c2ebe9e into valkey-io:unstable Apr 9, 2024
PatrickJS pushed a commit to PatrickJS/placeholderkv that referenced this pull request Apr 24, 2024
Add sharded-pubsub to the supported TCL tests.

Signed-off-by: Harkrishn Patro <harkrisp@amazon.com>
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