You are viewing a plain text version of this content. The canonical link for it is here.
Posted to gitbox@yetus.apache.org by aw...@apache.org on 2022/05/04 17:15:46 UTC

[yetus] 11/24: Patch process shouldn't -1 documentation patch for not having any tests.

This is an automated email from the ASF dual-hosted git repository.

aw pushed a commit to tag prehistory
in repository https://gitbox.apache.org/repos/asf/yetus.git

commit a0edec78758b9804d0b07dabde8f188380a54d4c
Author: Nigel Daley <ni...@apache.org>
AuthorDate: Fri Jul 18 07:00:40 2008 +0000

    Patch process shouldn't -1 documentation patch for not having any tests.
    
    git-svn-id: https://svn.apache.org/repos/asf/hadoop/core/trunk@677839 13f79535-47bb-0310-9956-ffa450edef68
---
 src/test/bin/test-patch.sh | 10 ++++++++++
 1 file changed, 10 insertions(+)

diff --git a/src/test/bin/test-patch.sh b/src/test/bin/test-patch.sh
index 47af83cc..c4ca7fec 100755
--- a/src/test/bin/test-patch.sh
+++ b/src/test/bin/test-patch.sh
@@ -214,6 +214,16 @@ checkTests () {
   testReferences=`$GREP -c -i '/test' $PATCH_DIR/patch`
   echo "There appear to be $testReferences test files referenced in the patch."
   if [[ $testReferences == 0 ]] ; then
+    if [[ $HUDSON == "true" ]] ; then
+      patchIsDoc=`$GREP -c -i 'title="documentation' $PATCH_DIR/jira`
+      if [[ $patchIsDoc != 0 ]] ; then
+        echo "The patch appears to be a documentation patch that doesn't require tests."
+        JIRA_COMMENT="$JIRA_COMMENT
+
+    +0 tests included.  The patch appears to be a documentation patch that doesn't require tests."
+        return 0
+      fi
+    fi
     JIRA_COMMENT="$JIRA_COMMENT
 
     -1 tests included.  The patch doesn't appear to include any new or modified tests.