Replace list with set for roles

Author: Sander0542

docs/data-sources/user.md

@@ -36,7 +36,7 @@ data "nginxproxymanager_user" "user" {
 - `name` (String) The name of the user.
 - `nickname` (String) The nickname of the user.
 - `permissions` (Attributes) The permissions of the user. (see [below for nested schema](#nestedatt--permissions))
-- `roles` (List of String) The roles of the user.
+- `roles` (Set of String) The roles of the user.
 
 <a id="nestedatt--permissions"></a>
 ### Nested Schema for `permissions`

docs/data-sources/user_me.md

@@ -31,7 +31,7 @@ data "nginxproxymanager_user_me" "user_me" {}
 - `name` (String) The name of the user.
 - `nickname` (String) The nickname of the user.
 - `permissions` (Attributes) The permissions of the user. (see [below for nested schema](#nestedatt--permissions))
-- `roles` (List of String) The roles of the user.
+- `roles` (Set of String) The roles of the user.
 
 <a id="nestedatt--permissions"></a>
 ### Nested Schema for `permissions`

docs/data-sources/users.md

@@ -38,7 +38,7 @@ Read-Only:
 - `name` (String) The name of the user.
 - `nickname` (String) The nickname of the user.
 - `permissions` (Attributes) The permissions of the user. (see [below for nested schema](#nestedatt--users--permissions))
-- `roles` (List of String) The roles of the user.
+- `roles` (Set of String) The roles of the user.
 
 <a id="nestedatt--users--permissions"></a>
 ### Nested Schema for `users.permissions`

internal/provider/models/user.go

@@ -21,7 +21,7 @@ type User struct {
 	Email       types.String `tfsdk:"email"`
 	Avatar      types.String `tfsdk:"avatar"`
 	IsDisabled  types.Bool   `tfsdk:"is_disabled"`
-	Roles       types.List   `tfsdk:"roles"`
+	Roles       types.Set    `tfsdk:"roles"`
 	Permissions types.Object `tfsdk:"permissions"`
 }
 
@@ -35,7 +35,7 @@ func (_ User) GetType() attr.Type {
 		"email":       types.StringType,
 		"avatar":      types.StringType,
 		"is_disabled": types.BoolType,
-		"roles":       types.ListType{ElemType: types.StringType},
+		"roles":       types.SetType{ElemType: types.StringType},
 		"permissions": types.ObjectType{AttrTypes: UserPermissions{}.GetType().AttributeTypes()},
 	})
 }
@@ -53,7 +53,7 @@ func (m *User) Write(ctx context.Context, user *nginxproxymanager.GetAccessLists
 	m.Avatar = types.StringValue(user.GetAvatar())
 	m.IsDisabled = types.BoolValue(user.GetIsDisabled())
 
-	m.Roles, tmpDiags = types.ListValueFrom(ctx, types.StringType, user.GetRoles())
+	m.Roles, tmpDiags = types.SetValueFrom(ctx, types.StringType, user.GetRoles())
 	diags.Append(tmpDiags...)
 
 	if user.HasPermissions() {

internal/provider/user_data_source.go

@@ -65,7 +65,7 @@ func (d *UserDataSource) Schema(ctx context.Context, req datasource.SchemaReques
 				MarkdownDescription: "Whether the user is disabled.",
 				Computed:            true,
 			},
-			"roles": schema.ListAttribute{
+			"roles": schema.SetAttribute{
 				MarkdownDescription: "The roles of the user.",
 				Computed:            true,
 				ElementType:         types.StringType,

internal/provider/user_data_source_test.go

@@ -70,7 +70,7 @@ func TestAccUserDataSource(t *testing.T) {
 					statecheck.ExpectKnownValue(
 						"data.nginxproxymanager_user.test",
 						tfjsonpath.New("roles"),
-						knownvalue.ListExact([]knownvalue.Check{
+						knownvalue.SetExact([]knownvalue.Check{
 							knownvalue.StringExact("admin"),
 						}),
 					),

internal/provider/user_me_data_source.go

@@ -65,7 +65,7 @@ func (d *UserMeDataSource) Schema(ctx context.Context, req datasource.SchemaRequ
 				MarkdownDescription: "Whether the user is disabled.",
 				Computed:            true,
 			},
-			"roles": schema.ListAttribute{
+			"roles": schema.SetAttribute{
 				MarkdownDescription: "The roles of the user.",
 				Computed:            true,
 				ElementType:         types.StringType,

internal/provider/user_me_data_source_test.go

@@ -70,7 +70,7 @@ func TestAccUserMeDataSource(t *testing.T) {
 					statecheck.ExpectKnownValue(
 						"data.nginxproxymanager_user_me.test",
 						tfjsonpath.New("roles"),
-						knownvalue.ListExact([]knownvalue.Check{
+						knownvalue.SetExact([]knownvalue.Check{
 							knownvalue.StringExact("admin"),
 						}),
 					),

internal/provider/users_data_source.go

@@ -70,7 +70,7 @@ func (d *UsersDataSource) Schema(ctx context.Context, req datasource.SchemaReque
 							MarkdownDescription: "Whether the user is disabled.",
 							Computed:            true,
 						},
-						"roles": schema.ListAttribute{
+						"roles": schema.SetAttribute{
 							MarkdownDescription: "The roles of the user.",
 							Computed:            true,
 							ElementType:         types.StringType,

internal/provider/users_data_source_test.go

@@ -40,7 +40,7 @@ func TestAccUsersDataSource(t *testing.T) {
 								"email":       knownvalue.StringExact("admin@example.com"),
 								"avatar":      knownvalue.StringExact(""),
 								"is_disabled": knownvalue.Bool(false),
-								"roles": knownvalue.ListExact([]knownvalue.Check{
+								"roles": knownvalue.SetExact([]knownvalue.Check{
 									knownvalue.StringExact("admin"),
 								}),
 								"permissions": knownvalue.ObjectExact(map[string]knownvalue.Check{