From 369935515b166d89d8157f8bd93d4c59ebf55e3f Mon Sep 17 00:00:00 2001 From: Breadway Date: Fri, 17 Jul 2026 08:32:17 +0800 Subject: [PATCH] breadpad-shared: fix stale test expectation from the bread-theme v0.2.10 bump MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit css_defines_bg_color still expected the pre-v0.2.10 Catppuccin Mocha default (#1e1e2e). bread-theme v0.2.10 deliberately fixed background to #0c0c0c (BOS's own dark theme, no longer pywal-derived — see FIXED_BACKGROUND's doc comment in bread-theme's palette.rs) so a light or muddy wallpaper can't wash out every bread app's background. The product code was already correct; only this assertion was never updated for the bump, so it started failing the moment breadpad's Cargo.toml pin moved to v0.2.10 (confirmed via git stash: it fails identically on the original commit). --- breadpad-shared/src/theme.rs | 8 +++++++- 1 file changed, 7 insertions(+), 1 deletion(-) diff --git a/breadpad-shared/src/theme.rs b/breadpad-shared/src/theme.rs index 8cc2512..1b6cb7f 100644 --- a/breadpad-shared/src/theme.rs +++ b/breadpad-shared/src/theme.rs @@ -284,8 +284,14 @@ mod tests { #[test] fn css_defines_bg_color() { + // #1e1e2e (Catppuccin Mocha) was the default before bread-theme + // v0.2.10, which deliberately fixed background/surface/overlay to + // BOS's own dark theme (#0c0c0c etc.) instead of tracking pywal — + // see bread-theme's `palette.rs` `FIXED_BACKGROUND` doc comment. + // This assertion was never updated for that bump, so it started + // failing the moment breadpad's Cargo.toml pin moved to v0.2.10. let css = build_css(&Palette::default(), None); - assert!(css.contains("@define-color bg #1e1e2e"), "css missing bg: {}", &css[..300]); + assert!(css.contains("@define-color bg #0c0c0c"), "css missing bg: {}", &css[..300]); } #[test]