codenotary / immudb / 25361461455

85%

master: 85%

Build Type

Pull #2094

gh-ci

Committed by vchaindz

Commit Message

test: cover lazy_remote_reader stubs and count_row_reader getters

Adds two test files for zero-coverage interface stubs flagged by
Coveralls on PR #2094. Test-only - no production-code changes.

embedded/appendable/remoteapp/lazy_remote_reader_test.go: covers all
16 blocks of lazy_remote_reader.go (ReadAt open-error caching, open-
once semantics, the 3 no-op shims, and require.Panics on the 9
"unimplemented" stubs). Adds a test-local fakeAppendable.

embedded/sql/count_row_reader_test.go: exercises OrderBy / ScanSpecs
/ Columns / colsBySelector / Tx / TableAlias / Parameters /
InferParameters / onClose / Close on countingRowReader and
keyFilterCountingRowReader, constructed via newRawRowReader against a
freshly-created table (same harness as joint_row_reader_test.go).
Read path already covered by TestCountStarIndexedWhere.

Verified: go test on both packages green; remoteapp coverage 81.3%
-> 83.6%; count_row_reader.go all blocks >= 80%.

Pull Request Pull Request #2094: perf(s3): wave 1-4 remote-storage performance + sql index-only COUNT + hot-path fixes

Coverage Stats

613 of 855 new or added lines in 16 files covered. (71.7%)

6 existing lines in 3 files now uncovered.

45156 of 53102 relevant lines covered (85.04%)

126286.47 hits per line