From mboxrd@z Thu Jan 1 00:00:00 1970 From: Stephan Beyer Subject: [PATCH 1/2] t3404: extra checks and s/! git/test_must_fail git/ Date: Fri, 20 Jun 2008 20:30:13 +0200 Message-ID: <1213986614-19536-1-git-send-email-s-beyer@gmx.net> References: Cc: Johannes Schindelin , Christian Couder , Stephan Beyer To: git@vger.kernel.org X-From: git-owner@vger.kernel.org Fri Jun 20 20:31:26 2008 Return-path: Envelope-to: gcvg-git-2@gmane.org Received: from vger.kernel.org ([209.132.176.167]) by lo.gmane.org with esmtp (Exim 4.50) id 1K9lOO-0004Gb-SK for gcvg-git-2@gmane.org; Fri, 20 Jun 2008 20:31:25 +0200 Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1752161AbYFTSaZ (ORCPT ); Fri, 20 Jun 2008 14:30:25 -0400 Received: (majordomo@vger.kernel.org) by vger.kernel.org id S1752037AbYFTSaZ (ORCPT ); Fri, 20 Jun 2008 14:30:25 -0400 Received: from mail.gmx.net ([213.165.64.20]:53533 "HELO mail.gmx.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with SMTP id S1751325AbYFTSaY (ORCPT ); Fri, 20 Jun 2008 14:30:24 -0400 Received: (qmail invoked by alias); 20 Jun 2008 18:30:22 -0000 Received: from q137.fem.tu-ilmenau.de (EHLO leksak.fem-net) [141.24.46.137] by mail.gmx.net (mp029) with SMTP; 20 Jun 2008 20:30:22 +0200 X-Authenticated: #1499303 X-Provags-ID: V01U2FsdGVkX1/hLhZHtX7hEjn/wrogKvrov7u0IlWNPc15zmc9T5 xdJ1QwjKrfFwxG Received: from sbeyer by leksak.fem-net with local (Exim 4.69) (envelope-from ) id 1K9lNG-00055T-VQ; Fri, 20 Jun 2008 20:30:14 +0200 X-Mailer: git-send-email 1.5.6.167.g86f2 In-Reply-To: X-Y-GMX-Trusted: 0 Sender: git-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: git@vger.kernel.org Archived-At: This patch adds some extra checks (especially branch checks) test cases and changes "! git ..." into "test_must_fail git". Also a --onto test case is added. Signed-off-by: Stephan Beyer --- Dscho, as you've comman^Wwished ;-) (except the analism) Note that I've also added a --onto test. t/t3404-rebase-interactive.sh | 31 ++++++++++++++++++++++++------- 1 files changed, 24 insertions(+), 7 deletions(-) diff --git a/t/t3404-rebase-interactive.sh b/t/t3404-rebase-interactive.sh index e6f3fad..96985ff 100755 --- a/t/t3404-rebase-interactive.sh +++ b/t/t3404-rebase-interactive.sh @@ -107,6 +107,7 @@ chmod a+x fake-editor.sh test_expect_success 'no changes are a nop' ' git rebase -i F && + test "$(git symbolic-ref -q HEAD)" = "refs/heads/branch2" && test $(git rev-parse I) = $(git rev-parse HEAD) ' @@ -115,14 +116,26 @@ test_expect_success 'test the [branch] option' ' git rm file6 && git commit -m "stop here" && git rebase -i F branch2 && + test "$(git symbolic-ref -q HEAD)" = "refs/heads/branch2" && + test $(git rev-parse I) = $(git rev-parse branch2) && test $(git rev-parse I) = $(git rev-parse HEAD) ' +test_expect_success 'test --onto ' ' + git checkout -b test-onto branch2 && + git rebase -i --onto branch1 F && + test "$(git symbolic-ref -q HEAD)" = "refs/heads/test-onto" && + test $(git rev-parse HEAD^) = $(git rev-parse branch1) && + test $(git rev-parse I) = $(git rev-parse branch2) +' + test_expect_success 'rebase on top of a non-conflicting commit' ' git checkout branch1 && git tag original-branch1 && git rebase -i branch2 && test file6 = $(git diff --name-only original-branch1) && + test "$(git symbolic-ref -q HEAD)" = "refs/heads/branch1" && + test $(git rev-parse I) = $(git rev-parse branch2) && test $(git rev-parse I) = $(git rev-parse HEAD~2) ' @@ -155,9 +168,12 @@ EOF test_expect_success 'stop on conflicting pick' ' git tag new-branch1 && - ! git rebase -i master && + test_must_fail git rebase -i master && + test "$(git rev-parse HEAD~3)" = "$(git rev-parse master)" && test_cmp expect .git/.dotest-merge/patch && test_cmp expect2 file1 && + test "$(git-diff --name-status | + sed -n -e "/^U/s/^U[^a-z]*//p")" = file1 && test 4 = $(grep -v "^#" < .git/.dotest-merge/done | wc -l) && test 0 = $(grep -c "^[^#]" < .git/.dotest-merge/git-rebase-todo) ' @@ -165,6 +181,7 @@ test_expect_success 'stop on conflicting pick' ' test_expect_success 'abort' ' git rebase --abort && test $(git rev-parse new-branch1) = $(git rev-parse HEAD) && + test "$(git symbolic-ref -q HEAD)" = "refs/heads/branch1" && ! test -d .git/.dotest-merge ' @@ -331,7 +348,7 @@ test_expect_success 'interactive -t preserves tags' ' test_expect_success '--continue tries to commit' ' git checkout to-be-rebased && test_tick && - ! git rebase -i --onto new-branch1 HEAD^ && + test_must_fail git rebase -i --onto new-branch1 HEAD^ && echo resolved > file1 && git add file1 && FAKE_COMMIT_MESSAGE="chouette!" git rebase --continue && @@ -342,7 +359,7 @@ test_expect_success '--continue tries to commit' ' test_expect_success 'verbose flag is heeded, even after --continue' ' git reset --hard HEAD@{1} && test_tick && - ! git rebase -v -i --onto new-branch1 HEAD^ && + test_must_fail git rebase -v -i --onto new-branch1 HEAD^ && echo resolved > file1 && git add file1 && git rebase --continue > output && @@ -380,7 +397,7 @@ test_expect_success 'interrupted squash works as expected' ' ! FAKE_LINES="1 squash 3 2" git rebase -i HEAD~3 && (echo one; echo two; echo four) > conflict && git add conflict && - ! git rebase --continue && + test_must_fail git rebase --continue && echo resolved > conflict && git add conflict && git rebase --continue && @@ -398,10 +415,10 @@ test_expect_success 'interrupted squash works as expected (case 2)' ' ! FAKE_LINES="3 squash 1 2" git rebase -i HEAD~3 && (echo one; echo four) > conflict && git add conflict && - ! git rebase --continue && + test_must_fail git rebase --continue && (echo one; echo two; echo four) > conflict && git add conflict && - ! git rebase --continue && + test_must_fail git rebase --continue && echo resolved > conflict && git add conflict && git rebase --continue && @@ -449,7 +466,7 @@ test_expect_success 'rebase a commit violating pre-commit' ' chmod a+x $PRE_COMMIT && echo "monde! " >> file1 && test_tick && - ! git commit -m doesnt-verify file1 && + test_must_fail git commit -m doesnt-verify file1 && git commit -m doesnt-verify --no-verify file1 && test_tick && FAKE_LINES=2 git rebase -i HEAD~2 -- 1.5.6.167.g86f2