uber-go / goleak

Goroutine leak detector
MIT License
4.55k stars 151 forks source link

Ignore t.Parallel goroutines, they represent queued tests #25

Closed prashantv closed 6 years ago

prashantv commented 6 years ago

When t.Parallel is used, tests are queued to run after all serially executed tests are run. This causes issues if the serial test tries to verify goroutines.

Fixes #16

CLAassistant commented 6 years ago

CLA assistant check
All committers have signed the CLA.

codecov[bot] commented 6 years ago

Codecov Report

Merging #25 into master will not change coverage. The diff coverage is 100%.

Impacted file tree graph

@@           Coverage Diff           @@
##           master      #25   +/-   ##
=======================================
  Coverage   95.03%   95.03%           
=======================================
  Files           4        4           
  Lines         141      141           
=======================================
  Hits          134      134           
  Misses          4        4           
  Partials        3        3
Impacted Files Coverage Δ
options.go 100% <100%> (ø) :arrow_up:

Continue to review full report at Codecov.

Legend - Click here to learn more Δ = absolute <relative> (impact), ø = not affected, ? = missing data Powered by Codecov. Last update d13b03e...85db571. Read the comment docs.