nextstrain / augur

Pipeline components for real-time phylodynamic analysis
https://docs.nextstrain.org/projects/augur/
GNU Affero General Public License v3.0
268 stars 128 forks source link

augur curate I/O: validate records have same fields #1518

Closed joverlee521 closed 5 days ago

joverlee521 commented 5 days ago

Description of proposed changes

Validate records have the same fields before and after the subcommand modifies the records.

Includes a functional test to check for the input error, but there's no good way to check for the output error. Hopefully we just never see it!

Related issue(s)

Resolves #1510

Checklist

codecov[bot] commented 5 days ago

Codecov Report

All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 69.60%. Comparing base (c679fde) to head (d84a0f4).

Additional details and impacted files ```diff @@ Coverage Diff @@ ## master #1518 +/- ## ========================================== + Coverage 69.54% 69.60% +0.05% ========================================== Files 73 73 Lines 7786 7801 +15 Branches 1904 1908 +4 ========================================== + Hits 5415 5430 +15 Misses 2086 2086 Partials 285 285 ```

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

joverlee521 commented 5 days ago

Rebased on to master to ensure no merge conflicts in changelog and updated changelog.