diff options
| author | Craig Jennings <c@cjennings.net> | 2026-06-29 04:08:27 -0400 |
|---|---|---|
| committer | Craig Jennings <c@cjennings.net> | 2026-06-29 04:08:27 -0400 |
| commit | 8c91731f5ba177d9f0f7c078576ddd91c606ed7c (patch) | |
| tree | 69601ceb305b5dad1e55a22b61a2dc12fd388a20 | |
| parent | 3f46bda7047a3b6bb9fe778887174209b835d504 (diff) | |
| download | dotemacs-8c91731f5ba177d9f0f7c078576ddd91c606ed7c.tar.gz dotemacs-8c91731f5ba177d9f0f7c078576ddd91c606ed7c.zip | |
chore(todo): file restart-emacs no-service test failure
| -rw-r--r-- | todo.org | 3 |
1 files changed, 3 insertions, 0 deletions
@@ -55,6 +55,9 @@ Tags are additive. For example, a small wrong-behavior fix can be =:bug:quick:=, and a feature that requires internal restructuring can be =:feature:refactor:=. * Emacs Open Work +** TODO [#C] restart-emacs no-service test fails on machines with emacs.service :test:bug: +=test-system-cmd-restart-emacs-no-service-aborts= (=tests/test-system-commands-resolve-and-run.el:172=) fails on velox and passes where no =emacs.service= exists. It mocks =cj/system-cmd--emacs-service-available-p= to return nil via =cl-letf=, but =cj/system-cmd-restart-emacs= (in =system-commands.el=, natively compiled) calls that helper through an intra-file direct reference in its =.eln=, which bypasses the symbol-function redefinition. So the real check runs, finds =emacs.service=, and restart proceeds instead of aborting -- the =should-error= sees "did not signal an error". Same native-comp mocking trap noted in CLAUDE.md, here for a same-file defun rather than a subr. Fix: mock at the boundary the helper uses (=executable-find= / the =systemctl= =call-process-shell-command=) the way the passing sibling tests =service-available-false-on-nonzero-exit= and =service-available-false-when-systemctl-absent= do, instead of mocking the helper directly; or inject the check. Pre-existing (fails on clean HEAD), surfaced during the 2026-06-29 structural speedrun. + ** PROJECT [#A] Manual testing and validation Exercised once the phases above land. *** DONE F12 opens the eshell-through-EAT terminal (dock, visual, colors, prompt, z) |
