beenull/ladybird
1
0
Fork 0
mirror of https://github.com/LadybirdBrowser/ladybird.git synced 2024-11-21 23:20:20 +00:00
Code Issues Projects Releases Packages Wiki Activity Actions 6

headless-browser: Do not log skipped tests by default

Browse source 
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.
This commit is contained in:
Timothy Flynn 2024-11-11 07:55:59 -05:00 committed by Andreas Kling
parent d2306efaea
commit aa0811d24e
Notes: github-actions[bot] 2024-11-11 15:55:45 +00:00 
Author: https://github.com/trflynn89
Commit: https://github.com/LadybirdBrowser/ladybird/commit/aa0811d24e5
Pull-request: https://github.com/LadybirdBrowser/ladybird/pull/2283
1 changed files with 5 additions and 1 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

6
UI/Headless/Test.cpp
View file

@ -501,8 +501,12 @@ ErrorOr<void> run_tests(Core::AnonymousBuffer const& theme, Gfx::IntSize window_
outln("Pass: {}, Fail: {}, Skipped: {}, Timeout: {}", pass_count, fail_count, skipped_count, timeout_count); outln("Pass: {}, Fail: {}, Skipped: {}, Timeout: {}", pass_count, fail_count, skipped_count, timeout_count);
outln("=================================================="); 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); outln("{}: {}", test_result_to_string(non_passing_test.result), non_passing_test.test.input_path);
}
if (app.verbose) { if (app.verbose) {
auto tests_to_print = min(10uz, tests.size()); auto tests_to_print = min(10uz, tests.size());