-
Notifications
You must be signed in to change notification settings - Fork 2.1k
[Improve][E2E][connector-redis-e2e] Transfer e2e test cases to unit tests #10160
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.
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
base: dev
Are you sure you want to change the base?
[Improve][E2E][connector-redis-e2e] Transfer e2e test cases to unit tests #10160
Conversation
|
issue: |
LiJie20190102
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
|
@corgy-w @zhangshenghang Help me take a look |
| } | ||
|
|
||
| @Test | ||
| public void testFakeToToRedisDeleteZSetTest() throws IOException { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
| public void testFakeToRedisDeleteZSetTest() throws IOException { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for the review.
It seems this suggested change doesn’t introduce any actual diff.
Please let me know if there is anything specific you’d like me to adjust.
Purpose of this pull request
Does this PR introduce any user-facing change?
How was this patch tested?
Check list
New License Guide
incompatible-changes.mdto describe the incompatibility caused by this PR.