git/list[1] front-page[2] threads[3] people[4] search[5] about
 

[PATCH v6 2/4] commit test: Use write_script

From
Caleb Thompson <caleb@calebthompson.io>
Date
Jun 17, 2014, 19:38 UTC
Message-ID
<fa7260f688bccac09138c8ef5ef1982cee0f0dde.1403033723.git.caleb@calebthompson.io>
In-Reply-To
<cover.1403033723.git.caleb@calebthompson.io>

Use write_script from t/test-lib-functions.sh instead of cat, shebang, and chmod. This aids in readability for creating the script by using the named function and allows us to turn off interpolation in the heredoc of the script body to avoid extra escaping, since $SHELL_PATH is handled for us.

Signed-off-by: Caleb Thompson <caleb@calebthompson.io>
Reviewed-by: Jeremiah Mahler <jmmahler@gmail.com>
---
 t/t7507-commit-verbose.sh | 6 ++----
 1 file changed, 2 insertions(+), 4 deletions(-)
diff --git a/t/t7507-commit-verbose.sh b/t/t7507-commit-verbose.sh
index 6d778ed..db09107 100755
--- a/t/t7507-commit-verbose.sh
+++ b/t/t7507-commit-verbose.sh
@@ -3,11 +3,9 @@
 test_description='verbose commit template'
 . ./test-lib.sh
 
-cat >check-for-diff <<EOF
-#!$SHELL_PATH
-exec grep '^diff --git' "\$1"
+write_script check-for-diff <<-'EOF'
+	exec grep '^diff --git' "$1"
 EOF
-chmod +x check-for-diff
 test_set_editor "$PWD/check-for-diff"
 
 cat >message <<'EOF'
-- 
2.0.0
Previous: Junio C HamanoNext: Caleb Thompson
Message 5 of 12 in “commit: Add commit.verbose configuration”
  1. 0/4 commit: Add commit.verbose configurationCaleb Thompson, Jun 17, 2014
  2. 1/4 commit test: Use test_config instead of git-configCaleb Thompson, Jun 17, 2014
  3. Junio C HamanoJun 17, 2014
  4. Junio C HamanoJun 17, 2014
  5. 2/4 commit test: Use write_scriptCaleb Thompson, Jun 17, 2014
  6. 3/4 commit test: test_set_editor in each testCaleb Thompson, Jun 17, 2014
  7. 4/4 commit: Add commit.verbose configurationCaleb Thompson, Jun 17, 2014
  8. Jeff KingJun 17, 2014
  9. Junio C HamanoJun 17, 2014
  10. Jeff KingJun 17, 2014
  11. Torstein HeggeFeb 27, 2015
  12. Caleb ThompsonMar 2, 2015

Read the whole thread, see it on lore, or plain text.

$ cat FOOTERMessages come from the public archive at lore.kernel.org/git, fetched every hour. The front page is chosen and written each morning by an AI editor and can be wrong; the threads themselves are the record. About and API. For agents: an MCP server at https://gitlist.dev/mcp, and any thread, story or person page as Markdown by adding .md to its URL (or sending Accept: text/markdown). Details in /llms.txt.