[WIP] test: let's use vite-jest for testing #82
Conversation
|
This pull request is being automatically deployed with Vercel (learn more). 🔍 Inspect: https://vercel.com/mutoe/vue3-realworld-example-app/EGDdwLaVHXYH6AfyWMzzPFEe9z4W |
|
@sodatea Sounds great, bro. I don't know if you represent the official Vite, but before it is officially released (at least the beta), I think this can only be pending, or merged into a separate branch for tracking :) |
| jest.mock('src/services/comment/getComments') | ||
| jest.mock('src/services/comment/postComment') | ||
| jest.unstable_mockModule('src/services/comment/getComments', () => ({ | ||
| getCommentsByArticle: jest.fn(), |
There was a problem hiding this comment.
Whether I still need to declare the mocked function here? Is it necessary?
There was a problem hiding this comment.
Yes, it is necessary. The jest mock API for ES modules only accepts a factory function for now, so the exports must be mocked and returned from that factory function.
Yeah, it makes sense. I'll ping you again when the package reaches beta. |
# Conflicts: # src/components/ArticleDetail.spec.ts # src/components/ArticleDetailComments.spec.ts # src/components/ArticlesList.spec.ts # src/components/ArticlesList.vue
mutoe
changed the title
|
Hi @sodatea , after the release of Vitest version 1.0, this project has been migrated to Vitest. Also this PR has been too long, close it for now. Welcome to open it again if necessary :D |
First, thanks for the great example app!
I was developing vite-jest and used this repository as a real-world test case.
Now after v0.1.3 all the current tests are passing. So I'm opening this PR in case you are interested in joining the experiment of this new vite-jest integration.
Main changes in this PR:
babel-jest,ts-jestin favor ofvite-jestjestobject is imported from@jest/globalsjest.unstable_mockModuleinstead ofjest.mock@ts-expect-errorcomments to pass the git pre-commit hook. These type checkings seem to have been failing before my refactor.The coverage report is not working at the moment, though.