Skip to content

Commit 98d13ac

Browse files
committed
Merge branch 'ds/scalar-ignore-cron-error' into next
Allow "scalar" to warn but continue when its periodic maintenance feature cannot be enabled. * ds/scalar-ignore-cron-error: scalar: only warn when background maintenance fails t921*: test scalar behavior starting maintenance t: allow 'scalar' in test_must_fail
2 parents d423119 + dea6308 commit 98d13ac

File tree

4 files changed

+15
-2
lines changed

4 files changed

+15
-2
lines changed

scalar.c

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -262,7 +262,7 @@ static int register_dir(void)
262262
return error(_("could not set recommended config"));
263263

264264
if (toggle_maintenance(1))
265-
return error(_("could not turn on maintenance"));
265+
warning(_("could not turn on maintenance"));
266266

267267
if (have_fsmonitor_support() && start_fsmonitor_daemon()) {
268268
return error(_("could not start the FSMonitor daemon"));

t/t9210-scalar.sh

Lines changed: 7 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -104,6 +104,13 @@ test_expect_success FSMONITOR_DAEMON 'scalar register starts fsmon daemon' '
104104
test_cmp_config -C test/src true core.fsmonitor
105105
'
106106

107+
test_expect_success 'scalar register warns when background maintenance fails' '
108+
git init register-repo &&
109+
GIT_TEST_MAINT_SCHEDULER="crontab:false,launchctl:false,schtasks:false" \
110+
scalar register register-repo 2>err &&
111+
grep "could not turn on maintenance" err
112+
'
113+
107114
test_expect_success 'scalar unregister' '
108115
git init vanish/src &&
109116
scalar register vanish/src &&

t/t9211-scalar-clone.sh

Lines changed: 6 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -174,4 +174,10 @@ test_expect_success 'progress without tty' '
174174
cleanup_clone $enlistment
175175
'
176176

177+
test_expect_success 'scalar clone warns when background maintenance fails' '
178+
GIT_TEST_MAINT_SCHEDULER="crontab:false,launchctl:false,schtasks:false" \
179+
scalar clone "file://$(pwd)/to-clone" maint-fail 2>err &&
180+
grep "could not turn on maintenance" err
181+
'
182+
177183
test_done

t/test-lib-functions.sh

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1016,7 +1016,7 @@ test_must_fail_acceptable () {
10161016
fi
10171017

10181018
case "$1" in
1019-
git|__git*|test-tool|test_terminal)
1019+
git|__git*|scalar|test-tool|test_terminal)
10201020
return 0
10211021
;;
10221022
*)

0 commit comments

Comments
 (0)