Skip to content

Commit

Permalink
xtask: Fix test-more not handling the new wasm workspace separation.
Browse files Browse the repository at this point in the history
  • Loading branch information
kpreid committed Nov 27, 2023
1 parent 06fc0b3 commit 1857bf8
Showing 1 changed file with 7 additions and 1 deletion.
8 changes: 7 additions & 1 deletion tools/xtask/src/xtask.rs
Original file line number Diff line number Diff line change
Expand Up @@ -409,8 +409,8 @@ const ALL_NONTEST_PACKAGES: [&str; 9] = [
"all-is-cubes-content",
"all-is-cubes-port",
"all-is-cubes-desktop",
"all-is-cubes-wasm",
"all-is-cubes-server",
"all-is-cubes-wasm",
];

const CHECK_SUBCMD: &str = "clippy";
Expand Down Expand Up @@ -554,6 +554,12 @@ fn do_for_all_packages(
// To test with limited features, we need to run commands separately for each
// package, as otherwise they will enable dependencies' features.
for package_name in ALL_NONTEST_PACKAGES {
if package_name == "all-is-cubes-wasm" {
// not in main workspace, and incidentally also has no features to test
// TODO: make this exemption data-driven
continue;
}

let _t = CaptureTime::new(time_log, format!("{op:?} --package {package_name}"));
op.cargo_cmd(config)
.args(["--package", package_name, "--no-default-features"])
Expand Down

0 comments on commit 1857bf8

Please sign in to comment.