additional skipped tests

Author: VenelinMartinov

pkg/pf/tests/diff_test/diff_test.go

@@ -1,9 +1,11 @@
 package tfbridgetests
 
 import (
+	"context"
 	"math/big"
 	"testing"
 
+	"github.com/hashicorp/terraform-plugin-framework/resource"
 	rschema "github.com/hashicorp/terraform-plugin-framework/resource/schema"
 	"github.com/hashicorp/terraform-plugin-framework/resource/schema/planmodifier"
 	"github.com/hashicorp/terraform-plugin-framework/resource/schema/stringplanmodifier"
@@ -188,22 +190,96 @@ func TestPFDetailedDiffDynamicType(t *testing.T) {
 			},
 		},
 	}
+	res := pb.NewResource(pb.NewResourceArgs{
+		ResourceSchema: attributeSchema,
+	})
 
 	t.Run("no change", func(t *testing.T) {
-		crosstests.Diff(t, pb.NewResource(pb.NewResourceArgs{
-			ResourceSchema: attributeSchema,
-		}), map[string]cty.Value{"key": cty.StringVal("value")}, map[string]cty.Value{"key": cty.StringVal("value")})
+		crosstests.Diff(t, res,
+			map[string]cty.Value{"key": cty.StringVal("value")},
+			map[string]cty.Value{"key": cty.StringVal("value")},
+		)
 	})
 
 	t.Run("change", func(t *testing.T) {
-		crosstests.Diff(t, pb.NewResource(pb.NewResourceArgs{
-			ResourceSchema: attributeSchema,
-		}), map[string]cty.Value{"key": cty.StringVal("value")}, map[string]cty.Value{"key": cty.StringVal("value1")})
+		crosstests.Diff(t, res,
+			map[string]cty.Value{"key": cty.StringVal("value")},
+			map[string]cty.Value{"key": cty.StringVal("value1")},
+		)
 	})
 
 	t.Run("int no change", func(t *testing.T) {
-		crosstests.Diff(t, pb.NewResource(pb.NewResourceArgs{
-			ResourceSchema: attributeSchema,
-		}), map[string]cty.Value{"key": cty.NumberVal(big.NewFloat(1))}, map[string]cty.Value{"key": cty.NumberVal(big.NewFloat(1))})
+		crosstests.Diff(t, res,
+			map[string]cty.Value{"key": cty.NumberVal(big.NewFloat(1))},
+			map[string]cty.Value{"key": cty.NumberVal(big.NewFloat(1))},
+		)
+	})
+
+	t.Run("type change", func(t *testing.T) {
+		// TODO[pulumi/pulumi-terraform-bridge#3078]
+		t.Skip(`Error converting tftypes.Number<"1"> (value2) at "AttributeName(\"key\")": can't unmarshal tftypes.Number into *string, expected string`)
+		crosstests.Diff(t, res,
+			map[string]cty.Value{"key": cty.StringVal("value")},
+			map[string]cty.Value{"key": cty.NumberVal(big.NewFloat(1))},
+		)
+	})
+}
+
+func TestPFDetailedDiffDynamicTypeWithMigration(t *testing.T) {
+	t.Parallel()
+	// TODO[pulumi/pulumi-terraform-bridge#3078]
+	t.Skip("DynamicPseudoType is not supported")
+
+	attributeSchema := rschema.Schema{
+		Attributes: map[string]rschema.Attribute{
+			"key": rschema.DynamicAttribute{
+				Optional: true,
+			},
+		},
+	}
+	res1 := pb.NewResource(pb.NewResourceArgs{
+		ResourceSchema: attributeSchema,
+	})
+
+	schema2 := rschema.Schema{
+		Attributes: map[string]rschema.Attribute{
+			"key": rschema.DynamicAttribute{
+				Optional: true,
+			},
+		},
+		Version: 1,
+	}
+	res2 := pb.NewResource(pb.NewResourceArgs{
+		ResourceSchema: schema2,
+		UpgradeStateFunc: func(ctx context.Context) map[int64]resource.StateUpgrader {
+			return map[int64]resource.StateUpgrader{
+				0: {
+					PriorSchema: &res1.ResourceSchema,
+					StateUpgrader: func(ctx context.Context, usr1 resource.UpgradeStateRequest, usr2 *resource.UpgradeStateResponse) {
+						usr2.State = *usr1.State
+					},
+				},
+			}
+		},
+	})
+
+	t.Run("no change", func(t *testing.T) {
+		crosstests.Diff(t, res1,
+			map[string]cty.Value{"key": cty.StringVal("value")},
+			map[string]cty.Value{"key": cty.StringVal("value")},
+			crosstests.DiffProviderUpgradedSchema(
+				res2,
+			),
+		)
+	})
+
+	t.Run("change", func(t *testing.T) {
+		crosstests.Diff(t, res1,
+			map[string]cty.Value{"key": cty.StringVal("value")},
+			map[string]cty.Value{"key": cty.StringVal("value1")},
+			crosstests.DiffProviderUpgradedSchema(
+				res2,
+			),
+		)
 	})
 }

pkg/pf/tfbridge/resource_state.go

@@ -273,7 +273,6 @@ func (p *provider) parseAndUpgradeResourceState(
 		return nil, fmt.Errorf("[pf/tfbridge] Error calling NewRawState: %w", err)
 	}
 
-
 	return p.upgradeResourceState(ctx, rh, rawState, parsedMeta.PrivateState, stateVersion)
 }