Skip to content

Commit c4e3af6

Browse files
pks-tgitster
authored andcommitted
t1405: remove unneeded cleanup step
In 5e00514 (t1405: explictly delete reflogs for reftable, 2022-01-31) we have added a test that explicitly deletes the reflog when not using the "files" backend. This was required because back then, the "reftable" backend didn't yet delete reflogs when deleting their corresponding branches, and thus subsequent tests would fail because some unexpected reflogs still exist. The "reftable" backend was eventually changed though so that it behaves the same as the "files" backend and deletes reflogs when deleting refs. This was done to make the "reftable" backend behave like the "files" backend as closely as possible so that it can act as a drop-in replacement. The cleanup-style test is thus not required anymore. Remove it. Signed-off-by: Patrick Steinhardt <[email protected]> Signed-off-by: Junio C Hamano <[email protected]>
1 parent 3f87bb2 commit c4e3af6

File tree

1 file changed

+0
-6
lines changed

1 file changed

+0
-6
lines changed

t/t1405-main-ref-store.sh

Lines changed: 0 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -33,12 +33,6 @@ test_expect_success 'delete_refs(FOO, refs/tags/new-tag)' '
3333
test_must_fail git rev-parse refs/tags/new-tag --
3434
'
3535

36-
# In reftable, we keep the reflogs around for deleted refs.
37-
test_expect_success !REFFILES 'delete-reflog(FOO, refs/tags/new-tag)' '
38-
$RUN delete-reflog FOO &&
39-
$RUN delete-reflog refs/tags/new-tag
40-
'
41-
4236
test_expect_success 'rename_refs(main, new-main)' '
4337
git rev-parse main >expected &&
4438
$RUN rename-ref refs/heads/main refs/heads/new-main &&

0 commit comments

Comments
 (0)