You are viewing a plain text version of this content. The canonical link for it is here.
Posted to common-commits@hadoop.apache.org by aw...@apache.org on 2015/08/13 22:35:39 UTC

hadoop git commit: HADOOP-12316. Potential false-positive and false-negative in parsing TAP output (Kengo Seki via aw)

Repository: hadoop
Updated Branches:
  refs/heads/HADOOP-12111 3e25a6839 -> f64f68a3b


HADOOP-12316. Potential false-positive and false-negative in parsing TAP output (Kengo Seki via aw)


Project: http://git-wip-us.apache.org/repos/asf/hadoop/repo
Commit: http://git-wip-us.apache.org/repos/asf/hadoop/commit/f64f68a3
Tree: http://git-wip-us.apache.org/repos/asf/hadoop/tree/f64f68a3
Diff: http://git-wip-us.apache.org/repos/asf/hadoop/diff/f64f68a3

Branch: refs/heads/HADOOP-12111
Commit: f64f68a3b745f3940b80c304e5c737b008976b67
Parents: 3e25a68
Author: Allen Wittenauer <aw...@apache.org>
Authored: Thu Aug 13 13:35:37 2015 -0700
Committer: Allen Wittenauer <aw...@apache.org>
Committed: Thu Aug 13 13:35:37 2015 -0700

----------------------------------------------------------------------
 dev-support/test-patch.d/tap.sh | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/hadoop/blob/f64f68a3/dev-support/test-patch.d/tap.sh
----------------------------------------------------------------------
diff --git a/dev-support/test-patch.d/tap.sh b/dev-support/test-patch.d/tap.sh
index c6796a8..f94a14f 100755
--- a/dev-support/test-patch.d/tap.sh
+++ b/dev-support/test-patch.d/tap.sh
@@ -49,7 +49,7 @@ function tap_process_tests
   declare module_failed_tests
   declare filenames
 
-  filenames=$(find "${TAP_LOG_DIR}" -type f -exec "${GREP}" -l -E "not ok " {} \;)
+  filenames=$(find "${TAP_LOG_DIR}" -type f -exec "${GREP}" -l -E "^not ok" {} \;)
 
   if [[ -n "${filenames}" ]]; then
     module_failed_tests=$(echo "${filenames}" \
@@ -73,7 +73,7 @@ function tap_finalize_results
 
   if [[ -n "${TAP_FAILED_TESTS}" ]] ; then
     # shellcheck disable=SC2086
-    populate_test_table "${jdk}Failed junit tests" ${TAP_FAILED_TESTS}
+    populate_test_table "${jdk}Failed TAP tests" ${TAP_FAILED_TESTS}
     TAP_FAILED_TESTS=""
     add_footer_table "TAP logs" "${TAP_LOGS}"
   fi