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

Add support for "tparallel" #1371

Closed
moricho opened this issue Sep 17, 2020 · 1 comment · Fixed by #1380
Closed

Add support for "tparallel" #1371

moricho opened this issue Sep 17, 2020 · 1 comment · Fixed by #1380
Labels
enhancement New feature or improvement

Comments

@moricho
Copy link

moricho commented Sep 17, 2020

tparallel finds inappropriate or inefficient usage of t.Parallel() method in Go test codes.
It detects the following:

  • insufficient parallelization: t.Parallel() is called in either a top-level test function or a sub-test function only
  • inappropriate teardown process: Although t.Parallel() is called in the sub-test function, it is post-processed by defer instead of t.Cleanup()

Particularly in tests that access the database, the communication time to the database takes up a large portion of the test time, so we can reduce the test time by executing the test code in parallel appropriately.

@moricho moricho added the enhancement New feature or improvement label Sep 17, 2020
@boring-cyborg
Copy link

boring-cyborg bot commented Sep 17, 2020

Hey, thank you for opening your first Issue ! 🙂 If you would like to contribute we have a guide for contributors.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or improvement
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant