Handle duplicate sid in policy document

[sunilkumarmohanty]

Fixes #6659

Changes proposed in this pull request:

• Checks for duplicate SID

[sunilkumarmohanty]

Added Acceptance test

 make testacc TEST=./aws TESTARGS='-run=TestAccAWSDataSourceIAMPolicyDocument_duplicateSid'
==> Checking that code complies with gofmt requirements...
TF_ACC=1 go test ./aws -v -parallel 20 -run=TestAccAWSDataSourceIAMPolicyDocument_duplicateSid -timeout 120m
=== RUN   TestAccAWSDataSourceIAMPolicyDocument_duplicateSid
=== PAUSE TestAccAWSDataSourceIAMPolicyDocument_duplicateSid
=== CONT  TestAccAWSDataSourceIAMPolicyDocument_duplicateSid
--- PASS: TestAccAWSDataSourceIAMPolicyDocument_duplicateSid (23.93s)
PASS
ok      github.com/sunilkumarmohanty/terraform-provider-aws/aws (cached)

[bflad]

LGTM, thanks @sunilkumarmohanty! 🚀

--- PASS: TestAccAWSDataSourceIAMPolicyDocument_override (4.02s)
--- PASS: TestAccAWSDataSourceIAMPolicyDocument_noStatementOverride (4.13s)
--- PASS: TestAccAWSDataSourceIAMPolicyDocument_sourceConflicting (4.21s)
--- PASS: TestAccAWSDataSourceIAMPolicyDocument_noStatementMerge (4.22s)
--- PASS: TestAccAWSDataSourceIAMPolicyDocument_basic (4.26s)
--- PASS: TestAccAWSDataSourceIAMPolicyDocument_duplicateSid (4.38s)
--- PASS: TestAccAWSDataSourceIAMPolicyDocument_source (6.39s)

[bflad]

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

[ghost]

I'm going to lock this issue because it has been closed for 30 days ⏳. This helps our maintainers find and focus on the active issues.

If you feel this issue should be reopened, we encourage creating a new issue linking back to this one for added context. Thanks!