-
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
Refactor integration tests to be table-driven #118
Conversation
zbuchheit
commented
Sep 18, 2024
- Refactor integration tests to be table-drive and all check for a diff-less preview
- Rename some tests to accurately reflect their resources
- Make resources in tests consistently have hyphenated pulumi names
Add additional arguments to the site-settings-ts test for a more robust test. Also check for a spurious diff and expect no changes.
The pulumi names of the resources were inconsistently hyphenated and underscored. Changing these resources to all be hyphenated. Also, rename all the wlan ssid to be named after their test.
Does the PR have any schema changes?Looking good! No breaking changes found. Maintainer note: consult the runbook for dealing with any breaking changes. |
4abf50f
to
c17caf0
Compare
Rename the site-wlan-ts test and site-wlan-isolation-ts test to reflect the resources inside. They are not org wlans.
The new version of providertest requires you to include t when you call pulumitest operations.
c17caf0
to
93f9c48
Compare
I missed some linting, that's why the force push |
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.
LGTM, the change towards table-driven tests looks great. I don't know much about the details of other changes, but no concern from me.
This PR has been shipped in release v0.0.24. |