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

DXCDT-470: Updating ordering of value.difference, updating tests #650

Merged
merged 1 commit into from
Jun 22, 2023

Conversation

willvedd
Copy link
Contributor

@willvedd willvedd commented Jun 21, 2023

🔧 Changes

Necessary reordering the add and remove operations returned form value.Difference(). It is necessary in cases where non-primitive values are process to remove entities before adding them. The changes here are largely to maintain consistency across the codebase and ensure correct operation.

This change was sparked by a related change in this PR: https://github.com/auth0/terraform-provider-auth0/pull/632/files#diff-8fd4b238e4ecbadc212d2f60d719e9c3666cf53523acecf785f03284f47132bbR83.

🔬 Testing

Added acceptance test assertions.

📝 Checklist

  • All new/changed/fixed functionality is covered by tests (or N/A)
  • I have added documentation for all new/changed functionality (or N/A)

@willvedd willvedd requested a review from a team as a code owner June 21, 2023 18:40
@sergiught sergiught force-pushed the DXCDT-470-value.difference-order-operations branch from e8a1954 to b229663 Compare June 22, 2023 07:59
@sergiught sergiught merged commit e78599d into main Jun 22, 2023
@sergiught sergiught deleted the DXCDT-470-value.difference-order-operations branch June 22, 2023 09:16
sergiught pushed a commit that referenced this pull request Jun 25, 2023
Co-authored-by: Will Vedder <will.vedder@okta.com>
sergiught pushed a commit that referenced this pull request Jun 25, 2023
Co-authored-by: Will Vedder <will.vedder@okta.com>
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.

2 participants