mirror of
https://github.com/git/git
synced 2024-08-28 12:09:19 +00:00
![David Aguilar](/assets/img/avatar_default.png)
run_merge_tool() was not setting $status, which prevented the exit code for builtin tools from being forwarded to the caller. Capture the exit status and add a test to guarantee the behavior. Reported-by: Adria Farres <14farresa@gmail.com> Signed-off-by: David Aguilar <davvid@gmail.com> Signed-off-by: Junio C Hamano <gitster@pobox.com>
508 lines
14 KiB
Bash
Executable file
508 lines
14 KiB
Bash
Executable file
#!/bin/sh
|
|
#
|
|
# Copyright (c) 2009, 2010, 2012, 2013 David Aguilar
|
|
#
|
|
|
|
test_description='git-difftool
|
|
|
|
Testing basic diff tool invocation
|
|
'
|
|
|
|
. ./test-lib.sh
|
|
|
|
difftool_test_setup ()
|
|
{
|
|
test_config diff.tool test-tool &&
|
|
test_config difftool.test-tool.cmd 'cat "$LOCAL"' &&
|
|
test_config difftool.bogus-tool.cmd false
|
|
}
|
|
|
|
prompt_given ()
|
|
{
|
|
prompt="$1"
|
|
test "$prompt" = "Launch 'test-tool' [Y/n]: branch"
|
|
}
|
|
|
|
# Create a file on master and change it on branch
|
|
test_expect_success PERL 'setup' '
|
|
echo master >file &&
|
|
git add file &&
|
|
git commit -m "added file" &&
|
|
|
|
git checkout -b branch master &&
|
|
echo branch >file &&
|
|
git commit -a -m "branch changed file" &&
|
|
git checkout master
|
|
'
|
|
|
|
# Configure a custom difftool.<tool>.cmd and use it
|
|
test_expect_success PERL 'custom commands' '
|
|
difftool_test_setup &&
|
|
test_config difftool.test-tool.cmd "cat \"\$REMOTE\"" &&
|
|
echo master >expect &&
|
|
git difftool --no-prompt branch >actual &&
|
|
test_cmp expect actual &&
|
|
|
|
test_config difftool.test-tool.cmd "cat \"\$LOCAL\"" &&
|
|
echo branch >expect &&
|
|
git difftool --no-prompt branch >actual &&
|
|
test_cmp expect actual
|
|
'
|
|
|
|
test_expect_success PERL 'custom tool commands override built-ins' '
|
|
test_config difftool.vimdiff.cmd "cat \"\$REMOTE\"" &&
|
|
echo master >expect &&
|
|
git difftool --tool vimdiff --no-prompt branch >actual &&
|
|
test_cmp expect actual
|
|
'
|
|
|
|
test_expect_success PERL 'difftool ignores bad --tool values' '
|
|
: >expect &&
|
|
test_must_fail \
|
|
git difftool --no-prompt --tool=bad-tool branch >actual &&
|
|
test_cmp expect actual
|
|
'
|
|
|
|
test_expect_success PERL 'difftool forwards arguments to diff' '
|
|
difftool_test_setup &&
|
|
>for-diff &&
|
|
git add for-diff &&
|
|
echo changes>for-diff &&
|
|
git add for-diff &&
|
|
: >expect &&
|
|
git difftool --cached --no-prompt -- for-diff >actual &&
|
|
test_cmp expect actual &&
|
|
git reset -- for-diff &&
|
|
rm for-diff
|
|
'
|
|
|
|
test_expect_success PERL 'difftool ignores exit code' '
|
|
test_config difftool.error.cmd false &&
|
|
git difftool -y -t error branch
|
|
'
|
|
|
|
test_expect_success PERL 'difftool forwards exit code with --trust-exit-code' '
|
|
test_config difftool.error.cmd false &&
|
|
test_must_fail git difftool -y --trust-exit-code -t error branch
|
|
'
|
|
|
|
test_expect_success PERL 'difftool forwards exit code with --trust-exit-code for built-ins' '
|
|
test_config difftool.vimdiff.path false &&
|
|
test_must_fail git difftool -y --trust-exit-code -t vimdiff branch
|
|
'
|
|
|
|
test_expect_success PERL 'difftool honors difftool.trustExitCode = true' '
|
|
test_config difftool.error.cmd false &&
|
|
test_config difftool.trustExitCode true &&
|
|
test_must_fail git difftool -y -t error branch
|
|
'
|
|
|
|
test_expect_success PERL 'difftool honors difftool.trustExitCode = false' '
|
|
test_config difftool.error.cmd false &&
|
|
test_config difftool.trustExitCode false &&
|
|
git difftool -y -t error branch
|
|
'
|
|
|
|
test_expect_success PERL 'difftool ignores exit code with --no-trust-exit-code' '
|
|
test_config difftool.error.cmd false &&
|
|
test_config difftool.trustExitCode true &&
|
|
git difftool -y --no-trust-exit-code -t error branch
|
|
'
|
|
|
|
test_expect_success PERL 'difftool stops on error with --trust-exit-code' '
|
|
test_when_finished "rm -f for-diff .git/fail-right-file" &&
|
|
test_when_finished "git reset -- for-diff" &&
|
|
write_script .git/fail-right-file <<-\EOF &&
|
|
echo "$2"
|
|
exit 1
|
|
EOF
|
|
>for-diff &&
|
|
git add for-diff &&
|
|
echo file >expect &&
|
|
test_must_fail git difftool -y --trust-exit-code \
|
|
--extcmd .git/fail-right-file branch >actual &&
|
|
test_cmp expect actual
|
|
'
|
|
|
|
test_expect_success PERL 'difftool honors --gui' '
|
|
difftool_test_setup &&
|
|
test_config merge.tool bogus-tool &&
|
|
test_config diff.tool bogus-tool &&
|
|
test_config diff.guitool test-tool &&
|
|
|
|
echo branch >expect &&
|
|
git difftool --no-prompt --gui branch >actual &&
|
|
test_cmp expect actual
|
|
'
|
|
|
|
test_expect_success PERL 'difftool --gui last setting wins' '
|
|
difftool_test_setup &&
|
|
: >expect &&
|
|
git difftool --no-prompt --gui --no-gui >actual &&
|
|
test_cmp expect actual &&
|
|
|
|
test_config merge.tool bogus-tool &&
|
|
test_config diff.tool bogus-tool &&
|
|
test_config diff.guitool test-tool &&
|
|
echo branch >expect &&
|
|
git difftool --no-prompt --no-gui --gui branch >actual &&
|
|
test_cmp expect actual
|
|
'
|
|
|
|
test_expect_success PERL 'difftool --gui works without configured diff.guitool' '
|
|
difftool_test_setup &&
|
|
echo branch >expect &&
|
|
git difftool --no-prompt --gui branch >actual &&
|
|
test_cmp expect actual
|
|
'
|
|
|
|
# Specify the diff tool using $GIT_DIFF_TOOL
|
|
test_expect_success PERL 'GIT_DIFF_TOOL variable' '
|
|
difftool_test_setup &&
|
|
git config --unset diff.tool &&
|
|
echo branch >expect &&
|
|
GIT_DIFF_TOOL=test-tool git difftool --no-prompt branch >actual &&
|
|
test_cmp expect actual
|
|
'
|
|
|
|
# Test the $GIT_*_TOOL variables and ensure
|
|
# that $GIT_DIFF_TOOL always wins unless --tool is specified
|
|
test_expect_success PERL 'GIT_DIFF_TOOL overrides' '
|
|
difftool_test_setup &&
|
|
test_config diff.tool bogus-tool &&
|
|
test_config merge.tool bogus-tool &&
|
|
|
|
echo branch >expect &&
|
|
GIT_DIFF_TOOL=test-tool git difftool --no-prompt branch >actual &&
|
|
test_cmp expect actual &&
|
|
|
|
test_config diff.tool bogus-tool &&
|
|
test_config merge.tool bogus-tool &&
|
|
GIT_DIFF_TOOL=bogus-tool \
|
|
git difftool --no-prompt --tool=test-tool branch >actual &&
|
|
test_cmp expect actual
|
|
'
|
|
|
|
# Test that we don't have to pass --no-prompt to difftool
|
|
# when $GIT_DIFFTOOL_NO_PROMPT is true
|
|
test_expect_success PERL 'GIT_DIFFTOOL_NO_PROMPT variable' '
|
|
difftool_test_setup &&
|
|
echo branch >expect &&
|
|
GIT_DIFFTOOL_NO_PROMPT=true git difftool branch >actual &&
|
|
test_cmp expect actual
|
|
'
|
|
|
|
# git-difftool supports the difftool.prompt variable.
|
|
# Test that GIT_DIFFTOOL_PROMPT can override difftool.prompt = false
|
|
test_expect_success PERL 'GIT_DIFFTOOL_PROMPT variable' '
|
|
difftool_test_setup &&
|
|
test_config difftool.prompt false &&
|
|
echo >input &&
|
|
GIT_DIFFTOOL_PROMPT=true git difftool branch <input >output &&
|
|
prompt=$(tail -1 <output) &&
|
|
prompt_given "$prompt"
|
|
'
|
|
|
|
# Test that we don't have to pass --no-prompt when difftool.prompt is false
|
|
test_expect_success PERL 'difftool.prompt config variable is false' '
|
|
difftool_test_setup &&
|
|
test_config difftool.prompt false &&
|
|
echo branch >expect &&
|
|
git difftool branch >actual &&
|
|
test_cmp expect actual
|
|
'
|
|
|
|
# Test that we don't have to pass --no-prompt when mergetool.prompt is false
|
|
test_expect_success PERL 'difftool merge.prompt = false' '
|
|
difftool_test_setup &&
|
|
test_might_fail git config --unset difftool.prompt &&
|
|
test_config mergetool.prompt false &&
|
|
echo branch >expect &&
|
|
git difftool branch >actual &&
|
|
test_cmp expect actual
|
|
'
|
|
|
|
# Test that the -y flag can override difftool.prompt = true
|
|
test_expect_success PERL 'difftool.prompt can overridden with -y' '
|
|
difftool_test_setup &&
|
|
test_config difftool.prompt true &&
|
|
echo branch >expect &&
|
|
git difftool -y branch >actual &&
|
|
test_cmp expect actual
|
|
'
|
|
|
|
# Test that the --prompt flag can override difftool.prompt = false
|
|
test_expect_success PERL 'difftool.prompt can overridden with --prompt' '
|
|
difftool_test_setup &&
|
|
test_config difftool.prompt false &&
|
|
echo >input &&
|
|
git difftool --prompt branch <input >output &&
|
|
prompt=$(tail -1 <output) &&
|
|
prompt_given "$prompt"
|
|
'
|
|
|
|
# Test that the last flag passed on the command-line wins
|
|
test_expect_success PERL 'difftool last flag wins' '
|
|
difftool_test_setup &&
|
|
echo branch >expect &&
|
|
git difftool --prompt --no-prompt branch >actual &&
|
|
test_cmp expect actual &&
|
|
echo >input &&
|
|
git difftool --no-prompt --prompt branch <input >output &&
|
|
prompt=$(tail -1 <output) &&
|
|
prompt_given "$prompt"
|
|
'
|
|
|
|
# git-difftool falls back to git-mergetool config variables
|
|
# so test that behavior here
|
|
test_expect_success PERL 'difftool + mergetool config variables' '
|
|
test_config merge.tool test-tool &&
|
|
test_config mergetool.test-tool.cmd "cat \$LOCAL" &&
|
|
echo branch >expect &&
|
|
git difftool --no-prompt branch >actual &&
|
|
test_cmp expect actual &&
|
|
|
|
# set merge.tool to something bogus, diff.tool to test-tool
|
|
test_config merge.tool bogus-tool &&
|
|
test_config diff.tool test-tool &&
|
|
git difftool --no-prompt branch >actual &&
|
|
test_cmp expect actual
|
|
'
|
|
|
|
test_expect_success PERL 'difftool.<tool>.path' '
|
|
test_config difftool.tkdiff.path echo &&
|
|
git difftool --tool=tkdiff --no-prompt branch >output &&
|
|
lines=$(grep file output | wc -l) &&
|
|
test "$lines" -eq 1
|
|
'
|
|
|
|
test_expect_success PERL 'difftool --extcmd=cat' '
|
|
echo branch >expect &&
|
|
echo master >>expect &&
|
|
git difftool --no-prompt --extcmd=cat branch >actual &&
|
|
test_cmp expect actual
|
|
'
|
|
|
|
test_expect_success PERL 'difftool --extcmd cat' '
|
|
echo branch >expect &&
|
|
echo master >>expect &&
|
|
git difftool --no-prompt --extcmd=cat branch >actual &&
|
|
test_cmp expect actual
|
|
'
|
|
|
|
test_expect_success PERL 'difftool -x cat' '
|
|
echo branch >expect &&
|
|
echo master >>expect &&
|
|
git difftool --no-prompt -x cat branch >actual &&
|
|
test_cmp expect actual
|
|
'
|
|
|
|
test_expect_success PERL 'difftool --extcmd echo arg1' '
|
|
echo file >expect &&
|
|
git difftool --no-prompt \
|
|
--extcmd sh\ -c\ \"echo\ \$1\" branch >actual &&
|
|
test_cmp expect actual
|
|
'
|
|
|
|
test_expect_success PERL 'difftool --extcmd cat arg1' '
|
|
echo master >expect &&
|
|
git difftool --no-prompt \
|
|
--extcmd sh\ -c\ \"cat\ \$1\" branch >actual &&
|
|
test_cmp expect actual
|
|
'
|
|
|
|
test_expect_success PERL 'difftool --extcmd cat arg2' '
|
|
echo branch >expect &&
|
|
git difftool --no-prompt \
|
|
--extcmd sh\ -c\ \"cat\ \$2\" branch >actual &&
|
|
test_cmp expect actual
|
|
'
|
|
|
|
# Create a second file on master and a different version on branch
|
|
test_expect_success PERL 'setup with 2 files different' '
|
|
echo m2 >file2 &&
|
|
git add file2 &&
|
|
git commit -m "added file2" &&
|
|
|
|
git checkout branch &&
|
|
echo br2 >file2 &&
|
|
git add file2 &&
|
|
git commit -a -m "branch changed file2" &&
|
|
git checkout master
|
|
'
|
|
|
|
test_expect_success PERL 'say no to the first file' '
|
|
(echo n && echo) >input &&
|
|
git difftool -x cat branch <input >output &&
|
|
grep m2 output &&
|
|
grep br2 output &&
|
|
! grep master output &&
|
|
! grep branch output
|
|
'
|
|
|
|
test_expect_success PERL 'say no to the second file' '
|
|
(echo && echo n) >input &&
|
|
git difftool -x cat branch <input >output &&
|
|
grep master output &&
|
|
grep branch output &&
|
|
! grep m2 output &&
|
|
! grep br2 output
|
|
'
|
|
|
|
test_expect_success PERL 'ending prompt input with EOF' '
|
|
git difftool -x cat branch </dev/null >output &&
|
|
! grep master output &&
|
|
! grep branch output &&
|
|
! grep m2 output &&
|
|
! grep br2 output
|
|
'
|
|
|
|
test_expect_success PERL 'difftool --tool-help' '
|
|
git difftool --tool-help >output &&
|
|
grep tool output
|
|
'
|
|
|
|
test_expect_success PERL 'setup change in subdirectory' '
|
|
git checkout master &&
|
|
mkdir sub &&
|
|
echo master >sub/sub &&
|
|
git add sub/sub &&
|
|
git commit -m "added sub/sub" &&
|
|
echo test >>file &&
|
|
echo test >>sub/sub &&
|
|
git add file sub/sub &&
|
|
git commit -m "modified both"
|
|
'
|
|
|
|
run_dir_diff_test () {
|
|
test_expect_success PERL "$1 --no-symlinks" "
|
|
symlinks=--no-symlinks &&
|
|
$2
|
|
"
|
|
test_expect_success PERL,SYMLINKS "$1 --symlinks" "
|
|
symlinks=--symlinks &&
|
|
$2
|
|
"
|
|
}
|
|
|
|
run_dir_diff_test 'difftool -d' '
|
|
git difftool -d $symlinks --extcmd ls branch >output &&
|
|
grep sub output &&
|
|
grep file output
|
|
'
|
|
|
|
run_dir_diff_test 'difftool --dir-diff' '
|
|
git difftool --dir-diff $symlinks --extcmd ls branch >output &&
|
|
grep sub output &&
|
|
grep file output
|
|
'
|
|
|
|
run_dir_diff_test 'difftool --dir-diff ignores --prompt' '
|
|
git difftool --dir-diff $symlinks --prompt --extcmd ls branch >output &&
|
|
grep sub output &&
|
|
grep file output
|
|
'
|
|
|
|
run_dir_diff_test 'difftool --dir-diff from subdirectory' '
|
|
(
|
|
cd sub &&
|
|
git difftool --dir-diff $symlinks --extcmd ls branch >output &&
|
|
grep sub output &&
|
|
grep file output
|
|
)
|
|
'
|
|
|
|
run_dir_diff_test 'difftool --dir-diff when worktree file is missing' '
|
|
test_when_finished git reset --hard &&
|
|
rm file2 &&
|
|
git difftool --dir-diff $symlinks --extcmd ls branch master >output &&
|
|
grep file2 output
|
|
'
|
|
|
|
write_script .git/CHECK_SYMLINKS <<\EOF
|
|
for f in file file2 sub/sub
|
|
do
|
|
echo "$f"
|
|
readlink "$2/$f"
|
|
done >actual
|
|
EOF
|
|
|
|
test_expect_success PERL,SYMLINKS 'difftool --dir-diff --symlink without unstaged changes' '
|
|
cat >expect <<-EOF &&
|
|
file
|
|
$(pwd)/file
|
|
file2
|
|
$(pwd)/file2
|
|
sub/sub
|
|
$(pwd)/sub/sub
|
|
EOF
|
|
git difftool --dir-diff --symlink \
|
|
--extcmd "./.git/CHECK_SYMLINKS" branch HEAD &&
|
|
test_cmp actual expect
|
|
'
|
|
|
|
write_script modify-right-file <<\EOF
|
|
echo "new content" >"$2/file"
|
|
EOF
|
|
|
|
run_dir_diff_test 'difftool --dir-diff syncs worktree with unstaged change' '
|
|
test_when_finished git reset --hard &&
|
|
echo "orig content" >file &&
|
|
git difftool -d $symlinks --extcmd "$(pwd)/modify-right-file" branch &&
|
|
echo "new content" >expect &&
|
|
test_cmp expect file
|
|
'
|
|
|
|
run_dir_diff_test 'difftool --dir-diff syncs worktree without unstaged change' '
|
|
test_when_finished git reset --hard &&
|
|
git difftool -d $symlinks --extcmd "$(pwd)/modify-right-file" branch &&
|
|
echo "new content" >expect &&
|
|
test_cmp expect file
|
|
'
|
|
|
|
write_script modify-file <<\EOF
|
|
echo "new content" >file
|
|
EOF
|
|
|
|
test_expect_success PERL 'difftool --no-symlinks does not overwrite working tree file ' '
|
|
echo "orig content" >file &&
|
|
git difftool --dir-diff --no-symlinks --extcmd "$(pwd)/modify-file" branch &&
|
|
echo "new content" >expect &&
|
|
test_cmp expect file
|
|
'
|
|
|
|
write_script modify-both-files <<\EOF
|
|
echo "wt content" >file &&
|
|
echo "tmp content" >"$2/file" &&
|
|
echo "$2" >tmpdir
|
|
EOF
|
|
|
|
test_expect_success PERL 'difftool --no-symlinks detects conflict ' '
|
|
(
|
|
TMPDIR=$TRASH_DIRECTORY &&
|
|
export TMPDIR &&
|
|
echo "orig content" >file &&
|
|
test_must_fail git difftool --dir-diff --no-symlinks --extcmd "$(pwd)/modify-both-files" branch &&
|
|
echo "wt content" >expect &&
|
|
test_cmp expect file &&
|
|
echo "tmp content" >expect &&
|
|
test_cmp expect "$(cat tmpdir)/file"
|
|
)
|
|
'
|
|
|
|
test_expect_success PERL 'difftool properly honors gitlink and core.worktree' '
|
|
git submodule add ./. submod/ule &&
|
|
(
|
|
cd submod/ule &&
|
|
test_config diff.tool checktrees &&
|
|
test_config difftool.checktrees.cmd '\''
|
|
test -d "$LOCAL" && test -d "$REMOTE" && echo good
|
|
'\'' &&
|
|
echo good >expect &&
|
|
git difftool --tool=checktrees --dir-diff HEAD~ >actual &&
|
|
test_cmp expect actual
|
|
)
|
|
'
|
|
|
|
test_done
|