Skip to content

Switch to Vitest because it supports ESM#2066

Merged
BobrImperator merged 3 commits intomainmatter:masterfrom
NullVoxPopuli:switch-to-vitest
Oct 9, 2023
Merged

Switch to Vitest because it supports ESM#2066
BobrImperator merged 3 commits intomainmatter:masterfrom
NullVoxPopuli:switch-to-vitest

Conversation

@NullVoxPopuli
Copy link
Contributor

@NullVoxPopuli NullVoxPopuli commented Oct 9, 2023

While working on #2065
I was reminded that jest does not have adequate TS / ESM support (it still thinks everything is cjs -- and just technically can be set up to work with native ESM, but it's a huuuuge pain, and no one has time for that).

So here is jest replaced by vitest.
I had the option of keeping the "globals" support, but it's better practice to import the things used rather than rely on globals (makes types and eslint easier when imported)

Both before and after this change are 39 test files with 329 tests.

@NullVoxPopuli NullVoxPopuli marked this pull request as ready for review October 9, 2023 01:36
@BobrImperator BobrImperator merged commit ce21365 into mainmatter:master Oct 9, 2023
@NullVoxPopuli NullVoxPopuli deleted the switch-to-vitest branch October 9, 2023 20:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants