Skip to content

Commit dd55990

Browse files
committed
Expand unit testing on CI.
Now using an exclusion approach instead of an inclusion approach, to increase coverage.
1 parent 9bc2a74 commit dd55990

2 files changed

Lines changed: 8 additions & 8 deletions

File tree

.github/workflows/ci_linux.yml

Lines changed: 5 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -158,16 +158,15 @@ jobs:
158158
cargo clippy
159159
'
160160
161-
# Very limited testing because we can only run tests that don't rely on having a CUDA GPU.
161+
# Exclude crates with tests that require an NVIDIA GPU: blastoff, cudnn, cust.
162162
- name: Test
163163
run: |
164164
docker exec "$CONTAINER_NAME" bash -lc 'set -euo pipefail
165165
export RUSTFLAGS=-Dwarnings
166-
cargo test \
167-
-p cuda_std \
168-
-p gpu_rand \
169-
-p nvvm \
170-
-p rustc_codegen_nvvm
166+
cargo test --workspace \
167+
--exclude blastoff \
168+
--exclude cudnn \
169+
--exclude cust
171170
'
172171
173172
- name: Check documentation

.github/workflows/ci_windows.yml

Lines changed: 3 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -96,9 +96,10 @@ jobs:
9696
RUSTFLAGS: -Dwarnings
9797
run: cargo clippy --workspace --exclude "cudnn*" --exclude "gemm*" --exclude "i128*" --exclude "sha2*" --exclude "vecadd*"
9898

99-
# Very limited testing because we can only run tests that don't rely on having a CUDA GPU.
99+
# Exclude crates with tests that require an NVIDIA GPU: blastoff, cudnn, cust.
100+
# Exclude crates that require cuDNN, not available on Windows CI: cudnn, cudnn-sys.
100101
- name: Test
101-
run: cargo test -p cuda_std -p gpu_rand -p nvvm -p rustc_codegen_nvvm
102+
run: cargo test --workspace --exclude blastoff --exclude cudnn --exclude cudnn-sys --exclude cust
102103

103104
- name: Check documentation
104105
env:

0 commit comments

Comments
 (0)