SuppressEquivalentJsonDiffs on redrive_allow_policy for aws_sqs_queue_redrive_allow_policy

[t0yv0]

Description

redrive_allow_policy field on aws_sqs_queue_redrive_allow_policy is made more robust to ignore non-semantic JSON whitespace changes for purposes of planning.

Relations

Closes #40119

References

Output from Acceptance Testing

> TF_ACC=1 go test -run TestAccSQSQueueRedriveAllowPolicy_by_queue -test.v
2024/12/17 16:22:20 Initializing Terraform AWS Provider...
=== RUN   TestAccSQSQueueRedriveAllowPolicy_by_queue
=== PAUSE TestAccSQSQueueRedriveAllowPolicy_by_queue
=== CONT  TestAccSQSQueueRedriveAllowPolicy_by_queue
--- PASS: TestAccSQSQueueRedriveAllowPolicy_by_queue (124.55s)
PASS

[github-actions]

Community Note

Voting for Prioritization

• Please vote on this pull request by adding a 👍 reaction to the original post to help the community and maintainers prioritize this pull request.
• Please see our prioritization guide for information on how we prioritize.
• Please do not leave "+1" or other comments that do not add relevant new information or questions, they generate extra noise for issue followers and do not help prioritize the request.

For Submitters

• Review the contribution guide relating to the type of change you are making to ensure all of the necessary steps have been taken.
• For new resources and data sources, use skaff to generate scaffolding with comments detailing common expectations.
• Whether or not the branch has been rebased will not impact prioritization, but doing so is always a welcome surprise.

[ewbankkit]

LGTM 🚀.

% make testacc TESTARGS='-run=TestAccSQSQueueRedriveAllowPolicy_\|TestAccSQSQueueDataSource_' PKG=sqs ACCTEST_PARALLELISM=3
make: Verifying source code with gofmt...
==> Checking that code complies with gofmt requirements...
TF_ACC=1 go1.23.3 test ./internal/service/sqs/... -v -count 1 -parallel 3  -run=TestAccSQSQueueRedriveAllowPolicy_\|TestAccSQSQueueDataSource_ -timeout 360m
2024/12/18 14:34:30 Initializing Terraform AWS Provider...
=== RUN   TestAccSQSQueueDataSource_tags
=== PAUSE TestAccSQSQueueDataSource_tags
=== RUN   TestAccSQSQueueDataSource_tags_NullMap
=== PAUSE TestAccSQSQueueDataSource_tags_NullMap
=== RUN   TestAccSQSQueueDataSource_tags_EmptyMap
=== PAUSE TestAccSQSQueueDataSource_tags_EmptyMap
=== RUN   TestAccSQSQueueDataSource_tags_DefaultTags_nonOverlapping
=== PAUSE TestAccSQSQueueDataSource_tags_DefaultTags_nonOverlapping
=== RUN   TestAccSQSQueueDataSource_tags_IgnoreTags_Overlap_DefaultTag
=== PAUSE TestAccSQSQueueDataSource_tags_IgnoreTags_Overlap_DefaultTag
=== RUN   TestAccSQSQueueDataSource_tags_IgnoreTags_Overlap_ResourceTag
=== PAUSE TestAccSQSQueueDataSource_tags_IgnoreTags_Overlap_ResourceTag
=== RUN   TestAccSQSQueueDataSource_basic
=== PAUSE TestAccSQSQueueDataSource_basic
=== RUN   TestAccSQSQueueRedriveAllowPolicy_basic
=== PAUSE TestAccSQSQueueRedriveAllowPolicy_basic
=== RUN   TestAccSQSQueueRedriveAllowPolicy_disappears
=== PAUSE TestAccSQSQueueRedriveAllowPolicy_disappears
=== RUN   TestAccSQSQueueRedriveAllowPolicy_Disappears_queue
=== PAUSE TestAccSQSQueueRedriveAllowPolicy_Disappears_queue
=== RUN   TestAccSQSQueueRedriveAllowPolicy_update
=== PAUSE TestAccSQSQueueRedriveAllowPolicy_update
=== RUN   TestAccSQSQueueRedriveAllowPolicy_byQueue
=== PAUSE TestAccSQSQueueRedriveAllowPolicy_byQueue
=== CONT  TestAccSQSQueueDataSource_tags
=== CONT  TestAccSQSQueueDataSource_basic
=== CONT  TestAccSQSQueueDataSource_tags_DefaultTags_nonOverlapping
--- PASS: TestAccSQSQueueDataSource_tags (41.01s)
=== CONT  TestAccSQSQueueDataSource_tags_IgnoreTags_Overlap_ResourceTag
--- PASS: TestAccSQSQueueDataSource_tags_DefaultTags_nonOverlapping (41.02s)
=== CONT  TestAccSQSQueueDataSource_tags_IgnoreTags_Overlap_DefaultTag
--- PASS: TestAccSQSQueueDataSource_basic (41.42s)
=== CONT  TestAccSQSQueueRedriveAllowPolicy_Disappears_queue
--- PASS: TestAccSQSQueueDataSource_tags_IgnoreTags_Overlap_DefaultTag (39.47s)
=== CONT  TestAccSQSQueueRedriveAllowPolicy_byQueue
--- PASS: TestAccSQSQueueDataSource_tags_IgnoreTags_Overlap_ResourceTag (39.66s)
=== CONT  TestAccSQSQueueRedriveAllowPolicy_update
--- PASS: TestAccSQSQueueRedriveAllowPolicy_Disappears_queue (133.18s)
=== CONT  TestAccSQSQueueRedriveAllowPolicy_disappears
--- PASS: TestAccSQSQueueRedriveAllowPolicy_byQueue (118.37s)
=== CONT  TestAccSQSQueueRedriveAllowPolicy_basic
--- PASS: TestAccSQSQueueRedriveAllowPolicy_update (151.87s)
=== CONT  TestAccSQSQueueDataSource_tags_NullMap
--- PASS: TestAccSQSQueueDataSource_tags_NullMap (39.20s)
=== CONT  TestAccSQSQueueDataSource_tags_EmptyMap
--- PASS: TestAccSQSQueueDataSource_tags_EmptyMap (39.32s)
--- PASS: TestAccSQSQueueRedriveAllowPolicy_basic (125.54s)
--- PASS: TestAccSQSQueueRedriveAllowPolicy_disappears (164.93s)
PASS
ok  	github.com/hashicorp/terraform-provider-aws/internal/service/sqs	344.877s

[t0yv0]

Is this needed as part of the change? Thanks.

[jar-b]

LGTM 🚀

[ewbankkit]

@t0yv0 Thanks for the contribution 🎉 👏.

[github-actions]

This functionality has been released in v5.82.0 of the Terraform AWS Provider. Please see the Terraform documentation on provider versioning or reach out if you need any assistance upgrading.

For further feature requests or bug reports with this functionality, please create a new GitHub issue following the template. Thank you!

[github-actions]

I'm going to lock this pull request because it has been closed for 30 days ⏳. This helps our maintainers find and focus on the active issues.
If you have found a problem that seems related to this change, please open a new issue and complete the issue template so we can capture all the details necessary to investigate further.