-
Notifications
You must be signed in to change notification settings - Fork 459
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
Refactor Integration Tests to Jest #567
Conversation
5c33922
to
11a14a3
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good. Thanks for adding that test.
I'm going to lock this pull request because it has been closed for 30 days. This helps our maintainers find and focus on the active issues. If you've found a problem that seems related to this change, please open a new issue and complete the issue template so we can capture all the details necessary to investigate further. |
While working on the Ink v3 upgrade (#559) I got pretty annoyed by the current state of integration tests. As part of #559, I made a little proof of concept for using Jest as the test orchestrator for the integration tests and it turned out pretty good. This PR migrates all existing integration tests.
The main benefits include:
This is supposed to be merged into #559