summaryrefslogtreecommitdiffhomepage
path: root/test/test-manager/src
diff options
context:
space:
mode:
authorSebastian Holmin <sebastian.holmin@mullvad.net>2024-08-05 13:49:16 +0200
committerSebastian Holmin <sebastian.holmin@mullvad.net>2024-08-16 11:19:07 +0200
commit2aa120d8944ddc4575da6d73b9e36766849c1ba0 (patch)
tree6a2fb5e1d49089d496c2bd5cb3e6f2507b241d78 /test/test-manager/src
parent5f9d41613b0938e3ba779d01682651ff06d123af (diff)
downloadmullvadvpn-2aa120d8944ddc4575da6d73b9e36766849c1ba0.tar.xz
mullvadvpn-2aa120d8944ddc4575da6d73b9e36766849c1ba0.zip
Remove `must_succede` test macro parameter
This simplifes handling of test results.
Diffstat (limited to 'test/test-manager/src')
-rw-r--r--test/test-manager/src/summary.rs6
-rw-r--r--test/test-manager/src/tests/test_metadata.rs1
2 files changed, 1 insertions, 6 deletions
diff --git a/test/test-manager/src/summary.rs b/test/test-manager/src/summary.rs
index ab217642d3..1beaef91b9 100644
--- a/test/test-manager/src/summary.rs
+++ b/test/test-manager/src/summary.rs
@@ -250,11 +250,7 @@ pub async fn print_summary_table<P: AsRef<Path>>(summary_files: &[P]) {
for test in &tests {
println!("<tr>");
- println!(
- "<td>{}{}</td>",
- test.name,
- if test.must_succeed { " *" } else { "" }
- );
+ println!("<td>{}</td>", test.name,);
let mut failed_platforms = vec![];
for summary in &summaries {
diff --git a/test/test-manager/src/tests/test_metadata.rs b/test/test-manager/src/tests/test_metadata.rs
index c403795a2b..d3d7a40407 100644
--- a/test/test-manager/src/tests/test_metadata.rs
+++ b/test/test-manager/src/tests/test_metadata.rs
@@ -9,7 +9,6 @@ pub struct TestMetadata {
pub func: TestWrapperFunction,
pub priority: Option<i32>,
pub always_run: bool,
- pub must_succeed: bool,
}
impl TestMetadata {