From mboxrd@z Thu Jan 1 00:00:00 1970 From: Jonathan Nieder Subject: [PATCH v2] t7610 (mergetool): more nitpicks Date: Tue, 24 Aug 2010 19:25:52 -0500 Message-ID: <20100825002552.GI2376@burratino> References: <1282617444-641-1-git-send-email-brian@gernhardtsoftware.com> <20100824030524.GF17406@burratino> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Cc: Git List , Junio C Hamano , Charles Bailey To: Brian Gernhardt X-From: git-owner@vger.kernel.org Wed Aug 25 02:27:43 2010 Return-path: Envelope-to: gcvg-git-2@lo.gmane.org Received: from vger.kernel.org ([209.132.180.67]) by lo.gmane.org with esmtp (Exim 4.69) (envelope-from ) id 1Oo3qA-0000Up-2o for gcvg-git-2@lo.gmane.org; Wed, 25 Aug 2010 02:27:42 +0200 Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1754321Ab0HYA1h (ORCPT ); Tue, 24 Aug 2010 20:27:37 -0400 Received: from mail-qw0-f46.google.com ([209.85.216.46]:41709 "EHLO mail-qw0-f46.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752464Ab0HYA1f (ORCPT ); Tue, 24 Aug 2010 20:27:35 -0400 Received: by qwh6 with SMTP id 6so6669064qwh.19 for ; Tue, 24 Aug 2010 17:27:34 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=gamma; h=domainkey-signature:received:received:date:from:to:cc:subject :message-id:references:mime-version:content-type:content-disposition :in-reply-to:user-agent; bh=8r8ZsGwILP7del9ppni47Oql0P/y5y9OzHVP1QPX8VM=; b=PK/VZ/c6ga6iZnG6lH9QuaoKI7ayDoaJ0XX4q92JEiq3tF7ZBbFAQEaO0JggpmdFFS 4DS8dqxwSypKs7kjLcpOb2z0rhLKNID7QcEC7hwaIVvaK+sWKTIc8vvj2WI6i7kISX+i YAklGFAHMKK2/CJeUzterM7fayE/524+m2K1o= DomainKey-Signature: a=rsa-sha1; c=nofws; d=gmail.com; s=gamma; h=date:from:to:cc:subject:message-id:references:mime-version :content-type:content-disposition:in-reply-to:user-agent; b=fxvr9Lzroc199MxykIVAiwJhNWYu+E1y8QNmW+lkrylb+ZVa+kokpnx9l8FaIZh7iS yCqx/4es0fObaEG9HTS95HAl8l1TcIOiwv+LxFjpXDTB0HMbsXKAmMs9Pz8TEq+HYgQC qS42bkpiUAHntBX4qDGaJ9uO/hbUdWyrO2QZc= Received: by 10.229.189.74 with SMTP id dd10mr5486874qcb.73.1282696054523; Tue, 24 Aug 2010 17:27:34 -0700 (PDT) Received: from burratino (dhcp-11-17.cs.uchicago.edu [128.135.11.176]) by mx.google.com with ESMTPS id f15sm818538qcr.1.2010.08.24.17.27.33 (version=SSLv3 cipher=RC4-MD5); Tue, 24 Aug 2010 17:27:33 -0700 (PDT) Content-Disposition: inline In-Reply-To: <20100824030524.GF17406@burratino> User-Agent: Mutt/1.5.20 (2009-06-14) Sender: git-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: git@vger.kernel.org Archived-At: - use tabs to indent - do not redirect output away unnecessarily - avoid a subshell for 'yes "" | git mergetool file3' - use test_tick for reproducible, increasing timestamps - use test_cmp instead of 'test $foo = bar'; the former is much nicer to debug with --verbose since it produces a diff. Helped-by: Junio C Hamano Signed-off-by: Jonathan Nieder --- Jonathan Nieder wrote: > test_expect_success 'mergetool crlf' ' > git config core.autocrlf true && > + test_when_finished "git config --unset core.autocrlf" && > + echo master updated | append_cr >file1.expected && > + echo master new | append_cr >file2.expected && > + echo master new sub | append_cr >sub.expected && > + > - git checkout -b test2 branch1 > + git checkout -b test2 branch1 && > - test_must_fail git merge master >/dev/null 2>&1 && > + test_must_fail git merge master && > + test_when_finished "git reset --hard" && > - ( yes "" | git mergetool file1 >/dev/null 2>&1 ) && > - ( yes "" | git mergetool file2 >/dev/null 2>&1 ) && > - ( yes "" | git mergetool subdir/file3 >/dev/null 2>&1 ) && > + yes "" | git mergetool file1 && > + yes "" | git mergetool file2 && > + yes "" | git mergetool subdir/file3 && > - test "$(printf x | cat file1 -)" = "$(printf "master updated\r\nx")" && > - test "$(printf x | cat file2 -)" = "$(printf "master new\r\nx")" && > - test "$(printf x | cat subdir/file3 -)" = "$(printf "master new sub\r\nx")" && > + > + test_cmp file1.expected file1 && > + test_cmp file2.expected file2 && > + test_cmp sub.expected subdir/file3 && > - git commit -m "branch1 resolved with mergetool - autocrlf" && > + > + git commit -m "branch1 resolved with mergetool - autocrlf" > - git config core.autocrlf false && > - git reset --hard > ' As Junio noticed, the net effect is to run "reset --hard" before the "[core] autocrlf" configuration is removed, which could be an undesirable behavior change. So this version runs "reset --hard" after unsetting autocrlf for good measure. t/t7610-mergetool.sh | 187 +++++++++++++++++++++++++++++--------------------- 1 files changed, 108 insertions(+), 79 deletions(-) diff --git a/t/t7610-mergetool.sh b/t/t7610-mergetool.sh index 3bd7404..a7f05b0 100755 --- a/t/t7610-mergetool.sh +++ b/t/t7610-mergetool.sh @@ -3,112 +3,141 @@ # Copyright (c) 2008 Charles Bailey # -test_description='git mergetool +test_description='Testing basic merge tool invocation -Testing basic merge tool invocation' +All the mergetool tests work by checking out a temporary branch based +off branch1 and then merging in master and checking the results of +running mergetool. +' . ./test-lib.sh -# All the mergetool test work by checking out a temporary branch based -# off 'branch1' and then merging in master and checking the results of -# running mergetool - test_expect_success 'setup' ' - git config rerere.enabled true && - echo master >file1 && - mkdir subdir && - echo master sub >subdir/file3 && - git add file1 subdir/file3 && - git commit -m "added file1" && + git config rerere.enabled true && + echo master >file1 && + mkdir subdir && + echo master sub >subdir/file3 && + git add file1 subdir/file3 && + test_tick && + git commit -m "added file1" && - git checkout -b branch1 master && - echo branch1 change >file1 && - echo branch1 newfile >file2 && - echo branch1 sub >subdir/file3 && - git add file1 file2 subdir/file3 && - git commit -m "branch1 changes" && + git checkout -b branch1 master && + echo branch1 change >file1 && + echo branch1 newfile >file2 && + echo branch1 sub >subdir/file3 && + git add file1 file2 subdir/file3 && + test_tick && + git commit -m "branch1 changes" && - git checkout master && - echo master updated >file1 && - echo master new >file2 && - echo master new sub >subdir/file3 && - git add file1 file2 subdir/file3 && - git commit -m "master updates" && + git checkout master && + echo master updated >file1 && + echo master new >file2 && + echo master new sub >subdir/file3 && + git add file1 file2 subdir/file3 && + test_tick && + git commit -m "master updates" && - git config merge.tool mytool && - git config mergetool.mytool.cmd "cat \"\$REMOTE\" >\"\$MERGED\"" && - git config mergetool.mytool.trustExitCode true + git config merge.tool mytool && + git config mergetool.mytool.cmd "cat \"\$REMOTE\" >\"\$MERGED\"" && + git config mergetool.mytool.trustExitCode true ' test_expect_success 'custom mergetool' ' - git checkout -b test1 branch1 && - test_must_fail git merge master >/dev/null 2>&1 && - ( yes "" | git mergetool file1 >/dev/null 2>&1 ) && - ( yes "" | git mergetool file2 >/dev/null 2>&1 ) && - ( yes "" | git mergetool subdir/file3 >/dev/null 2>&1 ) && - test "$(cat file1)" = "master updated" && - test "$(cat file2)" = "master new" && - test "$(cat subdir/file3)" = "master new sub" && - git commit -m "branch1 resolved with mergetool" + echo master updated >file1.expected && + echo master new >file2.expected && + echo master new sub >sub.expected && + + git checkout -b test1 branch1 && + test_must_fail git merge master && + yes "" | git mergetool file1 && + yes "" | git mergetool file2 && + yes "" | git mergetool subdir/file3 && + + test_cmp file1.expected file1 && + test_cmp file2.expected file2 && + test_cmp sub.expected subdir/file3 && + + git commit -m "branch1 resolved with mergetool" ' test_expect_success 'mergetool crlf' ' - git config core.autocrlf true && - git checkout -b test2 branch1 - test_must_fail git merge master >/dev/null 2>&1 && - ( yes "" | git mergetool file1 >/dev/null 2>&1 ) && - ( yes "" | git mergetool file2 >/dev/null 2>&1 ) && - ( yes "" | git mergetool subdir/file3 >/dev/null 2>&1 ) && - test "$(printf x | cat file1 -)" = "$(printf "master updated\r\nx")" && - test "$(printf x | cat file2 -)" = "$(printf "master new\r\nx")" && - test "$(printf x | cat subdir/file3 -)" = "$(printf "master new sub\r\nx")" && - git commit -m "branch1 resolved with mergetool - autocrlf" && - git config core.autocrlf false && - git reset --hard + git config core.autocrlf true && + test_when_finished " + git config --unset core.autocrlf && + git reset --hard + " && + echo master updated | append_cr >file1.expected && + echo master new | append_cr >file2.expected && + echo master new sub | append_cr >sub.expected && + + git checkout -b test2 branch1 && + test_must_fail git merge master && + test_when_finished "git reset --hard" && + yes "" | git mergetool file1 && + yes "" | git mergetool file2 && + yes "" | git mergetool subdir/file3 && + + test_cmp file1.expected file1 && + test_cmp file2.expected file2 && + test_cmp sub.expected subdir/file3 && + + git commit -m "branch1 resolved with mergetool - autocrlf" ' test_expect_success 'mergetool in subdir' ' - git checkout -b test3 branch1 && - ( - cd subdir && - test_must_fail git merge master >/dev/null 2>&1 && - ( yes "" | git mergetool file3 >/dev/null 2>&1 ) && - test "$(cat file3)" = "master new sub" - ) + echo master new sub >sub.expected && + git checkout -b test3 branch1 && + ( + cd subdir && + test_must_fail git merge master && + yes "" | git mergetool file3 && + test_cmp ../sub.expected file3 + ) ' test_expect_success 'mergetool on file in parent dir' ' - ( - cd subdir && - ( yes "" | git mergetool ../file1 >/dev/null 2>&1 ) && - ( yes "" | git mergetool ../file2 >/dev/null 2>&1 ) && - test "$(cat ../file1)" = "master updated" && - test "$(cat ../file2)" = "master new" && - git commit -m "branch1 resolved with mergetool - subdir" - ) + echo master updated >file1.expected && + echo master new >file2.expected && + ( + cd subdir && + yes "" | git mergetool ../file1 && + yes "" | git mergetool ../file2 && + test_cmp ../file1.expected ../file1 && + test_cmp ../file2.expected ../file2 && + git commit -m "branch1 resolved with mergetool - subdir" + ) ' test_expect_success 'mergetool skips autoresolved' ' - git checkout -b test4 branch1 && - test_must_fail git merge master && - test -n "$(git ls-files -u)" && - output="$(git mergetool --no-prompt)" && - test "$output" = "No files need merging" && - git reset --hard + echo "No files need merging" >expected && + git checkout -b test4 branch1 && + test_must_fail git merge master && + test_when_finished "git reset --hard" && + test_must_fail git update-index --refresh && + git mergetool --no-prompt >actual && + test_cmp expected actual ' test_expect_success 'mergetool merges all from subdir' ' - ( - cd subdir && + echo master updated >file1.expected && + echo master new >file2.expected && + echo master new sub >sub.expected && + git config rerere.enabled false && - test_must_fail git merge master && - git mergetool --no-prompt && - test "$(cat ../file1)" = "master updated" && - test "$(cat ../file2)" = "master new" && - test "$(cat file3)" = "master new sub" && - git add ../file1 ../file2 file3 && - git commit -m "branch2 resolved by mergetool from subdir" - ) + test_when_finished "git config rerere.enabled true" && + mv .git/rr-cache .git/rr-cache-moved && + test_when_finished "mv .git/rr-cache-moved .git/rr-cache" && + ( + cd subdir && + test_must_fail git merge master && + test_when_finished "git reset --hard" && + git mergetool --no-prompt && + test_cmp ../file1.expected ../file1 && + test_cmp ../file2.expected ../file2 && + test_cmp ../sub.expected file3 && + git add ../file1 ../file2 file3 && + git commit -m "branch2 resolved by mergetool from subdir" + ) ' test_done -- 1.7.2.2