pirl-unc / tcrsift / 25830380843

Build # 25830380843

Build Type

push

github

Committed by web-flow

Commit Message

Fix #44: make `tcrsift data download --db` repeatable (#61)

Single-value `--db` on `data download` (and `data clear`) silently
dropped all but the last value, so `--db vdjdb --db iedb` downloaded
only `iedb`. Switched to `action="extend", nargs="+"` so both forms
accumulate into a list:

  tcrsift data download --db vdjdb iedb       # space-separated
  tcrsift data download --db vdjdb --db iedb  # repeated flag (original bug)
  tcrsift data download                       # all DBs (unchanged default)

Same fix applied to `data clear --db` for symmetry (the issue's
explicit bonus ask). Handlers dedupe in-order so a repeated name
doesn't trigger a second download / second remove.

`choices=` still applies — unknown names fail argparse validation
rather than silently doing nothing.

Six new parser-layer tests and three handler-layer tests in
`TestDataDownloadParser` / `TestDataDownloadCommand` cover:

- no `--db` → None → handler iterates the URL-having DBs
- space-separated multi-value
- repeated-flag multi-value (the exact bug from #44)
- mixed forms in one invocation
- repeated names deduplicated (in order)
- `data clear` accepts the same forms
- unknown names still rejected

Bumps to 0.8.15. Closes #44.

Coverage Stats

4831 of 6745 relevant lines covered (71.62%)

2.86 hits per line

Source File

Press 'n' to go to next uncovered line, 'b' for previous

Source Not Available