diff options
| author | Craig Jennings <c@cjennings.net> | 2026-05-14 07:51:12 -0500 |
|---|---|---|
| committer | Craig Jennings <c@cjennings.net> | 2026-05-14 07:51:12 -0500 |
| commit | d8d83dc0347af0fdc8704ed1c486867e02779a6d (patch) | |
| tree | 78ebed2ca65b637a085d983f046d743134a0b3d6 /modules/dev-fkeys.el | |
| parent | 847de11a8075851d51d4abe2c10ed3dd64bdbfc5 (diff) | |
| download | dotemacs-d8d83dc0347af0fdc8704ed1c486867e02779a6d.tar.gz dotemacs-d8d83dc0347af0fdc8704ed1c486867e02779a6d.zip | |
refactor(system-commands): use string interactive spec so undercover instruments
`cj/system-cmd' had `(interactive (list (read-shell-command "System
command: ")))' -- the destructured-list interactive spec. Undercover.el
relies on edebug instrumentation, which doesn't see past that form, so
the entire function body registered as 0 hits under coverage even
though tests call the function directly.
Switch to the equivalent string spec `(interactive "sSystem command: ")'.
Same UX (prompt, history, single-string result), but the body now
instruments correctly and coverage moves from 34/49 to 50/51.
Add one more test that captures the `run-at-time' lambda in
`cj/system-cmd-restart-emacs' and invokes it directly so the inner
`call-process-shell-command' branch registers, taking coverage to 51/51.
Diffstat (limited to 'modules/dev-fkeys.el')
0 files changed, 0 insertions, 0 deletions