Skip to content

Commit 586ddf9

Browse files
committed
Change skip to fail if emscripten is not installed for test
1 parent 84726cf commit 586ddf9

1 file changed

Lines changed: 6 additions & 3 deletions

File tree

crates/smoketests/tests/smoketests/csharp_aot_module.rs

Lines changed: 6 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -17,11 +17,14 @@ fn test_build_csharp_module_aot() {
1717
return;
1818
}
1919

20-
// Check for emscripten - skip if not available
20+
// Check for emscripten - fail with helpful message if not available
2121
let emscripten_check = Command::new("emcc").arg("--version").output();
2222
if emscripten_check.is_err() || !emscripten_check.unwrap().status.success() {
23-
eprintln!("Skipping AOT test - emscripten not available");
24-
return;
23+
panic!(
24+
"NativeAOT-LLVM test requires emscripten but it was not found.\n\
25+
Install from: https://emscripten.org/docs/getting_started/downloads.html\n\
26+
Or ensure `emcc` is in your PATH."
27+
);
2528
}
2629

2730
let workspace = workspace_root();

0 commit comments

Comments
 (0)