aboutsummaryrefslogtreecommitdiff
path: root/tests/test-org-drill-determine-next-interval-sm5.el
diff options
context:
space:
mode:
Diffstat (limited to 'tests/test-org-drill-determine-next-interval-sm5.el')
-rw-r--r--tests/test-org-drill-determine-next-interval-sm5.el13
1 files changed, 8 insertions, 5 deletions
diff --git a/tests/test-org-drill-determine-next-interval-sm5.el b/tests/test-org-drill-determine-next-interval-sm5.el
index e0ac464..de476a4 100644
--- a/tests/test-org-drill-determine-next-interval-sm5.el
+++ b/tests/test-org-drill-determine-next-interval-sm5.el
@@ -248,11 +248,14 @@ The SM5 floor is shared with SM2 via `org-drill-modify-e-factor'.")
"Assert BODY signals a cl-assertion-failed via condition-case.
Mirrors the simple8 test file's helper. See its commentary for why
-this avoids `should-error' / `should' on Emacs 29.4."
- `(condition-case _err
- (progn ,@body
- (ert-fail "expected cl-assertion-failed signal, got none"))
- (cl-assertion-failed nil)))
+shadowing `signal-hook-function' is necessary on Emacs 29.4."
+ `(let ((caught
+ (let ((signal-hook-function nil))
+ (condition-case _err
+ (progn ,@body 'no-error)
+ (cl-assertion-failed 'caught)))))
+ (unless (eq caught 'caught)
+ (ert-fail "expected cl-assertion-failed signal, got none"))))
(ert-deftest test-org-drill-determine-next-interval-sm5-error-negative-n ()
"Error: n=-1 violates the (cl-assert (> n 0)) precondition."