Uh oh!
There was an error while loading.Please reload this page.
- Notifications
You must be signed in to change notification settings - Fork2.8k
Closed
Description
Suggestion
Post-Vitest Migration: Fixes & Improvements
- Use the
@nx/vite:test
executor inproject.json
to align with other workspaces. - Fix path-related issues that cause
vitest --project=eslint-plugin
to fail. - Replace
.each
with the new.for
syntax to allow running the tests concurrently viavitest --sequence.concurrent=true
, which does speed up local testing. - Replace
fs
withfs/promises
to take advantage ofvitest
’s better support for async/ESM code (vs.jest
's preference for sync/CJS). - Remove the manual
__dirname
indocs.test.mts
.vitest
backportsimport.meta.dirname
,import.meta.filename
as well asimport.meta.resolve()
for older versions of Node.js. - Convert manually thrown errors in unit tests to standard
vitest
assertions. - Fix issue related to
invalidCases
inno-unnecessary-template-expression.test.ts
. - Refactor remaining loops to use
.for
syntax. - Add a
test-single
target toproject.json
. - Correct the
outDir
path intsconfig.spec.json
.
Additional Info
I will submit a draft PR for this shortly.