]> Piment Noir Git Repositories - e-mobility-charging-stations-simulator.git/commit
fix(ui-web): skip dev-only diagnostics under Vitest (#1897)
authorJérôme Benoit <jerome.benoit@piment-noir.org>
Sun, 14 Jun 2026 01:29:02 +0000 (03:29 +0200)
committerGitHub <noreply@github.com>
Sun, 14 Jun 2026 01:29:02 +0000 (03:29 +0200)
commit18c3847df54a099e645e24561a5b55c295dba4ca
tree863bfe8a5028ca9c5a0afc5bfc7cbe2ae092af88
parent0eb89546c237200c00b3a607fba348d894b0b356
fix(ui-web): skip dev-only diagnostics under Vitest (#1897)

Extract isDev() helper in core/env.ts that returns true only in real
DEV (excludes production build and Vitest). Migrate the 7 existing
import.meta.env.DEV check sites to the helper.

Primary motivation: validateTokenContract schedules a
requestAnimationFrame whose body calls console.warn for missing CSS
tokens. jsdom does not resolve --color-* tokens, so every theme/skin
switch under Vitest 4 queued a console.warn that fired after
environment teardown — surfacing as EnvironmentTeardownError on
Linux/Node 22; passes on macOS/Node 24.

Side effect: silences console.debug noise from storage.ts,
providers.ts, ToggleButton.vue under Vitest. dev/build behaviour
unchanged.

Adds regression test in tests/unit/shared/tokens/contract.test.ts.
ui/web/src/core/env.ts [new file with mode: 0644]
ui/web/src/core/index.ts
ui/web/src/core/providers.ts
ui/web/src/core/storage.ts
ui/web/src/main.ts
ui/web/src/shared/tokens/contract.ts
ui/web/src/skins/classic/components/buttons/ToggleButton.vue
ui/web/tests/unit/shared/tokens/contract.test.ts