From 6fe89e45f44df216911de48958aed38965b0de1c Mon Sep 17 00:00:00 2001 From: Aaron Hill Date: Tue, 19 Nov 2019 15:55:12 -0500 Subject: [PATCH] Disable #[should_panic] test on Windows We should re-enable this once https://github.com/rust-lang/miri/issues/1059 is fixed --- test-cargo-miri/tests/test.rs | 7 ++++++- 1 file changed, 6 insertions(+), 1 deletion(-) diff --git a/test-cargo-miri/tests/test.rs b/test-cargo-miri/tests/test.rs index bc00cb6a765c..5d1beaec9806 100644 --- a/test-cargo-miri/tests/test.rs +++ b/test-cargo-miri/tests/test.rs @@ -42,8 +42,13 @@ fn num_cpus() { assert_eq!(num_cpus::get(), 1); } + +// FIXME: Remove this `cfg` once we fix https://github.com/rust-lang/miri/issues/1059 +// We cfg-gate the `should_panic` attribute and the `panic!` itself, so that the test +// stdout does not depend on the platform #[test] -#[should_panic] +#[cfg_attr(not(windows), should_panic)] fn do_panic() { // In large, friendly letters :) + #[cfg(not(windows))] panic!("Explicit panic from test!"); }