Reverting the renaming for now

Author: rainkwan

internal/fromproto5/upgraderesourceidentity.go

@@ -15,7 +15,7 @@ import (
 
 // UpgradeResourceIdentityRequest returns the *fwserver.UpgradeResourceIdentityRequest
 // equivalent of a *tfprotov5.UpgradeResourceIdentityRequest.
-func UpgradeIdentityRequest(ctx context.Context, proto5 *tfprotov5.UpgradeResourceIdentityRequest, resource resource.Resource, identitySchema fwschema.Schema) (*fwserver.UpgradeIdentityRequest, diag.Diagnostics) {
+func UpgradeResourceIdentityRequest(ctx context.Context, proto5 *tfprotov5.UpgradeResourceIdentityRequest, resource resource.Resource, identitySchema fwschema.Schema) (*fwserver.UpgradeResourceIdentityRequest, diag.Diagnostics) {
 	if proto5 == nil {
 		return nil, nil
 	}
@@ -36,7 +36,7 @@ func UpgradeIdentityRequest(ctx context.Context, proto5 *tfprotov5.UpgradeResour
 		return nil, diags
 	}
 
-	fw := &fwserver.UpgradeIdentityRequest{
+	fw := &fwserver.UpgradeResourceIdentityRequest{
 		RawState:       (*tfprotov6.RawState)(proto5.RawIdentity),
 		IdentitySchema: identitySchema,
 		Resource:       resource,

internal/fromproto5/upgraderesourceidentity_test.go

@@ -33,7 +33,7 @@ func TestUpgradeResourceIdentityRequest(t *testing.T) {
 		input               *tfprotov5.UpgradeResourceIdentityRequest
 		identitySchema      fwschema.Schema
 		resource            resource.Resource
-		expected            *fwserver.UpgradeIdentityRequest
+		expected            *fwserver.UpgradeResourceIdentityRequest
 		expectedDiagnostics diag.Diagnostics
 	}{
 		"nil": {
@@ -47,7 +47,7 @@ func TestUpgradeResourceIdentityRequest(t *testing.T) {
 				}),
 			},
 			identitySchema: testIdentitySchema,
-			expected: &fwserver.UpgradeIdentityRequest{
+			expected: &fwserver.UpgradeResourceIdentityRequest{
 				RawState: testNewTfprotov6RawState(t, map[string]interface{}{
 					"test_attribute": "test-value",
 				}),
@@ -57,7 +57,7 @@ func TestUpgradeResourceIdentityRequest(t *testing.T) {
 		"resourceschema": {
 			input:          &tfprotov5.UpgradeResourceIdentityRequest{},
 			identitySchema: testIdentitySchema,
-			expected: &fwserver.UpgradeIdentityRequest{
+			expected: &fwserver.UpgradeResourceIdentityRequest{
 				IdentitySchema: testIdentitySchema,
 			},
 		},
@@ -79,7 +79,7 @@ func TestUpgradeResourceIdentityRequest(t *testing.T) {
 				Version: 123,
 			},
 			identitySchema: testIdentitySchema,
-			expected: &fwserver.UpgradeIdentityRequest{
+			expected: &fwserver.UpgradeResourceIdentityRequest{
 				IdentitySchema: testIdentitySchema,
 				Version:        123,
 			},
@@ -90,7 +90,7 @@ func TestUpgradeResourceIdentityRequest(t *testing.T) {
 		t.Run(name, func(t *testing.T) {
 			t.Parallel()
 
-			got, diags := fromproto5.UpgradeIdentityRequest(context.Background(), testCase.input, testCase.resource, testCase.identitySchema)
+			got, diags := fromproto5.UpgradeResourceIdentityRequest(context.Background(), testCase.input, testCase.resource, testCase.identitySchema)
 
 			if diff := cmp.Diff(got, testCase.expected); diff != "" {
 				t.Errorf("unexpected difference: %s", diff)

internal/fromproto6/upgraderesourceidentity.go

@@ -12,9 +12,9 @@ import (
 	"github.com/hashicorp/terraform-plugin-go/tfprotov6"
 )
 
-// UpgradeIdentityRequest returns the *fwserver.UpgradeIdentityRequest
-// equivalent of a *tfprotov6.UpgradeIdentityRequest.
-func UpgradeIdentityRequest(ctx context.Context, proto6 *tfprotov6.UpgradeResourceIdentityRequest, resource resource.Resource, identitySchema fwschema.Schema) (*fwserver.UpgradeIdentityRequest, diag.Diagnostics) {
+// UpgradeResourceIdentityRequest returns the *fwserver.UpgradeResourceIdentityRequest
+// equivalent of a *tfprotov6.UpgradeResourceIdentityRequest.
+func UpgradeResourceIdentityRequest(ctx context.Context, proto6 *tfprotov6.UpgradeResourceIdentityRequest, resource resource.Resource, identitySchema fwschema.Schema) (*fwserver.UpgradeResourceIdentityRequest, diag.Diagnostics) {
 	if proto6 == nil {
 		return nil, nil
 	}
@@ -35,7 +35,7 @@ func UpgradeIdentityRequest(ctx context.Context, proto6 *tfprotov6.UpgradeResour
 		return nil, diags
 	}
 
-	fw := &fwserver.UpgradeIdentityRequest{
+	fw := &fwserver.UpgradeResourceIdentityRequest{
 		RawState:       proto6.RawIdentity,
 		IdentitySchema: identitySchema,
 		Resource:       resource,

internal/fromproto6/upgraderesourceidentity_test.go

@@ -18,7 +18,7 @@ import (
 	"github.com/hashicorp/terraform-plugin-go/tfprotov6"
 )
 
-func TestUpgradeIdentityRequest(t *testing.T) {
+func TestUpgradeResourceIdentityRequest(t *testing.T) {
 	t.Parallel()
 
 	testIdentitySchema := identityschema.Schema{
@@ -33,7 +33,7 @@ func TestUpgradeIdentityRequest(t *testing.T) {
 		input               *tfprotov6.UpgradeResourceIdentityRequest
 		identitySchema      fwschema.Schema
 		resource            resource.Resource
-		expected            *fwserver.UpgradeIdentityRequest
+		expected            *fwserver.UpgradeResourceIdentityRequest
 		expectedDiagnostics diag.Diagnostics
 	}{
 		"nil": {
@@ -47,7 +47,7 @@ func TestUpgradeIdentityRequest(t *testing.T) {
 				}),
 			},
 			identitySchema: testIdentitySchema,
-			expected: &fwserver.UpgradeIdentityRequest{
+			expected: &fwserver.UpgradeResourceIdentityRequest{
 				RawState: testNewRawState(t, map[string]interface{}{
 					"test_attribute": "test-value",
 				}),
@@ -57,7 +57,7 @@ func TestUpgradeIdentityRequest(t *testing.T) {
 		"resourceschema": {
 			input:          &tfprotov6.UpgradeResourceIdentityRequest{},
 			identitySchema: testIdentitySchema,
-			expected: &fwserver.UpgradeIdentityRequest{
+			expected: &fwserver.UpgradeResourceIdentityRequest{
 				IdentitySchema: testIdentitySchema,
 			},
 		},
@@ -79,7 +79,7 @@ func TestUpgradeIdentityRequest(t *testing.T) {
 				Version: 123,
 			},
 			identitySchema: testIdentitySchema,
-			expected: &fwserver.UpgradeIdentityRequest{
+			expected: &fwserver.UpgradeResourceIdentityRequest{
 				IdentitySchema: testIdentitySchema,
 				Version:        123,
 			},
@@ -90,7 +90,7 @@ func TestUpgradeIdentityRequest(t *testing.T) {
 		t.Run(name, func(t *testing.T) {
 			t.Parallel()
 
-			got, diags := fromproto6.UpgradeIdentityRequest(context.Background(), testCase.input, testCase.resource, testCase.identitySchema)
+			got, diags := fromproto6.UpgradeResourceIdentityRequest(context.Background(), testCase.input, testCase.resource, testCase.identitySchema)
 
 			if diff := cmp.Diff(got, testCase.expected); diff != "" {
 				t.Errorf("unexpected difference: %s", diff)

internal/fwserver/server_upgraderesourceidentity.go

@@ -16,9 +16,9 @@ import (
 	"github.com/hashicorp/terraform-plugin-framework/tfsdk"
 )
 
-// UpgradeIdentityRequest is the framework server request for the
-// UpgradeIdentity RPC.
-type UpgradeIdentityRequest struct {
+// UpgradeResourceIdentityRequest is the framework server request for the
+// UpgradeResourceIdentity RPC.
+type UpgradeResourceIdentityRequest struct {
 	Resource       resource.Resource
 	IdentitySchema fwschema.Schema
 	// TypeName is the type of resource that Terraform needs to upgrade the
@@ -36,15 +36,15 @@ type UpgradeIdentityRequest struct {
 	RawState *tfprotov6.RawState
 }
 
-// UpgradeIdentityResponse is the framework server response for the
-// UpgradeIdentity RPC.
-type UpgradeIdentityResponse struct {
+// UpgradeResourceIdentityResponse is the framework server response for the
+// UpgradeResourceIdentity RPC.
+type UpgradeResourceIdentityResponse struct {
 	UpgradedIdentity *tfsdk.ResourceIdentity
 	Diagnostics      diag.Diagnostics
 }
 
-// UpgradeIdentity implements the framework server UpgradeIdentity RPC.
-func (s *Server) UpgradeIdentity(ctx context.Context, req *UpgradeIdentityRequest, resp *UpgradeIdentityResponse) {
+// UpgradeResourceIdentity implements the framework server UpgradeResourceIdentity RPC.
+func (s *Server) UpgradeResourceIdentity(ctx context.Context, req *UpgradeResourceIdentityRequest, resp *UpgradeResourceIdentityResponse) {
 	if req == nil {
 		return
 	}
@@ -94,23 +94,23 @@ func (s *Server) UpgradeIdentity(ctx context.Context, req *UpgradeIdentityReques
 		}
 	}
 
-	resourceWithUpgradeIdentity, ok := req.Resource.(resource.ResourceWithUpgradeIdentity)
+	resourceWithUpgradeResourceIdentity, ok := req.Resource.(resource.ResourceWithUpgradeResourceIdentity)
 
 	if !ok {
 		resp.Diagnostics.AddError(
 			"Unable to Upgrade Resource Identity",
-			"This resource was implemented without an UpgradeIdentity() method, "+
+			"This resource was implemented without an UpgradeResourceIdentity() method, "+
 				fmt.Sprintf("however Terraform was expecting an implementation for version %d upgrade.\n\n", req.Version)+
 				"This is always an issue with the Terraform Provider and should be reported to the provider developer.",
 		)
 		return
 	}
 
-	logging.FrameworkTrace(ctx, "Resource implements ResourceWithUpgradeIdentity")
+	logging.FrameworkTrace(ctx, "Resource implements ResourceWithUpgradeResourceIdentity")
 
-	logging.FrameworkTrace(ctx, "Calling provider defined Resource UpgradeIdentity")
-	resourceIdentityUpgraders := resourceWithUpgradeIdentity.UpgradeIdentity(ctx)
-	logging.FrameworkTrace(ctx, "Called provider defined Resource UpgradeIdentity")
+	logging.FrameworkTrace(ctx, "Calling provider defined Resource UpgradeResourceIdentity")
+	resourceIdentityUpgraders := resourceWithUpgradeResourceIdentity.UpgradeResourceIdentity(ctx)
+	logging.FrameworkTrace(ctx, "Called provider defined Resource UpgradeResourceIdentity")
 
 	// Panic prevention
 	if resourceIdentityUpgraders == nil {
@@ -122,27 +122,27 @@ func (s *Server) UpgradeIdentity(ctx context.Context, req *UpgradeIdentityReques
 	if !ok {
 		resp.Diagnostics.AddError(
 			"Unable to Upgrade Resource Identity",
-			"This resource was implemented with an UpgradeIdentity() method, "+
+			"This resource was implemented with an UpgradeResourceIdentity() method, "+
 				fmt.Sprintf("however Terraform was expecting an implementation for version %d upgrade.\n\n", req.Version)+
 				"This is always an issue with the Terraform Provider and should be reported to the provider developer.",
 		)
 		return
 	}
 
-	upgradeResourceIdentityRequest := resource.UpgradeIdentityRequest{
+	upgradeResourceIdentityRequest := resource.UpgradeResourceIdentityRequest{
 		RawIdentity: req.RawState,
 	}
 
 	if resourceIdentityUpgrader.PriorSchema != nil {
-		logging.FrameworkTrace(ctx, "Initializing populated UpgradeIdentityRequest Identity from provider defined prior schema and request RawState")
+		logging.FrameworkTrace(ctx, "Initializing populated UpgradeResourceIdentityRequest Identity from provider defined prior schema and request RawState")
 
 		priorSchemaType := resourceIdentityUpgrader.PriorSchema.Type().TerraformType(ctx)
 
 		rawIdentityValue, err := req.RawState.UnmarshalWithOpts(priorSchemaType, unmarshalOpts)
 
 		if err != nil {
 			resp.Diagnostics.AddError(
-				"Unable to Read Previously Saved Identity for UpgradeIdentity",
+				"Unable to Read Previously Saved Identity for UpgradeResourceIdentity",
 				fmt.Sprintf("There was an error reading the saved resource Identity using the prior resource schema defined for version %d upgrade.\n\n", req.Version)+
 					"Please report this to the provider developer:\n\n"+err.Error(),
 			)
@@ -156,7 +156,7 @@ func (s *Server) UpgradeIdentity(ctx context.Context, req *UpgradeIdentityReques
 
 	}
 
-	upgradeResourceIdentityResponse := resource.UpgradeIdentityResponse{
+	upgradeResourceIdentityResponse := resource.UpgradeResourceIdentityResponse{
 		Identity: &tfsdk.ResourceIdentity{
 			Schema: req.IdentitySchema,
 			// Raw is intentionally not set.