diff --git a/tests/functional/new_test.rs b/tests/functional/new_test.rs index 175288f..a6a9976 100644 --- a/tests/functional/new_test.rs +++ b/tests/functional/new_test.rs @@ -2,12 +2,14 @@ //! data, so it can't be range-matched. The selection layer is supposed to //! list it via nextest and tag it `(new)` in the verbose output. //! -//! Two failure modes are guarded here. A perpetually-`#[ignore]`d test is -//! always listed but never collected, so it must not read as "new" forever. -//! And the listing must build with the same cargo features as the run, or a -//! feature-gated new test is invisible to detection — checked for `run` and -//! for `status`, which has to reach the same answer to be worth anything as a -//! dry run. +//! Several failure modes are guarded here. A perpetually-`#[ignore]`d test is +//! always listed but never collected, so it must not read as "new" forever. A +//! test that becomes `#[ignore]`d after collection keeps its coverage rows, so +//! it must be dropped from the `affected` selection too — a separate path from +//! new-test detection. And the listing must build with the same cargo features +//! as the run, or a feature-gated new test is invisible to detection — checked +//! for `run` and for `status`, which has to reach the same answer to be worth +//! anything as a dry run. use std::path::Path;