From 280c3261d67ecbc9d6b6df89b151a9096266fa72 Mon Sep 17 00:00:00 2001
From: Alexandre Aubin <alex.aubin@mailoo.org>
Date: Tue, 9 Mar 2021 06:08:56 +0100
Subject: [PATCH] Uhoh tests were not failing anymore

---
 lib/tests_coordination.sh | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/lib/tests_coordination.sh b/lib/tests_coordination.sh
index 0583461..c4ed0c6 100644
--- a/lib/tests_coordination.sh
+++ b/lib/tests_coordination.sh
@@ -323,6 +323,8 @@ TEST_LAUNCHER () {
     # Execute the test
     $test_type $test_arg
 
+    [ $? -eq 0 ] && SET_RESULT "success" main_result || SET_RESULT "failure" main_result
+
     # Check that the number of warning ain't higher than a treshold
     local n_warnings=$(grep --extended-regexp '^[0-9]+\s+.{1,15}WARNING' $current_test_log | wc -l)
     # (we ignore this test for upgrade from older commits to avoid having to patch older commits for this)
@@ -333,8 +335,6 @@ TEST_LAUNCHER () {
         SET_RESULT "failure" too_many_warnings
     fi
 
-    [ $? -eq 0 ] && SET_RESULT "success" main_result || SET_RESULT "failure" main_result
-
     local test_duration=$(echo $(( $(date +%s) - $global_start_timer )))
     SET_RESULT "$test_duration" test_duration