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 format-dates: accept masked dates #1501

Closed joverlee521 closed 6 days ago

joverlee521 commented 1 week ago

Description of proposed changes

Add DEFAULT_EXPECTED_DATE_FORMATS to augur curate format-dates. These are default date formats that the command should parse without additional input from the user, which includes the ISO 8601 date format and it's various masked forms.

Also includes some minor restructuring of tests for format-dates to be more modular cram tests.

Related issue(s)

Resolves #1496

Checklist

codecov[bot] commented 1 week ago

Codecov Report

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

Project coverage is 69.40%. Comparing base (a9d1b3a) to head (2cd7111).

:exclamation: Current head 2cd7111 differs from pull request most recent head 25f4993

Please upload reports for the commit 25f4993 to get more accurate results.

Additional details and impacted files ```diff @@ Coverage Diff @@ ## master #1501 +/- ## ========================================== - Coverage 69.53% 69.40% -0.13% ========================================== Files 73 72 -1 Lines 7788 7756 -32 Branches 1905 1900 -5 ========================================== - Hits 5415 5383 -32 Misses 2087 2087 Partials 286 286 ```

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

joverlee521 commented 6 days ago

Rebased on to master to resolve merge conflicts in the changelog.