Skip to content
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

limit test cover packages #687

Merged
merged 4 commits into from
Jul 24, 2019

Conversation

xiaojingchen
Copy link
Contributor

@xiaojingchen xiaojingchen commented Jul 24, 2019

What problem does this PR solve?

current unit test cover generated codes and client codes, that influencing real test coverage.
this PR limit the unit test cover packages.

What is changed and how does it work?

  • Run make test
  • Run go tool cover -V -func=coverage.txt -o coverage.out && cat coverage.out

after limit the cover packages the test coverage increased from 61.4% to 74.3%

Check List

Code changes

  • Has CI related scripts change

Side effects

NONE

Does this PR introduce a user-facing change?:

NONE

@onlymellb @tennix @weekface PTAL

@xiaojingchen xiaojingchen requested a review from onlymellb July 24, 2019 07:38
@xiaojingchen
Copy link
Contributor Author

/run-e2e-test

@@ -22,6 +22,7 @@ filenametags
*.coverprofile

coverage.txt
coverage.out
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

where does this file come from?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

run go tool cover -V -func=coverage.txt -o coverage.out will create a file which named coverage.out

@weekface weekface added this to the v1.0.0 milestone Jul 24, 2019
@xiaojingchen xiaojingchen merged commit 0f253d7 into pingcap:master Jul 24, 2019
weekface pushed a commit that referenced this pull request Jul 29, 2019
* limit test cover packages

* remove coverage.out

(cherry picked from commit 0f253d7)
yahonda pushed a commit that referenced this pull request Dec 27, 2021
* add tag-check scripts; fix some inline codes

* test ci

* Revert "test ci"

This reverts commit 24358ce98b983096d7565a2672beb54d30f8726b.

* Update check-tags.py

* Update ci.yaml

* test ci

* make an exit

* exit 1

* Update check-tags.py

* Revert "test ci"

This reverts commit 8427ac09a97820547b6f103ccdd0374aae6eb67d.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants