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

[EPIC] Cross-testing coverage for Update #2843

Open
3 tasks
iwahbe opened this issue Jan 17, 2025 · 1 comment
Open
3 tasks

[EPIC] Cross-testing coverage for Update #2843

iwahbe opened this issue Jan 17, 2025 · 1 comment
Labels
kind/epic Large new features or investments

Comments

@iwahbe
Copy link
Member

iwahbe commented Jan 17, 2025

Pursuant to #2773, we are introducing integration level cross-tests (SDKv2, PF) for bridged provider operations. These allow us to easily verify that the behavior of bridged TF providers with Pulumi match the behavior of native terraform providers with Terraform. crosstests are written on a verb by verb basis. This EPIC describes the tests we need for the crosstest.Update verb.

As a pre-requisite to implementing any of these tests, we need:

  • TODO: [SDKv2] Implement crosstest.Update
  • TODO: [PF] Implement crosstest.Update

Some of these tests need:

We won't attempt to be as through with testing inputs as we were for #2537, since we presume that inputs and outputs will flow appropriately.

Issues:

  • Verify default values:
    • Going from an empty value to a non-empty value
    • Going from a non-empty value to an empty value
    • Verify changing a value works as expected
    • Verify changing a nested value {List,Set,Map}[T] works as expected
    • Verify that changing a {List,Set,Map} from non-empty to empty and vice versa works as expected
    • Verify that adding or removing an element from a {List,Set,Map} (without hitting empty) works as expected
    • Verify that List re-orders work as expected
    • Verify that Set re-orders work as expected
    • Verify that a no-op update works as expected
    • Verify that changes within an object work as expected
@pulumi-bot pulumi-bot added the needs-triage Needs attention from the triage team label Jan 17, 2025
@iwahbe iwahbe added kind/epic Large new features or investments and removed needs-triage Needs attention from the triage team labels Jan 17, 2025
@VenelinMartinov
Copy link
Contributor

I wonder if we can somehow re-use the tests for Diff here - the setup should be similar.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
kind/epic Large new features or investments
Projects
None yet
Development

No branches or pull requests

3 participants