Skip to content

Commit

Permalink
chore: minor formatting for consistency
Browse files Browse the repository at this point in the history 
Signed-off-by: Achim Christ <[email protected]>
  • Loading branch information
@acch
acch committed Nov 7, 2024
1 parent 1c1bf97 commit 24c1882
Show file tree
Hide file tree
Showing 3 changed files with 42 additions and 38 deletions.
66 changes: 33 additions & 33 deletions internal/interfaces/networking_broadcast_domain.go
View file
Original file line number Diff line number Diff line change
Expand Up @@ -62,26 +62,26 @@ func GetBroadcastDomain(errorHandler *utils.ErrorHandler, r restclient.RestClien

statusCode, response, err := r.GetNilOrOneRecord(api, query, nil)
if err == nil && response == nil {
err = fmt.Errorf("no broadcast-domain with id '%s' found", id)
err = fmt.Errorf("no broadcast domain with id '%s' found", id)
}
if err != nil {
return nil, errorHandler.MakeAndReportError(
"error reading broadcast-domain info",
fmt.Sprintf("error on GET %s: %s, statusCode %d", api, err, statusCode),
"Error Reading Broadcast Domain Info",
fmt.Sprintf("Error on GET %s: %s, statusCode %d.", api, err, statusCode),
)
}

var dataONTAP BroadcastDomainGetDataModelONTAP
if err := mapstructure.Decode(response, &dataONTAP); err != nil {
return nil, errorHandler.MakeAndReportError(
fmt.Sprintf("failed to decode response from GET %s", api),
fmt.Sprintf("error: %s, statusCode %d, response %#v", err, statusCode, response),
fmt.Sprintf("Failed To Decode Response From GET %s", api),
fmt.Sprintf("Error: %s, statusCode %d, response %#v.", err, statusCode, response),
)
}

tflog.Debug(
errorHandler.Ctx,
fmt.Sprintf("Read broadcast_domain data source: %#v", dataONTAP),
fmt.Sprintf("Read broadcast domain data source: %#v", dataONTAP),
)

return &dataONTAP, nil
Expand All @@ -104,26 +104,26 @@ func GetBroadcastDomainByName(errorHandler *utils.ErrorHandler, r restclient.Res

statusCode, response, err := r.GetNilOrOneRecord(api, query, nil)
if err == nil && response == nil {
err = fmt.Errorf("no broadcast-domain with ipspace '%s' and name '%s' found", ipspace, name)
err = fmt.Errorf("no broadcast domain with ipspace '%s' and name '%s' found", ipspace, name)
}
if err != nil {
return nil, errorHandler.MakeAndReportError(
"error reading broadcast-domain info",
fmt.Sprintf("error on GET %s: %s, statusCode %d", api, err, statusCode),
"Error Reading Broadcast Domain Info",
fmt.Sprintf("Error on GET %s: %s, statusCode %d.", api, err, statusCode),
)
}

var dataONTAP BroadcastDomainGetDataModelONTAP
if err := mapstructure.Decode(response, &dataONTAP); err != nil {
return nil, errorHandler.MakeAndReportError(
fmt.Sprintf("failed to decode response from GET %s", api),
fmt.Sprintf("error: %s, statusCode %d, response %#v", err, statusCode, response),
fmt.Sprintf("Failed To Decode Response From GET %s", api),
fmt.Sprintf("Error: %s, statusCode %d, response %#v.", err, statusCode, response),
)
}

tflog.Debug(
errorHandler.Ctx,
fmt.Sprintf("Read broadcast_domain data source: %#v", dataONTAP),
fmt.Sprintf("Read broadcast domain data source: %#v", dataONTAP),
)

return &dataONTAP, nil
Expand Down Expand Up @@ -153,12 +153,12 @@ func GetListBroadcastDomains(errorHandler *utils.ErrorHandler, r restclient.Rest

statusCode, response, err := r.GetZeroOrMoreRecords(api, query, nil)
if err == nil && response == nil {
err = fmt.Errorf("no broadcast-domains with ipspace '%s' and name '%s' found", filter.IPspace, filter.Name)
err = fmt.Errorf("no broadcast domains with ipspace '%s' and name '%s' found", filter.IPspace, filter.Name)
}
if err != nil {
return nil, errorHandler.MakeAndReportError(
"error reading broadcast-domains info",
fmt.Sprintf("error on GET %s: %s, statusCode %d", api, err, statusCode),
"Error Reading Broadcast Domains Info",
fmt.Sprintf("Error on GET %s: %s, statusCode %d.", api, err, statusCode),
)
}

Expand All @@ -167,16 +167,16 @@ func GetListBroadcastDomains(errorHandler *utils.ErrorHandler, r restclient.Rest
var record BroadcastDomainGetDataModelONTAP
if err := mapstructure.Decode(info, &record); err != nil {
return nil, errorHandler.MakeAndReportError(
fmt.Sprintf("failed to decode response from GET %s", api),
fmt.Sprintf("error: %s, statusCode %d, info %#v", err, statusCode, info),
fmt.Sprintf("Failed To Decode Response From GET %s", api),
fmt.Sprintf("Error: %s, statusCode %d, info %#v.", err, statusCode, info),
)
}
dataONTAP = append(dataONTAP, record)
}

tflog.Debug(
errorHandler.Ctx,
fmt.Sprintf("Read broadcast_domain data source: %#v", dataONTAP),
fmt.Sprintf("Read broadcast domain data source: %#v", dataONTAP),
)

return dataONTAP, nil
Expand All @@ -189,8 +189,8 @@ func CreateBroadcastDomain(errorHandler *utils.ErrorHandler, r restclient.RestCl
var bodyMap map[string]interface{}
if err := mapstructure.Decode(body, &bodyMap); err != nil {
return nil, errorHandler.MakeAndReportError(
"error encoding broadcast-domain body",
fmt.Sprintf("error on encoding %s body: %s, body: %#v", api, err, body),
"Error Encoding Broadcast Domain Body",
fmt.Sprintf("Error on encoding %s body: %s, body: %#v.", api, err, body),
)
}
query := r.NewQuery()
Expand All @@ -199,22 +199,22 @@ func CreateBroadcastDomain(errorHandler *utils.ErrorHandler, r restclient.RestCl
statusCode, response, err := r.CallCreateMethod(api, query, bodyMap)
if err != nil {
return nil, errorHandler.MakeAndReportError(
"error creating broadcast-domain",
fmt.Sprintf("error on POST %s: %s, statusCode %d", api, err, statusCode),
"Error Creating Broadcast Domain",
fmt.Sprintf("Error on POST %s: %s, statusCode %d.", api, err, statusCode),
)
}

var dataONTAP BroadcastDomainGetDataModelONTAP
if err := mapstructure.Decode(response.Records[0], &dataONTAP); err != nil {
return nil, errorHandler.MakeAndReportError(
"error decoding broadcast-domain info",
fmt.Sprintf("error on decode broadcast-domain info: %s, statusCode %d, response %#v", err, statusCode, response),
"Error Decoding Broadcast Domain Info",
fmt.Sprintf("Error on decode broadcast domain info: %s, statusCode %d, response %#v.", err, statusCode, response),
)
}

tflog.Debug(
errorHandler.Ctx,
fmt.Sprintf("Create broadcast_domain resource: %#v", dataONTAP),
fmt.Sprintf("Create broadcast domain resource: %#v", dataONTAP),
)

return &dataONTAP, nil
Expand All @@ -227,22 +227,22 @@ func UpdateBroadcastDomain(errorHandler *utils.ErrorHandler, r restclient.RestCl
var bodyMap map[string]interface{}
if err := mapstructure.Decode(body, &bodyMap); err != nil {
return errorHandler.MakeAndReportError(
"error encoding broadcast-domain body",
fmt.Sprintf("error on encoding %s body: %s, body: %#v", api, err, body),
"Error Encoding Broadcast Domain Body",
fmt.Sprintf("Error on encoding %s body: %s, body: %#v.", api, err, body),
)
}

statusCode, _, err := r.CallUpdateMethod(api, nil, bodyMap)
if err != nil {
return errorHandler.MakeAndReportError(
"error updating broadcast-domain",
fmt.Sprintf("error on PATCH %s: %s, statusCode %d", api, err, statusCode),
"Error Updating Broadcast Domain",
fmt.Sprintf("Error on PATCH %s: %s, statusCode %d.", api, err, statusCode),
)
}

tflog.Debug(
errorHandler.Ctx,
fmt.Sprintf("Update broadcast_domain resource: %#v", bodyMap),
fmt.Sprintf("Update broadcast domain resource: %#v", bodyMap),
)

return nil
Expand All @@ -256,14 +256,14 @@ func DeleteBroadcastDomain(errorHandler *utils.ErrorHandler, r restclient.RestCl
statusCode, _, err := r.CallDeleteMethod(api, nil, nil)
if err != nil {
return errorHandler.MakeAndReportError(
"error deleting broadcast-domain",
fmt.Sprintf("error on DELETE %s: %s, statusCode %d", api, err, statusCode),
"Error Deleting Broadcast Domain",
fmt.Sprintf("Error on DELETE %s: %s, statusCode %d.", api, err, statusCode),
)
}

tflog.Debug(
errorHandler.Ctx,
fmt.Sprintf("Delete broadcast_domain resource: %s", id),
fmt.Sprintf("Delete broadcast domain resource: %s", id),
)

return nil
Expand Down
6 changes: 4 additions & 2 deletions internal/provider/networking/network_broadcast_domain_data_source.go
View file
Original file line number Diff line number Diff line change
Expand Up @@ -133,8 +133,10 @@ func (d *BroadcastDomainDataSource) Read(ctx context.Context, req datasource.Rea
}
if restInfo == nil {
errorHandler.MakeAndReportError(
"No broadcast domain found",
fmt.Sprintf("No broadcast-domain '%s' found.", data.Name.ValueString()))
"No Broadcast Domain Found",
fmt.Sprintf("No broadcast domain '%s' found.", data.Name.ValueString()),
)

return
}

Expand Down
8 changes: 5 additions & 3 deletions internal/provider/networking/network_broadcast_domain_resource.go
View file
Original file line number Diff line number Diff line change
Expand Up @@ -165,9 +165,10 @@ func (r *BroadcastDomainResource) Read(ctx context.Context, req resource.ReadReq
}
if restInfo == nil {
errorHandler.MakeAndReportError(
"No broadcast domain found",
fmt.Sprintf("No broadcast-domain, %s found.", data.Name.ValueString()),
"No Broadcast Domain Found",
fmt.Sprintf("No broadcast domain '%s' found.", data.Name.ValueString()),
)

return
}

Expand Down Expand Up @@ -300,7 +301,8 @@ func (r *BroadcastDomainResource) Delete(ctx context.Context, req resource.Delet

// Ensure that ID in known
if data.ID.IsNull() {
errorHandler.MakeAndReportError("ID is null", "broadcast_domain ID is null")
errorHandler.MakeAndReportError("ID Is Null", "Broadcast domain ID is null.")

return
}

Expand Down

0 comments on commit 24c1882

Please sign in to comment.