1 files changed, 148 insertions, 0 deletions
diff --git a/tests/test-chime-check-interval.el b/tests/test-chime-check-interval.el
new file mode 100644
index 0000000..0a51e2e
--- /dev/null
+++ b/tests/test-chime-check-interval.el
@@ -0,0 +1,148 @@
+;;; test-chime-check-interval.el --- Tests for chime-check-interval -*- lexical-binding: t; -*-
+
+;; Copyright (C) 2024 Craig Jennings
+
+;; Author: Craig Jennings <c@cjennings.net>
+
+;; This program is free software: you can redistribute it and/or modify
+;; it under the terms of the GNU General Public License as published by
+;; the Free Software Foundation, either version 3 of the License, or
+;; (at your option) any later version.
+
+;; This program is distributed in the hope that it will be useful,
+;; but WITHOUT ANY WARRANTY; without even the implied warranty of
+;; MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
+;; GNU General Public License for more details.
+
+;; You should have received a copy of the GNU General Public License
+;; along with this program.  If not, see <http://www.gnu.org/licenses/>.
+
+;;; Commentary:
+
+;; Unit tests for chime-check-interval customization variable.
+;; Tests cover normal cases, boundary cases, and error cases.
+
+;;; Code:
+
+;; Initialize package system for batch mode
+(when noninteractive
+  (package-initialize))
+
+(require 'ert)
+
+;; Load dependencies required by chime
+(require 'dash)
+(require 'alert)
+(require 'async)
+(require 'org-agenda)
+
+;; Load chime from parent directory
+(load (expand-file-name "../chime.el") nil t)
+
+;; Load test utilities
+(require 'testutil-general (expand-file-name "testutil-general.el"))
+
+;;; Setup and Teardown
+
+(defun test-chime-check-interval-setup ()
+  "Setup function run before each test."
+  (chime-create-test-base-dir)
+  ;; Reset to default
+  (setq chime-check-interval 60))
+
+(defun test-chime-check-interval-teardown ()
+  "Teardown function run after each test."
+  (chime-delete-test-base-dir)
+  ;; Reset to default
+  (setq chime-check-interval 60))
+
+;;; Normal Cases
+
+(ert-deftest test-chime-check-interval-normal-default-is-60 ()
+  "Test that default check interval is 60 seconds."
+  (test-chime-check-interval-setup)
+  (unwind-protect
+      (progn
+        (should (equal chime-check-interval 60)))
+    (test-chime-check-interval-teardown)))
+
+(ert-deftest test-chime-check-interval-normal-can-set-30-seconds ()
+  "Test that check interval can be set to 30 seconds."
+  (test-chime-check-interval-setup)
+  (unwind-protect
+      (progn
+        (setq chime-check-interval 30)
+        (should (equal chime-check-interval 30)))
+    (test-chime-check-interval-teardown)))
+
+(ert-deftest test-chime-check-interval-normal-can-set-300-seconds ()
+  "Test that check interval can be set to 300 seconds (5 minutes)."
+  (test-chime-check-interval-setup)
+  (unwind-protect
+      (progn
+        (setq chime-check-interval 300)
+        (should (equal chime-check-interval 300)))
+    (test-chime-check-interval-teardown)))
+
+(ert-deftest test-chime-check-interval-normal-can-set-10-seconds ()
+  "Test that check interval can be set to 10 seconds (minimum recommended)."
+  (test-chime-check-interval-setup)
+  (unwind-protect
+      (progn
+        (setq chime-check-interval 10)
+        (should (equal chime-check-interval 10)))
+    (test-chime-check-interval-teardown)))
+
+;;; Boundary Cases
+
+(ert-deftest test-chime-check-interval-boundary-one-second-triggers-warning ()
+  "Test that 1 second interval triggers warning but is allowed."
+  (test-chime-check-interval-setup)
+  (unwind-protect
+      (progn
+        ;; Setting to 1 should trigger a warning but succeed
+        ;; We can't easily test the warning, but we can verify it's set
+        (setq chime-check-interval 1)
+        (should (equal chime-check-interval 1)))
+    (test-chime-check-interval-teardown)))
+
+(ert-deftest test-chime-check-interval-boundary-large-value-accepted ()
+  "Test that large interval values are accepted (e.g., 1 hour)."
+  (test-chime-check-interval-setup)
+  (unwind-protect
+      (progn
+        (setq chime-check-interval 3600)  ; 1 hour
+        (should (equal chime-check-interval 3600)))
+    (test-chime-check-interval-teardown)))
+
+;;; Error Cases
+
+(ert-deftest test-chime-check-interval-error-zero-rejected ()
+  "Test that zero interval is rejected."
+  (test-chime-check-interval-setup)
+  (unwind-protect
+      (progn
+        (should-error (customize-set-variable 'chime-check-interval 0)
+                      :type 'user-error))
+    (test-chime-check-interval-teardown)))
+
+(ert-deftest test-chime-check-interval-error-negative-rejected ()
+  "Test that negative interval is rejected."
+  (test-chime-check-interval-setup)
+  (unwind-protect
+      (progn
+        (should-error (customize-set-variable 'chime-check-interval -60)
+                      :type 'user-error))
+    (test-chime-check-interval-teardown)))
+
+(ert-deftest test-chime-check-interval-error-non-integer-rejected ()
+  "Test that non-integer values are rejected."
+  (test-chime-check-interval-setup)
+  (unwind-protect
+      (progn
+        (should-error (customize-set-variable 'chime-check-interval "60")
+                      :type 'user-error))
+    (test-chime-check-interval-teardown)))
+
+(provide 'test-chime-check-interval)
+;;; test-chime-check-interval.el ends here