theodesp / go-heaps

Reference implementations of heap data structures in Go - treap, skew, leftlist, pairing, fibonacci
MIT License
96 stars 29 forks source link

feat(test): Merge all coverage in one file to parse it with codecov.io #37

Closed lhauspie closed 5 years ago

lhauspie commented 5 years ago

Even after the fix about $$(go list ./... | grep -v example) in the Makefile. I was not able to run make test on local because of

GOPATH=/home/lhauspie/go go test -race $(go list ./... | grep -v example) -coverprofile=coverage.txt -covermode=atomic
cannot use test profile flag with multiple packages
Makefile:13: recipe for target 'test' failed
make: *** [test] Error 1

So I replaced this one command line by a loop over the go sub packages and then run test on each one and then merge the cover profile files into a final coverage.txt file that will be readable by codecov.io

Now everybody will be able to run tests locally.

codecov-io commented 5 years ago

Codecov Report

Merging #37 into master will not change coverage. The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##           master      #37   +/-   ##
=======================================
  Coverage   84.14%   84.14%           
=======================================
  Files           7        7           
  Lines         694      694           
=======================================
  Hits          584      584           
  Misses         79       79           
  Partials       31       31

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 181ab88...a55b1a0. Read the comment docs.