Skip to content

Commit d7f584e

Browse files
committed
pkg/coveragedb: fix data unpacking bug
Current tests check the amount of data transfered to the mock. But they didn't check the data correctness. Because of this bug every batch have the same coverage which is nonsense.
1 parent 1ac6aa8 commit d7f584e

File tree

1 file changed

+1
-1
lines changed

1 file changed

+1
-1
lines changed

pkg/coveragedb/coveragedb.go

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -81,8 +81,8 @@ func SaveMergeResult(ctx context.Context, client spannerclient.SpannerClient, de
8181
session := uuid.New().String()
8282
mutations := []*spanner.Mutation{}
8383

84-
var mcr MergedCoverageRecord
8584
for {
85+
var mcr MergedCoverageRecord
8686
err := dec.Decode(&mcr)
8787
if err == io.EOF {
8888
break

0 commit comments

Comments
 (0)