-
Notifications
You must be signed in to change notification settings - Fork 5.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
executor: improve UT coverage of executor package #10728
Conversation
@@ -240,101 +240,6 @@ func (s *testSerialSuite) TestRecoverTableByJobID(c *C) { | |||
c.Assert(gcEnable, Equals, false) | |||
} | |||
|
|||
func (s *testSerialSuite) TestRecoverTableByTableName(c *C) { |
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.
This UT is moved to package executor
because:
- Code about this test is almost in
executor/ddl.go
and - UT coverage of package
ddl
doesn't decrease.
Codecov Report
@@ Coverage Diff @@
## master #10728 +/- ##
================================================
+ Coverage 79.5733% 79.6772% +0.1038%
================================================
Files 415 415
Lines 87993 87980 -13
================================================
+ Hits 70019 70100 +81
+ Misses 12793 12699 -94
Partials 5181 5181 |
Codecov Report
@@ Coverage Diff @@
## master #10728 +/- ##
================================================
+ Coverage 79.8572% 79.8935% +0.0363%
================================================
Files 415 415
Lines 88270 88310 +40
================================================
+ Hits 70490 70554 +64
+ Misses 12572 12551 -21
+ Partials 5208 5205 -3 |
/run-all-tests |
/run-all-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
/bench |
Bench result:
Bench topology:
|
/bench |
Bench result:
Bench topology:
|
/bench |
Bench result:
Bench topology:
|
Bench result:
Bench topology:
|
Bench result:
Bench topology:
|
/bench |
3 similar comments
/bench |
/bench |
/bench |
Bench result:
Bench topology:
|
475a83c
to
5c92c12
Compare
/run-all-tests |
1 similar comment
/run-all-tests |
/run-all-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
What problem does this PR solve?
improve UT coverage of executor package to about 82%.
What is changed and how it works?
add more tests.
Check List
Tests