reconhub / aweek

Convert dates to arbitrary week definitions :calendar:
https://www.repidemicsconsortium.org/aweek
Other
17 stars 4 forks source link

Test failing every seven days #5

Closed zkamvar closed 5 years ago

zkamvar commented 5 years ago

It was a bad idea to use Sys.Date() for a test :disappointed:

Version: 0.1.0 Check: tests Result: ERROR Running ‘testthat.R’ Running the tests in ‘tests/testthat.R’ failed. Complete output:

library(testthat) library(aweek)

test_check("aweek") ── 1. Failure: aweek can be converted to POSIXlt (@test_conversion.R#45) ────── Expectation did not fail

 ══ testthat results ═══════════════════════════════════════════════════════════
 OK: 31 SKIPPED: 1 FAILED: 1
 1. Failure: aweek can be converted to POSIXlt (@test_conversion.R#45)

 Error: testthat unit tests failed
 Execution halted

Flavors: r-devel-linux-x86_64-fedora-clang, r-devel-linux-x86_64-fedora-gcc, r-patched-solaris-x86

zkamvar commented 5 years ago

Fixed as of 8380b8d6527cd41a3ce4180c0ccdc9d5546c5aa6