diff options
| author | Taras Madan <tarasmadan@google.com> | 2025-02-13 11:17:21 +0100 |
|---|---|---|
| committer | Taras Madan <tarasmadan@google.com> | 2025-02-13 10:28:51 +0000 |
| commit | a98a8417f2c0dfdab1e8d3d49e8a7605340ada89 (patch) | |
| tree | 85bbf7bac0b0b1cae6a25225b6f00e0043c9cf8e /pkg/coveragedb/coveragedb.go | |
| parent | 7320a65fa13ffdebd45bb5eb12aee17ca9753f94 (diff) | |
pkg/coveragedb: fix tests flakiness
Happy path testing rely on iter.Stop() call to be done before we close errCh.
Diffstat (limited to 'pkg/coveragedb/coveragedb.go')
| -rw-r--r-- | pkg/coveragedb/coveragedb.go | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/pkg/coveragedb/coveragedb.go b/pkg/coveragedb/coveragedb.go index e0a139371..50fbd0633 100644 --- a/pkg/coveragedb/coveragedb.go +++ b/pkg/coveragedb/coveragedb.go @@ -416,9 +416,9 @@ func FilesCoverageStream(ctx context.Context, client spannerclient.SpannerClient resCh := make(chan *FileCoverageWithLineInfo) errCh := make(chan error) go func() { - defer iter.Stop() - defer close(resCh) defer close(errCh) + defer close(resCh) + defer iter.Stop() if err := readIterToChan(ctx, iter, resCh); err != nil { errCh <- fmt.Errorf("readIterToChan: %w", err) } |