From d7f584ee3c24504bb07d04526a23b7d8df38b8ed Mon Sep 17 00:00:00 2001 From: Taras Madan Date: Fri, 20 Dec 2024 21:35:10 +0100 Subject: 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. --- pkg/coveragedb/coveragedb.go | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/pkg/coveragedb/coveragedb.go b/pkg/coveragedb/coveragedb.go index 0a6175ad5..4488d1b6b 100644 --- a/pkg/coveragedb/coveragedb.go +++ b/pkg/coveragedb/coveragedb.go @@ -81,8 +81,8 @@ func SaveMergeResult(ctx context.Context, client spannerclient.SpannerClient, de session := uuid.New().String() mutations := []*spanner.Mutation{} - var mcr MergedCoverageRecord for { + var mcr MergedCoverageRecord err := dec.Decode(&mcr) if err == io.EOF { break -- cgit mrf-deployment