Merge pull request #11 from enthought/fix-organization-team

ddebeau · web-flow · commit 1fc48227ff69 · 2024-09-06T13:43:23.000-05:00
Change OrganizationTeam members from a list to a set
diff --git a/code_generator/provider_code_spec.json b/code_generator/provider_code_spec.json
@@ -114,7 +114,7 @@
           },
           {
             "name": "members",
-            "list": {
+            "set": {
               "computed_optional_required": "optional",
               "description": "List of team members",
               "element_type": {
@@ -413,7 +413,7 @@
           },
           {
             "name": "members",
-            "list": {
+            "set": {
               "computed_optional_required": "computed",
               "description": "List of team members",
               "element_type": {
diff --git a/docs/data-sources/organization_team.md b/docs/data-sources/organization_team.md
@@ -45,5 +45,5 @@ data "quay_organization_team" "team" {
 ### Read-Only
 
 - `description` (String) Markdown description
-- `members` (List of String) List of team members
+- `members` (Set of String) List of team members
 - `role` (String) Team role
diff --git a/docs/resources/organization_team.md b/docs/resources/organization_team.md
@@ -45,7 +45,7 @@ resource "quay_organization_team" "admin" {
 ### Optional
 
 - `description` (String) Markdown description
-- `members` (List of String) List of team members
+- `members` (Set of String) List of team members
 
 ## Import
 
diff --git a/internal/datasource_organization_team/organization_team_data_source_gen.go b/internal/datasource_organization_team/organization_team_data_source_gen.go
diff --git a/internal/provider/organization_team_data_source.go b/internal/provider/organization_team_data_source.go
@@ -78,7 +78,7 @@ func (d *organizationTeamDataSource) Read(ctx context.Context, req datasource.Re
 	for _, member := range resTeamData.Members {
 		memList = append(memList, member.Name)
 	}
-	members, diags := types.ListValueFrom(ctx, types.StringType, memList)
+	members, diags := types.SetValueFrom(ctx, types.StringType, memList)
 
 	resp.Diagnostics.Append(diags...)
 	if resp.Diagnostics.HasError() {
diff --git a/internal/provider/organization_team_resource.go b/internal/provider/organization_team_resource.go
@@ -152,7 +152,7 @@ func (r *organizationTeamResource) Read(ctx context.Context, req resource.ReadRe
 	for _, member := range resTeamData.Members {
 		memList = append(memList, member.Name)
 	}
-	members, diags := types.ListValueFrom(ctx, types.StringType, memList)
+	members, diags := types.SetValueFrom(ctx, types.StringType, memList)
 
 	resp.Diagnostics.Append(diags...)
 	if resp.Diagnostics.HasError() {
diff --git a/internal/resource_organization_team/organization_team_resource_gen.go b/internal/resource_organization_team/organization_team_resource_gen.go

Original file line number	Diff line number	Diff line change
`@@ -78,7 +78,7 @@ func (d *organizationTeamDataSource) Read(ctx context.Context, req datasource.Re`
`78`	`78`	`for _, member := range resTeamData.Members {`
`79`	`79`	`memList = append(memList, member.Name)`
`80`	`80`	`}`
`81`		`- members, diags := types.ListValueFrom(ctx, types.StringType, memList)`
	`81`	`+ members, diags := types.SetValueFrom(ctx, types.StringType, memList)`
`82`	`82`
`83`	`83`	`resp.Diagnostics.Append(diags...)`
`84`	`84`	`if resp.Diagnostics.HasError() {`
Original file line number	Diff line number	Diff line change
`@@ -152,7 +152,7 @@ func (r *organizationTeamResource) Read(ctx context.Context, req resource.ReadRe`
`152`	`152`	`for _, member := range resTeamData.Members {`
`153`	`153`	`memList = append(memList, member.Name)`
`154`	`154`	`}`
`155`		`- members, diags := types.ListValueFrom(ctx, types.StringType, memList)`
	`155`	`+ members, diags := types.SetValueFrom(ctx, types.StringType, memList)`
`156`	`156`
`157`	`157`	`resp.Diagnostics.Append(diags...)`
`158`	`158`	`if resp.Diagnostics.HasError() {`