-
Notifications
You must be signed in to change notification settings - Fork 8.7k
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
optimize: disable SAGA
tests
#4403
optimize: disable SAGA
tests
#4403
Conversation
Codecov Report
@@ Coverage Diff @@
## develop #4403 +/- ##
==========================================
Coverage 49.00% 49.00%
- Complexity 3952 3953 +1
==========================================
Files 733 733
Lines 24978 24978
Branches 3092 3091 -1
==========================================
Hits 12240 12240
Misses 11440 11440
Partials 1298 1298
|
SAGA
testsSAGA
tests
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
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
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
禁用SAGA单测:
SAGA
的测试用例,避免失败的测试用例不会记录cost日志。SAGA
单测:禁用三个非常耗时的单测、以及两个经常出错的单测,并标记FIXME。另外:
DefaultCoreTest
偶发的ConcurrentModificationException
Review建议:
5个单测的代码变更行比较多,但实际上就是包装了一层日志打印,实际的测试代码并没有变更。建议在review界面勾选
Hide whitespace
后再开始review