-
Notifications
You must be signed in to change notification settings - Fork 6.8k
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
Add more unit tests for WhereSegmentBinder #28513
Comments
There hasn't been any activity on this issue recently, and in order to prioritize active issues, it will be marked as stale. |
hi @strongduanmu can i take this one up? although github-actions bot marked this as stale, this still needs to be worked on right? |
Hi @strongduanmu I have created a PR for this issue.Can you please review it? Here's the link to my PR #29111 |
There hasn't been any activity on this issue recently, and in order to prioritize active issues, it will be marked as stale. |
Hi @strongduanmu I have raised a new PR for this issue. Could you review it? Here's the link to it #30852 |
* Added unit test for WhereSegmentBinder class methods * Added unit test for WhereSegmentBinder class methods
Hi community,
This issue is for #28487.
Aim
Add more unit tests for WhereSegmentBinder to test its public functions to improve unit test coverage.
Unit test coverage can be viewed at: https://app.codecov.io/gh/apache/shardingsphere
Basic Qualifications
The text was updated successfully, but these errors were encountered: