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 bo...@apache.org on 2012/04/05 15:57:09 UTC

svn commit: r1309864 - /hadoop/common/trunk/dev-support/test-patch.sh

Author: bobby
Date: Thu Apr  5 13:57:09 2012
New Revision: 1309864

URL: http://svn.apache.org/viewvc?rev=1309864&view=rev
Log:
HADOOP-7757. Test file reference count is at least 3x actual value (Jon Eagles via bobby)

Modified:
    hadoop/common/trunk/dev-support/test-patch.sh

Modified: hadoop/common/trunk/dev-support/test-patch.sh
URL: http://svn.apache.org/viewvc/hadoop/common/trunk/dev-support/test-patch.sh?rev=1309864&r1=1309863&r2=1309864&view=diff
==============================================================================
--- hadoop/common/trunk/dev-support/test-patch.sh (original)
+++ hadoop/common/trunk/dev-support/test-patch.sh Thu Apr  5 13:57:09 2012
@@ -311,7 +311,7 @@ checkTests () {
   echo "======================================================================"
   echo ""
   echo ""
-  testReferences=`$GREP -c -i '/test' $PATCH_DIR/patch`
+  testReferences=`$GREP -c -i -e '^+++.*/test' $PATCH_DIR/patch`
   echo "There appear to be $testReferences test files referenced in the patch."
   if [[ $testReferences == 0 ]] ; then
     if [[ $JENKINS == "true" ]] ; then
@@ -333,7 +333,7 @@ checkTests () {
   fi
   JIRA_COMMENT="$JIRA_COMMENT
 
-    +1 tests included.  The patch appears to include $testReferences new or modified tests."
+    +1 tests included.  The patch appears to include $testReferences new or modified test files."
   return 0
 }