feat(harness): Give callers responsibility for ignoreds (#121)

epage · web-flow · commit f9a851ab4bad · 2025-09-11T16:33:41.000-05:00
diff --git a/crates/libtest2-harness/src/case.rs b/crates/libtest2-harness/src/case.rs
@@ -69,14 +69,16 @@ impl RunError {
         Self::with_cause(Message(cause.to_string()))
     }
 
-    pub(crate) fn ignore() -> Self {
+    /// Should not be called with `libtest_lexarg::RunIgnored::Yes`
+    pub fn ignore() -> Self {
         Self {
             status: notify::MessageKind::Ignored,
             cause: None,
         }
     }
 
-    pub(crate) fn ignore_for(reason: String) -> Self {
+    /// Should not be called with `libtest_lexarg::RunIgnored::Yes`
+    pub fn ignore_for(reason: String) -> Self {
         Self {
             status: notify::MessageKind::Ignored,
             cause: Some(Box::new(Message(reason))),

Original file line number	Diff line number	Diff line change
`@@ -69,14 +69,16 @@ impl RunError {`
`69`	`69`	`Self::with_cause(Message(cause.to_string()))`
`70`	`70`	`}`
`71`	`71`
`72`		`- pub(crate) fn ignore() -> Self {`
	`72`	+ /// Should not be called with `libtest_lexarg::RunIgnored::Yes`
	`73`	`+ pub fn ignore() -> Self {`
`73`	`74`	`Self {`
`74`	`75`	`status: notify::MessageKind::Ignored,`
`75`	`76`	`cause: None,`
`76`	`77`	`}`
`77`	`78`	`}`
`78`	`79`
`79`		`- pub(crate) fn ignore_for(reason: String) -> Self {`
	`80`	+ /// Should not be called with `libtest_lexarg::RunIgnored::Yes`
	`81`	`+ pub fn ignore_for(reason: String) -> Self {`
`80`	`82`	`Self {`
`81`	`83`	`status: notify::MessageKind::Ignored,`
`82`	`84`	`cause: Some(Box::new(Message(reason))),`