-
Notifications
You must be signed in to change notification settings - Fork 57
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
Replace Karma with another runner for browser tests #232
Comments
its probably a little out dated but jasmine was my pick back in the day. Looking now Jest seems more popular (if that counts for anything) |
Yea, Jest is still popular but it seems a lot of projects are migrating away. One of the reason some seem to be gravitating towards Vitest is that the API is fully Jest compatible. And it also supports the expect style tests from Chai, which means we wouldn't have to rewrite many of our tests to migrate. Also Moe hates Jest so we had steered away from it early on 😄 |
Did a deep dive and my conclusion is Jest is the way to go |
Found a way to replace Karma with zero changes to the test suite: https://github.com/TBD54566975/web5-js/pull/316/files. Still not as good as using Jest. But very simple to do. |
Closing since Karma was replaced with Web Test Runner in #316 |
Rationale:
Work to be done:
The text was updated successfully, but these errors were encountered: