-
Notifications
You must be signed in to change notification settings - Fork 3.9k
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
feat(integ-runner): add missing features from the integ manifest #19969
Merged
Commits on Apr 19, 2022
-
feat(integ-runner): add missing features from the integ manifest
Originally the integ-runner was built to work with the "legacy" integration tests since that is all we currently have. Since then we have published the `integ-tests` library which adds support for the integ manifest. This PR adds additional coverage for the integ manifest. To make the implementation a little cleaner I've extracted the logic that deals with the manifest into `IntegTestCases` and `LegacyIntegTestCases` classes. This allows the runner to just deal with the manifest and not have to worry about handling where it came from. I've also split up the `runners.ts`/`runners.test.ts` into `integ-test-runner.ts`, `runner-base.ts`, and `snapshot-test-runner.ts` Coverage added: 1. `stackUpdateWorkflow` - This was moved from the individual `testCase` to the manifest. This means that we will enable or disable the update workflow at the test level instead of the individual test case level. This also required a bump in the schema version. 2. `diffAssets` 3. `hooks`
Configuration menu - View commit details
-
Copy full SHA for 9d28779 - Browse repository at this point
Copy the full SHA 9d28779View commit details -
Configuration menu - View commit details
-
Copy full SHA for 2d82eef - Browse repository at this point
Copy the full SHA 2d82eefView commit details -
Configuration menu - View commit details
-
Copy full SHA for b70a526 - Browse repository at this point
Copy the full SHA b70a526View commit details -
Configuration menu - View commit details
-
Copy full SHA for d88b368 - Browse repository at this point
Copy the full SHA d88b368View commit details -
Configuration menu - View commit details
-
Copy full SHA for 79ecd37 - Browse repository at this point
Copy the full SHA 79ecd37View commit details
Commits on Apr 20, 2022
-
Configuration menu - View commit details
-
Copy full SHA for 3d6d492 - Browse repository at this point
Copy the full SHA 3d6d492View commit details -
Configuration menu - View commit details
-
Copy full SHA for 59fdf75 - Browse repository at this point
Copy the full SHA 59fdf75View commit details -
Configuration menu - View commit details
-
Copy full SHA for 5be019d - Browse repository at this point
Copy the full SHA 5be019dView commit details -
Configuration menu - View commit details
-
Copy full SHA for 08d5107 - Browse repository at this point
Copy the full SHA 08d5107View commit details -
Configuration menu - View commit details
-
Copy full SHA for df591f9 - Browse repository at this point
Copy the full SHA df591f9View commit details -
Configuration menu - View commit details
-
Copy full SHA for 7439fa9 - Browse repository at this point
Copy the full SHA 7439fa9View commit details -
Configuration menu - View commit details
-
Copy full SHA for 61240bd - Browse repository at this point
Copy the full SHA 61240bdView commit details
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.