New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Improve rueidiscompat test coverage #784

Merged

rueian merged 2 commits into redis:main from Beadko:feat/pubsub_test

Feb 28, 2025

Contributor

Beadko commented Feb 27, 2025

Relates to #679

Tests added:

TestWithChannelSize
TestSubscriptionString
TestMessageString


          Improve rueidiscompat test coverage

d273808

Collaborator

rueian commented Feb 27, 2025

Hi @Beadko, thank you for adding the tests! But could you do the tests without importing testify?


          Update tests to maintain the current style

710ed85

rueian merged commit 83000d5 into redis:main

27 checks passed

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet