diff options
| author | Taras Madan <tarasmadan@google.com> | 2024-12-20 21:35:10 +0100 |
|---|---|---|
| committer | Taras Madan <tarasmadan@google.com> | 2024-12-20 21:00:06 +0000 |
| commit | d7f584ee3c24504bb07d04526a23b7d8df38b8ed (patch) | |
| tree | 4d4b5120b2f865c26a2b452901dc66f1a29e0bec | |
| parent | 1ac6aa80f6bb0b4deb3de864f2e1dc7b26a8ab4e (diff) | |
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.
| -rw-r--r-- | pkg/coveragedb/coveragedb.go | 2 |
1 files changed, 1 insertions, 1 deletions
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 |