From aa0811d24e503930806509a1f6111a448172afe2 Mon Sep 17 00:00:00 2001 From: Timothy Flynn Date: Mon, 11 Nov 2024 07:55:59 -0500 Subject: [PATCH] headless-browser: Do not log skipped tests by default We now skip so many tests that the list of skipped tests exceeds the height of my terminal. Let's skip logging these by default, as it is too noisy to find actually relevant information. --- UI/Headless/Test.cpp | 6 +++++- 1 file changed, 5 insertions(+), 1 deletion(-) diff --git a/UI/Headless/Test.cpp b/UI/Headless/Test.cpp index 6bb687f58cb..33005543ee4 100644 --- a/UI/Headless/Test.cpp +++ b/UI/Headless/Test.cpp @@ -501,8 +501,12 @@ ErrorOr run_tests(Core::AnonymousBuffer const& theme, Gfx::IntSize window_ outln("Pass: {}, Fail: {}, Skipped: {}, Timeout: {}", pass_count, fail_count, skipped_count, timeout_count); outln("=================================================="); - for (auto const& non_passing_test : non_passing_tests) + for (auto const& non_passing_test : non_passing_tests) { + if (non_passing_test.result == TestResult::Skipped && !app.verbose) + continue; + outln("{}: {}", test_result_to_string(non_passing_test.result), non_passing_test.test.input_path); + } if (app.verbose) { auto tests_to_print = min(10uz, tests.size());