Merge pull request EFForg#3448 from jwilk/test-rules-heredoc

semenko · semenko · commit 744b9d62a5c7 · 2015-12-16T18:06:48.000-06:00
test/rules.sh: Use heredoc for printing multi-line message
diff --git a/test/rules.sh b/test/rules.sh
@@ -29,14 +29,16 @@ if [ $# -gt 0 ] ; then
 fi
 if ! python2.7 test/rules/src/https_everywhere_checker/check_rules.py \
       test/rules/coverage.checker.config; then
-  echo "Ruleset test coverage was insufficient."
-  echo ""
-  echo "Under the new ruleset testing rules (February 2015), any modified ruleset"
-  echo "must have sufficient test coverage. You can often improve test coverage by"
-  echo "adding <test url='...' /> tags, or by restructuring the rule to avoid"
-  echo "wildcard <target> tags. See these documents:"
-  echo "https://github.com/EFForg/https-everywhere/blob/master/ruleset-testing.md"
-  echo "https://github.com/EFForg/https-everywhere/blob/master/ruleset-style.md"
+  cat <<EOF
+Ruleset test coverage was insufficient.
+
+Under the new ruleset testing rules (February 2015), any modified ruleset
+must have sufficient test coverage. You can often improve test coverage by
+adding <test url='...' /> tags, or by restructuring the rule to avoid
+wildcard <target> tags. See these documents:
+https://github.com/EFForg/https-everywhere/blob/master/ruleset-testing.md
+https://github.com/EFForg/https-everywhere/blob/master/ruleset-style.md
+EOF
   exit 1
 else
   exit 0
