Jeff King <peff@xxxxxxxx> writes: > ... At any rate, since reuse_worktree_file() > is meant to be an optimization that may or may not trigger, our test > should be robust either way. > > Instead of checking the filename, let's just make sure we got a single > line of output (which would not be true if we continued after the first > failure). Makes sense. Thanks for spotting. > Signed-off-by: Jeff King <peff@xxxxxxxx> > --- > t/t7800-difftool.sh | 5 ++--- > 1 file changed, 2 insertions(+), 3 deletions(-) > > diff --git a/t/t7800-difftool.sh b/t/t7800-difftool.sh > index 6bac9ed180..29b92907e2 100755 > --- a/t/t7800-difftool.sh > +++ b/t/t7800-difftool.sh > @@ -125,15 +125,14 @@ test_expect_success '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" > + echo failed > 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_line_count = 1 actual > ' > > test_expect_success 'difftool honors exit status if command not found' '
