cudf icon indicating copy to clipboard operation
cudf copied to clipboard

Add environment-agnostic scripts for running ctests and pytests

Open trxcllnt opened this issue 1 year ago • 2 comments

Description

This PR adds environment-agnostic run_*_{ctests,pytests}.sh scripts, and updates test_*_{cpp,python}.sh to call them.

The test_*_{cpp,python}.sh scripts assume they're running in our CI environment, and they do more than just run the tests.

This PR allows devs and downstream consumers to only run the tests, and skip the unrelated logic in test_*_{cpp,python}.sh.

Checklist

  • [x] I am familiar with the Contributing Guidelines.
  • [x] New or existing tests cover these changes.
  • [ ] The documentation is up to date with these changes.

trxcllnt avatar Feb 07 '24 07:02 trxcllnt

@trxcllnt Just FYI, there are several PRs across RAPIDS adding --no-tests=error to our ctest invocations. These could conflict. See #14983, https://github.com/rapidsai/build-planning/issues/18.

bdice avatar Feb 07 '24 15:02 bdice

@bdice yes this will help with running tests in devcontainers, but the primary motivation for this PR (and all the other similar ones I filed last night) is to provide a test script we can run in the DLFW/NVAIE builds. The test recipes change so frequently that it's difficult to keep the DLFW tests in sync for all repos.

trxcllnt avatar Feb 07 '24 17:02 trxcllnt

/merge

bdice avatar Feb 26 '24 19:02 bdice