bread-theme: test that every shipped theme actually resolves #6

Merged
Breadway merged 1 commit from fix/theme-load-coverage into main 2026-09-03 13:15:23 +08:00
Owner

CI gap that let the shell-theme breadbar ship with only glass-workbench verified: a builtin theme.toml can parse but fail resolve_theme at runtime, and load() silently falls back to liquid-motion (breadbar has no tracing subscriber, so the warn is invisible).

3 tests over builtin::ALL: every builtin load_named()s with matching id/name/anchors; BREAD_SHELL_THEME=<each> → load() returns that id not the fallback; unknown id still falls back.

CI gap that let the shell-theme breadbar ship with only glass-workbench verified: a builtin theme.toml can parse but fail `resolve_theme` at runtime, and `load()` silently falls back to liquid-motion (breadbar has no tracing subscriber, so the warn is invisible). 3 tests over `builtin::ALL`: every builtin `load_named()`s with matching id/name/anchors; `BREAD_SHELL_THEME=<each>` → `load()` returns that id not the fallback; unknown id still falls back.
Sign in to join this conversation.
No reviewers
No milestone
No project
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference
Breadway/bread-ecosystem!6
No description provided.