-
Notifications
You must be signed in to change notification settings - Fork 5k
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
Consolidate e2e Repro Specs #43505
Labels
Comments
This was referenced Jun 3, 2024
One down side to this project is that some developers find working with large files to be rather cumbersome. One alternative approach suggested by @romeovs is keeping test files small, but running them via imports into larger files, which should solve the performance problem. see discussion and proof of concept PR: One open question:
One blocker:
|
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
There is significant (4-6 second) setup overhead in separate test files in CI. combining dozens of reproduction specs that all have 1 or 2 tests, provides several minutes of CI savings in many cases
We should consolidate all of these, even when there are only a couple offenders to eliminate the pattern from the codebase and to keep it from continuing.
Tasks
The text was updated successfully, but these errors were encountered: