Skip to content
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

Diff cross tests for provider upgrades #2910

Open
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

VenelinMartinov
Copy link
Contributor

@VenelinMartinov VenelinMartinov commented Feb 25, 2025

This PR adds a DiffProviderUpgradedSchema option to Diff cross-tests which allows us to test transitions between two different resource schemas.

It uses the same pulumitest mechanism (t.Run) as the upgrade tests.

I've also added a repro for #1667

Part of #2914

Copy link

codecov bot commented Feb 25, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 67.75%. Comparing base (55d6d45) to head (dc3f267).

Additional details and impacted files
@@           Coverage Diff           @@
##           master    #2910   +/-   ##
=======================================
  Coverage   67.74%   67.75%           
=======================================
  Files         331      331           
  Lines       42210    42210           
=======================================
+ Hits        28596    28599    +3     
+ Misses      12019    12017    -2     
+ Partials     1595     1594    -1     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@VenelinMartinov
Copy link
Contributor Author

VenelinMartinov commented Feb 25, 2025

@VenelinMartinov VenelinMartinov force-pushed the vvm/diff_cross_tests_for_provider_upgrades branch from a144941 to dc3f267 Compare March 3, 2025 14:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant