Update Vitest #3867
Merged
Update Vitest #3867
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
amanmahajan7
commented
Oct 23, 2025
| await resizeHandle.hover({ | ||
| position: { x: 5, y: 5 } | ||
| }); | ||
| await page.mouse.move(x + 5, y + 5); |
Collaborator
Author
There was a problem hiding this comment.
This gives more accurate dimensions
| name: 'browser', | ||
| include: ['test/browser/**/*.test.*'], | ||
| browser: { | ||
| ui: false, |
Collaborator
Author
There was a problem hiding this comment.
headed tests fail without it. Thanks to @nstepien
nstepien
reviewed
Oct 23, 2025
Comment on lines
+12
to
+14
| .vitest-attachments | ||
|
|
||
| __traces__ |
Collaborator
There was a problem hiding this comment.
What do each one contain?
Collaborator
Author
There was a problem hiding this comment.
https://vitest.dev/guide/browser/trace-view
https://vitest.dev/config/#attachmentsdir (not sure what these files are used for)
Co-authored-by: Nicolas Stepien <[email protected]>
Co-authored-by: Nicolas Stepien <[email protected]>
This reverts commit 41445b1.
nstepien
approved these changes
Oct 24, 2025
royue
pushed a commit
to royue/react-data-grid
that referenced
this pull request
Dec 17, 2025
* Vitest 4 beta * Use SHA * Set up workflow * first test * run workflow on lael * format * Tweak * manually run workflow * typo * main * Tweak * fix syntax * revert * Fix step * Update command * Do we need this command? * Revert "Do we need this command?" This reverts commit 7a66594. * Does this remove label? * set env * try `actions-ecosystem/action-remove-labels` * New approach * Does this work * Update visual regression screenshots Co-authored-by: amanmahajan7 * Does this work? * Try this * Update visual regression screenshots Co-authored-by: amanmahajan7 * Simplify * remove name * Update visual regression screenshots Co-authored-by: amanmahajan7 * Tweaks * Fix permissions * Tweak * test * Does this work? * Update screenshots Co-authored-by: amanmahajan7 * Tweaks * Tweak coverage * Tweak coverage * Does this work? * Does this work? * Does this work? * Does this work? * Fix watch * different coverage directories? * Fix typo * Another * `beta.12` * Run VRT in FF also * install ff * Update screenshots Co-authored-by: amanmahajan7 * Fix tests * update package * update test * Update screenshots Co-authored-by: amanmahajan7 * add formatter * Update screenshots Co-authored-by: amanmahajan7 * pin * Still need this :( * unpin * Update deps * Import from `vitest/browser` * update deps * Add back clsx * Fix biome/eslint * update deps * Use async render * Move async * Fix eslint * Enable tracing * Remove VRT changes * Revert .gitignore changes * Fix types * fix 2 tests * set scale * Fix tests for now * Revert commands * typo * Update tsconfig.test.json Co-authored-by: Nicolas Stepien <[email protected]> * Update package.json Co-authored-by: Nicolas Stepien <[email protected]> * Revert "Update package.json" This reverts commit 41445b1. * remove @vitest/browser --------- Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com> Co-authored-by: Nicolas Stepien <[email protected]> Co-authored-by: Nicolas Stepien <[email protected]>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
https://voidzero.dev/posts/announcing-vitest-4
https://vitest.dev/guide/migration.html
Had to add special cases for FF due to scale being set to
0.6666666vitest-dev/vitest#8783