Handles 204 responses from AWX.

.gitignore

@@ -42,3 +42,5 @@ terraform
 
 develop/.env
 develop/terraform-provider-*
+
+test.go

VERSION

@@ -1 +1 @@
-v1.0.3
+v1.0.4

internal/awx/resource_team.go

@@ -105,7 +105,7 @@ func resourceTeamCreate(ctx context.Context, d *schema.ResourceData, m interface
 	return resourceTeamRead(ctx, d, m)
 }
 
-func roleTeamEntitlementUpdate(m interface{}, team_id int, roles []interface{}, remove bool) error {
+func roleTeamEntitlementUpdate(m interface{}, teamID int, roles []interface{}, remove bool) error {
 	client := m.(*awx.AWX)
 	awxService := client.TeamService
 
@@ -118,7 +118,7 @@ func roleTeamEntitlementUpdate(m interface{}, team_id int, roles []interface{},
 			payload["disassociate"] = true // presence of key triggers removal
 		}
 
-		_, err := awxService.UpdateTeamRoleEntitlement(team_id, payload, make(map[string]string))
+		_, err := awxService.UpdateTeamRoleEntitlement(teamID, payload, make(map[string]string))
 		if err != nil {
 			return err
 		}

tools/goawx/request.go

@@ -107,14 +107,14 @@ func (r *Requester) Do(ar *APIRequest, responseStruct interface{}, options ...in
 
 	response, err := r.Client.Do(req)
 	if err != nil {
-		return nil, err
+		return nil, fmt.Errorf("Do.Request: %v", err)
 	}
 
 	//nolint:gomnd
 	if response.StatusCode == 400 { // Bad Request
 		var errorString string
 
-		errorList := map[string][]string{}
+		errorList := map[string]interface{}{}
 		if err := json.NewDecoder(response.Body).Decode(&errorList); err != nil {
 			return response, err
 		}
@@ -127,6 +127,11 @@ func (r *Requester) Do(ar *APIRequest, responseStruct interface{}, options ...in
 		return response, errors.New(errorString)
 	}
 
+	// If there is no response body, or if the response is of type `No Content` bypass the decode process.
+	if responseStruct == nil || response.StatusCode == 204 { //nolint:gomnd
+		return response, nil
+	}
+
 	switch responseStruct.(type) {
 	case *string:
 		return r.ReadRawResponse(response, responseStruct)

tools/goawx/teams.go

@@ -318,12 +318,12 @@ func (t *TeamService) UpdateTeam(id int, data map[string]interface{}, params map
 }
 
 // UpdateTeamRoleEntitlement updates the role entitlements for a team.
-func (t *TeamService) UpdateTeamRoleEntitlement(id int, data map[string]interface{}, params map[string]string) (interface{}, error) {
+func (t *TeamService) UpdateTeamRoleEntitlement(id int, data map[string]interface{}, _ map[string]string) (interface{}, error) {
 	result := new(interface{})
 	endpoint := fmt.Sprintf("%s%d/roles/", teamsAPIEndpoint, id)
 	payload, err := json.Marshal(data)
 	if err != nil {
-		return nil, err
+		return nil, fmt.Errorf("UpdateTeamRoleEntitlement:json.Marshal error in payload. %s", err)
 	}
 
 	resp, err := t.client.Requester.PostJSON(endpoint, bytes.NewReader(payload), result, nil)
@@ -335,7 +335,7 @@ func (t *TeamService) UpdateTeamRoleEntitlement(id int, data map[string]interfac
 		}()
 	}
 	if err != nil {
-		return nil, err
+		return nil, fmt.Errorf("UpdateTeamRoleEntitlement:Requester.PostJSON error. %s", err)
 	}
 
 	if err := CheckResponse(resp); err != nil {