ref: make some optional client key fields computed

Author: jianyuan

docs/resources/key.md

@@ -33,7 +33,7 @@ resource "sentry_key" "default" {
 
 ### Optional
 
-- `javascript_loader_script` (Block, Optional) The JavaScript loader script configuration. (see [below for nested schema](#nestedblock--javascript_loader_script))
+- `javascript_loader_script` (Attributes) The JavaScript loader script configuration. (see [below for nested schema](#nestedatt--javascript_loader_script))
 - `rate_limit_count` (Number) Number of events that can be reported within the rate limit window.
 - `rate_limit_window` (Number) Length of time in seconds that will be considered when checking the rate limit.
 
@@ -48,7 +48,7 @@ resource "sentry_key" "default" {
 - `public` (String) The public key.
 - `secret` (String) The secret key.
 
-<a id="nestedblock--javascript_loader_script"></a>
+<a id="nestedatt--javascript_loader_script"></a>
 ### Nested Schema for `javascript_loader_script`
 
 Optional:

internal/provider/data_source_all_client_keys.go

@@ -21,14 +21,13 @@ type AllClientKeysDataSourceModel struct {
 	Keys         []ClientKeyResourceModel `tfsdk:"keys"`
 }
 
-func (m *AllClientKeysDataSourceModel) Fill(keys []apiclient.ProjectKey) error {
+func (m *AllClientKeysDataSourceModel) Fill(ctx context.Context, keys []apiclient.ProjectKey) error {
 	m.Keys = make([]ClientKeyResourceModel, len(keys))
 	for i, key := range keys {
-		if err := m.Keys[i].FillAll(
-			m.Organization.ValueString(),
-			m.Project.ValueString(),
-			key,
-		); err != nil {
+		m.Keys[i].Organization = types.StringValue(m.Organization.ValueString())
+		m.Keys[i].Project = types.StringValue(m.Organization.ValueString())
+
+		if err := m.Keys[i].Fill(ctx, key); err != nil {
 			return err
 		}
 	}
@@ -194,7 +193,7 @@ func (d *AllClientKeysDataSource) Read(ctx context.Context, req datasource.ReadR
 		}
 	}
 
-	if err := data.Fill(allKeys); err != nil {
+	if err := data.Fill(ctx, allKeys); err != nil {
 		resp.Diagnostics.Append(diagutils.NewFillError(err))
 		return
 	}

internal/provider/data_source_client_key_test.go

@@ -229,7 +229,11 @@ data "sentry_key" "test" {
 }
 
 func testAccClientKeyDataSourceConfig(teamName, projectName, keyName, extras string) string {
-	return testAccClientKeyResourceConfig(teamName, projectName, keyName, "") + fmt.Sprintf(`
+	return testAccClientKeyResourceConfig(testAccClientKeyResourceConfigData{
+		TeamName:    teamName,
+		ProjectName: projectName,
+		KeyName:     keyName,
+	}) + fmt.Sprintf(`
 data "sentry_key" "test" {
 	organization = sentry_project.test.organization
 	project      = sentry_project.test.id