ingitdb / ingitdb-cli / 25745305140

96%

Build Type

push

github

Committed by inGitDB Dev

Commit Message

refactor(cli/select): apply post-review polish

Two small fixes from the final code review:

1. Remove dead `_ = colDef` pattern in runSelectFromSet — the lookup
   was only being used to check collection existence; collapse into
   `if _, ok := def.Collections[from]; !ok { ... }` so no unused
   binding leaks into the function.

2. Add TestSelect_SetMode_EmptyResult_MD covering the only
   format-mode pair missing direct empty-set assertion. Markdown
   empty output is header + separator only (no data rows).

No behavior change; both fixes are polish.

Coverage Stats

1 of 1 new or added line in 1 file covered. (100.0%)

21 existing lines in 3 files now uncovered.

7202 of 8705 relevant lines covered (82.73%)

0.97 hits per line