1 files changed, 47 insertions, 0 deletions
diff --git a/tests/test-calendar-sync--batch-failures.el b/tests/test-calendar-sync--batch-failures.el
new file mode 100644
index 00000000..3190be21
--- /dev/null
+++ b/tests/test-calendar-sync--batch-failures.el
@@ -0,0 +1,47 @@
+;;; test-calendar-sync--batch-failures.el --- Batch failure filter tests  -*- lexical-binding: t; -*-
+
+;;; Commentary:
+;; `calendar-sync--batch-failures' picks the rows that did not finish cleanly.
+;; The batch runner's exit code is derived from it, and systemd reads that exit
+;; code, so the rule is deliberately strict: only `ok' passes.  A calendar left
+;; `syncing' at the timeout, or one that never started, is a failure -- both
+;; states mean the org file on disk is not the calendar's current contents,
+;; which is exactly the silent staleness the timer exists to prevent.
+
+;;; Code:
+
+(require 'ert)
+
+(add-to-list 'load-path (expand-file-name "modules" user-emacs-directory))
+(require 'calendar-sync)
+
+(ert-deftest test-calendar-sync-batch-failures-keeps-only-non-ok ()
+  "Normal: an errored calendar is returned and a healthy one is not."
+  (should (equal (calendar-sync--batch-failures
+                  '(("google" . ok) ("proton" . error)))
+                 '(("proton" . error)))))
+
+(ert-deftest test-calendar-sync-batch-failures-all-ok-is-empty ()
+  "Normal: a fully successful run reports no failures."
+  (should (equal (calendar-sync--batch-failures
+                  '(("google" . ok) ("proton" . ok)))
+                 '())))
+
+(ert-deftest test-calendar-sync-batch-failures-empty-input-is-empty ()
+  "Boundary: no rows in, no rows out."
+  (should (equal (calendar-sync--batch-failures '()) '())))
+
+(ert-deftest test-calendar-sync-batch-failures-timeout-counts-as-failure ()
+  "Error: a calendar still `syncing' when the wait expired is a failure.
+Its org file was not rewritten, so reporting success would hide the staleness."
+  (should (equal (calendar-sync--batch-failures
+                  '(("google" . ok) ("proton" . syncing)))
+                 '(("proton" . syncing)))))
+
+(ert-deftest test-calendar-sync-batch-failures-never-counts-as-failure ()
+  "Error: a calendar that never started is a failure, not a skip."
+  (should (equal (calendar-sync--batch-failures '(("google" . never)))
+                 '(("google" . never)))))
+
+(provide 'test-calendar-sync--batch-failures)
+;;; test-calendar-sync--batch-failures.el ends here