extendr / rextendr

An R package that helps scaffolding extendr-enabled packages or compiling Rust code dynamically
https://extendr.github.io/rextendr/
Other
190 stars 28 forks source link

Using `use_dev_extendr` in tests is contentious #341

Closed CGMossa closed 6 months ago

CGMossa commented 7 months ago

Currently, part of extendr's CI, is to run an "integration" test that involves rextendr, and its test-suite (R CMD check). This is problematic when using use_dev_extendr in a few rextendr-tests, because then we are not testing against the extendr version in the CI instance.

Suggested solution:

  1. Invent an environment variable that we can set, if we wish for tests to use use_dev_extendr, i.e. REXTENDR_TESTS_USE_DEV_EXTENDR, or similar
  2. Go through all tests that uses use_dev_extendr = TRUE and inject logic to check for this environment variable
  3. Document this maybe