r/pagerduty_user: Set time_zone as optional

pagerduty/resource_pagerduty_user.go

@@ -21,15 +21,18 @@ func resourcePagerDutyUser() *schema.Resource {
 				Type:     schema.TypeString,
 				Required: true,
 			},
+
 			"email": {
 				Type:     schema.TypeString,
 				Required: true,
 			},
+
 			"color": {
 				Type:     schema.TypeString,
 				Optional: true,
 				Computed: true,
 			},
+
 			"role": {
 				Type:     schema.TypeString,
 				Optional: true,
@@ -43,14 +46,17 @@ func resourcePagerDutyUser() *schema.Resource {
 					"user",
 				}),
 			},
+
 			"job_title": {
 				Type:     schema.TypeString,
 				Optional: true,
 			},
+
 			"avatar_url": {
 				Type:     schema.TypeString,
 				Computed: true,
 			},
+
 			"teams": {
 				Type:     schema.TypeSet,
 				Optional: true,
@@ -59,18 +65,23 @@ func resourcePagerDutyUser() *schema.Resource {
 				},
 				Set: schema.HashString,
 			},
+
 			"time_zone": {
 				Type:     schema.TypeString,
+				Optional: true,
 				Computed: true,
 			},
+
 			"html_url": {
 				Type:     schema.TypeString,
 				Computed: true,
 			},
+
 			"invitation_sent": {
 				Type:     schema.TypeBool,
 				Computed: true,
 			},
+
 			"description": {
 				Type:     schema.TypeString,
 				Optional: true,
@@ -90,6 +101,10 @@ func buildUserStruct(d *schema.ResourceData) *pagerduty.User {
 		user.Color = attr.(string)
 	}
 
+	if attr, ok := d.GetOk("time_zone"); ok {
+		user.TimeZone = attr.(string)
+	}
+
 	if attr, ok := d.GetOk("role"); ok {
 		role := attr.(string)
 		// Skip setting the role if the user is the owner of the account.

pagerduty/resource_pagerduty_user_test.go

@@ -191,6 +191,7 @@ resource "pagerduty_user" "foo" {
   role        = "user"
   job_title   = "foo"
   description = "foo"
+  time_zone   = "Europe/Berlin"
 }`, username, email)
 }
 
@@ -203,6 +204,7 @@ resource "pagerduty_user" "foo" {
   role        = "team_responder"
   job_title   = "bar"
   description = "bar"
+  time_zone   = "Europe/Dublin"
 }`, username, email)
 }