You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
in #33, a test suite was added to test the validity of the raw data. This is a good thing! The icky thing is the fact that the package tests were never really set up for this situation, which leads to things like getwd() sneaking into the code.
Given that these tests are designed to test the data generation and will only ever run locally, would it make more sense to move them inside of the script?
library(testthat)
library(dplyr)
## Data fetching and generation
test_that("data looks good", {
expect_is(...)
})
test_that("data smells good", {
expect_equal(...)
})
## Data writing and stuff.
The text was updated successfully, but these errors were encountered:
Just to add that if it is data integrity we are interested in, I.e. checking the data for certain characteristics before saving) then the assertr package might be more appropriate. https://docs.ropensci.org/assertr/
in #33, a test suite was added to test the validity of the raw data. This is a good thing! The icky thing is the fact that the package tests were never really set up for this situation, which leads to things like
getwd()
sneaking into the code.Given that these tests are designed to test the data generation and will only ever run locally, would it make more sense to move them inside of the script?
The text was updated successfully, but these errors were encountered: