diff --git a/builtin/reflog.c b/builtin/reflog.c index 1211c58fa4..49e5453dd9 100644 --- a/builtin/reflog.c +++ b/builtin/reflog.c @@ -279,7 +279,10 @@ static int cmd_reflog_expire(int argc, const char **argv, const char *prefix, for (i = 0; i < argc; i++) { char *ref; - struct expire_reflog_policy_cb cb = { .opts = opts }; + struct expire_reflog_policy_cb cb = { + .opts = opts, + .dry_run = !!(flags & EXPIRE_REFLOGS_DRY_RUN), + }; if (!repo_dwim_log(the_repository, argv[i], strlen(argv[i]), NULL, &ref)) { status |= error(_("reflog could not be found: '%s'"), argv[i]); diff --git a/t/t1410-reflog.sh b/t/t1410-reflog.sh index 93b5b49e1d..998a7e8667 100755 --- a/t/t1410-reflog.sh +++ b/t/t1410-reflog.sh @@ -240,6 +240,40 @@ test_expect_success 'reflog expire --dry-run should not touch reflog' ' check_fsck "missing blob $F" ' +# A dry run only ever says "keep" or "would prune". +check_dry_run_output () { + test_grep "^would prune " "$1" && + sed -e "/^keep /d" -e "/^would prune /d" \ + -e "/^Marking reachable objects/d" "$1" >unexpected && + test_must_be_empty unexpected +} + +test_expect_success 'reflog expire --dry-run --verbose with a named ref' ' + git reflog refs/heads/main >before && + git reflog expire --dry-run --verbose \ + --expire=$(($test_tick - 10000)) \ + --expire-unreachable=$(($test_tick - 10000)) \ + --stale-fix \ + refs/heads/main >output && + check_dry_run_output output && + + git reflog refs/heads/main >after && + test_cmp before after +' + +test_expect_success 'reflog expire --dry-run --verbose --all' ' + git reflog refs/heads/main >before && + git reflog expire --dry-run --verbose \ + --expire=$(($test_tick - 10000)) \ + --expire-unreachable=$(($test_tick - 10000)) \ + --stale-fix \ + --all >output && + check_dry_run_output output && + + git reflog refs/heads/main >after && + test_cmp before after +' + test_expect_success 'reflog expire' ' git reflog expire --verbose \