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

[PATCH 3/5] t7610-mergetool: add test cases for mergetool.writeToTemp

From
David Aguilar <davvid@gmail.com>
Date
Oct 15, 2014, 08:35 UTC
Message-ID
<1413362121-17361-3-git-send-email-davvid@gmail.com>
In-Reply-To
<1413362121-17361-2-git-send-email-davvid@gmail.com>

Add tests to ensure that filenames start with "./" when mergetool.writeToTemp is false and do not start with "./" when mergetool.writeToTemp is true.

Signed-off-by: David Aguilar <davvid@gmail.com>
---
 t/t7610-mergetool.sh | 23 +++++++++++++++++++++++
 1 file changed, 23 insertions(+)
diff --git a/t/t7610-mergetool.sh b/t/t7610-mergetool.sh
index 214edfb..1a15e06 100755
--- a/t/t7610-mergetool.sh
+++ b/t/t7610-mergetool.sh
@@ -516,4 +516,27 @@ test_expect_success 'custom commands override built-ins' '
 	git reset --hard master >/dev/null 2>&1
 '
 
+test_expect_success 'filenames seen by tools start with ./' '
+	git checkout -b test15 branch1 &&
+	test_config mergetool.writeToTemp false &&
+	test_config mergetool.myecho.cmd "echo \"\$LOCAL\"" &&
+	test_config mergetool.myecho.trustExitCode true &&
+	test_must_fail git merge master &&
+	git mergetool --no-prompt --tool myecho -- both >actual &&
+	grep ^\./both_LOCAL_ actual >/dev/null &&
+	git reset --hard master >/dev/null 2>&1
+'
+
+test_expect_success 'temporary filenames are used with mergetool.writeToTemp' '
+	git checkout -b test16 branch1 &&
+	test_config mergetool.writeToTemp true &&
+	test_config mergetool.myecho.cmd "echo \"\$LOCAL\"" &&
+	test_config mergetool.myecho.trustExitCode true &&
+	test_must_fail git merge master &&
+	git mergetool --no-prompt --tool myecho -- both >actual &&
+	test_must_fail grep ^\./both_LOCAL_ actual >/dev/null &&
+	grep /both_LOCAL_ actual >/dev/null &&
+	git reset --hard master >/dev/null 2>&1
+'
+
 test_done
-- 
2.1.2.453.g1b015e3
Previous: David AguilarNext: David Aguilar
Message 3 of 7 in “t7610-mergetool: use tabs instead of a mix of tabs and spaces”
  1. 1/5 t7610-mergetool: use tabs instead of a mix of tabs and spacesDavid Aguilar, Oct 15, 2014
  2. 2/5 t7610-mergetool: add missing && and remove commented-out codeDavid Aguilar, Oct 15, 2014
  3. 3/5 t7610-mergetool: add test cases for mergetool.writeToTempDavid Aguilar, Oct 15, 2014
  4. 4/5 t7610-mergetool: prefer test_config over git configDavid Aguilar, Oct 15, 2014
  5. 5/5 test-lib-functions: adjust style to match CodingGuidelinesDavid Aguilar, Oct 15, 2014
  6. Junio C HamanoOct 15, 2014
  7. 4/5 t7610-mergetool: use test_config to isolate testsDavid Aguilar, Oct 16, 2014

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.