From 474ada2bd9c00f504c7c49e2f9af11f1ea07e418 Mon Sep 17 00:00:00 2001 From: Alexandre Aubin Date: Tue, 23 Mar 2021 23:37:22 +0100 Subject: [PATCH] Only handle shitload-of-warnings as an actual error if it's > 200 --- lib/tests_coordination.sh | 11 ++++++++--- 1 file changed, 8 insertions(+), 3 deletions(-) diff --git a/lib/tests_coordination.sh b/lib/tests_coordination.sh index 9d3f385..836cbf1 100644 --- a/lib/tests_coordination.sh +++ b/lib/tests_coordination.sh @@ -341,9 +341,14 @@ TEST_LAUNCHER () { # (we ignore this test for upgrade from older commits to avoid having to patch older commits for this) if [ "$n_warnings" -gt 50 ] && [ "$test_type" != "TEST_UPGRADE" -o "$test_arg" == "" ] then - log_error "There's a shitload of warnings in the output ! If those warnings are coming from some app build step and ain't actual warnings, please redirect them to the standard output instead of the error output ...!" - log_report_test_failed - SET_RESULT "failure" too_many_warnings + if [ "$n_warnings" -gt 200 ] + then + log_error "There's A SHITLOAD of warnings in the output ! If those warnings are coming from some app build step and ain't actual warnings, please redirect them to the standard output instead of the error output ...!" + log_report_test_failed + SET_RESULT "failure" too_many_warnings + else + log_error "There's quite a lot of warnings in the output ! If those warnings are coming from some app build step and ain't actual warnings, please redirect them to the standard output instead of the error output ...!" + fi fi local test_duration=$(echo $(( $(date +%s) - $global_start_timer )))