emacs-wttrin/tests/test-wttrin--mode-line-helpers.el, branch v0.3.2Emacs frontend for Igor Chubin's wttr.in weather service
https://git.cjennings.net/emacs-wttrin/atom?h=v0.3.22026-04-26T22:59:59+00:00test: regression lock for nil :foreground in --make-emoji-icon2026-04-26T22:59:59+00:00Craig Jenningsc@cjennings.net2026-04-26T22:59:59+00:00urn:sha1:8f3c770e062a9aefec17f6b5f4f0a5ed927ac603
When wttrin-mode-line-emoji-font is set and the cache is fresh, --make-emoji-icon was emitting a face plist of the form (:family ... :height 1.0 :foreground nil). Emacs validates faces on every redisplay, so a single fresh-cache state produced hundreds of "Invalid face attribute :foreground nil" warnings in *Messages*.
The new boundary test asserts the face plist omits :foreground entirely when the caller passes nil. The assertion uses plist-member, not plist-get. plist-get returns nil for both a missing key and a present key bound to nil. That's the exact distinction Emacs's redisplay validator cares about.
doc: update copyright years to 2026 and add missing author field2026-04-04T17:51:32+00:00Craig Jenningsc@cjennings.net2026-04-04T17:51:32+00:00urn:sha1:622ae39ad1e2544980cff73af8a5d5b2a64dacc8
Update copyright headers across all 37 .el files to include 2026.
Add missing Author field to testutil-wttrin.el for consistency.
refactor: extract mode-line icon and string helpers2026-04-04T17:42:46+00:00Craig Jenningsc@cjennings.net2026-04-04T17:42:46+00:00urn:sha1:130bbc076b17f8a5a275bea00e5aa2a354baad49
Add wttrin--make-emoji-icon and wttrin--set-mode-line-string to
eliminate three near-identical propertize blocks across
update-placeholder-error, set-placeholder, and update-display.
Reduces wttrin--mode-line-update-display from 40 lines to 25.