aboutsummaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorCraig Jennings <c@cjennings.net>2026-05-25 17:35:28 -0500
committerCraig Jennings <c@cjennings.net>2026-05-25 17:35:28 -0500
commitc414a6346f970f0ac49f92222471efeaf7e922d0 (patch)
tree337e5bd96deeff8d9c865ba60d94f52c4078aa72
parent8c7b90c7758790722870ea5635d6f0c56fee6b32 (diff)
downloaddotemacs-c414a6346f970f0ac49f92222471efeaf7e922d0.tar.gz
dotemacs-c414a6346f970f0ac49f92222471efeaf7e922d0.zip
docs(todo): log prog-lisp smoke coverage and the prog-general/prog-training dispositions
-rw-r--r--todo.org28
1 files changed, 16 insertions, 12 deletions
diff --git a/todo.org b/todo.org
index 99268162..924dbae9 100644
--- a/todo.org
+++ b/todo.org
@@ -1927,18 +1927,22 @@ Completion review 2026-05-15:
process handling, shell-script executable policy, and formatter process
boundaries.
-**** TODO [#C] Add smoke coverage for lightweight programming modules with no direct tests :tests:solo:
-
-Several low-risk programming modules currently have little or no direct test
-surface, especially =prog-general.el=, =prog-lisp.el=, and
-=prog-training.el=. Most behavior is package/hook configuration, so this should
-stay narrow:
-- require the modules with package side effects stubbed,
-- assert key hooks/settings that this config owns,
-- ensure batch loading does not trigger external installs or downloads.
-
-Prioritize this after the LSP and tree-sitter policy tasks, because those
-changes will define the stable assertions.
+**** 2026-05-25 Mon @ 17:35:02 -0500 Added prog-lisp smoke coverage; assessed the other two
+
+Added =tests/test-prog-lisp.el= (4 tests) covering the config prog-lisp owns
+directly: cj/elisp-setup and cj/common-lisp-setup are each registered on their
+mode hook and apply the right buffer locals (4-space/no-tabs/fill-120 for
+elisp, 2-space/fill-100 for Common Lisp). The module loads with use-package
+stubbed to a no-op, so nothing installs or downloads in batch.
+
+=prog-training.el= got no test: it's entirely deferred use-package config with
+no top-level surface, and its only owned settings (leetcode language/dir) live
+in a deferred =:config= that would make the test package-dependent for no real
+value. Forcing a test there would be coverage theater.
+
+=prog-general.el= is deferred: it's the one of the three that touches LSP and
+tree-sitter, and the task itself says to wait for the LSP/tree-sitter policy
+tasks to land before fixing its assertions. Its smoke coverage rides with those.
**** TODO [#B] Revisit F4 project classification vs actual project capabilities :ux: