Skip to content

Commit 12a0836

Browse files
authored
Merge pull request #44392 from hashicorp/b-generate-existing-resource-norefresh-nochange-test-v6-nulls
Generates `Identity_ExistingResource_NoRefresh_NoChange` for resource types with v6.0 `null` Identity bug
2 parents 1f5ca9b + 630192c commit 12a0836

File tree

74 files changed

+3471
-50
lines changed

Some content is hidden

Large Commits have some content hidden by default. Use the searchbox below for content that may be hidden.

74 files changed

+3471
-50
lines changed

internal/generate/identitytests/resource_test.go.gtpl

Lines changed: 47 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -862,6 +862,53 @@ func {{ template "testname" . }}_Identity_RegionOverride(t *testing.T) {
862862
},
863863
})
864864
}
865+
866+
func {{ template "testname" . }}_Identity_ExistingResource_NoRefresh_NoChange(t *testing.T) {
867+
{{- template "Init" . }}
868+
869+
{{ template "Test" . }}(ctx, t, resource.TestCase{
870+
{{ template "TestCaseSetupNoProviders" . }}
871+
AdditionalCLIOptions: &resource.AdditionalCLIOptions{
872+
Plan: resource.PlanOptions{
873+
NoRefresh: true,
874+
},
875+
},
876+
Steps: []resource.TestStep{
877+
{{ $step := 1 -}}
878+
// Step {{ $step }}: Create pre-Identity
879+
{
880+
ConfigDirectory: config.StaticDirectory("testdata/{{ .Name }}/basic_v5.100.0/"),
881+
ConfigVariables: config.Variables{ {{ if .Generator }}
882+
acctest.CtRName: config.StringVariable(rName),{{ end }}
883+
{{ template "AdditionalTfVars" . }}
884+
},
885+
{{ if .HasExistsFunc -}}
886+
Check: resource.ComposeAggregateTestCheckFunc(
887+
{{- template "ExistsCheck" . -}}
888+
),
889+
{{ end -}}
890+
ConfigStateChecks: []statecheck.StateCheck{
891+
tfstatecheck.ExpectNoIdentity(resourceName),
892+
},
893+
},
894+
895+
// Step {{ ($step = inc $step) | print }}: Current version
896+
{
897+
ProtoV5ProviderFactories: acctest.ProtoV5ProviderFactories,
898+
ConfigDirectory: config.StaticDirectory("testdata/{{ .Name }}/basic/"),
899+
ConfigVariables: config.Variables{ {{ if .Generator }}
900+
acctest.CtRName: config.StringVariable(rName),{{ end }}
901+
{{ template "AdditionalTfVars" . }}
902+
},
903+
{{ if .HasExistsFunc -}}
904+
Check: resource.ComposeAggregateTestCheckFunc(
905+
{{- template "ExistsCheck" . -}}
906+
),
907+
{{ end -}}
908+
},
909+
},
910+
})
911+
}
865912
{{ else if .PreIdentityVersion }}
866913
{{ if .PreIdentityVersion.GreaterThanOrEqual (NewVersion "6.0.0") }}
867914
// Resource Identity was added after v{{ .PreIdentityVersion }}

internal/service/acm/certificate_identity_gen_test.go

Lines changed: 52 additions & 0 deletions
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

internal/service/acmpca/certificate_authority_certificate_identity_gen_test.go

Lines changed: 49 additions & 0 deletions
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

internal/service/acmpca/certificate_authority_identity_gen_test.go

Lines changed: 49 additions & 0 deletions
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

internal/service/acmpca/certificate_identity_gen_test.go

Lines changed: 48 additions & 0 deletions
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

internal/service/acmpca/policy_identity_gen_test.go

Lines changed: 43 additions & 0 deletions
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

internal/service/amp/rule_group_namespace_identity_gen_test.go

Lines changed: 49 additions & 0 deletions
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

0 commit comments

Comments
 (0)