]> git.ipfire.org Git - thirdparty/git.git/commitdiff
git-p4: add failing test for P4EDITOR handling
authorLuke Diamand <luke@diamand.org>
Tue, 19 May 2015 22:23:16 +0000 (23:23 +0100)
committerJunio C Hamano <gitster@pobox.com>
Sun, 24 May 2015 18:50:12 +0000 (11:50 -0700)
Add test case that git-p4 handles a setting of P4EDITOR
that takes arguments, e.g. "gvim -f". This currently fails.

Signed-off-by: Luke Diamand <luke@diamand.org>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
t/t9820-git-p4-editor-handling.sh [new file with mode: 0755]

diff --git a/t/t9820-git-p4-editor-handling.sh b/t/t9820-git-p4-editor-handling.sh
new file mode 100755 (executable)
index 0000000..abaa066
--- /dev/null
@@ -0,0 +1,38 @@
+#!/bin/sh
+
+test_description='git p4 handling of EDITOR'
+
+. ./lib-git-p4.sh
+
+test_expect_success 'start p4d' '
+       start_p4d
+'
+
+test_expect_success 'init depot' '
+       (
+               cd "$cli" &&
+               echo file1 >file1 &&
+               p4 add file1 &&
+               p4 submit -d "file1"
+       )
+'
+
+# Check that the P4EDITOR argument can be given command-line
+# options, which git-p4 will then pass through to the shell.
+test_expect_failure 'EDITOR with options' '
+       git p4 clone --dest="$git" //depot &&
+       test_when_finished cleanup_git &&
+       (
+               cd "$git" &&
+               echo change >file1 &&
+               git commit -m "change" file1 &&
+               P4EDITOR=": >\"$git/touched\" && test-chmtime +5" git p4 submit &&
+               test_path_is_file "$git/touched"
+       )
+'
+
+test_expect_success 'kill p4d' '
+       kill_p4d
+'
+
+test_done