b/aws_organizations_policy_allow_nonSCP_content

[RonaldTechnative]

Community Note

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

Closes #25977

Output from acceptance testing:

These tests do not test DiffSuppressFunc switch. Can someone help me with that?

ronald@Ronalds-MacBook-Pro terraform-provider-aws % make testacc TESTS=TestAccPolicy_type PKG=organizations 
==> Checking that code complies with gofmt requirements...
TF_ACC=1 go test ./internal/service/organizations/... -v -count 1 -parallel 20 -run='TestAccPolicy_type'  -timeout 180m
=== RUN   TestAccPolicy_type_AI_OPT_OUT
--- PASS: TestAccPolicy_type_AI_OPT_OUT (32.89s)
=== RUN   TestAccPolicy_type_Backup
--- PASS: TestAccPolicy_type_Backup (29.82s)
=== RUN   TestAccPolicy_type_SCP
--- PASS: TestAccPolicy_type_SCP (47.49s)
=== RUN   TestAccPolicy_type_Tag
--- PASS: TestAccPolicy_type_Tag (30.45s)
PASS
ok      github.com/hashicorp/terraform-provider-aws/internal/service/organizations      142.246s

...

[github-actions]

Hey @RonaldTechnative 👋 Thank you very much for your contribution! At times, our maintainers need to make direct edits to pull requests in order to help get it ready to be merged. Your current settings do not allow maintainers to make such edits. To help facilitate this, update your pull request to allow such edits as described in GitHub's Allowing changes to a pull request branch created from a fork documentation. (If you're using a fork owned by an organization, your organization may not allow you to change this setting. If that is the case, let us know.)

[RonaldTechnative]

Hey @RonaldTechnative 👋 Thank you very much for your contribution! At times, our maintainers need to make direct edits to pull requests in order to help get it ready to be merged. Your current settings do not allow maintainers to make such edits. To help facilitate this, update your pull request to allow such edits as described in GitHub's Allowing changes to a pull request branch created from a fork documentation. (If you're using a fork owned by an organization, your organization may not allow you to change this setting. If that is the case, let us know.)

I'm unable to do this. I'm not able to find this, I can give individual maintainers access if required.

[ewbankkit]

@RonaldTechnative Thanks for the contribution 🎉 👏.
As maintainers do not have permissions to push changes to this fork I created a new PR - #26456 - with your changes, reverting the final commit that changes the test names.

[RonaldTechnative]

@RonaldTechnative Thanks for the contribution 🎉 👏. As maintainers do not have permissions to push changes to this fork I created a new PR - #26456 - with your changes, reverting the final commit that changes the test names.

If you revert the test name changes then they will not be included when trying to test this module. Is that your intended goal of the revert?

[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.