On Thu, Mar 9, 2017 at 8:03 PM, Prathamesh Chavan <pc44800@xxxxxxxxx> wrote: > From: Prathamesh <pc44800@xxxxxxxxx> > > Whenever a git command is present in the upstream of a pipe, its failure > gets masked by piping and hence it should be avoided for testing the > upstream git command. By writing out the output of the git command to > a file, we can test the exit codes of both the commands as a failure exit > code in any command is able to stop the && chain. > > Signed-off-by: Prathamesh <pc44800@xxxxxxxxx> > --- When you post a new version of a patch or a patch series, could you do the following: - add v2 or v3, or ... after "PATCH" in the subject, so that we know which version it is (see the mailing list archive and the format-patch documentation to see how it should appear and how to do it) - tell what you changed since the previous version, and maybe also why you made those changes, if it has not already been explained or discussed (you can do it after the "---" above and before the file stats below, or in a separate email replying to the patch, or in the cover letter of the patch series if you are sending a patch series) > t/t2027-worktree-list.sh | 18 +++++++++++------- > 1 file changed, 11 insertions(+), 7 deletions(-) > > diff --git a/t/t2027-worktree-list.sh b/t/t2027-worktree-list.sh > index 848da5f36..d8b3907e0 100755 > --- a/t/t2027-worktree-list.sh > +++ b/t/t2027-worktree-list.sh > @@ -31,7 +31,8 @@ test_expect_success '"list" all worktrees from main' ' > test_when_finished "rm -rf here && git worktree prune" && > git worktree add --detach here master && > echo "$(git -C here rev-parse --show-toplevel) $(git rev-parse --short HEAD) (detached HEAD)" >>expect && > - git worktree list | sed "s/ */ /g" >actual && > + git worktree list >out && > + sed "s/ */ /g" <out >actual && > test_cmp expect actual > ' [...] > @@ -118,9 +122,9 @@ test_expect_success 'broken main worktree still at the top' ' > cd linked && > echo "worktree $(pwd)" >expected && > echo "ref: .broken" >../.git/HEAD && > - git worktree list --porcelain | head -n 3 >actual && > + git worktree list --porcelain >out && head -n 3 out >actual && > test_cmp ../expected actual && > - git worktree list | head -n 1 >actual.2 && > + git worktree list >out && head -n 1 out >actual.2 && I think it would be better if the 'head' commands above and the 'grep' command below were also on their own line. > grep -F "(error)" actual.2 > ) > ' > @@ -134,7 +138,7 @@ test_expect_success 'linked worktrees are sorted' ' > test_commit new && > git worktree add ../first && > git worktree add ../second && > - git worktree list --porcelain | grep ^worktree >actual > + git worktree list --porcelain >out && grep ^worktree out >actual > ) && > cat >expected <<-EOF && > worktree $(pwd)/sorted/main > -- > 2.11.0 >