diff --git a/src/ui.rs b/src/ui.rs index 6153764..0108cbc 100644 --- a/src/ui.rs +++ b/src/ui.rs @@ -406,20 +406,17 @@ impl OutputBuffer { result_url_base: &str, ) -> Vec> { let status_part = match status { - // Note - cancellation is an "error" in the type system but we - // don't treat it as an error in the UI. - TestStatus::Finished(Err(TestInconclusive::Error(msg))) => { - Span::new(msg).with_class(Class::Error) - } + TestStatus::Enqueued => Span::new("⏳"), + TestStatus::Started => Span::new("🔄"), TestStatus::Finished(Ok(result)) => { if result.exit_code == 0 { - Span::new("success").with_class(Class::Success) + Span::new("✅") } else { - Span::new(format!("failed (status {})", result.exit_code)) - .with_class(Class::Failure) + Span::new("❌") } } - _ => Span::new(status.to_string()), + TestStatus::Finished(Err(TestInconclusive::Canceled)) => Span::new("⛔"), + TestStatus::Finished(Err(_)) => Span::new("🔥"), } .with_url(format!( "{}/{}/{}", @@ -543,9 +540,9 @@ mod tests { *strip_ansi_escapes::strip_str(str::from_utf8(buf.as_bytes()).unwrap()), eq(format!( "* {commit3} 3\n\ - | my_test1: Enqueued my_test2: success \n\ + | my_test1: ⏳ my_test2: ✅ \n\ * {commit2} 2\n\ - | my_test1: oh no my_test2: Started \n", + | my_test1: 🔥 my_test2: 🔄 \n", commit3 = abbrev(&commit3), commit2 = abbrev(&commit2) )) @@ -604,13 +601,13 @@ mod tests { |\\ \\ \n\ | | | \n\ | | * {commit2} 2\n\ - | | my_test1: oh no my_test2: Started \n\ + | | my_test1: 🔥 my_test2: 🔄 \n\ | * {commit1} 1\n\ | | \n\ | * {join} join\n\ | \n\ * {commit3} 3\n\ - | my_test1: Enqueued my_test2: success \n", + | my_test1: ⏳ my_test2: ✅ \n", merge = abbrev(&merge), commit3 = abbrev(&commit3), commit2 = abbrev(&commit2),