YETUS-147 Failed test table mixes result from different JDKs Signed-off-by: Allen Wittenauer <[email protected]>
Project: http://git-wip-us.apache.org/repos/asf/yetus/repo Commit: http://git-wip-us.apache.org/repos/asf/yetus/commit/36e58112 Tree: http://git-wip-us.apache.org/repos/asf/yetus/tree/36e58112 Diff: http://git-wip-us.apache.org/repos/asf/yetus/diff/36e58112 Branch: refs/heads/YETUS-83 Commit: 36e581125eb3211cbb8fdb5368c8208d5e41e1a4 Parents: 7c1face Author: Marco Zuehlke <[email protected]> Authored: Thu Oct 29 22:33:15 2015 +0100 Committer: Allen Wittenauer <[email protected]> Committed: Thu Oct 29 15:21:41 2015 -0700 ---------------------------------------------------------------------- precommit/test-patch.sh | 14 +++++++------- 1 file changed, 7 insertions(+), 7 deletions(-) ---------------------------------------------------------------------- http://git-wip-us.apache.org/repos/asf/yetus/blob/36e58112/precommit/test-patch.sh ---------------------------------------------------------------------- diff --git a/precommit/test-patch.sh b/precommit/test-patch.sh index 93874a2..fb1b4d0 100755 --- a/precommit/test-patch.sh +++ b/precommit/test-patch.sh @@ -1943,6 +1943,13 @@ function check_unittests ((i=i+1)) done + for testsys in ${TESTFORMATS}; do + if declare -f ${testsys}_finalize_results >/dev/null; then + yetus_debug "Calling ${testsys}_finalize_results" + "${testsys}_finalize_results" "${statusjdk}" + fi + done + done JAVA_HOME=${savejavahome} @@ -1954,13 +1961,6 @@ function check_unittests add_footer_table "${statusjdk} Test Results" "${BUILD_URL}testReport/" fi - for testsys in ${TESTFORMATS}; do - if declare -f ${testsys}_finalize_results >/dev/null; then - yetus_debug "Calling ${testsys}_finalize_results" - "${testsys}_finalize_results" "${statusjdk}" - fi - done - if [[ ${result} -gt 0 ]]; then return 1 fi
