;;; test-chime-timestamp-within-interval-p.el --- Tests for chime--timestamp-within-interval-p -*- lexical-binding: t; -*-

;; Copyright (C) 2024-2026 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--timestamp-within-interval-p function.
;; 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"))
(require 'testutil-time (expand-file-name "testutil-time.el"))

;;; Setup and Teardown

(defun test-chime-timestamp-within-interval-p-setup ()
  "Setup function run before each test."
  (chime-create-test-base-dir))

(defun test-chime-timestamp-within-interval-p-teardown ()
  "Teardown function run after each test."
  (chime-delete-test-base-dir))

;;; Normal Cases

(ert-deftest test-chime-timestamp-within-interval-p-exactly-at-interval-returns-t ()
  "Test that timestamp exactly at interval returns t.

REFACTORED: Uses dynamic timestamps and with-test-time"
  (test-chime-timestamp-within-interval-p-setup)
  (unwind-protect
      (let* ((now (test-time-today-at 14 0))
             ;; Timestamp at 14:10 (10 minutes from 14:00)
             (timestamp (test-time-today-at 14 10))
             (interval 10))
        (with-test-time now
          (let ((result (chime--timestamp-within-interval-p timestamp interval)))
            (should result))))
    (test-chime-timestamp-within-interval-p-teardown)))

(ert-deftest test-chime-timestamp-within-interval-p-zero-interval-returns-t ()
  "Test that zero interval (notify now) returns t for current time.

REFACTORED: Uses dynamic timestamps and with-test-time"
  (test-chime-timestamp-within-interval-p-setup)
  (unwind-protect
      (let* ((now (test-time-today-at 14 30))
             ;; Timestamp at exactly current time (14:30)
             (timestamp (test-time-today-at 14 30))
             (interval 0))
        (with-test-time now
          (let ((result (chime--timestamp-within-interval-p timestamp interval)))
            (should result))))
    (test-chime-timestamp-within-interval-p-teardown)))

(ert-deftest test-chime-timestamp-within-interval-p-five-minutes-returns-t ()
  "Test that 5-minute interval works correctly.

REFACTORED: Uses dynamic timestamps and with-test-time"
  (test-chime-timestamp-within-interval-p-setup)
  (unwind-protect
      (let* ((now (test-time-today-at 14 25))
             ;; Timestamp at 14:30 (5 minutes from 14:25)
             (timestamp (test-time-today-at 14 30))
             (interval 5))
        (with-test-time now
          (let ((result (chime--timestamp-within-interval-p timestamp interval)))
            (should result))))
    (test-chime-timestamp-within-interval-p-teardown)))

(ert-deftest test-chime-timestamp-within-interval-p-sixty-minutes-returns-t ()
  "Test that 60-minute (1 hour) interval works correctly.

REFACTORED: Uses dynamic timestamps and with-test-time"
  (test-chime-timestamp-within-interval-p-setup)
  (unwind-protect
      (let* ((now (test-time-today-at 14 0))
             ;; Timestamp at 15:00 (60 minutes from 14:00)
             (timestamp (test-time-today-at 15 0))
             (interval 60))
        (with-test-time now
          (let ((result (chime--timestamp-within-interval-p timestamp interval)))
            (should result))))
    (test-chime-timestamp-within-interval-p-teardown)))

(ert-deftest test-chime-timestamp-within-interval-p-large-interval-returns-t ()
  "Test that large interval (1 day = 1440 minutes) works correctly.

REFACTORED: Uses dynamic timestamps and with-test-time"
  (test-chime-timestamp-within-interval-p-setup)
  (unwind-protect
      (let* ((now (test-time-today-at 14 0))
             ;; Timestamp at 14:00 next day (1440 minutes from now)
             ;; Add 86400 seconds (1440 minutes = 1 day) to now
             ;; Convert to list format for compatibility
             (timestamp (apply #'encode-time (decode-time (time-add now (seconds-to-time 86400)))))
             (interval 1440))
        (with-test-time now
          (let ((result (chime--timestamp-within-interval-p timestamp interval)))
            (should result))))
    (test-chime-timestamp-within-interval-p-teardown)))

(ert-deftest test-chime-timestamp-within-interval-p-thirty-minutes-returns-t ()
  "Test that 30-minute interval works correctly.

REFACTORED: Uses dynamic timestamps and with-test-time"
  (test-chime-timestamp-within-interval-p-setup)
  (unwind-protect
      (let* ((now (test-time-today-at 14 15))
             ;; Timestamp at 14:45 (30 minutes from 14:15)
             (timestamp (test-time-today-at 14 45))
             (interval 30))
        (with-test-time now
          (let ((result (chime--timestamp-within-interval-p timestamp interval)))
            (should result))))
    (test-chime-timestamp-within-interval-p-teardown)))

;;; Boundary Cases

(ert-deftest test-chime-timestamp-within-interval-p-one-minute-before-returns-nil ()
  "Test that timestamp 1 minute before interval returns nil.

REFACTORED: Uses dynamic timestamps and with-test-time"
  (test-chime-timestamp-within-interval-p-setup)
  (unwind-protect
      (let* ((now (test-time-today-at 14 0))
             ;; Timestamp at 14:09 (9 minutes from 14:00, not 10)
             (timestamp (test-time-today-at 14 9))
             (interval 10))
        (with-test-time now
          (let ((result (chime--timestamp-within-interval-p timestamp interval)))
            (should-not result))))
    (test-chime-timestamp-within-interval-p-teardown)))

(ert-deftest test-chime-timestamp-within-interval-p-one-minute-after-returns-nil ()
  "Test that timestamp 1 minute after interval returns nil.

REFACTORED: Uses dynamic timestamps and with-test-time"
  (test-chime-timestamp-within-interval-p-setup)
  (unwind-protect
      (let* ((now (test-time-today-at 14 0))
             ;; Timestamp at 14:11 (11 minutes from 14:00, not 10)
             (timestamp (test-time-today-at 14 11))
             (interval 10))
        (with-test-time now
          (let ((result (chime--timestamp-within-interval-p timestamp interval)))
            (should-not result))))
    (test-chime-timestamp-within-interval-p-teardown)))

(ert-deftest test-chime-timestamp-within-interval-p-crossing-midnight-returns-t ()
  "Test that interval crossing midnight works correctly.

REFACTORED: Uses dynamic timestamps and with-test-time"
  (test-chime-timestamp-within-interval-p-setup)
  (unwind-protect
      (let* ((now (test-time-today-at 23 50))
             ;; Timestamp at 00:00 next day (10 minutes from 23:50)
             ;; Add 600 seconds (10 minutes) to 23:50 to get 00:00 next day
             ;; Convert to list format for compatibility
             (timestamp (apply #'encode-time (decode-time (time-add now (seconds-to-time 600)))))
             (interval 10))
        (with-test-time now
          (let ((result (chime--timestamp-within-interval-p timestamp interval)))
            (should result))))
    (test-chime-timestamp-within-interval-p-teardown)))

(ert-deftest test-chime-timestamp-within-interval-p-crossing-day-boundary-returns-t ()
  "Test that interval crossing to next day works correctly.

REFACTORED: Uses dynamic timestamps and with-test-time"
  (test-chime-timestamp-within-interval-p-setup)
  (unwind-protect
      (let* ((now (test-time-today-at 23 30))
             ;; Timestamp at 00:30 next day (60 minutes from 23:30)
             ;; Add 3600 seconds (60 minutes) to 23:30 to get 00:30 next day
             ;; Convert to list format for compatibility
             (timestamp (apply #'encode-time (decode-time (time-add now (seconds-to-time 3600)))))
             (interval 60))
        (with-test-time now
          (let ((result (chime--timestamp-within-interval-p timestamp interval)))
            (should result))))
    (test-chime-timestamp-within-interval-p-teardown)))

(ert-deftest test-chime-timestamp-within-interval-p-week-interval-returns-t ()
  "Test that very large interval (1 week = 10080 minutes) works.

REFACTORED: Uses dynamic timestamps and with-test-time"
  (test-chime-timestamp-within-interval-p-setup)
  (unwind-protect
      (let* ((now (test-time-today-at 14 0))
             ;; Timestamp at 14:00 one week later (10080 minutes = 7 days from now)
             ;; Add 604800 seconds (10080 minutes = 7 days) to now
             ;; Convert to list format for compatibility
             (timestamp (apply #'encode-time (decode-time (time-add now (seconds-to-time 604800)))))
             (interval 10080))
        (with-test-time now
          (let ((result (chime--timestamp-within-interval-p timestamp interval)))
            (should result))))
    (test-chime-timestamp-within-interval-p-teardown)))

(ert-deftest test-chime-timestamp-within-interval-p-at-midnight-returns-t ()
  "Test that timestamp at exact midnight works correctly.

REFACTORED: Uses dynamic timestamps and with-test-time"
  (test-chime-timestamp-within-interval-p-setup)
  (unwind-protect
      (let* ((now (test-time-today-at 23 50))
             ;; Timestamp at midnight (10 minutes from 23:50)
             ;; Add 600 seconds (10 minutes) to 23:50 to get 00:00 next day
             ;; Convert to list format for compatibility
             (timestamp (apply #'encode-time (decode-time (time-add now (seconds-to-time 600)))))
             (interval 10))
        (with-test-time now
          (let ((result (chime--timestamp-within-interval-p timestamp interval)))
            (should result))))
    (test-chime-timestamp-within-interval-p-teardown)))

;;; Error Cases

(ert-deftest test-chime-timestamp-within-interval-p-nil-timestamp-returns-nil ()
  "Test that nil timestamp returns nil.

REFACTORED: Uses dynamic timestamps and with-test-time"
  (test-chime-timestamp-within-interval-p-setup)
  (unwind-protect
      (let* ((now (test-time-today-at 14 0))
             (timestamp nil)
             (interval 10))
        (with-test-time now
          (let ((result (chime--timestamp-within-interval-p timestamp interval)))
            (should-not result))))
    (test-chime-timestamp-within-interval-p-teardown)))

(ert-deftest test-chime-timestamp-within-interval-p-nil-interval-returns-nil ()
  "Test that nil interval returns nil.

REFACTORED: Uses dynamic timestamps and with-test-time"
  (test-chime-timestamp-within-interval-p-setup)
  (unwind-protect
      (let* ((now (test-time-today-at 14 0))
             (timestamp (test-time-today-at 14 10))
             (interval nil))
        (with-test-time now
          (let ((result (chime--timestamp-within-interval-p timestamp interval)))
            (should-not result))))
    (test-chime-timestamp-within-interval-p-teardown)))

(ert-deftest test-chime-timestamp-within-interval-p-negative-interval-returns-nil ()
  "Test that negative interval returns nil (past timestamps).

REFACTORED: Uses dynamic timestamps and with-test-time"
  (test-chime-timestamp-within-interval-p-setup)
  (unwind-protect
      (let* ((now (test-time-today-at 14 0))
             ;; Timestamp 10 minutes in the past (13:50)
             (timestamp (test-time-today-at 13 50))
             (interval -10))
        (with-test-time now
          (let ((result (chime--timestamp-within-interval-p timestamp interval)))
            (should result))))
    (test-chime-timestamp-within-interval-p-teardown)))

(ert-deftest test-chime-timestamp-within-interval-p-invalid-timestamp-returns-nil ()
  "Test that invalid timestamp format returns nil.

REFACTORED: Uses dynamic timestamps and with-test-time"
  (test-chime-timestamp-within-interval-p-setup)
  (unwind-protect
      (let* ((now (test-time-today-at 14 0))
             (timestamp "not-a-timestamp")
             (interval 10))
        (with-test-time now
          (let ((result (chime--timestamp-within-interval-p timestamp interval)))
            (should-not result))))
    (test-chime-timestamp-within-interval-p-teardown)))

(ert-deftest test-chime-timestamp-within-interval-p-float-interval-works ()
  "Test that float interval gets converted properly.

REFACTORED: Uses dynamic timestamps and with-test-time"
  (test-chime-timestamp-within-interval-p-setup)
  (unwind-protect
      (let* ((now (test-time-today-at 14 0))
             ;; Timestamp at 14:10 (10 minutes from 14:00)
             (timestamp (test-time-today-at 14 10))
             (interval 10.5))
        (with-test-time now
          (let ((result (chime--timestamp-within-interval-p timestamp interval)))
            (should result))))
    (test-chime-timestamp-within-interval-p-teardown)))

(provide 'test-chime-timestamp-within-interval-p)
;;; test-chime-timestamp-within-interval-p.el ends here