fix(desktop): green up desktop CI on main
Two breakages surfaced when CI started running desktop tests:
- Root vitest.config.ts replaced default exclude with `["**/.claude/**"]`,
losing `**/node_modules/**`. Desktop has no local vitest config, so
vitest scanned into `node_modules/zod/src/v4/classic/tests/*.test.ts`
and failed on missing peers (`@web-std/file`, `recheck`). Spread
`configDefaults.exclude` so node_modules stays excluded.
- window-manager dark titlebar color changed from #18181c to #181B1A in
963c7926 (white-flash fix) but the test was not updated. Update the
expected color to match the new intended behavior.
This commit is contained in:
@@ -54,7 +54,7 @@ describe("window-manager", () => {
|
|||||||
|
|
||||||
it("returns dark title bar overlay colors", () => {
|
it("returns dark title bar overlay colors", () => {
|
||||||
expect(getTitleBarOverlayOptions("dark")).toEqual({
|
expect(getTitleBarOverlayOptions("dark")).toEqual({
|
||||||
color: "#18181c",
|
color: "#181B1A",
|
||||||
symbolColor: "#e4e4e7",
|
symbolColor: "#e4e4e7",
|
||||||
height: 29,
|
height: 29,
|
||||||
});
|
});
|
||||||
@@ -66,11 +66,11 @@ describe("window-manager", () => {
|
|||||||
expect(
|
expect(
|
||||||
readWindowControlsOverlayUpdate({
|
readWindowControlsOverlayUpdate({
|
||||||
height: 48,
|
height: 48,
|
||||||
backgroundColor: "#18181c",
|
backgroundColor: "#181B1A",
|
||||||
}),
|
}),
|
||||||
).toEqual({
|
).toEqual({
|
||||||
height: 48,
|
height: 48,
|
||||||
backgroundColor: "#18181c",
|
backgroundColor: "#181B1A",
|
||||||
});
|
});
|
||||||
});
|
});
|
||||||
|
|
||||||
@@ -107,7 +107,7 @@ describe("window-manager", () => {
|
|||||||
win: { setTitleBarOverlay },
|
win: { setTitleBarOverlay },
|
||||||
current: state,
|
current: state,
|
||||||
update: {
|
update: {
|
||||||
backgroundColor: "#18181c",
|
backgroundColor: "#181B1A",
|
||||||
foregroundColor: "#e4e4e7",
|
foregroundColor: "#e4e4e7",
|
||||||
},
|
},
|
||||||
});
|
});
|
||||||
@@ -120,16 +120,16 @@ describe("window-manager", () => {
|
|||||||
|
|
||||||
expect(state).toEqual({
|
expect(state).toEqual({
|
||||||
height: 48,
|
height: 48,
|
||||||
backgroundColor: "#18181c",
|
backgroundColor: "#181B1A",
|
||||||
foregroundColor: "#e4e4e7",
|
foregroundColor: "#e4e4e7",
|
||||||
});
|
});
|
||||||
expect(setTitleBarOverlay).toHaveBeenNthCalledWith(1, {
|
expect(setTitleBarOverlay).toHaveBeenNthCalledWith(1, {
|
||||||
color: "#18181c",
|
color: "#181B1A",
|
||||||
symbolColor: "#e4e4e7",
|
symbolColor: "#e4e4e7",
|
||||||
height: 28,
|
height: 28,
|
||||||
});
|
});
|
||||||
expect(setTitleBarOverlay).toHaveBeenNthCalledWith(2, {
|
expect(setTitleBarOverlay).toHaveBeenNthCalledWith(2, {
|
||||||
color: "#18181c",
|
color: "#181B1A",
|
||||||
symbolColor: "#e4e4e7",
|
symbolColor: "#e4e4e7",
|
||||||
height: 47,
|
height: 47,
|
||||||
});
|
});
|
||||||
@@ -148,7 +148,7 @@ describe("window-manager", () => {
|
|||||||
frame: false,
|
frame: false,
|
||||||
autoHideMenuBar: true,
|
autoHideMenuBar: true,
|
||||||
titleBarOverlay: {
|
titleBarOverlay: {
|
||||||
color: "#18181c",
|
color: "#181B1A",
|
||||||
symbolColor: "#e4e4e7",
|
symbolColor: "#e4e4e7",
|
||||||
height: 29,
|
height: 29,
|
||||||
},
|
},
|
||||||
|
|||||||
+2
-2
@@ -1,6 +1,6 @@
|
|||||||
import fs from "node:fs";
|
import fs from "node:fs";
|
||||||
import path from "node:path";
|
import path from "node:path";
|
||||||
import { defineConfig } from "vitest/config";
|
import { configDefaults, defineConfig } from "vitest/config";
|
||||||
|
|
||||||
const appDir = path.resolve(__dirname, "packages/app");
|
const appDir = path.resolve(__dirname, "packages/app");
|
||||||
const appNodeModules = path.resolve(appDir, "node_modules");
|
const appNodeModules = path.resolve(appDir, "node_modules");
|
||||||
@@ -57,6 +57,6 @@ export default defineConfig({
|
|||||||
],
|
],
|
||||||
},
|
},
|
||||||
test: {
|
test: {
|
||||||
exclude: ["**/.claude/**"],
|
exclude: [...configDefaults.exclude, "**/.claude/**"],
|
||||||
},
|
},
|
||||||
});
|
});
|
||||||
|
|||||||
Reference in New Issue
Block a user