Merge pull request #15744 from terraform-providers/t-gov-hardarn-org

tests/provider: Fix hardcoded ARN (Organizations)
hashicorp · Oct 27, 2020 · 83ed83e · 83ed83e
2 parents 7ea75ce + d26c2a9
commit 83ed83e
Showing 1 changed file with 5 additions and 5 deletions.
diff --git a/aws/resource_aws_organizations_policy_test.go b/aws/resource_aws_organizations_policy_test.go
@@ -220,7 +220,7 @@ func testAccAwsOrganizationsPolicy_type_Backup(t *testing.T) {
 	rName := acctest.RandomWithPrefix("tf-acc-test")
 	resourceName := "aws_organizations_policy.test"
 	// Reference: https://docs.aws.amazon.com/organizations/latest/userguide/orgs_manage_policies_backup_syntax.html
-	backupPolicyContent := `{
+	backupPolicyContent := fmt.Sprintf(`{
    "plans":{
       "PII_Backup_Plan":{
          "regions":{
@@ -253,9 +253,9 @@ func testAccAwsOrganizationsPolicy_type_Backup(t *testing.T) {
                   "@@assign":"FortKnox"
                },
                "copy_actions":{
-                  "arn:aws:backup:us-east-1:$account:backup-vault:secondary_vault":{
+                  "arn:%[1]s:backup:us-east-1:$account:backup-vault:secondary_vault":{
                      "target_backup_vault_arn":{
-                        "@@assign":"arn:aws:backup:us-east-1:$account:backup-vault:secondary_vault"
+                        "@@assign":"arn:%[1]s:backup:us-east-1:$account:backup-vault:secondary_vault"
                      },
                      "lifecycle":{
                         "delete_after_days":{
@@ -273,7 +273,7 @@ func testAccAwsOrganizationsPolicy_type_Backup(t *testing.T) {
             "tags":{
                "datatype":{
                   "iam_role_arn":{
-                     "@@assign":"arn:aws:iam::$account:role/MyIamRole"
+                     "@@assign":"arn:%[1]s:iam::$account:role/MyIamRole"
                   },
                   "tag_key":{
                      "@@assign":"dataType"
@@ -289,7 +289,7 @@ func testAccAwsOrganizationsPolicy_type_Backup(t *testing.T) {
          }
       }
    }
-}`
+}`, testAccGetPartition())
 
 	resource.Test(t, resource.TestCase{
 		PreCheck:     func() { testAccPreCheck(t); testAccOrganizationsAccountPreCheck(t) },