Add skip conditions to CTS runner #7949

andyleiserson · 2025-07-15T00:16:54Z

Adds a simple mechanism to skip tests on a particular platform in the CTS runner. It is not used in this PR, but I plan to use it for #7948.

Testing
Tested manually.

Squash or Rebase? Squash

Checklist

Run cargo fmt.
Run taplo format.
Run cargo clippy --tests. If applicable, add:
- --target wasm32-unknown-unknown
Run cargo xtask test to run tests.
If this contains user-facing changes, add a CHANGELOG.md entry.

cwfitzgerald

G2G with nit

cwfitzgerald · 2025-07-15T01:29:10Z

xtask/src/cts.rs

+        match (&test.fails_if, &running_on_backend) {
+            (Some(backend), Some(running_on_backend)) if backend == running_on_backend => {


Nit: could we early continue in this case to prevent the rightward drift?

Add skip conditions to CTS runner

4ade390

andyleiserson requested a review from a team as a code owner July 15, 2025 00:16

cwfitzgerald approved these changes Jul 15, 2025

View reviewed changes

Fix nit

f7efb8e

andyleiserson merged commit 1da2177 into gfx-rs:trunk Jul 15, 2025
40 checks passed

andyleiserson deleted the cts-test-conditions branch July 15, 2025 17:06

Provide feedback

		match (&test.fails_if, &running_on_backend) {
		(Some(backend), Some(running_on_backend)) if backend == running_on_backend => {