From fda1e989e0c22433b0214ab537a61852d906fd4b Mon Sep 17 00:00:00 2001 From: Craig Jennings Date: Tue, 5 May 2026 05:12:11 -0500 Subject: test: tag integration suite :slow and unstick test-all MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit I tagged the 15 end-to-end tests across `test-integration-chime-mode.el`, `test-integration-recurring-events-tooltip.el`, and `test-integration-startup.el` with `:tags '(:slow)`. The default `make test` now runs only the 652 fast unit tests; `make test-all` runs the full 667. While verifying, I hit a pre-existing bug in `tests/Makefile` where `test-all`'s `--eval '(ert-run-tests-batch-and-exit)'` came back from eask's argv passthrough with its quotes stripped, which broke shell parsing. Passing the explicit `t` selector dodges whatever eask does to no-arg single-quoted forms. The bug only mattered once integration tests were `:slow` — before that, `test-all` overlapped fully with `test-unit` plus `test-integration`, so nothing actually depended on it working. --- tests/test-integration-chime-mode.el | 6 ++++++ 1 file changed, 6 insertions(+) (limited to 'tests/test-integration-chime-mode.el') diff --git a/tests/test-integration-chime-mode.el b/tests/test-integration-chime-mode.el index 69ad176..cd1fec5 100644 --- a/tests/test-integration-chime-mode.el +++ b/tests/test-integration-chime-mode.el @@ -36,6 +36,7 @@ (ert-deftest test-integration-chime-mode-enable-adds-to-global-mode-string () "Enabling chime-mode should add chime-modeline-string to global-mode-string." + :tags '(:slow) (let ((chime-enable-modeline t) (chime-modeline-lookahead-minutes 60)) (unwind-protect @@ -47,6 +48,7 @@ (ert-deftest test-integration-chime-mode-disable-removes-from-global-mode-string () "Disabling chime-mode should remove chime-modeline-string from global-mode-string and set it to nil." + :tags '(:slow) (let ((chime-enable-modeline t) (chime-modeline-lookahead-minutes 60)) (chime-mode 1) @@ -56,6 +58,7 @@ and set it to nil." (ert-deftest test-integration-chime-mode-disable-nils-timer () "Disabling chime-mode should set chime--timer to nil." + :tags '(:slow) (let ((chime-enable-modeline t) (chime-modeline-lookahead-minutes 60)) (chime-mode 1) @@ -68,6 +71,7 @@ and set it to nil." (ert-deftest test-integration-chime-mode-enable-sets-modeline-string-immediately () "Enabling chime-mode should set chime-modeline-string to a non-nil value immediately, before the first async check completes." + :tags '(:slow) (let ((chime-enable-modeline t) (chime-modeline-lookahead-minutes 120) (chime-modeline-no-events-text " ⏰")) @@ -81,6 +85,7 @@ immediately, before the first async check completes." (ert-deftest test-integration-chime-mode-enable-immediate-string-has-tooltip () "The immediate modeline string should have a help-echo tooltip." + :tags '(:slow) (let ((chime-enable-modeline t) (chime-modeline-lookahead-minutes 120) (chime-modeline-no-events-text " ⏰")) @@ -93,6 +98,7 @@ immediately, before the first async check completes." (ert-deftest test-integration-chime-mode-validation-failure-keeps-icon-visible () "When validation fails, modeline should still show the icon with error info in the tooltip, not go blank." + :tags '(:slow) (let ((chime-enable-modeline t) (chime-modeline-lookahead-minutes 120) (chime-modeline-no-events-text " ⏰") -- cgit v1.2.3