-
Notifications
You must be signed in to change notification settings - Fork 0
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
Make wrapito agnostic of testing framework #56
base: master
Are you sure you want to change the base?
Conversation
We need to change the matcher to use .toUpperCase for the method, to keep consistency with regular wrapito |
My two cents on this matter is that it shouldn't matter if you use |
Makes sense to me, just changed the types and made the matcher use uppercase to verify methods like it was in wrapito |
🎩 What?
Make wrapito agnostic of testing framework, i.e it will work with
jest
,vitest
and other test frameworks if needed.🤔 Why?
wrapito-vitest
🧪 How has this been tested? / 💥 How will I know if this breaks?
All tests must pass
closes #54
Migration steps
1.- Uninstall wrapito and install wrapito-vitests (changing all imports from wrapito to wrapito-vitest)
2.- Change all post/get/patch methods to uppercase
3.- Change in "src/setupTests.js" assertions with matchers
Initial tests in ACMOFy
First iteration
Expect API compatible
ACMOFy
BeeGs
Curation (vitest)
Curation (jest)