You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Does this issue reproduce with the latest release?
yes. This issue might be a problem with upcoming Go 1.10 release.
What operating system and processor architecture are you using (go env)?
N/A
What did you do?
Parse the test cover profile result with x/tools/cover package.
What did you expect to see?
Multiple samples from the same location will be merged when parsing cover
profile in the same way as src/cmd/cover/profile.go1.
What did you see instead?
The implementation between x/tools/cover and src/cmd/cover/profile.go is
different and x/tools/cover won't handle multiple samples from the same location.
Now that go test supported -coverprofile with multiple packages (#6909),
x/tools/cover/profile should also handle multiple samples from the same
location.
Because go test -coverprofile with multiple packages just merges the cover
profile result by appending cover profile2, if users specify -coverpkg flag, the
merged profile will contain the multiple samples from the same location.
mikioh
changed the title
x/tools/cover doesn't handle multiple sample from the same location in cover profile.
x/tools/cover: doesn't handle multiple sample from the same location in cover profile.
Jan 5, 2018
What version of Go are you using (
go version
)?N/A
Does this issue reproduce with the latest release?
yes. This issue might be a problem with upcoming Go 1.10 release.
What operating system and processor architecture are you using (
go env
)?N/A
What did you do?
Parse the test cover profile result with x/tools/cover package.
What did you expect to see?
Multiple samples from the same location will be merged when parsing cover
profile in the same way as src/cmd/cover/profile.go1.
What did you see instead?
The implementation between x/tools/cover and src/cmd/cover/profile.go is
different and x/tools/cover won't handle multiple samples from the same location.
Now that go test supported -coverprofile with multiple packages (#6909),
x/tools/cover/profile should also handle multiple samples from the same
location.
Because go test -coverprofile with multiple packages just merges the cover
profile result by appending cover profile2, if users specify -coverpkg flag, the
merged profile will contain the multiple samples from the same location.
How to reproduce
go1.10beta1 test -v -coverprofile=coverage.out -coverpkg=./... ./...
under this directory https://github.com/haya14busa/goverage/tree/master/example/rootOutput:
Expect:
I'll send a CL soon.
The text was updated successfully, but these errors were encountered: