mirror of
https://github.com/zadam/trilium.git
synced 2026-09-12 11:40:19 +05:00
e8eeb68b50
Vitest 5 handed the webdriverio provider to community maintenance, and no
stable 5.x of `@vitest/browser-webdriverio` was ever published — npm `latest`
is still 4.1.11, with only a 5.0.0-rc.1 from before vitest 5.0.0 shipped.
Pairing that v4 provider with `@vitest/browser` 5 kills the suite at startup:
TypeError: Cannot read properties of undefined (reading 'project')
at createBrowserServer (@vitest/browser/dist/index.js:7900:26)
`@vitest/browser-playwright` 5.0.0 is stable and first-party, and Playwright
1.62.1 is already here for the e2e suites, so the browser toolchain costs
nothing new. Only two of the 129 specs touch provider-visible API at all, both
through the provider-agnostic `userEvent`.
Playwright drives the browser over CDP with no separate driver, so the
chromedriver half of the NixOS workaround goes away: the dev shell keeps
`pkgs.chromium` and `CHROME_BIN`, which now reaches the provider as
`launchOptions.executablePath`. CI installs the browser in a step of its own
rather than inside the test step, whose 15-minute cap exists to catch a browser
session that never starts and should not also have to cover a 190 MB download.
Two specs depended on webdriverio behaviour and are fixed rather than skipped:
- The token-cost assertion read `1.234` where it wanted `1,234`.
`toLocaleString()` takes the browser's locale, and Playwright inherits the
host's where the old Chrome defaulted to en-US. Pinning `contextOptions.locale`
keeps the suite from depending on the developer's machine.
- The format painter's drag-selection selected nothing. Playwright's
`dragAndDrop` turns on drag interception, so the press reaches the page as an
HTML5 drag intent instead of selecting text. Driving the press, move and
release over CDP restores what the test is actually for: proving a *native*
pointer interaction has updated the model selection by the time the `mouseup`
listener runs.
Vitest 5 also moved failure screenshots from `.vitest-attachments` to
`.vitest`, which needs ignoring.
Verified: 127 files / 1626 tests green, coverage 99.91/99.54/100/99.98 against
the 99.5 gate. The Nix path is unexercised — `executablePath` is ordinary
Playwright, but it was not run from a dev shell.
Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
67 lines
1.0 KiB
Plaintext
67 lines
1.0 KiB
Plaintext
# See https://docs.github.com/en/get-started/getting-started-with-git/ignoring-files for more about ignoring files.
|
|
/.cache
|
|
|
|
# compiled output
|
|
dist
|
|
tmp
|
|
out-tsc
|
|
|
|
# dependencies
|
|
node_modules
|
|
|
|
# IDEs and editors
|
|
/.idea
|
|
.idea
|
|
.project
|
|
.classpath
|
|
.c9/
|
|
*.launch
|
|
.settings/
|
|
*.sublime-workspace
|
|
|
|
# misc
|
|
/.sass-cache
|
|
/connect.lock
|
|
/coverage
|
|
/libpeerconnection.log
|
|
npm-debug.log
|
|
yarn-error.log
|
|
testem.log
|
|
/typings
|
|
|
|
# System Files
|
|
.DS_Store
|
|
Thumbs.db
|
|
|
|
vite.config.*.timestamp*
|
|
vitest.config.*.timestamp*
|
|
test-output
|
|
.vitest-reports
|
|
.vitest
|
|
__screenshots__
|
|
.vitest-attachments
|
|
|
|
apps/*/data*
|
|
apps/*/out
|
|
upload
|
|
|
|
.rollup.cache
|
|
*.tsbuildinfo
|
|
|
|
/.direnv
|
|
/result
|
|
|
|
# docs
|
|
site/
|
|
apps/*/coverage
|
|
scripts/translation/.language*.json
|
|
|
|
# Capacitor regenerates this on every `cap sync` and produces different
|
|
# output on Windows vs macOS, causing constant churn — keep it local.
|
|
apps/mobile/ios/App/CapApp-SPM/Package.swift
|
|
# Ignore local Claude Code settings, but keep shared project skills.
|
|
.claude/*
|
|
!.claude/skills/
|
|
.pnpm-store
|
|
apps/edit-docs/output.zip
|