Skip to content
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 our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Test for removed records, switch to R CMD CHECK workflow for tests, use default testthat setup #246

Draft
wants to merge 24 commits into
base: main
Choose a base branch
from

Conversation

PietrH
Copy link
Member

@PietrH PietrH commented Dec 9, 2024

  • Add function to get the new, current, local to the repo, version of occurrence.csv
  • Add function to get the reference, published, on github, version of occurrence.csv
  • test get_current
  • test get_reference
  • get coverage on new helpers
  • Adapt tests to use new helper functions
  • Change test setup to use default testthat dirs
  • Use R CMD CHECK Github action to run tests instead of running single script, so package context (exported data and functions) can be used in tests
  • Move occurrence.csv to inst/data/processed/occurrence.csv instead of package root data folder, THIS REQUIRES A CHANGE ON THE IPT

  • Adapt IPT dataset

  • pass r cmd check
  • pass tests
  • pass lintr

@PietrH PietrH self-assigned this Jan 3, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant