fix(workers_script): Obtain migrations directly from config instead of plan

Fixes #5898

Author: 1000hz

internal/services/workers_script/resource.go

@@ -19,6 +19,7 @@ import (
 	"github.com/cloudflare/terraform-provider-cloudflare/internal/importpath"
 	"github.com/cloudflare/terraform-provider-cloudflare/internal/logging"
 	"github.com/hashicorp/terraform-plugin-framework/diag"
+	"github.com/hashicorp/terraform-plugin-framework/path"
 	"github.com/hashicorp/terraform-plugin-framework/resource"
 	"github.com/hashicorp/terraform-plugin-framework/types"
 	"github.com/jinzhu/copier"
@@ -65,6 +66,7 @@ func (r *WorkersScriptResource) Create(ctx context.Context, req resource.CreateR
 	var data *WorkersScriptModel
 
 	resp.Diagnostics.Append(req.Plan.Get(ctx, &data)...)
+	resp.Diagnostics.Append(req.Config.GetAttribute(ctx, path.Root("migrations"), &data.Migrations)...)
 
 	if resp.Diagnostics.HasError() {
 		return
@@ -126,6 +128,7 @@ func (r *WorkersScriptResource) Update(ctx context.Context, req resource.UpdateR
 	var data *WorkersScriptModel
 
 	resp.Diagnostics.Append(req.Plan.Get(ctx, &data)...)
+	resp.Diagnostics.Append(req.Config.GetAttribute(ctx, path.Root("migrations"), &data.Migrations)...)
 
 	if resp.Diagnostics.HasError() {
 		return

internal/services/workers_script/resource_test.go

@@ -339,6 +339,38 @@ func TestAccCloudflareWorkerScript_PythonWorker(t *testing.T) {
 	})
 }
 
+func TestAccCloudflareWorkerScript_ModuleWithDurableObject(t *testing.T) {
+	t.Parallel()
+
+	rnd := utils.GenerateRandomResourceName()
+	name := "cloudflare_workers_script." + rnd
+	accountID := os.Getenv("CLOUDFLARE_ACCOUNT_ID")
+
+	resource.Test(t, resource.TestCase{
+		PreCheck: func() {
+			acctest.TestAccPreCheck(t)
+			acctest.TestAccPreCheck_AccountID(t)
+		},
+		ProtoV6ProviderFactories: acctest.TestAccProtoV6ProviderFactories,
+		Steps: []resource.TestStep{
+			{
+				Config: acctest.LoadTestCase("module_with_durable_object.tf", rnd, accountID),
+				ConfigStateChecks: []statecheck.StateCheck{
+					statecheck.ExpectKnownValue(name, tfjsonpath.New("script_name"), knownvalue.StringExact(rnd)),
+					statecheck.ExpectKnownValue(name, tfjsonpath.New("main_module"), knownvalue.StringExact("worker.js")),
+				},
+			},
+			{
+				ResourceName:            name,
+				ImportStateIdPrefix:     fmt.Sprintf("%s/", accountID),
+				ImportState:             true,
+				ImportStateVerify:       true,
+				ImportStateVerifyIgnore: []string{"bindings.0.namespace_id", "has_modules", "main_module", "startup_time_ms"},
+			},
+		},
+	})
+}
+
 func testAccCheckCloudflareWorkerScriptConfigServiceWorkerInitial(rnd, accountID string) string {
 	return acctest.LoadTestCase("service_worker_initial.tf", rnd, scriptContent1, accountID)
 }

internal/services/workers_script/testdata/module_with_durable_object.tf

@@ -0,0 +1,21 @@
+resource "cloudflare_workers_script" "%[1]s" {
+  account_id = "%[2]s"
+  script_name = "%[1]s"
+  content = <<-EOT
+    import {DurableObject} from "cloudflare:workers"
+    export class MyDurableObject extends DurableObject {}
+    export default { fetch() {return new Response()} }
+  EOT
+  main_module = "worker.js"
+  migrations = {
+    new_tag = "v1"
+    new_sqlite_classes = ["MyDurableObject"]
+  }
+  bindings = [
+    {
+      name = "MY_DO"
+      type = "durable_object_namespace"
+      class_name = "MyDurableObject"
+    }
+  ]
+}