diff options
| author | Craig Jennings <c@cjennings.net> | 2026-05-03 17:39:10 -0500 |
|---|---|---|
| committer | Craig Jennings <c@cjennings.net> | 2026-05-03 17:39:10 -0500 |
| commit | f19f14758ed03a62f3ca67c95f1dd650cbccdb72 (patch) | |
| tree | 4f009365cfbfe68873c2d1826b9a31ae44198414 /githooks | |
| parent | 8efa929a3a2dd8d9e8f99a0d5ffd472c22014f6d (diff) | |
| download | dotemacs-f19f14758ed03a62f3ca67c95f1dd650cbccdb72.tar.gz dotemacs-f19f14758ed03a62f3ca67c95f1dd650cbccdb72.zip | |
fix(dev-fkeys): F6 elisp runner uses basename, not rel-path
I shipped Phase 2a with `cj/--f6-test-runner-cmd-for' building `make test-file FILE=<rel-path>' for elisp test files (e.g., FILE=tests/test-foo.el). The project Makefile prepends `tests/' to FILE itself, so the full invocation expands to `tests/tests/test-foo.el' and emacs reports "Cannot open load file". The bug surfaced on a live-test in step 7 of the Phase 2a smoke plan.
Fix: pass `(file-name-nondirectory rel-path)' so the Makefile gets just `test-foo.el' and re-prepends `tests/' itself.
Two unit tests in `test-dev-fkeys--f6-test-runner-cmd-for.el' had encoded the wrong expectation (the rel-path form). Two orchestrator tests in `test-dev-fkeys--f6-current-file-tests-impl.el' inherited the same wrong assertion via integration. Updated all four to assert the basename form.
Verified: full suite green, including the 4 updated tests. Live re-test on `tests/test-dev-fkeys--f6-language-detect.el' should now produce the working `make test-file FILE=test-dev-fkeys--f6-language-detect.el'.
Diffstat (limited to 'githooks')
0 files changed, 0 insertions, 0 deletions