diff options
| author | Craig Jennings <c@cjennings.net> | 2026-05-05 09:37:12 -0500 |
|---|---|---|
| committer | Craig Jennings <c@cjennings.net> | 2026-05-05 09:37:12 -0500 |
| commit | b83865a65eb744f6d36c1c672a4d83a59d6383f8 (patch) | |
| tree | 02619079973580147ccf2cf2e7e4567b7d69e022 /scripts/coverage-summary.py | |
| parent | 14195e10b01fec3d9ded6e664a416c5cf3d33cc9 (diff) | |
| download | emacs-wttrin-b83865a65eb744f6d36c1c672a4d83a59d6383f8.tar.gz emacs-wttrin-b83865a65eb744f6d36c1c672a4d83a59d6383f8.zip | |
fix(ci): rewrite mode-init-order test and retry deps install
Two CI failures from the first run, neither a real bug in production code.
The Emacs-snapshot job failed on
`test-wttrin-mode-initialization-order-normal-mode-before-buffer-local-vars-calls-mode-first`.
The original test mocked the `set' primitive to detect when
`xterm-color--state' was first set. That worked on the byte-code path some
older Emacs versions used, but `setq-local' doesn't go through the `set'
function on Emacs master, so the mock never fired and the assertion read nil.
The test was already brittle in isolation locally too.
Rewrote the test to use `advice-add :before' on `wttrin-mode' and
`make-local-variable'. Both are ordinary advisable functions, and
`make-local-variable' is on the code path for every form that defines a
buffer-local binding (`setq-local', `defvar-local', etc.) so the observation
holds across Emacs versions. Renamed the test to drop "calls-mode-first"
and use "mode-runs-before-xterm-color-state-binding" since that's what the
new advice actually observes.
The Emacs 26.3 job failed with a transient DNS error from elpa.gnu.org
during `make deps'. Wrapped the install step in a 3-attempt retry with a
15-second backoff so a runner-side network blip doesn't fail the build.
Applied to all three jobs (test, lint, coverage) since they all hit the
same archives.
Diffstat (limited to 'scripts/coverage-summary.py')
0 files changed, 0 insertions, 0 deletions
