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
[Improvement-15692][task-datasync] rewrite the mockito test file for DatasyncTaskTest. #15693
Conversation
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## dev #15693 +/- ##
============================================
+ Coverage 38.96% 39.04% +0.08%
- Complexity 4840 4844 +4
============================================
Files 1316 1316
Lines 45010 45010
Branches 4818 4818
============================================
+ Hits 17539 17576 +37
+ Misses 25575 25527 -48
- Partials 1896 1907 +11 ☔ View full report in Codecov by Sentry. |
Quality Gate passedIssues Measures |
Hi @Tianqi-Dotes can you add some description for why need to rewrite this UT? |
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
Purpose of the pull request
Due to the previous test code having defects and misusing the mockito, rewrite the mockito test file for the task-datasync.
Brief change log
Rewrite the mockito test file for the task-datasync.
Verify this pull request
I have tested locally.