shared/tester: retain test failure status

If a test has called tester_test_failed, consider the test failed, even
if the test also called tester_test_passed/abort.

This avoids reporting success for misbehaving tests that call the
status report functions multiple times.

diff --git a/src/shared/tester.c b/src/shared/tester.c
index 20e99bce40228a285b51c15acf61ec597338fc57..f6aff866901a7a899eb36f213235fa5affc374ed 100755 (executable)
--- a/src/shared/tester.c
+++ b/src/shared/tester.c
@@ -621,6 +621,9 @@ static void test_result(enum test_result result)
                test->timeout_id = 0;
        }
 
+       if (test->result == TEST_RESULT_FAILED)
+               result = TEST_RESULT_FAILED;
+
        test->result = result;
        switch (result) {
        case TEST_RESULT_PASSED: