Only handle shitload-of-warnings as an actual error if it's > 200

This commit is contained in:
Alexandre Aubin 2021-03-23 23:37:22 +01:00 committed by GitHub
parent 2474602713
commit 474ada2bd9
No known key found for this signature in database
GPG key ID: 4AEE18F83AFDEB23

View file

@ -341,9 +341,14 @@ TEST_LAUNCHER () {
# (we ignore this test for upgrade from older commits to avoid having to patch older commits for this) # (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" == "" ] if [ "$n_warnings" -gt 50 ] && [ "$test_type" != "TEST_UPGRADE" -o "$test_arg" == "" ]
then 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 ...!" if [ "$n_warnings" -gt 200 ]
log_report_test_failed then
SET_RESULT "failure" too_many_warnings 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 fi
local test_duration=$(echo $(( $(date +%s) - $global_start_timer ))) local test_duration=$(echo $(( $(date +%s) - $global_start_timer )))