Skip to content

Commit

Permalink
Implementation of "active" property on resource "aws-sso-scim_user" (#15
Browse files Browse the repository at this point in the history
)

* Implementation of "active" property on resource "aws-sso-scim_user"
  • Loading branch information
manospasj authored Sep 6, 2021
1 parent 701e6da commit e937ed0
Show file tree
Hide file tree
Showing 3 changed files with 12 additions and 4 deletions.
5 changes: 1 addition & 4 deletions docs/resources/user.md
Original file line number Diff line number Diff line change
Expand Up @@ -9,8 +9,6 @@ description: |-

SCIM User resource.



## Schema

### Required
Expand All @@ -19,9 +17,8 @@ SCIM User resource.
- **family_name** (String) Family Name
- **given_name** (String) Given Name
- **user_name** (String) Username
- **active** (Boolean) Active

### Read-only

- **id** (String) The ID of this resource.


9 changes: 9 additions & 0 deletions internal/provider/resource_user.go
Original file line number Diff line number Diff line change
Expand Up @@ -43,6 +43,12 @@ func resourceUser() *schema.Resource {
Type: schema.TypeString,
Required: true,
},
"active": {
Description: "Active",
Type: schema.TypeBool,
Optional: true,
Default: false,
},
},
}
}
Expand All @@ -58,6 +64,7 @@ func resourceUserCreate(ctx context.Context, d *schema.ResourceData, meta interf
FamilyName: d.Get("family_name").(string),
GivenName: d.Get("given_name").(string),
},
Active: d.Get("active").(bool),
}

user, err := client.CreateUser(&new_user)
Expand Down Expand Up @@ -95,6 +102,7 @@ func resourceUserRead(ctx context.Context, d *schema.ResourceData, meta interfac
d.Set("user_name", user.UserName)
d.Set("family_name", user.Name.FamilyName)
d.Set("given_name", user.Name.GivenName)
d.Set("active", user.Active)

return diags
}
Expand Down Expand Up @@ -128,6 +136,7 @@ func resourceUserUpdate(ctx context.Context, d *schema.ResourceData, meta interf
"display_name": "displayName",
"family_name": "name.familyName",
"given_name": "name.givenName",
"active": "active",
}

for attribute, path := range required_map {
Expand Down
2 changes: 2 additions & 0 deletions internal/provider/resource_user_test.go
Original file line number Diff line number Diff line change
Expand Up @@ -34,6 +34,7 @@ resource "aws-sso-scim_user" "foo" {
user_name = "terraform-test-temporary-user-user_name"
family_name = "terraform-test-temporary-user-family_name"
given_name = "terraform-test-temporary-user-given_name"
active = true
}
`

Expand All @@ -43,5 +44,6 @@ resource "aws-sso-scim_user" "foo" {
user_name = "terraform-test-temporary-user-user_name2"
family_name = "terraform-test-temporary-user-family_name2"
given_name = "terraform-test-temporary-user-given_name2"
active = false
}
`

0 comments on commit e937ed0

Please sign in to comment.