Set Computed field to true for common.Resource account_id attribute (#3310)

* Set `Computed` field to `true` in `common/resource.go` for `account_id`

* Add unit test to test NoDiff on `account_id` for `databricks_mws_private_access_settings`

Author: 840

common/resource.go

@@ -443,6 +443,7 @@ var NoAuth string = "default auth: cannot configure default credentials, " +
 func AddAccountIdField(s map[string]*schema.Schema) map[string]*schema.Schema {
 	s["account_id"] = &schema.Schema{
 		Type:       schema.TypeString,
+		Computed:   true,
 		Optional:   true,
 		Deprecated: "Configuring `account_id` at the resource-level is deprecated; please specify it in the `provider {}` configuration block instead",
 	}

mws/resource_mws_private_access_settings_test.go

@@ -3,6 +3,8 @@ package mws
 import (
 	"testing"
 
+	"github.com/hashicorp/terraform-plugin-sdk/v2/terraform"
+
 	"github.com/databricks/databricks-sdk-go/apierr"
 	"github.com/databricks/databricks-sdk-go/experimental/mocks"
 	"github.com/databricks/databricks-sdk-go/service/provisioning"
@@ -247,3 +249,26 @@ func TestResourcePASDelete_Error(t *testing.T) {
 	qa.AssertErrorStartsWith(t, err, "Internal error happened")
 	assert.Equal(t, "abc/pas_id", d.Id())
 }
+
+func TestResourcePASUpdateAccountIdNoDiff(t *testing.T) {
+	qa.ResourceFixture{
+		Resource: ResourceMwsPrivateAccessSettings(),
+		ID:       "abc",
+		InstanceState: map[string]string{
+			"account_id":                   "foo",
+			"private_access_settings_name": "pas_name",
+			"public_access_enabled":        "false",
+			"region":                       "eu-west-1",
+			"private_access_level":         "ENDPOINT",
+		},
+		ExpectedDiff: map[string]*terraform.ResourceAttrDiff{
+			"private_access_settings_id": {Old: "", New: "", NewComputed: true, NewRemoved: false, RequiresNew: false, Sensitive: false},
+		},
+		HCL: `
+		private_access_settings_name = "pas_name"
+		public_access_enabled = false
+		region = "eu-west-1"
+		private_access_level = "ENDPOINT"
+		`,
+	}.ApplyNoError(t)
+}