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 to...@apache.org on 2011/10/19 01:49:33 UTC

svn commit: r1185923 - /hadoop/common/branches/branch-0.23/dev-support/test-patch.sh

Author: tomwhite
Date: Tue Oct 18 23:49:33 2011
New Revision: 1185923

URL: http://svn.apache.org/viewvc?rev=1185923&view=rev
Log:
Merge -r 1185921:1185922 from trunk to branch-0.23. Fixes:HADOOP-7755.

Modified:
    hadoop/common/branches/branch-0.23/dev-support/test-patch.sh

Modified: hadoop/common/branches/branch-0.23/dev-support/test-patch.sh
URL: http://svn.apache.org/viewvc/hadoop/common/branches/branch-0.23/dev-support/test-patch.sh?rev=1185923&r1=1185922&r2=1185923&view=diff
==============================================================================
--- hadoop/common/branches/branch-0.23/dev-support/test-patch.sh (original)
+++ hadoop/common/branches/branch-0.23/dev-support/test-patch.sh Tue Oct 18 23:49:33 2011
@@ -595,20 +595,23 @@ runTests () {
   echo "======================================================================"
   echo ""
   echo ""
-  
+
   echo "$MVN clean install test -Pnative -D${PROJECT_NAME}PatchProcess"
   $MVN clean install test -Pnative -D${PROJECT_NAME}PatchProcess
   if [[ $? != 0 ]] ; then
     ### Find and format names of failed tests
     failed_tests=`find . -name 'TEST*.xml' | xargs $GREP  -l -E "<failure|<error" | sed -e "s|.*target/surefire-reports/TEST-|                  |g" | sed -e "s|\.xml||g"`
-  fi
-  
-  if [[ -n "$failed_tests" ]] ; then
-  
-    JIRA_COMMENT="$JIRA_COMMENT
+
+    if [[ -n "$failed_tests" ]] ; then
+      JIRA_COMMENT="$JIRA_COMMENT
 
     -1 core tests.  The patch failed these unit tests:
 $failed_tests"
+    else
+      JIRA_COMMENT="$JIRA_COMMENT
+
+    -1 core tests.  The patch failed the unit tests build"
+    fi
     return 1
   fi
   JIRA_COMMENT="$JIRA_COMMENT