Re: [PATCH 2/6] t3404-rebase-interactive: mark a test with REFFILES prereq
From: Ævar Arnfjörð Bjarmason <hidden>
Date: 2022-09-30 16:50:48
Subsystem:
the rest · Maintainer:
Linus Torvalds
On Fri, Sep 30 2022, SZEDER Gábor wrote:
quoted hunk ↗ jump to hunk
The test '--update-refs: check failed ref update' added in b3b1a21d1a (sequencer: rewrite update-refs as user edits todo list, 2022-07-19) directly modifies the contents of a ref file, so mark this test with the REFFILES prereq. Signed-off-by: SZEDER Gábor <redacted> --- t/t3404-rebase-interactive.sh | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)diff --git a/t/t3404-rebase-interactive.sh b/t/t3404-rebase-interactive.sh index 688b01e3eb..7f0df58628 100755 --- a/t/t3404-rebase-interactive.sh +++ b/t/t3404-rebase-interactive.sh@@ -1964,7 +1964,7 @@ test_expect_success 'respect user edits to update-ref steps' ' test_cmp_rev HEAD refs/heads/no-conflict-branch ' -test_expect_success '--update-refs: check failed ref update' ' +test_expect_success REFFILES '--update-refs: check failed ref update' ' git checkout -B update-refs-error no-conflict-branch && git branch -f base HEAD~4 && git branch -f first HEAD~3 &&
We had various tests that depended on .git/refs/* for a good reason, and some that didn't. I may be missing something, but in this case this seems to be a "no good reason" case. I.e. the fix seems to just be:
diff --git a/t/t3404-rebase-interactive.sh b/t/t3404-rebase-interactive.sh
index 688b01e3eb6..f1c021a7f7b 100755
--- a/t/t3404-rebase-interactive.sh
+++ b/t/t3404-rebase-interactive.sh@@ -1994,8 +1994,9 @@ test_expect_success '--update-refs: check failed ref update' ' # At this point, the values of first, second, and third are # recorded in the update-refs file. We will force-update the # "second" ref, but "git branch -f" will not work because of - # the lock in the update-refs file. - git rev-parse third >.git/refs/heads/second && + # the lock in the update-refs file, so we need to use + # "update-ref". + git update-ref refs/heads/second third && test_must_fail git rebase --continue 2>err && grep "update_ref failed for ref '\''refs/heads/second'\''" err &&
As the comment notes if you try that with "git branch" you'll get an error, even with --force, but update-ref works just fine...