hathitrust / holdings-backend / 26300123244

94%

Build Type

push

github

Committed by web-flow

Commit Message

ETT-1447 phctl report costreport should accept ht item count & ht item pd count (#443)

- Add `--ht_item_count` and `--ht_pd_item_count` parameters to `phctl report costreport`
- Add test for proper conversion to Integer and inclusion in params
- The ticket notes that writing this information to the frequency table file would be preferable;
however, I am a bit leery of introducing a new version of the freqtable file format if not needed.
Hence this simplistic solution.

Coverage Stats

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

6 existing lines in 1 file now uncovered.

6572 of 7022 relevant lines covered (93.59%)

122.96 hits per line