diff options
| author | Craig Jennings <c@cjennings.net> | 2026-07-09 16:12:44 -0500 |
|---|---|---|
| committer | Craig Jennings <c@cjennings.net> | 2026-07-09 16:12:44 -0500 |
| commit | 7e03862f6315af4b960cbbbe670764cbd49b09d1 (patch) | |
| tree | 5e7cb21a736414c8ea8d5fdf473a86ff1b131ad4 /tests/test-chime-validate-configuration.el | |
| parent | ce396bd2683b0ca4e11279af73f8e733dc16ddc6 (diff) | |
| download | chime-7e03862f6315af4b960cbbbe670764cbd49b09d1.tar.gz chime-7e03862f6315af4b960cbbbe670764cbd49b09d1.zip | |
fix(tests): report failure from test-file and validate
Both targets read $? after a pipeline, so they got the exit status of the last command in it rather than of Emacs.
test-file piped Emacs through tee, so it printed "All tests passed" over a wall of FAILED lines and exited 0. It now redirects to the log, saves the status, and cats the log back.
validate piped Emacs through grep, and grep matched the failure line it was looking for, so a file with unbalanced parens printed its error in green and counted as validated. It now captures the output and the status separately. Verified against a deliberately broken file: before the fix, "All 78 files validated successfully" and exit 0.
make test and make test-all were never affected. They shell out to test-unit and test-integration without a pipeline.
Diffstat (limited to 'tests/test-chime-validate-configuration.el')
0 files changed, 0 insertions, 0 deletions
