helper/resource: add ImportBlockWithResourceIdentity kind

.changes/unreleased/BREAKING CHANGES-20250404-170656.yaml

@@ -0,0 +1,5 @@
+kind: BREAKING CHANGES
+body: 'importstate: `ImportStatePersist` and `ImportStateVerify` are not supported for plannable import (`ImportBlockWith*`) and will return an error'
+time: 2025-04-04T17:06:56.900935-04:00
+custom:
+    Issue: "476"

.changes/unreleased/BREAKING CHANGES-20250404-171048.yaml

@@ -0,0 +1,5 @@
+kind: BREAKING CHANGES
+body: 'importstate: renamed `ImportStateWithId` to `ImportStateWithID` and renamed `ImportCommandWithId` to `ImportCommandWithID`.'
+time: 2025-04-04T17:10:48.525611-04:00
+custom:
+    Issue: "465"

.changes/unreleased/BUG FIXES-20250404-170734.yaml

@@ -0,0 +1,5 @@
+kind: BUG FIXES
+body: 'importstate: plannable import (`ImportBlockWith*`) fixed for a resource with a dependency'
+time: 2025-04-04T17:07:34.428542-04:00
+custom:
+    Issue: "476"

.changes/unreleased/NOTES-20250414-095537.yaml

@@ -0,0 +1,5 @@
+kind: NOTES
+body: 'This alpha pre-release adds support for managed resource identity, which can be used with Terraform v1.12.0-beta1. Acceptance tests can use the `ImportBlockWithResourceIdentity` kind to exercise the import of a managed resource using its resource identity object values instead of using a string identifier.'
+time: 2025-04-14T09:55:37.938453-04:00
+custom:
+    Issue: "480"

helper/resource/importstate/examplecloud_test.go

@@ -4,11 +4,14 @@
 package importstate_test
 
 import (
+	"context"
+
 	"github.com/hashicorp/terraform-plugin-go/tfprotov6"
 	"github.com/hashicorp/terraform-plugin-go/tftypes"
 	"github.com/hashicorp/terraform-plugin-testing/internal/testing/testprovider"
 	"github.com/hashicorp/terraform-plugin-testing/internal/testing/testsdk/datasource"
 	"github.com/hashicorp/terraform-plugin-testing/internal/testing/testsdk/resource"
+	"github.com/hashicorp/terraform-plugin-testing/internal/teststep"
 )
 
 func examplecloudDataSource() testprovider.DataSource {
@@ -29,11 +32,7 @@ func examplecloudDataSource() testprovider.DataSource {
 			Schema: &tfprotov6.Schema{
 				Block: &tfprotov6.SchemaBlock{
 					Attributes: []*tfprotov6.SchemaAttribute{
-						{
-							Name:     "id",
-							Type:     tftypes.String,
-							Computed: true,
-						},
+						ComputedStringAttribute("id"),
 					},
 				},
 			},
@@ -58,6 +57,16 @@ func examplecloudResource() testprovider.Resource {
 					"name":     tftypes.NewValue(tftypes.String, "somevalue"),
 				},
 			),
+			NewIdentity: teststep.Pointer(tftypes.NewValue(
+				tftypes.Object{
+					AttributeTypes: map[string]tftypes.Type{
+						"id": tftypes.String,
+					},
+				},
+				map[string]tftypes.Value{
+					"id": tftypes.NewValue(tftypes.String, "westeurope/somevalue"),
+				},
+			)),
 		},
 		ReadResponse: &resource.ReadResponse{
 			NewState: tftypes.NewValue(
@@ -74,6 +83,16 @@ func examplecloudResource() testprovider.Resource {
 					"name":     tftypes.NewValue(tftypes.String, "somevalue"),
 				},
 			),
+			NewIdentity: teststep.Pointer(tftypes.NewValue(
+				tftypes.Object{
+					AttributeTypes: map[string]tftypes.Type{
+						"id": tftypes.String,
+					},
+				},
+				map[string]tftypes.Value{
+					"id": tftypes.NewValue(tftypes.String, "westeurope/somevalue"),
+				},
+			)),
 		},
 		ImportStateResponse: &resource.ImportStateResponse{
 			State: tftypes.NewValue(
@@ -90,26 +109,122 @@ func examplecloudResource() testprovider.Resource {
 					"name":     tftypes.NewValue(tftypes.String, "somevalue"),
 				},
 			),
+			Identity: teststep.Pointer(tftypes.NewValue(
+				tftypes.Object{
+					AttributeTypes: map[string]tftypes.Type{
+						"id": tftypes.String,
+					},
+				},
+				map[string]tftypes.Value{
+					"id": tftypes.NewValue(tftypes.String, "westeurope/somevalue"),
+				},
+			)),
+		},
+		SchemaResponse: &resource.SchemaResponse{
+			Schema: &tfprotov6.Schema{
+				Block: &tfprotov6.SchemaBlock{
+					Attributes: []*tfprotov6.SchemaAttribute{
+						ComputedStringAttribute("id"),
+						RequiredStringAttribute("location"),
+						RequiredStringAttribute("name"),
+					},
+				},
+			},
+		},
+		IdentitySchemaResponse: &resource.IdentitySchemaResponse{
+			Schema: &tfprotov6.ResourceIdentitySchema{
+				Version: 1,
+				IdentityAttributes: []*tfprotov6.ResourceIdentitySchemaAttribute{
+					{
+						Name:              "id",
+						Type:              tftypes.String,
+						RequiredForImport: true,
+					},
+				},
+			},
+		},
+	}
+}
+
+// examplecloudZone is a test resource that mimics a DNS zone resource.
+func examplecloudZone() testprovider.Resource {
+	value := tftypes.NewValue(
+		tftypes.Object{
+			AttributeTypes: map[string]tftypes.Type{
+				"id":   tftypes.String,
+				"name": tftypes.String,
+			},
+		},
+		map[string]tftypes.Value{
+			"id":   tftypes.NewValue(tftypes.String, "5381dd14-6d75-4f32-9096-47f5500b1507"),
+			"name": tftypes.NewValue(tftypes.String, "example.net"),
+		},
+	)
+
+	return testprovider.Resource{
+		CreateResponse: &resource.CreateResponse{
+			NewState: value,
+		},
+		ReadResponse: &resource.ReadResponse{
+			NewState: value,
+		},
+		ImportStateResponse: &resource.ImportStateResponse{
+			State: value,
+		},
+		SchemaResponse: &resource.SchemaResponse{
+			Schema: &tfprotov6.Schema{
+				Block: &tfprotov6.SchemaBlock{
+					Attributes: []*tfprotov6.SchemaAttribute{
+						ComputedStringAttribute("id"),
+						RequiredStringAttribute("name"),
+					},
+				},
+			},
+		},
+	}
+}
+
+// examplecloudZoneRecord is a test resource that mimics a DNS zone record resource.
+// It models a resource dependency; specifically, it depends on a DNS zone ID and will
+// plan a replacement if the zone ID changes.
+func examplecloudZoneRecord() testprovider.Resource {
+	value := tftypes.NewValue(
+		tftypes.Object{
+			AttributeTypes: map[string]tftypes.Type{
+				"id":      tftypes.String,
+				"zone_id": tftypes.String,
+				"name":    tftypes.String,
+			},
+		},
+		map[string]tftypes.Value{
+			"id":      tftypes.NewValue(tftypes.String, "f00911be-e188-433d-9ccd-d0393a9f5d05"),
+			"zone_id": tftypes.NewValue(tftypes.String, "5381dd14-6d75-4f32-9096-47f5500b1507"),
+			"name":    tftypes.NewValue(tftypes.String, "www"),
+		},
+	)
+
+	return testprovider.Resource{
+		CreateResponse: &resource.CreateResponse{
+			NewState: value,
+		},
+		PlanChangeFunc: func(ctx context.Context, req resource.PlanChangeRequest, resp *resource.PlanChangeResponse) {
+			resp.RequiresReplace = []*tftypes.AttributePath{
+				tftypes.NewAttributePath().WithAttributeName("zone_id"),
+			}
+		},
+		ReadResponse: &resource.ReadResponse{
+			NewState: value,
+		},
+		ImportStateResponse: &resource.ImportStateResponse{
+			State: value,
 		},
 		SchemaResponse: &resource.SchemaResponse{
 			Schema: &tfprotov6.Schema{
 				Block: &tfprotov6.SchemaBlock{
 					Attributes: []*tfprotov6.SchemaAttribute{
-						{
-							Name:     "id",
-							Type:     tftypes.String,
-							Computed: true,
-						},
-						{
-							Name:     "location",
-							Type:     tftypes.String,
-							Required: true,
-						},
-						{
-							Name:     "name",
-							Type:     tftypes.String,
-							Required: true,
-						},
+						ComputedStringAttribute("id"),
+						RequiredStringAttribute("zone_id"),
+						RequiredStringAttribute("name"),
 					},
 				},
 			},

helper/resource/importstate/import_block_as_first_step_test.go

@@ -18,7 +18,7 @@ import (
 	r "github.com/hashicorp/terraform-plugin-testing/helper/resource"
 )
 
-func Test_ImportBlock_AsFirstStep(t *testing.T) {
+func TestImportBlock_AsFirstStep(t *testing.T) {
 	t.Parallel()
 
 	r.UnitTest(t, r.TestCase{
@@ -38,12 +38,10 @@ func Test_ImportBlock_AsFirstStep(t *testing.T) {
 				ImportStateId:   "examplecloud_container.test",
 				ImportState:     true,
 				ImportStateKind: r.ImportBlockWithID,
-				// ImportStateVerify: true,
 				Config: `resource "examplecloud_container" "test" {
 					name = "somevalue"
 					location = "westeurope"
 				}`,
-				ImportStatePersist: true,
 				ImportPlanChecks: r.ImportPlanChecks{
 					PreApply: []plancheck.PlanCheck{
 						plancheck.ExpectResourceAction("examplecloud_container.test", plancheck.ResourceActionNoop),

helper/resource/importstate/import_block_for_resource_with_a_dependency_test.go

@@ -0,0 +1,52 @@
+// Copyright (c) HashiCorp, Inc.
+// SPDX-License-Identifier: MPL-2.0
+
+package importstate_test
+
+import (
+	"testing"
+
+	"github.com/hashicorp/terraform-plugin-go/tfprotov6"
+	r "github.com/hashicorp/terraform-plugin-testing/helper/resource"
+	"github.com/hashicorp/terraform-plugin-testing/internal/testing/testprovider"
+	"github.com/hashicorp/terraform-plugin-testing/internal/testing/testsdk/providerserver"
+	"github.com/hashicorp/terraform-plugin-testing/tfversion"
+)
+
+func TestImportBlockForResourceWithADependency(t *testing.T) {
+	t.Parallel()
+
+	config := `
+resource "examplecloud_zone" "zone" {
+	name = "example.net"
+}
+
+resource "examplecloud_zone_record" "record" {
+	zone_id = examplecloud_zone.zone.id
+	name    = "www"
+}
+`
+	r.UnitTest(t, r.TestCase{
+		TerraformVersionChecks: []tfversion.TerraformVersionCheck{
+			tfversion.SkipBelow(tfversion.Version1_5_0), // ImportBlockWithID requires Terraform 1.5.0 or later
+		},
+		ProtoV6ProviderFactories: map[string]func() (tfprotov6.ProviderServer, error){
+			"examplecloud": providerserver.NewProviderServer(testprovider.Provider{
+				Resources: map[string]testprovider.Resource{
+					"examplecloud_zone":        examplecloudZone(),
+					"examplecloud_zone_record": examplecloudZoneRecord(),
+				},
+			}),
+		},
+		Steps: []r.TestStep{
+			{
+				Config: config,
+			},
+			{
+				ImportState:     true,
+				ImportStateKind: r.ImportBlockWithID,
+				ResourceName:    "examplecloud_zone_record.record",
+			},
+		},
+	})
+}

helper/resource/importstate/import_block_in_config_directory_test.go

@@ -16,7 +16,7 @@ import (
 	r "github.com/hashicorp/terraform-plugin-testing/helper/resource"
 )
 
-func Test_ImportBlock_InConfigDirectory(t *testing.T) {
+func TestImportBlock_InConfigDirectory(t *testing.T) {
 	t.Parallel()
 
 	r.UnitTest(t, r.TestCase{
@@ -35,11 +35,10 @@ func Test_ImportBlock_InConfigDirectory(t *testing.T) {
 				ConfigDirectory: config.StaticDirectory(`testdata/1`),
 			},
 			{
-				ResourceName:      "examplecloud_container.test",
-				ImportState:       true,
-				ImportStateKind:   r.ImportBlockWithID,
-				ImportStateVerify: true,
-				ConfigDirectory:   config.StaticDirectory(`testdata/2`),
+				ResourceName:    "examplecloud_container.test",
+				ImportState:     true,
+				ImportStateKind: r.ImportBlockWithID,
+				ConfigDirectory: config.StaticDirectory(`testdata/2`),
 			},
 		},
 	})

helper/resource/importstate/import_block_in_config_file_test.go

@@ -16,7 +16,7 @@ import (
 	r "github.com/hashicorp/terraform-plugin-testing/helper/resource"
 )
 
-func Test_ImportBlock_InConfigFile(t *testing.T) {
+func TestImportBlock_InConfigFile(t *testing.T) {
 	t.Parallel()
 
 	r.UnitTest(t, r.TestCase{
@@ -35,11 +35,10 @@ func Test_ImportBlock_InConfigFile(t *testing.T) {
 				ConfigFile: config.StaticFile(`testdata/1/examplecloud_container.tf`),
 			},
 			{
-				ResourceName:      "examplecloud_container.test",
-				ImportState:       true,
-				ImportStateKind:   r.ImportBlockWithID,
-				ImportStateVerify: true,
-				ConfigFile:        config.StaticFile(`testdata/2/examplecloud_container_import.tf`),
+				ResourceName:    "examplecloud_container.test",
+				ImportState:     true,
+				ImportStateKind: r.ImportBlockWithID,
+				ConfigFile:      config.StaticFile(`testdata/2/examplecloud_container_import.tf`),
 			},
 		},
 	})