Don't treat warnings as failures in the junit output

This matches Travis and the actual return code.

diff --git a/run-tests.php b/run-tests.php
index 4f29345d9fddc6f1ae62ef0574c2e84a1fbe8f9f..17930b29bc11699851466e94b02fb533ee1cd8d6 100755 (executable)
--- a/run-tests.php
+++ b/run-tests.php
@@ -2724,7 +2724,7 @@ function junit_mark_test_as($type, $file_name, $test_name, $time = null, $messag
 
        if (is_array($type)) {
                $output_type = $type[0] . 'ED';
-               $temp = array_intersect(array('XFAIL', 'FAIL'), $type);
+               $temp = array_intersect(array('XFAIL', 'FAIL', 'WARN'), $type);
                $type = reset($temp);
        } else {
                $output_type = $type . 'ED';
@@ -2738,6 +2738,9 @@ function junit_mark_test_as($type, $file_name, $test_name, $time = null, $messag
        } elseif ('SKIP' == $type) {
                junit_suite_record($suite, 'test_skip');
                $JUNIT['files'][$file_name]['xml'] .= "<skipped>$escaped_message</skipped>\n";
+       } elseif ('WARN' == $type) {
+               junit_suite_record($suite, 'test_warn');
+               $JUNIT['files'][$file_name]['xml'] .= "<warning>$escaped_message</warning>\n";
        } elseif('FAIL' == $type) {
                junit_suite_record($suite, 'test_fail');
                $JUNIT['files'][$file_name]['xml'] .= "<failure type='$output_type' message='$escaped_message'>$escaped_details</failure>\n";