Closed netsettler closed 1 year ago
Changes Missing Coverage | Covered Lines | Changed/Added Lines | % | ||
---|---|---|---|---|---|
dcicutils/ff_utils.py | 23 | 24 | 95.83% | ||
dcicutils/qa_utils.py | 45 | 46 | 97.83% | ||
<!-- | Total: | 72 | 74 | 97.3% | --> |
Totals | |
---|---|
Change from base Build 5894343897: | 0.2% |
Covered Lines: | 8234 |
Relevant Lines: | 10591 |
misc_utils.to_camelcase
.qa_utils.is_subdict
.ff_utils.get_schema
andff_utils.get_schemas
(C4-1089).xfail
).Other opportunistic changes:
ff_utils.dump_results_to_json
for style reasons, and repairs to its overly complex and error-prone unit test.ff_utils
because PyCharm grammar checker was fussing.NOTES TO REVIEWERS:
get_profile
function forff_utils
but I decided to call itget_schema
instead since that better describes what I want. If there is a reasonget_profile
is a better name, that would be useful to know.get_schemas
so am open to a suggestion to just not include that, though I can see why it might be useful. It would be more useful if the profile operation took (maybe comma-separated?) several filenames.