Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

test: migrate to vitest #1637

Merged
merged 16 commits into from
Jul 4, 2022
Merged

test: migrate to vitest #1637

merged 16 commits into from
Jul 4, 2022

Conversation

okxiaoliang4
Copy link
Contributor

@okxiaoliang4 okxiaoliang4 commented Jul 1, 2022

migrate to vitest

vitest test for source

Test Files  52 passed (52)
     Tests  403 passed | 1 todo (404)
      Time  16.35s (in thread 4.93s, 331.63%)

vitest test for product

Test Files  52 passed (52)
     Tests  403 passed | 1 todo (404)
      Time  14.89s (in thread 4.58s, 325.36%)

jest test for source

Test Suites: 53 passed, 53 total
Tests:       1 skipped, 1 todo, 404 passed, 406 total
Snapshots:   4 passed, 4 total
Time:        38.9 s
Ran all test suites matching /tests\\/i.

jest test for product

Test Suites: 53 passed, 53 total
Tests:       1 skipped, 1 todo, 404 passed, 406 total
Snapshots:   4 passed, 4 total
Time:        37.815 s
Ran all test suites matching /tests\\/i.

@netlify
Copy link

netlify bot commented Jul 1, 2022

Deploy Preview for vue-test-utils-docs ready!

Name Link
🔨 Latest commit 93ce9ff
🔍 Latest deploy log https://app.netlify.com/sites/vue-test-utils-docs/deploys/62c2e2ae19836e00090255d6
😎 Deploy Preview https://deploy-preview-1637--vue-test-utils-docs.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 site settings.

tsconfig.json Outdated Show resolved Hide resolved
@lmiller1990
Copy link
Member

Cool - happy to merge this if we can get everything green. Definitely not my biggest priority, but nice to see Vitest is getting more mature.

Copy link
Contributor Author

@okxiaoliang4 okxiaoliang4 left a comment

Choose a reason for hiding this comment

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

self reviewed :)

@okxiaoliang4 okxiaoliang4 marked this pull request as ready for review July 2, 2022 18:09
// TODO: Find out why this does not work
// Is it valid to have an AsyncComponent as the root? Was it ever?
it.skip('works when AsyncComponent is the root', async () => {
it('works when AsyncComponent is the root', async () => {
Copy link
Contributor Author

Choose a reason for hiding this comment

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

fix the test fail

@okxiaoliang4 okxiaoliang4 requested a review from lmiller1990 July 3, 2022 06:36
Copy link
Member

@cexbrayat cexbrayat left a comment

Choose a reason for hiding this comment

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

LGTM, thanks!

@cexbrayat
Copy link
Member

@okxiaoliang4 Would you mind rebasing the PR and fixing the conflicts please? I'll merge when it's done

@lmiller1990
Copy link
Member

🎉

I actually have no problem with Jest... but happy to adopt Vitest, much of a muchness, really.

@okxiaoliang4
Copy link
Contributor Author

@cexbrayat done.

@cexbrayat cexbrayat merged commit 31f66b9 into vuejs:main Jul 4, 2022
@okxiaoliang4 okxiaoliang4 deleted the feature/vitest branch July 4, 2022 14:53
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.

3 participants