From 1e2da6b1a463492ada31ce473414289e761519ca Mon Sep 17 00:00:00 2001 From: Craig Jennings Date: Tue, 28 Apr 2026 14:31:34 -0500 Subject: refactor: extract missing-glossary test helper Four tests across lookup, list, find-buffer-position, and first-call-creates-file shared the same boilerplate. Each let-bound gloss-file to a randomized nonexistent path, wrapped in unwind-protect, reset the cache, and cleaned up file and buffer afterward. Extracted as gloss-test--with-missing-glossary in testutil-gloss.el, parallel to the existing gloss-test--with-temp-glossary. The four call sites drop from 8-10 lines each to 2-3. Tested by running the full 32-test suite. All 32 pass in 0.21 seconds. --- tests/test-gloss-core--list.el | 10 ++-------- 1 file changed, 2 insertions(+), 8 deletions(-) (limited to 'tests/test-gloss-core--list.el') diff --git a/tests/test-gloss-core--list.el b/tests/test-gloss-core--list.el index 22c988e..2e5a04d 100644 --- a/tests/test-gloss-core--list.el +++ b/tests/test-gloss-core--list.el @@ -23,14 +23,8 @@ (ert-deftest test-gloss-core-list-missing-file-returns-nil () "Error: list before any save returns nil (file does not exist)." - (let ((gloss-file (concat temporary-file-directory "gloss-list-nonexistent-" - (number-to-string (random 100000)) ".org"))) - (unwind-protect - (progn - (gloss-core--cache-reset) - (should-not (gloss-core-list))) - (gloss-core--cache-reset) - (when (file-exists-p gloss-file) (delete-file gloss-file))))) + (gloss-test--with-missing-glossary + (should-not (gloss-core-list)))) (provide 'test-gloss-core--list) ;;; test-gloss-core--list.el ends here -- cgit v1.2.3