src/tool/subcommands/api_cmd/stateful_tests.rs (1)

109-120: Print the ignore reason and allow bypassing ignores in nightly runs.

Two small improvements:

• Include the ignore message (URL) in the “ignored” log to aid triage.
• Gate ignores behind an env flag (e.g., FOREST_RUN_IGNORED=1) so we can exercise flaky tests in scheduled jobs.

Proposed tweak to run_tests logging/flow:

if let Some(reason) = test.ignore {
    if std::env::var_os("FOREST_RUN_IGNORED").is_none() {
        ignored += 1;
        println!(
            "test {} ... ignored: {reason}",
            if let Some(name) = test.name { name.to_string() } else { format!("#{i}") },
        );
        continue;
    } else {
        println!(
            "test {} ... RUNNING (was ignored): {reason}",
            if let Some(name) = test.name { name.to_string() } else { format!("#{i}") },
        );
        // fall through to run the test
    }
}

This keeps CI green by default while ensuring ignored tests still get signal in nightly/scheduled workflows.

Configuration used: CodeRabbit UI

Review profile: CHILL

Plan: Pro

Learnt from: hanabi1224
PR: ChainSafe/forest#5930
File: build.rs:64-77
Timestamp: 2025-08-13T09:43:20.301Z
Learning: hanabi1224 prefers hard compile-time errors in build scripts rather than runtime safeguards or collision detection, believing it's better to fail fast and fix root causes of issues like malformed snapshot names.