Add reasons to #[ignore] attributes (#2797)
Some checks failed
CI / pages (push) Has been cancelled
CI / test (macos-latest) (push) Has been cancelled
CI / test (ubuntu-latest) (push) Has been cancelled
CI / lint (push) Has been cancelled
CI / msrv (push) Has been cancelled
CI / test (windows-latest) (push) Has been cancelled

This commit is contained in:
Casey Rodarmor 2025-06-28 00:41:21 -07:00 committed by GitHub
parent 407230f183
commit f401dfb18d
No known key found for this signature in database
GPG key ID: B5690EEEBB952194
2 changed files with 8 additions and 12 deletions

View file

@ -134,7 +134,6 @@ fn no_choosable_recipes() {
}
#[test]
#[ignore]
fn multiple_recipes() {
Test::new()
.arg("--choose")

View file

@ -42,7 +42,7 @@ fn interrupt_test(arguments: &[&str], justfile: &str) {
}
#[test]
#[ignore]
#[ignore = "Interrupt tests are flaky."]
fn interrupt_shebang() {
interrupt_test(
&[],
@ -55,7 +55,7 @@ fn interrupt_shebang() {
}
#[test]
#[ignore]
#[ignore = "Interrupt tests are flaky."]
fn interrupt_line() {
interrupt_test(
&[],
@ -67,7 +67,7 @@ fn interrupt_line() {
}
#[test]
#[ignore]
#[ignore = "Interrupt tests are flaky."]
fn interrupt_backtick() {
interrupt_test(
&[],
@ -81,16 +81,15 @@ fn interrupt_backtick() {
}
#[test]
#[ignore]
#[ignore = "Interrupt tests are flaky."]
fn interrupt_command() {
interrupt_test(&["--command", "sleep", "1"], "");
}
/// This test is sensitive to the process signal mask. Programs like
/// `watchexec` and `cargo-watch` change the signal mask to ignore `SIGHUP`,
/// which causes this test to fail.
#[test]
#[ignore]
#[ignore = "This test is sensitive to the process signal mask. Programs like \
`watchexec` and `cargo-watch` change the signal mask to ignore \
`SIGHUP`, which causes this test to fail."]
fn forwarding() {
let just = executable_path("just");
@ -155,10 +154,8 @@ fn forwarding() {
}
}
/// This test is ignored because it includes a 500ms wait, and because signal
/// tests are often flakey.
#[test]
#[ignore]
#[ignore = "Includes a 500ms wait is often flakey."]
#[cfg(any(
target_os = "dragonfly",
target_os = "freebsd",