Skip to content

test: add vitest prep - #707

Merged
JoviDeCroock merged 1 commit into
preactjs:vitest-migrationfrom
43081j:vitest-migration
Jun 28, 2025
Merged

test: add vitest prep#707
JoviDeCroock merged 1 commit into
preactjs:vitest-migrationfrom
43081j:vitest-migration

Conversation

@43081j

@43081j 43081j commented Jun 28, 2025

Copy link
Copy Markdown
Contributor

Adds vitest and associated scripts which will not succeed yet, as we
have no projects.

This is a base branch. We will merge all other vitest migration branches
into this one, and ultimately merge this into main when everything is
migrated.

Adds vitest and associated scripts which will not succeed yet, as we
have no projects.
@changeset-bot

changeset-bot Bot commented Jun 28, 2025

Copy link
Copy Markdown

⚠️ No Changeset found

Latest commit: f12b8ef

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

This PR includes no changesets

When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

@netlify

netlify Bot commented Jun 28, 2025

Copy link
Copy Markdown

Deploy Preview for preact-signals-demo ready!

Name Link
🔨 Latest commit f12b8ef
🔍 Latest deploy log https://app.netlify.com/projects/preact-signals-demo/deploys/686042f8d5623a0008eb3ce3
😎 Deploy Preview https://deploy-preview-707--preact-signals-demo.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

@43081j
43081j marked this pull request as draft June 28, 2025 19:31
@JoviDeCroock
JoviDeCroock changed the base branch from main to vitest-migration June 28, 2025 19:40
@43081j
43081j marked this pull request as ready for review June 28, 2025 19:41
@JoviDeCroock
JoviDeCroock merged commit 7639e98 into preactjs:vitest-migration Jun 28, 2025
5 of 6 checks passed
@43081j
43081j deleted the vitest-migration branch June 28, 2025 19:43
JoviDeCroock pushed a commit that referenced this pull request Jun 30, 2025
Adds vitest and associated scripts which will not succeed yet, as we
have no projects.
JoviDeCroock pushed a commit that referenced this pull request Oct 29, 2025
Adds vitest and associated scripts which will not succeed yet, as we
have no projects.
JoviDeCroock pushed a commit that referenced this pull request Jan 9, 2026
Adds vitest and associated scripts which will not succeed yet, as we
have no projects.
JoviDeCroock added a commit that referenced this pull request Jan 14, 2026
* test: add vitest prep (#707)

Adds vitest and associated scripts which will not succeed yet, as we
have no projects.

* fix: move to using a root vitest config (#713)

We were quickly going to end up with too many sub-configs, and all sorts
of problems with extending the various configs.

This switches to a single root config which globs each of the monorepo
packages instead.

* test: migrate preact/utils to vitest (#715)

* test: migrate preact/utils to vitest

This also adds `preact/utils` to the pnpm workspace so it can be
resolved properly via `@preact/signals-utils`.

* chore: revert workspace change

* fix: downgrade to vite 6 and revert the revert

* test: migrate signals package to vitest (#710)

Migrates the signals package to use vitest browser tests.

* test: migrate core tests to vitest (#708)

Migrates the core package to use vitest.

* test: migrate react tests to vitest (#716)

* test: migrate react tests to vitest

* test: migrate react-transform to vitest

* chore: remove mocha/sinon/chai/karma

* fix: add exact import

* fix: strongly type mocked components

* test: add the sketchy transform back

* fix: funky runtime path

* fix: resolve signals after runtime

* test: map things to places

* test: use vitest aliases

THIS MAY ACTUALLY WORK!

* chore: use react 16 in runtime too

* test: use correct vitest coverage globs (#779)

* chore: remove unused karma config

* test: correct coverage globs

* Fix some stragglers

* Move to browser

* test: de-dupe dependencies & use CDP GC (#786)

* chore: de-dupe babel dependencies

In the lockfile, we ended up with two versions of babel. This is
basically de-duping and dropping one of them.

* fix: use vitest assertions

* fix: dedupe vite

* test: use cdp to force garbage collection

---------

Co-authored-by: James Garbutt <43081j@users.noreply.github.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