- Notifications
You must be signed in to change notification settings - Fork741
test(touch): add coverage for -d option#925
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to ourterms of service andprivacy statement. We’ll occasionally send you account related emails.
Already on GitHub?Sign in to your account
Uh oh!
There was an error while loading.Please reload this page.
Conversation
No change to production logic.We never had coverage for `touch({'-d': ...})`, so this adds testcoverage. This also updates documentation to clarify the parametershould be an instance of the `Date` type.Test: `handles date argument` casecodecov-io commentedJan 4, 2019
Codecov Report
@@ Coverage Diff @@## master #925 +/- ##======================================= Coverage 97.26% 97.26% ======================================= Files 34 34 Lines 1281 1281 ======================================= Hits 1246 1246 Misses 35 35
Continue to review full report at Codecov.
|
This was covered by some |
No change to production logic.
We never had coverage for
touch({'-d': ...}), so this adds testcoverage. This also updates documentation to clarify the parameter
should be an instance of the
Datetype.Test:
handles date argumentcase