Skip to content

Add test commands to rapids-build-utils#630

Merged
bdice merged 3 commits intorapidsai:mainfrom
bdice:feature/add-test-commands
Dec 10, 2025
Merged

Add test commands to rapids-build-utils#630
bdice merged 3 commits intorapidsai:mainfrom
bdice:feature/add-test-commands

Conversation

@bdice
Copy link
Copy Markdown
Contributor

@bdice bdice commented Dec 9, 2025 •
edited
Loading

Summary

With help from claude, I am implementing my favorite feature from the old rapids-compose: test-* commands.

  • Add test-<lib>-cpp commands that run C++ test scripts (e.g., test-cudf-cpp runs ci/run_cudf_ctests.sh)
  • Add test-<lib>-python commands that run Python test scripts (e.g., test-rmm-python runs ci/run_pytests.sh)
  • Add test-<repo> commands that run both C++ and Python tests for a repo
  • Add test-all, test-all-cpp, and test-all-python commands

Test script paths are specified in manifest.yaml since they vary across repos. All arguments are forwarded to the underlying scripts.

Test plan

  • Rebuild devcontainer and verify test-* commands are generated
  • Run test-rmm-cpp -h to verify help text
  • Run test-cudf-cpp (via script content inspection) to verify C++ tests invoke correct CI scripts
  • Run test-rmm-python to verify Python tests invoke correct CI scripts
  • Run test-all -h to verify aggregated command works

Testing notes

Validated using rapidsai/devcontainers:26.02-cpp-rapids-build-utils-ubuntu24.04 with modified rapids-build-utils mounted:

# Commands generated correctly
$ ls /usr/bin/test-rmm*
/usr/bin/test-rmm
/usr/bin/test-rmm-cpp
/usr/bin/test-rmm-python

# Help text works
$ test-rmm-cpp -h
Usage:
 test-rmm-cpp [OPTION]...

Run rmm C++ tests.

Boolean options:
 -h,--help  Print this text.

# test-rmm-cpp correctly invokes ci/run_ctests.sh
$ test-rmm-cpp
./ci/run_ctests.sh: line 8: cd: /usr/bin/gtests/librmm/: No such file or directory

# test-rmm-python correctly invokes ci/run_pytests.sh
$ test-rmm-python
./ci/run_pytests.sh: line 10: pytest: command not found

# test-all commands work
$ test-all-cpp -h
Usage:
 test-all-cpp [OPTION]...

Runs test-<repo>-cpp for each repo in 'rmm' 'ucxx' ...

bdice added 2 commits December 8, 2025 22:06
@bdice
Add test commands to rapids-build-utils
8df7937 
Add test-<lib>-cpp, test-<lib>-python, test-<repo>, test-all,
test-all-cpp, and test-all-python commands that run CI test scripts.

- Add 6 new test templates (cpp.test, python.test, repo.test,
  all.test, all.cpp.test, all.python.test)
- Add test field to manifest.yaml for C++ and Python components
- Update generate-scripts.sh to generate test commands

Test commands forward all arguments to underlying CI scripts.
@bdice
Bump devcontainer feature version
caec34a
@trxcllnt
Copy link
Copy Markdown
Collaborator

trxcllnt commented Dec 9, 2025

Looks good to me!

@bdice
Copy link
Copy Markdown
Contributor Author

bdice commented Dec 9, 2025

Trying to test this locally before I open. I'd really like to have this for testing CCCL 3.2 updates. It saves a lot of brainpower.

@bdice bdice marked this pull request as ready for review December 10, 2025 19:17
@bdice bdice requested a review from a team as a code owner December 10, 2025 19:17
@bdice bdice requested review from KyleFromNVIDIA and removed request for a team December 10, 2025 19:17
@bdice
Copy link
Copy Markdown
Contributor Author

bdice commented Dec 10, 2025

I think this is pretty close to right. I'll merge it and then iterate to fix any issues.

@bdice bdice merged commit 2e4bcaf into rapidsai:main Dec 10, 2025
13 of 21 checks passed
This was referenced Dec 11, 2025
Merged
Merged
Merged
Merged
Merged
Merged
Merged
Merged
Merged
rapids-bot Bot pushed a commit to rapidsai/kvikio that referenced this pull request Dec 11, 2025
@bdice
Add devcontainer fallback for C++ test location (#887)
4883d4f 
## Summary
- Update `run_ctests.sh` to first try the installed test location (CI/conda environments) and fall back to the build directory (devcontainer environments)
- Enables testing in devcontainers with `test-kvikio-cpp`

xref: rapidsai/devcontainers#630

Authors:
  - Bradley Dice (https://github.com/bdice)

Approvers:
  - Kyle Edwards (https://github.com/KyleFromNVIDIA)

URL: #887
rapids-bot Bot pushed a commit to rapidsai/raft that referenced this pull request Dec 12, 2025
@bdice
Add devcontainer fallback for C++ test location (#2893)
eb4c383 
## Summary
- Update `run_ctests.sh` to first try the installed test location (CI/conda environments) and fall back to the build directory (devcontainer environments)
- Enables testing in devcontainers with `test-raft-cpp`

xref: rapidsai/devcontainers#630

Authors:
  - Bradley Dice (https://github.com/bdice)

Approvers:
  - Kyle Edwards (https://github.com/KyleFromNVIDIA)
  - Nate Rock (https://github.com/rockhowse)

URL: #2893
rapids-bot Bot pushed a commit to rapidsai/rmm that referenced this pull request Dec 12, 2025
@bdice
Add devcontainer fallback for C++ test location (#2181)
227d292 
## Summary
- Update `run_ctests.sh` to first try the installed test location (CI/conda environments) and fall back to the build directory (devcontainer environments)
- Enables testing in devcontainers with `test-rmm-cpp`

xref: rapidsai/devcontainers#630

Authors:
  - Bradley Dice (https://github.com/bdice)

Approvers:
  - Paul Taylor (https://github.com/trxcllnt)
  - Kyle Edwards (https://github.com/KyleFromNVIDIA)

URL: #2181
rapids-bot Bot pushed a commit to rapidsai/cugraph-gnn that referenced this pull request Dec 12, 2025
@bdice
Add devcontainer fallback for C++ test location (#370)
9bb2b94 
## Summary
- Update `run_ctests.sh` to first try the installed test location (CI/conda environments) and fall back to the build directory (devcontainer environments)
- Enables testing in devcontainers with `test-wholegraph-cpp`

xref: rapidsai/devcontainers#630

Authors:
  - Bradley Dice (https://github.com/bdice)

Approvers:
  - Kyle Edwards (https://github.com/KyleFromNVIDIA)

URL: #370
rapids-bot Bot pushed a commit to rapidsai/cugraph that referenced this pull request Dec 12, 2025
@bdice
Add devcontainer fallback for C++ test location (#5369)
d22062d 
## Summary
- Update `run_ctests.sh` to first try the installed test location (CI/conda environments) and fall back to the build directory (devcontainer environments)
- Enables testing in devcontainers with `test-cugraph-cpp`

xref: rapidsai/devcontainers#630

Authors:
  - Bradley Dice (https://github.com/bdice)

Approvers:
  - Kyle Edwards (https://github.com/KyleFromNVIDIA)

URL: #5369
rapids-bot Bot pushed a commit to rapidsai/cuml that referenced this pull request Dec 12, 2025
@bdice
Add devcontainer fallback for C++ test location (#7596)
412210b 
## Summary
- Update `run_ctests.sh` to first try the installed test location (CI/conda environments) and fall back to the build directory (devcontainer environments)
- Enables testing in devcontainers with `test-cuml-cpp`

xref: rapidsai/devcontainers#630

Authors:
  - Bradley Dice (https://github.com/bdice)

Approvers:
  - Kyle Edwards (https://github.com/KyleFromNVIDIA)

URL: #7596
rapids-bot Bot pushed a commit to rapidsai/rapidsmpf that referenced this pull request Dec 12, 2025
@bdice
Add devcontainer fallback for C++ test location (#734)
220e262 
## Summary
- Update `run_ctests.sh` to first try the installed test location (CI/conda environments) and fall back to the build directory (devcontainer environments)
- Enables testing in devcontainers with `test-rapidsmpf-cpp`

xref: rapidsai/devcontainers#630

Authors:
  - Bradley Dice (https://github.com/bdice)

Approvers:
  - Kyle Edwards (https://github.com/KyleFromNVIDIA)

URL: #734
rapids-bot Bot pushed a commit to rapidsai/cudf that referenced this pull request Dec 12, 2025
@bdice
Add devcontainer fallback for C++ test location (#20838)
0eadb8b 
## Summary
- Update `run_cudf_ctests.sh` and `run_cudf_kafka_ctests.sh` to first try the installed test location (CI/conda environments) and fall back to the build directory (devcontainer environments)
- Enables testing in devcontainers with `test-cudf-cpp`

xref: rapidsai/devcontainers#630

Authors:
  - Bradley Dice (https://github.com/bdice)

Approvers:
  - Kyle Edwards (https://github.com/KyleFromNVIDIA)
  - Karthikeyan (https://github.com/karthikeyann)

URL: #20838
rapids-bot Bot pushed a commit to rapidsai/cuvs that referenced this pull request Dec 12, 2025
@bdice
Add devcontainer fallback for C++ test location (#1631)
121071e 
## Summary
- Update `run_ctests.sh` to first try the installed test location (CI/conda environments) and fall back to the build directory (devcontainer environments)
- Enables testing in devcontainers with `test-cuvs-cpp`

xref: rapidsai/devcontainers#630

Authors:
  - Bradley Dice (https://github.com/bdice)

Approvers:
  - Gil Forsyth (https://github.com/gforsyth)
  - Kyle Edwards (https://github.com/KyleFromNVIDIA)

URL: #1631
@bdice bdice mentioned this pull request Dec 16, 2025
Merged
rapids-bot Bot pushed a commit to rapidsai/ucxx that referenced this pull request Dec 16, 2025
@bdice
Add devcontainer fallback for C++ test location (#561)
c0c6a7a 
## Summary
- Update `run_cpp.sh` to first try the installed test location (CI/conda environments) and fall back to the build directory (devcontainer environments)
- Enables testing in devcontainers with `test-ucxx-cpp`

xref: rapidsai/devcontainers#630

Authors:
  - Bradley Dice (https://github.com/bdice)

Approvers:
  - Kyle Edwards (https://github.com/KyleFromNVIDIA)

URL: #561
mani-builds pushed a commit to mani-builds/cuml that referenced this pull request Jan 11, 2026
@bdice
Add devcontainer fallback for C++ test location (rapidsai#7596)
e09e133 
## Summary
- Update `run_ctests.sh` to first try the installed test location (CI/conda environments) and fall back to the build directory (devcontainer environments)
- Enables testing in devcontainers with `test-cuml-cpp`

xref: rapidsai/devcontainers#630

Authors:
  - Bradley Dice (https://github.com/bdice)

Approvers:
  - Kyle Edwards (https://github.com/KyleFromNVIDIA)

URL: rapidsai#7596
@jameslamb jameslamb mentioned this pull request Jan 23, 2026
Open
@jameslamb jameslamb mentioned this pull request Apr 15, 2026
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@KyleFromNVIDIA KyleFromNVIDIA Awaiting requested review from KyleFromNVIDIA KyleFromNVIDIA is a code owner automatically assigned from rapidsai/packaging-codeowners

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@bdice @trxcllnt