Skip to content

chore: update vitest and related dependencies; refactor browser provi…#4575

Open
GordonSmith wants to merge 1 commit intohpcc-systems:mainfrom
GordonSmith:VITEST_BUMP
Open

chore: update vitest and related dependencies; refactor browser provi…#4575
GordonSmith wants to merge 1 commit intohpcc-systems:mainfrom
GordonSmith:VITEST_BUMP

Conversation

@GordonSmith
Copy link
Copy Markdown
Member

@GordonSmith GordonSmith commented Apr 9, 2026

…der to use playwright

  • Updated vitest and related packages to version 4.1.4 in package.json.
  • Refactored browser provider in vite-utils.ts to use the new playwright integration.
  • Changed test descriptions in react.browser.spec.ts from it() to describe() for better structure and readability.

Checklist:

  • The commit message is properly formatted and free of typos.
    • The commit message title makes sense in a changelog, by itself.
    • The commit message includes a "fixes" reference if appropriate.
    • The commit is signed.
  • The change has been fully tested:
    • I have viewed all related gallery items
    • I have viewed all related dermatology items
  • There are no similar instances of the same problem that should be addressed
    • I have addressed them here
    • I have raised new issues to address them separately

Testing:

Copy link
Copy Markdown
Contributor

@jeclrsg jeclrsg left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@GordonSmith seems like those tests won't run?

@GordonSmith
Copy link
Copy Markdown
Member Author

npm ci failing...

…der to use playwright

- Updated vitest and related packages to version 4.1.4 in package.json.
- Refactored browser provider in vite-utils.ts to use the new playwright integration.
- Changed test descriptions in react.browser.spec.ts from it() to describe() for better structure and readability.

Signed-off-by: Gordon Smith <GordonJSmith@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants