Skip to content

Commit

Permalink
Merge pull request #8012 from heyitsanthony/cov-corruption
Browse files Browse the repository at this point in the history
test: incrementally merge coverage files
  • Loading branch information
heyitsanthony committed Jun 1, 2017
2 parents 82153e8 + 00e5817 commit f80be42
Showing 1 changed file with 10 additions and 1 deletion.
11 changes: 10 additions & 1 deletion test
Original file line number Diff line number Diff line change
Expand Up @@ -129,7 +129,16 @@ function cov_pass {
# on leadership transfer timeout during gracefully shutdown
go test -tags cov -timeout 30m -v ${REPO_PATH}"/e2e" || failed="$failed e2e"

gocovmerge "$COVERDIR"/*.coverprofile >"$COVERDIR"/cover.out
# incrementally merge to get coverage data even if some coverage files are corrupted
# optimistically assume etcdserver package's coverage file is OK since gocovmerge
# expects to start with a non-empty file
cp "$COVERDIR"/etcdserver.coverprofile "$COVERDIR"/cover.out
for f in "$COVERDIR"/*.coverprofile; do
gocovmerge $f "$COVERDIR"/cover.out >"$COVERDIR"/cover.tmp || failed="$failed $f"
if [ -s "$COVERDIR"/cover.tmp ]; then
mv "$COVERDIR"/cover.tmp "$COVERDIR"/cover.out
fi
done
# strip out generated files (using GNU-style sed)
sed --in-place '/generated.go/d' "$COVERDIR"/cover.out || true

Expand Down

0 comments on commit f80be42

Please sign in to comment.