-
Notifications
You must be signed in to change notification settings - Fork 2
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
Joining efforts #1
Comments
Hello @DanielMSchmidt :-) About the binary/api.. yeah. I'm not sure either way. Starting up with API is slow.. I'm guessing is the same with binary. Those are the areas are mostly concerned about currently. |
Yes, both ways start the runner pretty slowly. I think reusing the runner makes sense in some cases, for us it doesn't on CI. (Our CI RAM limit is not big enough to run every test case with the same cypress instance, we need to "force" a shutdown to allow the chromium instance to release the memory) I would like to make it configurable that we have re-runs per test case. Integration tests can be flaky so let's take this into account 🙈 I would love to start it in visual mode, I think that would be a really nice thing |
For earlier discussions please see jestjs/jest#5706.
For me, the plus point of the binary was that I could use all the options we already configured and pass them to the runner directly. I thought this would make the migration easier, but I think using the node API gives us more power over what is happening.
The text was updated successfully, but these errors were encountered: