diff --git a/sdk/authentication/token_helper.go b/sdk/authentication/token_helper.go index e5d5ab66..6b9cb332 100644 --- a/sdk/authentication/token_helper.go +++ b/sdk/authentication/token_helper.go @@ -73,25 +73,14 @@ func GetToken(credentials settings.AuthenticationSettings, httpSettings *setting if err != nil { return localVarReturnValue, localVarHttpResponse, err } - if localVarHttpResponse.StatusCode < 300 { - err = decode(&localVarReturnValue, localVarBody, localVarHttpResponse.Header.Get("Content-Type")) - if err == nil { - return localVarReturnValue, localVarHttpResponse, err - } - } - if localVarHttpResponse.StatusCode >= 300 { + + if localVarHttpResponse.StatusCode < 200 || localVarHttpResponse.StatusCode >= 300 { newErr := fmt.Errorf(string(localVarBody)) - if localVarHttpResponse.StatusCode == 200 { - var v model.Task - err = decode(&v, localVarBody, localVarHttpResponse.Header.Get("Content-Type")) - if err != nil { - return localVarReturnValue, localVarHttpResponse, newErr - } - return localVarReturnValue, localVarHttpResponse, newErr - } return localVarReturnValue, localVarHttpResponse, newErr + } else { + err = decode(&localVarReturnValue, localVarBody, localVarHttpResponse.Header.Get("Content-Type")) } - return localVarReturnValue, localVarHttpResponse, nil + return localVarReturnValue, localVarHttpResponse, err } func prepareRequest( diff --git a/sdk/client/api_application_resource.go b/sdk/client/api_application_resource.go index 33a9d283..b961dfdc 100644 --- a/sdk/client/api_application_resource.go +++ b/sdk/client/api_application_resource.go @@ -82,20 +82,14 @@ func (a *ApplicationResourceApiService) AddRoleToApplicationUser(ctx context.Con return returnValue, httpResponse, err } - if httpResponse.StatusCode < 300 { - // If we succeed, return the data, otherwise pass on to decode error. + if isSuccessfulStatus(httpResponse.StatusCode) { err = a.decode(&returnValue, responseBody, httpResponse.Header.Get("Content-Type")) - if err == nil { - return returnValue, httpResponse, err - } - } - - if httpResponse.StatusCode >= 300 { + } else { newErr := NewGenericSwaggerError(responseBody, httpResponse.Status, nil, httpResponse.StatusCode) return returnValue, httpResponse, newErr } - return returnValue, httpResponse, nil + return returnValue, httpResponse, err } /* @@ -154,20 +148,14 @@ func (a *ApplicationResourceApiService) CreateAccessKey(ctx context.Context, id return nil, httpResponse, err } - if httpResponse.StatusCode < 300 { - // If we succeed, return the data, otherwise pass on to decode error. + if isSuccessfulStatus(httpResponse.StatusCode) { err = a.decode(&returnValue, responseBody, httpResponse.Header.Get("Content-Type")) - if err == nil { - return &returnValue, httpResponse, err - } - } - - if httpResponse.StatusCode >= 300 { + } else { newErr := NewGenericSwaggerError(responseBody, httpResponse.Status, nil, httpResponse.StatusCode) return nil, httpResponse, newErr } - return nil, httpResponse, nil + return &returnValue, httpResponse, err } /* @@ -227,20 +215,14 @@ func (a *ApplicationResourceApiService) CreateApplication(ctx context.Context, b return nil, httpResponse, err } - if httpResponse.StatusCode < 300 { - // If we succeed, return the data, otherwise pass on to decode error. + if isSuccessfulStatus(httpResponse.StatusCode) { err = a.decode(&returnValue, responseBody, httpResponse.Header.Get("Content-Type")) - if err == nil { - return &returnValue, httpResponse, err - } - } - - if httpResponse.StatusCode >= 300 { + } else { newErr := NewGenericSwaggerError(responseBody, httpResponse.Status, nil, httpResponse.StatusCode) return nil, httpResponse, newErr } - return nil, httpResponse, nil + return &returnValue, httpResponse, err } /* @@ -301,20 +283,13 @@ func (a *ApplicationResourceApiService) DeleteAccessKey(ctx context.Context, app return httpResponse, err } - if httpResponse.StatusCode < 300 { - // If we succeed, return the data, otherwise pass on to decode error. + if isSuccessfulStatus(httpResponse.StatusCode) { err = a.decode(&returnValue, responseBody, httpResponse.Header.Get("Content-Type")) - if err == nil { - return httpResponse, err - } - } - - if httpResponse.StatusCode >= 300 { - newErr := NewGenericSwaggerError(responseBody, httpResponse.Status, nil, httpResponse.StatusCode) - return httpResponse, newErr + } else { + return httpResponse, NewGenericSwaggerError(responseBody, httpResponse.Status, nil, httpResponse.StatusCode) } - return httpResponse, nil + return httpResponse, err } /* @@ -373,20 +348,14 @@ func (a *ApplicationResourceApiService) DeleteApplication(ctx context.Context, i return returnValue, httpResponse, err } - if httpResponse.StatusCode < 300 { - // If we succeed, return the data, otherwise pass on to decode error. + if isSuccessfulStatus(httpResponse.StatusCode) { err = a.decode(&returnValue, responseBody, httpResponse.Header.Get("Content-Type")) - if err == nil { - return returnValue, httpResponse, err - } - } - - if httpResponse.StatusCode >= 300 { + } else { newErr := NewGenericSwaggerError(responseBody, httpResponse.Status, nil, httpResponse.StatusCode) return returnValue, httpResponse, newErr } - return returnValue, httpResponse, nil + return returnValue, httpResponse, err } /* @@ -446,9 +415,8 @@ func (a *ApplicationResourceApiService) DeleteTagForApplication(ctx context.Cont return httpResponse, err } - if httpResponse.StatusCode >= 300 { - newErr := NewGenericSwaggerError(responseBody, httpResponse.Status, nil, httpResponse.StatusCode) - return httpResponse, newErr + if !isSuccessfulStatus(httpResponse.StatusCode) { + return httpResponse, NewGenericSwaggerError(responseBody, httpResponse.Status, nil, httpResponse.StatusCode) } return httpResponse, nil @@ -510,20 +478,14 @@ func (a *ApplicationResourceApiService) GetAccessKeys(ctx context.Context, id st return returnValue, httpResponse, err } - if httpResponse.StatusCode < 300 { - // If we succeed, return the data, otherwise pass on to decode error. + if isSuccessfulStatus(httpResponse.StatusCode) { err = a.decode(&returnValue, responseBody, httpResponse.Header.Get("Content-Type")) - if err == nil { - return returnValue, httpResponse, err - } - } - - if httpResponse.StatusCode >= 300 { + } else { newErr := NewGenericSwaggerError(responseBody, httpResponse.Status, nil, httpResponse.StatusCode) return returnValue, httpResponse, newErr } - return returnValue, httpResponse, nil + return returnValue, httpResponse, err } /* @@ -582,20 +544,14 @@ func (a *ApplicationResourceApiService) GetAppByAccessKeyId(ctx context.Context, return returnValue, httpResponse, err } - if httpResponse.StatusCode < 300 { - // If we succeed, return the data, otherwise pass on to decode error. + if isSuccessfulStatus(httpResponse.StatusCode) { err = a.decode(&returnValue, responseBody, httpResponse.Header.Get("Content-Type")) - if err == nil { - return returnValue, httpResponse, err - } - } - - if httpResponse.StatusCode >= 300 { + } else { newErr := NewGenericSwaggerError(responseBody, httpResponse.Status, nil, httpResponse.StatusCode) return returnValue, httpResponse, newErr } - return returnValue, httpResponse, nil + return returnValue, httpResponse, err } /* @@ -654,20 +610,14 @@ func (a *ApplicationResourceApiService) GetApplication(ctx context.Context, id s return nil, httpResponse, err } - if httpResponse.StatusCode < 300 { - // If we succeed, return the data, otherwise pass on to decode error. + if isSuccessfulStatus(httpResponse.StatusCode) { err = a.decode(&returnValue, responseBody, httpResponse.Header.Get("Content-Type")) - if err == nil { - return &returnValue, httpResponse, err - } - } - - if httpResponse.StatusCode >= 300 { + } else { newErr := NewGenericSwaggerError(responseBody, httpResponse.Status, nil, httpResponse.StatusCode) return nil, httpResponse, newErr } - return nil, httpResponse, nil + return &returnValue, httpResponse, err } /* @@ -726,20 +676,14 @@ func (a *ApplicationResourceApiService) GetTagsForApplication(ctx context.Contex return returnValue, httpResponse, err } - if httpResponse.StatusCode < 300 { - // If we succeed, return the data, otherwise pass on to decode error. + if isSuccessfulStatus(httpResponse.StatusCode) { err = a.decode(&returnValue, responseBody, httpResponse.Header.Get("Content-Type")) - if err == nil { - return returnValue, httpResponse, err - } - } - - if httpResponse.StatusCode >= 300 { + } else { newErr := NewGenericSwaggerError(responseBody, httpResponse.Status, nil, httpResponse.StatusCode) return returnValue, httpResponse, newErr } - return returnValue, httpResponse, nil + return returnValue, httpResponse, err } /* @@ -796,20 +740,14 @@ func (a *ApplicationResourceApiService) ListApplications(ctx context.Context) ([ return returnValue, httpResponse, err } - if httpResponse.StatusCode < 300 { - // If we succeed, return the data, otherwise pass on to decode error. + if isSuccessfulStatus(httpResponse.StatusCode) { err = a.decode(&returnValue, responseBody, httpResponse.Header.Get("Content-Type")) - if err == nil { - return returnValue, httpResponse, err - } - } - - if httpResponse.StatusCode >= 300 { + } else { newErr := NewGenericSwaggerError(responseBody, httpResponse.Status, nil, httpResponse.StatusCode) return returnValue, httpResponse, newErr } - return returnValue, httpResponse, nil + return returnValue, httpResponse, err } /* @@ -869,9 +807,8 @@ func (a *ApplicationResourceApiService) PutTagForApplication(ctx context.Context return httpResponse, err } - if httpResponse.StatusCode >= 300 { - newErr := NewGenericSwaggerError(responseBody, httpResponse.Status, nil, httpResponse.StatusCode) - return httpResponse, newErr + if !isSuccessfulStatus(httpResponse.StatusCode) { + return httpResponse, NewGenericSwaggerError(responseBody, httpResponse.Status, nil, httpResponse.StatusCode) } return httpResponse, nil @@ -935,20 +872,14 @@ func (a *ApplicationResourceApiService) RemoveRoleFromApplicationUser(ctx contex return returnValue, httpResponse, err } - if httpResponse.StatusCode < 300 { - // If we succeed, return the data, otherwise pass on to decode error. + if isSuccessfulStatus(httpResponse.StatusCode) { err = a.decode(&returnValue, responseBody, httpResponse.Header.Get("Content-Type")) - if err == nil { - return returnValue, httpResponse, err - } - } - - if httpResponse.StatusCode >= 300 { + } else { newErr := NewGenericSwaggerError(responseBody, httpResponse.Status, nil, httpResponse.StatusCode) return returnValue, httpResponse, newErr } - return returnValue, httpResponse, nil + return returnValue, httpResponse, err } /* @@ -1009,15 +940,7 @@ func (a *ApplicationResourceApiService) ToggleAccessKeyStatus(ctx context.Contex return returnValue, httpResponse, err } - if httpResponse.StatusCode < 300 { - // If we succeed, return the data, otherwise pass on to decode error. - err = a.decode(&returnValue, responseBody, httpResponse.Header.Get("Content-Type")) - if err == nil { - return returnValue, httpResponse, err - } - } - - if httpResponse.StatusCode >= 300 { + if !isSuccessfulStatus(httpResponse.StatusCode) { newErr := NewGenericSwaggerError(responseBody, httpResponse.Status, nil, httpResponse.StatusCode) return returnValue, httpResponse, newErr } @@ -1084,18 +1007,12 @@ func (a *ApplicationResourceApiService) UpdateApplication(ctx context.Context, b return nil, httpResponse, err } - if httpResponse.StatusCode < 300 { - // If we succeed, return the data, otherwise pass on to decode error. + if isSuccessfulStatus(httpResponse.StatusCode) { err = a.decode(&returnValue, responseBody, httpResponse.Header.Get("Content-Type")) - if err == nil { - return &returnValue, httpResponse, err - } - } - - if httpResponse.StatusCode >= 300 { + } else { newErr := NewGenericSwaggerError(responseBody, httpResponse.Status, nil, httpResponse.StatusCode) return nil, httpResponse, newErr } - return nil, httpResponse, nil + return &returnValue, httpResponse, err } diff --git a/sdk/client/api_authorization_resource.go b/sdk/client/api_authorization_resource.go index b7896a43..d7014361 100644 --- a/sdk/client/api_authorization_resource.go +++ b/sdk/client/api_authorization_resource.go @@ -81,20 +81,14 @@ func (a *AuthorizationResourceApiService) GetPermissions(ctx context.Context, ty return returnValue, httpResponse, err } - if httpResponse.StatusCode < 300 { - // If we succeed, return the data, otherwise pass on to decode error. + if isSuccessfulStatus(httpResponse.StatusCode) { err = a.decode(&returnValue, responseBody, httpResponse.Header.Get("Content-Type")) - if err == nil { - return returnValue, httpResponse, err - } - } - - if httpResponse.StatusCode >= 300 { + } else { newErr := NewGenericSwaggerError(responseBody, httpResponse.Status, nil, httpResponse.StatusCode) return returnValue, httpResponse, newErr } - return returnValue, httpResponse, nil + return returnValue, httpResponse, err } /* @@ -153,13 +147,8 @@ func (a *AuthorizationResourceApiService) GrantPermissions(ctx context.Context, return httpResponse, err } - if httpResponse.StatusCode < 300 { - return httpResponse, err - } - - if httpResponse.StatusCode >= 300 { - newErr := NewGenericSwaggerError(responseBody, httpResponse.Status, nil, httpResponse.StatusCode) - return httpResponse, newErr + if !isSuccessfulStatus(httpResponse.StatusCode) { + return httpResponse, NewGenericSwaggerError(responseBody, httpResponse.Status, nil, httpResponse.StatusCode) } return httpResponse, nil @@ -221,13 +210,8 @@ func (a *AuthorizationResourceApiService) RemovePermissions(ctx context.Context, return httpResponse, err } - if httpResponse.StatusCode < 300 { - return httpResponse, err - } - - if httpResponse.StatusCode >= 300 { - newErr := NewGenericSwaggerError(responseBody, httpResponse.Status, nil, httpResponse.StatusCode) - return httpResponse, newErr + if !isSuccessfulStatus(httpResponse.StatusCode) { + return httpResponse, NewGenericSwaggerError(responseBody, httpResponse.Status, nil, httpResponse.StatusCode) } return httpResponse, nil diff --git a/sdk/client/api_client.go b/sdk/client/api_client.go index 0793c724..a2f10614 100644 --- a/sdk/client/api_client.go +++ b/sdk/client/api_client.go @@ -128,6 +128,10 @@ func (c *APIClient) callAPI(request *http.Request) (*http.Response, error) { } func (c *APIClient) decode(v interface{}, b []byte, contentType string) (err error) { + if len(b) == 0 { + return nil + } + if strings.Contains(contentType, "application/xml") { if err = xml.Unmarshal(b, v); err != nil { return err @@ -135,6 +139,13 @@ func (c *APIClient) decode(v interface{}, b []byte, contentType string) (err err return nil } else if strings.Contains(contentType, "application/json") { if err = json.Unmarshal(b, v); err != nil { + // Hacky - if json unmarshalling fails, return a string. + // it's because the backend might reply with content-type: application/json and a string. + rv := reflect.ValueOf(v) + if rv.Kind() == reflect.Ptr && rv.Elem().Kind() == reflect.String { + rv.Elem().SetString(string(b)) + return nil + } return err } return nil @@ -146,6 +157,7 @@ func (c *APIClient) decode(v interface{}, b []byte, contentType string) (err err rv.Elem().SetString(string(b)) return nil } + return errors.New("undefined response type") } @@ -349,3 +361,7 @@ func addFile(w *multipart.Writer, fieldName, path string) error { return err } + +func isSuccessfulStatus(statusCode int) bool { + return statusCode >= 200 && statusCode < 300 +} diff --git a/sdk/client/api_environment_resource.go b/sdk/client/api_environment_resource.go index a6fdb6fe..dcd50afe 100644 --- a/sdk/client/api_environment_resource.go +++ b/sdk/client/api_environment_resource.go @@ -81,9 +81,8 @@ func (a *EnvironmentResourceApiService) CreateOrUpdateEnvVariable(ctx context.Co return httpResponse, err } - if httpResponse.StatusCode >= 300 { - newErr := NewGenericSwaggerError(responseBody, httpResponse.Status, nil, httpResponse.StatusCode) - return httpResponse, newErr + if !isSuccessfulStatus(httpResponse.StatusCode) { + return httpResponse, NewGenericSwaggerError(responseBody, httpResponse.Status, nil, httpResponse.StatusCode) } return httpResponse, nil @@ -145,20 +144,14 @@ func (a *EnvironmentResourceApiService) DeleteEnvVariable(ctx context.Context, k return returnValue, httpResponse, err } - if httpResponse.StatusCode < 300 { - // If we succeed, return the data, otherwise pass on to decode error. + if isSuccessfulStatus(httpResponse.StatusCode) { err = a.decode(&returnValue, responseBody, httpResponse.Header.Get("Content-Type")) - if err == nil { - return returnValue, httpResponse, err - } - } - - if httpResponse.StatusCode >= 300 { + } else { newErr := NewGenericSwaggerError(responseBody, httpResponse.Status, nil, httpResponse.StatusCode) return returnValue, httpResponse, newErr } - return returnValue, httpResponse, nil + return returnValue, httpResponse, err } /* @@ -218,9 +211,8 @@ func (a *EnvironmentResourceApiService) DeleteTagForEnvVar(ctx context.Context, return httpResponse, err } - if httpResponse.StatusCode >= 300 { - newErr := NewGenericSwaggerError(responseBody, httpResponse.Status, nil, httpResponse.StatusCode) - return httpResponse, newErr + if !isSuccessfulStatus(httpResponse.StatusCode) { + return httpResponse, NewGenericSwaggerError(responseBody, httpResponse.Status, nil, httpResponse.StatusCode) } return httpResponse, nil @@ -282,20 +274,14 @@ func (a *EnvironmentResourceApiService) Get(ctx context.Context, key string) (st return returnValue, httpResponse, err } - if httpResponse.StatusCode < 300 { - // If we succeed, return the data, otherwise pass on to decode error. + if isSuccessfulStatus(httpResponse.StatusCode) { err = a.decode(&returnValue, responseBody, httpResponse.Header.Get("Content-Type")) - if err == nil { - return returnValue, httpResponse, err - } - } - - if httpResponse.StatusCode >= 300 { + } else { newErr := NewGenericSwaggerError(responseBody, httpResponse.Status, nil, httpResponse.StatusCode) return returnValue, httpResponse, newErr } - return returnValue, httpResponse, nil + return returnValue, httpResponse, err } /* @@ -352,20 +338,14 @@ func (a *EnvironmentResourceApiService) GetAll(ctx context.Context) ([]model.Env return returnValue, httpResponse, err } - if httpResponse.StatusCode < 300 { - // If we succeed, return the data, otherwise pass on to decode error. + if isSuccessfulStatus(httpResponse.StatusCode) { err = a.decode(&returnValue, responseBody, httpResponse.Header.Get("Content-Type")) - if err == nil { - return returnValue, httpResponse, err - } - } - - if httpResponse.StatusCode >= 300 { + } else { newErr := NewGenericSwaggerError(responseBody, httpResponse.Status, nil, httpResponse.StatusCode) return returnValue, httpResponse, newErr } - return returnValue, httpResponse, nil + return returnValue, httpResponse, err } /* @@ -424,20 +404,14 @@ func (a *EnvironmentResourceApiService) GetTagsForEnvVar(ctx context.Context, na return returnValue, httpResponse, err } - if httpResponse.StatusCode < 300 { - // If we succeed, return the data, otherwise pass on to decode error. + if isSuccessfulStatus(httpResponse.StatusCode) { err = a.decode(&returnValue, responseBody, httpResponse.Header.Get("Content-Type")) - if err == nil { - return returnValue, httpResponse, err - } - } - - if httpResponse.StatusCode >= 300 { + } else { newErr := NewGenericSwaggerError(responseBody, httpResponse.Status, nil, httpResponse.StatusCode) return returnValue, httpResponse, newErr } - return returnValue, httpResponse, nil + return returnValue, httpResponse, err } /* @@ -497,9 +471,8 @@ func (a *EnvironmentResourceApiService) PutTagForEnvVar(ctx context.Context, bod return httpResponse, err } - if httpResponse.StatusCode >= 300 { - newErr := NewGenericSwaggerError(responseBody, httpResponse.Status, nil, httpResponse.StatusCode) - return httpResponse, newErr + if !isSuccessfulStatus(httpResponse.StatusCode) { + return httpResponse, NewGenericSwaggerError(responseBody, httpResponse.Status, nil, httpResponse.StatusCode) } return httpResponse, nil diff --git a/sdk/client/api_event_resource.go b/sdk/client/api_event_resource.go index 6cd5e7b7..88571922 100644 --- a/sdk/client/api_event_resource.go +++ b/sdk/client/api_event_resource.go @@ -78,11 +78,8 @@ func (a *EventResourceApiService) AddEventHandler(ctx context.Context, body mode return localVarHttpResponse, err } - if localVarHttpResponse.StatusCode >= 300 { - newErr := GenericSwaggerError{ - body: localVarBody, - error: localVarHttpResponse.Status, - } + if !isSuccessfulStatus(localVarHttpResponse.StatusCode) { + newErr := NewGenericSwaggerError(localVarBody, localVarHttpResponse.Status, nil, localVarHttpResponse.StatusCode) return localVarHttpResponse, newErr } @@ -144,11 +141,8 @@ func (a *EventResourceApiService) DeleteQueueConfig(ctx context.Context, queueTy return localVarHttpResponse, err } - if localVarHttpResponse.StatusCode >= 300 { - newErr := GenericSwaggerError{ - body: localVarBody, - error: localVarHttpResponse.Status, - } + if !isSuccessfulStatus(localVarHttpResponse.StatusCode) { + newErr := NewGenericSwaggerError(localVarBody, localVarHttpResponse.Status, nil, localVarHttpResponse.StatusCode) return localVarHttpResponse, newErr } @@ -209,33 +203,14 @@ func (a *EventResourceApiService) GetEventHandlers(ctx context.Context) ([]model return localVarReturnValue, localVarHttpResponse, err } - if localVarHttpResponse.StatusCode < 300 { - // If we succeed, return the data, otherwise pass on to decode error. + if isSuccessfulStatus(localVarHttpResponse.StatusCode) { err = a.decode(&localVarReturnValue, localVarBody, localVarHttpResponse.Header.Get("Content-Type")) - if err == nil { - return localVarReturnValue, localVarHttpResponse, err - } - } - - if localVarHttpResponse.StatusCode >= 300 { - newErr := GenericSwaggerError{ - body: localVarBody, - error: localVarHttpResponse.Status, - } - if localVarHttpResponse.StatusCode == 200 { - var v []model.EventHandler - err = a.decode(&v, localVarBody, localVarHttpResponse.Header.Get("Content-Type")) - if err != nil { - newErr.error = err.Error() - return localVarReturnValue, localVarHttpResponse, newErr - } - newErr.model = v - return localVarReturnValue, localVarHttpResponse, newErr - } + } else { + newErr := NewGenericSwaggerError(localVarBody, localVarHttpResponse.Status, nil, localVarHttpResponse.StatusCode) return localVarReturnValue, localVarHttpResponse, newErr } - return localVarReturnValue, localVarHttpResponse, nil + return localVarReturnValue, localVarHttpResponse, err } /* @@ -303,33 +278,14 @@ func (a *EventResourceApiService) GetEventHandlersForEvent(ctx context.Context, return localVarReturnValue, localVarHttpResponse, err } - if localVarHttpResponse.StatusCode < 300 { - // If we succeed, return the data, otherwise pass on to decode error. + if isSuccessfulStatus(localVarHttpResponse.StatusCode) { err = a.decode(&localVarReturnValue, localVarBody, localVarHttpResponse.Header.Get("Content-Type")) - if err == nil { - return localVarReturnValue, localVarHttpResponse, err - } - } - - if localVarHttpResponse.StatusCode >= 300 { - newErr := GenericSwaggerError{ - body: localVarBody, - error: localVarHttpResponse.Status, - } - if localVarHttpResponse.StatusCode == 200 { - var v []model.EventHandler - err = a.decode(&v, localVarBody, localVarHttpResponse.Header.Get("Content-Type")) - if err != nil { - newErr.error = err.Error() - return localVarReturnValue, localVarHttpResponse, newErr - } - newErr.model = v - return localVarReturnValue, localVarHttpResponse, newErr - } + } else { + newErr := NewGenericSwaggerError(localVarBody, localVarHttpResponse.Status, nil, localVarHttpResponse.StatusCode) return localVarReturnValue, localVarHttpResponse, newErr } - return localVarReturnValue, localVarHttpResponse, nil + return localVarReturnValue, localVarHttpResponse, err } /* @@ -390,33 +346,14 @@ func (a *EventResourceApiService) GetQueueConfig(ctx context.Context, queueType return localVarReturnValue, localVarHttpResponse, err } - if localVarHttpResponse.StatusCode < 300 { - // If we succeed, return the data, otherwise pass on to decode error. + if isSuccessfulStatus(localVarHttpResponse.StatusCode) { err = a.decode(&localVarReturnValue, localVarBody, localVarHttpResponse.Header.Get("Content-Type")) - if err == nil { - return localVarReturnValue, localVarHttpResponse, err - } - } - - if localVarHttpResponse.StatusCode >= 300 { - newErr := GenericSwaggerError{ - body: localVarBody, - error: localVarHttpResponse.Status, - } - if localVarHttpResponse.StatusCode == 200 { - var v map[string]interface{} - err = a.decode(&v, localVarBody, localVarHttpResponse.Header.Get("Content-Type")) - if err != nil { - newErr.error = err.Error() - return localVarReturnValue, localVarHttpResponse, newErr - } - newErr.model = v - return localVarReturnValue, localVarHttpResponse, newErr - } + } else { + newErr := NewGenericSwaggerError(localVarBody, localVarHttpResponse.Status, nil, localVarHttpResponse.StatusCode) return localVarReturnValue, localVarHttpResponse, newErr } - return localVarReturnValue, localVarHttpResponse, nil + return localVarReturnValue, localVarHttpResponse, err } /* @@ -473,33 +410,14 @@ func (a *EventResourceApiService) GetQueueNames(ctx context.Context) (map[string return localVarReturnValue, localVarHttpResponse, err } - if localVarHttpResponse.StatusCode < 300 { - // If we succeed, return the data, otherwise pass on to decode error. + if isSuccessfulStatus(localVarHttpResponse.StatusCode) { err = a.decode(&localVarReturnValue, localVarBody, localVarHttpResponse.Header.Get("Content-Type")) - if err == nil { - return localVarReturnValue, localVarHttpResponse, err - } - } - - if localVarHttpResponse.StatusCode >= 300 { - newErr := GenericSwaggerError{ - body: localVarBody, - error: localVarHttpResponse.Status, - } - if localVarHttpResponse.StatusCode == 200 { - var v map[string]string - err = a.decode(&v, localVarBody, localVarHttpResponse.Header.Get("Content-Type")) - if err != nil { - newErr.error = err.Error() - return localVarReturnValue, localVarHttpResponse, newErr - } - newErr.model = v - return localVarReturnValue, localVarHttpResponse, newErr - } + } else { + newErr := NewGenericSwaggerError(localVarBody, localVarHttpResponse.Status, nil, localVarHttpResponse.StatusCode) return localVarReturnValue, localVarHttpResponse, newErr } - return localVarReturnValue, localVarHttpResponse, nil + return localVarReturnValue, localVarHttpResponse, err } /* @@ -560,11 +478,8 @@ func (a *EventResourceApiService) PutQueueConfig(ctx context.Context, body strin return localVarHttpResponse, err } - if localVarHttpResponse.StatusCode >= 300 { - newErr := GenericSwaggerError{ - body: localVarBody, - error: localVarHttpResponse.Status, - } + if !isSuccessfulStatus(localVarHttpResponse.StatusCode) { + newErr := NewGenericSwaggerError(localVarBody, localVarHttpResponse.Status, nil, localVarHttpResponse.StatusCode) return localVarHttpResponse, newErr } @@ -624,11 +539,8 @@ func (a *EventResourceApiService) RemoveEventHandler(ctx context.Context, name s return localVarHttpResponse, err } - if localVarHttpResponse.StatusCode >= 300 { - newErr := GenericSwaggerError{ - body: localVarBody, - error: localVarHttpResponse.Status, - } + if !isSuccessfulStatus(localVarHttpResponse.StatusCode) { + newErr := NewGenericSwaggerError(localVarBody, localVarHttpResponse.Status, nil, localVarHttpResponse.StatusCode) return localVarHttpResponse, newErr } @@ -689,11 +601,8 @@ func (a *EventResourceApiService) UpdateEventHandler(ctx context.Context, body m return localVarHttpResponse, err } - if localVarHttpResponse.StatusCode >= 300 { - newErr := GenericSwaggerError{ - body: localVarBody, - error: localVarHttpResponse.Status, - } + if !isSuccessfulStatus(localVarHttpResponse.StatusCode) { + newErr := NewGenericSwaggerError(localVarBody, localVarHttpResponse.Status, nil, localVarHttpResponse.StatusCode) return localVarHttpResponse, newErr } diff --git a/sdk/client/api_group_resource.go b/sdk/client/api_group_resource.go index 3214fc4f..b0dc66b6 100644 --- a/sdk/client/api_group_resource.go +++ b/sdk/client/api_group_resource.go @@ -81,20 +81,14 @@ func (a *GroupResourceApiService) AddUserToGroup(ctx context.Context, groupId st return returnValue, httpResponse, err } - if httpResponse.StatusCode < 300 { - // If we succeed, return the data, otherwise pass on to decode error. + if isSuccessfulStatus(httpResponse.StatusCode) { err = a.decode(&returnValue, responseBody, httpResponse.Header.Get("Content-Type")) - if err == nil { - return returnValue, httpResponse, err - } - } - - if httpResponse.StatusCode >= 300 { + } else { newErr := NewGenericSwaggerError(responseBody, httpResponse.Status, nil, httpResponse.StatusCode) return returnValue, httpResponse, newErr } - return returnValue, httpResponse, nil + return returnValue, httpResponse, err } /* @@ -154,9 +148,8 @@ func (a *GroupResourceApiService) AddUsersToGroup(ctx context.Context, body []st return httpResponse, err } - if httpResponse.StatusCode >= 300 { - newErr := NewGenericSwaggerError(responseBody, httpResponse.Status, nil, httpResponse.StatusCode) - return httpResponse, newErr + if !isSuccessfulStatus(httpResponse.StatusCode) { + return httpResponse, NewGenericSwaggerError(responseBody, httpResponse.Status, nil, httpResponse.StatusCode) } return httpResponse, nil @@ -217,13 +210,8 @@ func (a *GroupResourceApiService) DeleteGroup(ctx context.Context, id string) (* return httpResponse, err } - if httpResponse.StatusCode < 300 { - return httpResponse, err - } - - if httpResponse.StatusCode >= 300 { - newErr := NewGenericSwaggerError(responseBody, httpResponse.Status, nil, httpResponse.StatusCode) - return httpResponse, newErr + if !isSuccessfulStatus(httpResponse.StatusCode) { + return httpResponse, NewGenericSwaggerError(responseBody, httpResponse.Status, nil, httpResponse.StatusCode) } return httpResponse, nil @@ -285,20 +273,14 @@ func (a *GroupResourceApiService) GetGrantedPermissions1(ctx context.Context, gr return returnValue, httpResponse, err } - if httpResponse.StatusCode < 300 { - // If we succeed, return the data, otherwise pass on to decode error. + if isSuccessfulStatus(httpResponse.StatusCode) { err = a.decode(&returnValue, responseBody, httpResponse.Header.Get("Content-Type")) - if err == nil { - return returnValue, httpResponse, err - } - } - - if httpResponse.StatusCode >= 300 { + } else { newErr := NewGenericSwaggerError(responseBody, httpResponse.Status, nil, httpResponse.StatusCode) return returnValue, httpResponse, newErr } - return returnValue, httpResponse, nil + return returnValue, httpResponse, err } /* @@ -357,20 +339,14 @@ func (a *GroupResourceApiService) GetGroup(ctx context.Context, id string) (inte return returnValue, httpResponse, err } - if httpResponse.StatusCode < 300 { - // If we succeed, return the data, otherwise pass on to decode error. + if isSuccessfulStatus(httpResponse.StatusCode) { err = a.decode(&returnValue, responseBody, httpResponse.Header.Get("Content-Type")) - if err == nil { - return returnValue, httpResponse, err - } - } - - if httpResponse.StatusCode >= 300 { + } else { newErr := NewGenericSwaggerError(responseBody, httpResponse.Status, nil, httpResponse.StatusCode) return returnValue, httpResponse, newErr } - return returnValue, httpResponse, nil + return returnValue, httpResponse, err } /* @@ -429,20 +405,14 @@ func (a *GroupResourceApiService) GetUsersInGroup(ctx context.Context, id string return returnValue, httpResponse, err } - if httpResponse.StatusCode < 300 { - // If we succeed, return the data, otherwise pass on to decode error. + if isSuccessfulStatus(httpResponse.StatusCode) { err = a.decode(&returnValue, responseBody, httpResponse.Header.Get("Content-Type")) - if err == nil { - return returnValue, httpResponse, err - } - } - - if httpResponse.StatusCode >= 300 { + } else { newErr := NewGenericSwaggerError(responseBody, httpResponse.Status, nil, httpResponse.StatusCode) return returnValue, httpResponse, newErr } - return returnValue, httpResponse, nil + return returnValue, httpResponse, err } /* @@ -499,20 +469,14 @@ func (a *GroupResourceApiService) ListGroups(ctx context.Context) ([]rbac.Group, return returnValue, httpResponse, err } - if httpResponse.StatusCode < 300 { - // If we succeed, return the data, otherwise pass on to decode error. + if isSuccessfulStatus(httpResponse.StatusCode) { err = a.decode(&returnValue, responseBody, httpResponse.Header.Get("Content-Type")) - if err == nil { - return returnValue, httpResponse, err - } - } - - if httpResponse.StatusCode >= 300 { + } else { newErr := NewGenericSwaggerError(responseBody, httpResponse.Status, nil, httpResponse.StatusCode) return returnValue, httpResponse, newErr } - return returnValue, httpResponse, nil + return returnValue, httpResponse, err } /* @@ -573,22 +537,17 @@ func (a *GroupResourceApiService) RemoveUserFromGroup(ctx context.Context, group return returnValue, httpResponse, err } - if httpResponse.StatusCode < 300 { - // If we succeed, return the data, otherwise pass on to decode error. + if isSuccessfulStatus(httpResponse.StatusCode) { err = a.decode(&returnValue, responseBody, httpResponse.Header.Get("Content-Type")) - if err == nil { - return returnValue, httpResponse, err - } - } - - if httpResponse.StatusCode >= 300 { + } else { newErr := NewGenericSwaggerError(responseBody, httpResponse.Status, nil, httpResponse.StatusCode) return returnValue, httpResponse, newErr } - return returnValue, httpResponse, nil + return returnValue, httpResponse, err } +//TODO test this method /* GroupResourceApiService Remove users from group * @param ctx context.Context - for authentication, logging, cancellation, deadlines, tracing, etc. Passed from http.Request or context.Background(). @@ -646,9 +605,8 @@ func (a *GroupResourceApiService) RemoveUsersFromGroup(ctx context.Context, body return httpResponse, err } - if httpResponse.StatusCode >= 300 { - newErr := NewGenericSwaggerError(responseBody, httpResponse.Status, nil, httpResponse.StatusCode) - return httpResponse, newErr + if !isSuccessfulStatus(httpResponse.StatusCode) { + return httpResponse, NewGenericSwaggerError(responseBody, httpResponse.Status, nil, httpResponse.StatusCode) } return httpResponse, nil @@ -713,18 +671,12 @@ func (a *GroupResourceApiService) UpsertGroup(ctx context.Context, body rbac.Ups return returnValue, httpResponse, err } - if httpResponse.StatusCode < 300 { - // If we succeed, return the data, otherwise pass on to decode error. + if isSuccessfulStatus(httpResponse.StatusCode) { err = a.decode(&returnValue, responseBody, httpResponse.Header.Get("Content-Type")) - if err == nil { - return returnValue, httpResponse, err - } - } - - if httpResponse.StatusCode >= 300 { + } else { newErr := NewGenericSwaggerError(responseBody, httpResponse.Status, nil, httpResponse.StatusCode) return returnValue, httpResponse, newErr } - return returnValue, httpResponse, nil + return returnValue, httpResponse, err } diff --git a/sdk/client/api_human_task.go b/sdk/client/api_human_task.go index abf57301..ead45ba5 100644 --- a/sdk/client/api_human_task.go +++ b/sdk/client/api_human_task.go @@ -97,20 +97,14 @@ func (a *HumanTaskApiService) AssignAndClaim(ctx context.Context, taskId string, return returnValue, httpResponse, err } - if httpResponse.StatusCode < 300 { - // If we succeed, return the data, otherwise pass on to decode error. + if isSuccessfulStatus(httpResponse.StatusCode) { err = a.decode(&returnValue, responseBody, httpResponse.Header.Get("Content-Type")) - if err == nil { - return returnValue, httpResponse, err - } - } - - if httpResponse.StatusCode >= 300 { + } else { newErr := NewGenericSwaggerError(responseBody, httpResponse.Status, nil, httpResponse.StatusCode) return returnValue, httpResponse, newErr } - return returnValue, httpResponse, nil + return returnValue, httpResponse, err } /* @@ -169,20 +163,14 @@ func (a *HumanTaskApiService) BackPopulateFullTextIndex(ctx context.Context, var return returnValue, httpResponse, err } - if httpResponse.StatusCode < 300 { - // If we succeed, return the data, otherwise pass on to decode error. + if isSuccessfulStatus(httpResponse.StatusCode) { err = a.decode(&returnValue, responseBody, httpResponse.Header.Get("Content-Type")) - if err == nil { - return returnValue, httpResponse, err - } - } - - if httpResponse.StatusCode >= 300 { + } else { newErr := NewGenericSwaggerError(responseBody, httpResponse.Status, nil, httpResponse.StatusCode) return returnValue, httpResponse, newErr } - return returnValue, httpResponse, nil + return returnValue, httpResponse, err } /* @@ -256,20 +244,14 @@ func (a *HumanTaskApiService) ClaimTask(ctx context.Context, taskId string, opti return returnValue, httpResponse, err } - if httpResponse.StatusCode < 300 { - // If we succeed, return the data, otherwise pass on to decode error. + if isSuccessfulStatus(httpResponse.StatusCode) { err = a.decode(&returnValue, responseBody, httpResponse.Header.Get("Content-Type")) - if err == nil { - return returnValue, httpResponse, err - } - } - - if httpResponse.StatusCode >= 300 { + } else { newErr := NewGenericSwaggerError(responseBody, httpResponse.Status, nil, httpResponse.StatusCode) return returnValue, httpResponse, newErr } - return returnValue, httpResponse, nil + return returnValue, httpResponse, err } /* @@ -327,9 +309,8 @@ func (a *HumanTaskApiService) DeleteTaskFromHumanTaskRecords(ctx context.Context return httpResponse, err } - if httpResponse.StatusCode >= 300 { - newErr := NewGenericSwaggerError(responseBody, httpResponse.Status, nil, httpResponse.StatusCode) - return httpResponse, newErr + if !isSuccessfulStatus(httpResponse.StatusCode) { + return httpResponse, NewGenericSwaggerError(responseBody, httpResponse.Status, nil, httpResponse.StatusCode) } return httpResponse, nil @@ -389,9 +370,8 @@ func (a *HumanTaskApiService) DeleteTaskFromHumanTaskRecords1(ctx context.Contex return httpResponse, err } - if httpResponse.StatusCode >= 300 { - newErr := NewGenericSwaggerError(responseBody, httpResponse.Status, nil, httpResponse.StatusCode) - return httpResponse, newErr + if !isSuccessfulStatus(httpResponse.StatusCode) { + return httpResponse, NewGenericSwaggerError(responseBody, httpResponse.Status, nil, httpResponse.StatusCode) } return httpResponse, nil @@ -451,9 +431,8 @@ func (a *HumanTaskApiService) DeleteTemplateByName(ctx context.Context, name str return httpResponse, err } - if httpResponse.StatusCode >= 300 { - newErr := NewGenericSwaggerError(responseBody, httpResponse.Status, nil, httpResponse.StatusCode) - return httpResponse, newErr + if !isSuccessfulStatus(httpResponse.StatusCode) { + return httpResponse, NewGenericSwaggerError(responseBody, httpResponse.Status, nil, httpResponse.StatusCode) } return httpResponse, nil @@ -515,9 +494,8 @@ func (a *HumanTaskApiService) DeleteTemplatesByNameAndVersion(ctx context.Contex return httpResponse, err } - if httpResponse.StatusCode >= 300 { - newErr := NewGenericSwaggerError(responseBody, httpResponse.Status, nil, httpResponse.StatusCode) - return httpResponse, newErr + if !isSuccessfulStatus(httpResponse.StatusCode) { + return httpResponse, NewGenericSwaggerError(responseBody, httpResponse.Status, nil, httpResponse.StatusCode) } return httpResponse, nil @@ -592,20 +570,14 @@ func (a *HumanTaskApiService) GetAllTemplates(ctx context.Context, optionals *Hu return returnValue, httpResponse, err } - if httpResponse.StatusCode < 300 { - // If we succeed, return the data, otherwise pass on to decode error. + if isSuccessfulStatus(httpResponse.StatusCode) { err = a.decode(&returnValue, responseBody, httpResponse.Header.Get("Content-Type")) - if err == nil { - return returnValue, httpResponse, err - } - } - - if httpResponse.StatusCode >= 300 { + } else { newErr := NewGenericSwaggerError(responseBody, httpResponse.Status, nil, httpResponse.StatusCode) return returnValue, httpResponse, newErr } - return returnValue, httpResponse, nil + return returnValue, httpResponse, err } /* @@ -674,20 +646,14 @@ func (a *HumanTaskApiService) GetTask1(ctx context.Context, taskId string, optio return returnValue, httpResponse, err } - if httpResponse.StatusCode < 300 { - // If we succeed, return the data, otherwise pass on to decode error. + if isSuccessfulStatus(httpResponse.StatusCode) { err = a.decode(&returnValue, responseBody, httpResponse.Header.Get("Content-Type")) - if err == nil { - return returnValue, httpResponse, err - } - } - - if httpResponse.StatusCode >= 300 { + } else { newErr := NewGenericSwaggerError(responseBody, httpResponse.Status, nil, httpResponse.StatusCode) return returnValue, httpResponse, newErr } - return returnValue, httpResponse, nil + return returnValue, httpResponse, err } /* @@ -746,20 +712,14 @@ func (a *HumanTaskApiService) GetTaskDisplayNames(ctx context.Context, searchTyp return returnValue, httpResponse, err } - if httpResponse.StatusCode < 300 { - // If we succeed, return the data, otherwise pass on to decode error. + if isSuccessfulStatus(httpResponse.StatusCode) { err = a.decode(&returnValue, responseBody, httpResponse.Header.Get("Content-Type")) - if err == nil { - return returnValue, httpResponse, err - } - } - - if httpResponse.StatusCode >= 300 { + } else { newErr := NewGenericSwaggerError(responseBody, httpResponse.Status, nil, httpResponse.StatusCode) return returnValue, httpResponse, newErr } - return returnValue, httpResponse, nil + return returnValue, httpResponse, err } /* @@ -820,20 +780,14 @@ func (a *HumanTaskApiService) GetTemplateByNameAndVersion(ctx context.Context, n return returnValue, httpResponse, err } - if httpResponse.StatusCode < 300 { - // If we succeed, return the data, otherwise pass on to decode error. + if isSuccessfulStatus(httpResponse.StatusCode) { err = a.decode(&returnValue, responseBody, httpResponse.Header.Get("Content-Type")) - if err == nil { - return returnValue, httpResponse, err - } - } - - if httpResponse.StatusCode >= 300 { + } else { newErr := NewGenericSwaggerError(responseBody, httpResponse.Status, nil, httpResponse.StatusCode) return returnValue, httpResponse, newErr } - return returnValue, httpResponse, nil + return returnValue, httpResponse, err } /* @@ -892,20 +846,14 @@ func (a *HumanTaskApiService) GetTemplateByTaskId(ctx context.Context, humanTask return returnValue, httpResponse, err } - if httpResponse.StatusCode < 300 { - // If we succeed, return the data, otherwise pass on to decode error. + if isSuccessfulStatus(httpResponse.StatusCode) { err = a.decode(&returnValue, responseBody, httpResponse.Header.Get("Content-Type")) - if err == nil { - return returnValue, httpResponse, err - } - } - - if httpResponse.StatusCode >= 300 { + } else { newErr := NewGenericSwaggerError(responseBody, httpResponse.Status, nil, httpResponse.StatusCode) return returnValue, httpResponse, newErr } - return returnValue, httpResponse, nil + return returnValue, httpResponse, err } /* @@ -965,9 +913,8 @@ func (a *HumanTaskApiService) ReassignTask(ctx context.Context, body []human.Hum return httpResponse, err } - if httpResponse.StatusCode >= 300 { - newErr := NewGenericSwaggerError(responseBody, httpResponse.Status, nil, httpResponse.StatusCode) - return httpResponse, newErr + if !isSuccessfulStatus(httpResponse.StatusCode) { + return httpResponse, NewGenericSwaggerError(responseBody, httpResponse.Status, nil, httpResponse.StatusCode) } return httpResponse, nil @@ -1027,9 +974,8 @@ func (a *HumanTaskApiService) ReleaseTask(ctx context.Context, taskId string) (* return httpResponse, err } - if httpResponse.StatusCode >= 300 { - newErr := NewGenericSwaggerError(responseBody, httpResponse.Status, nil, httpResponse.StatusCode) - return httpResponse, newErr + if !isSuccessfulStatus(httpResponse.StatusCode) { + return httpResponse, NewGenericSwaggerError(responseBody, httpResponse.Status, nil, httpResponse.StatusCode) } return httpResponse, nil @@ -1102,20 +1048,14 @@ func (a *HumanTaskApiService) SaveTemplate(ctx context.Context, body human.Human return returnValue, httpResponse, err } - if httpResponse.StatusCode < 300 { - // If we succeed, return the data, otherwise pass on to decode error. + if isSuccessfulStatus(httpResponse.StatusCode) { err = a.decode(&returnValue, responseBody, httpResponse.Header.Get("Content-Type")) - if err == nil { - return returnValue, httpResponse, err - } - } - - if httpResponse.StatusCode >= 300 { + } else { newErr := NewGenericSwaggerError(responseBody, httpResponse.Status, nil, httpResponse.StatusCode) return returnValue, httpResponse, newErr } - return returnValue, httpResponse, nil + return returnValue, httpResponse, err } /* @@ -1185,20 +1125,14 @@ func (a *HumanTaskApiService) SaveTemplates(ctx context.Context, body []human.Hu return returnValue, httpResponse, err } - if httpResponse.StatusCode < 300 { - // If we succeed, return the data, otherwise pass on to decode error. + if isSuccessfulStatus(httpResponse.StatusCode) { err = a.decode(&returnValue, responseBody, httpResponse.Header.Get("Content-Type")) - if err == nil { - return returnValue, httpResponse, err - } - } - - if httpResponse.StatusCode >= 300 { + } else { newErr := NewGenericSwaggerError(responseBody, httpResponse.Status, nil, httpResponse.StatusCode) return returnValue, httpResponse, newErr } - return returnValue, httpResponse, nil + return returnValue, httpResponse, err } /* @@ -1258,20 +1192,14 @@ func (a *HumanTaskApiService) Search(ctx context.Context, body human.HumanTaskSe return returnValue, httpResponse, err } - if httpResponse.StatusCode < 300 { - // If we succeed, return the data, otherwise pass on to decode error. + if isSuccessfulStatus(httpResponse.StatusCode) { err = a.decode(&returnValue, responseBody, httpResponse.Header.Get("Content-Type")) - if err == nil { - return returnValue, httpResponse, err - } - } - - if httpResponse.StatusCode >= 300 { + } else { newErr := NewGenericSwaggerError(responseBody, httpResponse.Status, nil, httpResponse.StatusCode) return returnValue, httpResponse, newErr } - return returnValue, httpResponse, nil + return returnValue, httpResponse, err } /* @@ -1339,9 +1267,8 @@ func (a *HumanTaskApiService) SkipTask(ctx context.Context, taskId string, optio return httpResponse, err } - if httpResponse.StatusCode >= 300 { - newErr := NewGenericSwaggerError(responseBody, httpResponse.Status, nil, httpResponse.StatusCode) - return httpResponse, newErr + if !isSuccessfulStatus(httpResponse.StatusCode) { + return httpResponse, NewGenericSwaggerError(responseBody, httpResponse.Status, nil, httpResponse.StatusCode) } return httpResponse, nil @@ -1415,9 +1342,8 @@ func (a *HumanTaskApiService) UpdateTaskOutput(ctx context.Context, body map[str return httpResponse, err } - if httpResponse.StatusCode >= 300 { - newErr := NewGenericSwaggerError(responseBody, httpResponse.Status, nil, httpResponse.StatusCode) - return httpResponse, newErr + if !isSuccessfulStatus(httpResponse.StatusCode) { + return httpResponse, NewGenericSwaggerError(responseBody, httpResponse.Status, nil, httpResponse.StatusCode) } return httpResponse, nil @@ -1498,9 +1424,8 @@ func (a *HumanTaskApiService) UpdateTaskOutputByRef(ctx context.Context, body ma return httpResponse, err } - if httpResponse.StatusCode >= 300 { - newErr := NewGenericSwaggerError(responseBody, httpResponse.Status, nil, httpResponse.StatusCode) - return httpResponse, newErr + if !isSuccessfulStatus(httpResponse.StatusCode) { + return httpResponse, NewGenericSwaggerError(responseBody, httpResponse.Status, nil, httpResponse.StatusCode) } return httpResponse, nil diff --git a/sdk/client/api_integration_resource.go b/sdk/client/api_integration_resource.go index 7e85d9f4..5189ebbb 100644 --- a/sdk/client/api_integration_resource.go +++ b/sdk/client/api_integration_resource.go @@ -84,11 +84,8 @@ func (a *IntegrationResourceApiService) AssociatePromptWithIntegration(ctx conte return localVarHttpResponse, err } - if localVarHttpResponse.StatusCode >= 300 { - newErr := GenericSwaggerError{ - body: localVarBody, - error: localVarHttpResponse.Status, - } + if !isSuccessfulStatus(localVarHttpResponse.StatusCode) { + newErr := NewGenericSwaggerError(localVarBody, localVarHttpResponse.Status, nil, localVarHttpResponse.StatusCode) return localVarHttpResponse, newErr } @@ -151,11 +148,8 @@ func (a *IntegrationResourceApiService) DeleteIntegrationApi(ctx context.Context return localVarHttpResponse, err } - if localVarHttpResponse.StatusCode >= 300 { - newErr := GenericSwaggerError{ - body: localVarBody, - error: localVarHttpResponse.Status, - } + if !isSuccessfulStatus(localVarHttpResponse.StatusCode) { + newErr := NewGenericSwaggerError(localVarBody, localVarHttpResponse.Status, nil, localVarHttpResponse.StatusCode) return localVarHttpResponse, newErr } @@ -216,11 +210,8 @@ func (a *IntegrationResourceApiService) DeleteIntegrationProvider(ctx context.Co return localVarHttpResponse, err } - if localVarHttpResponse.StatusCode >= 300 { - newErr := GenericSwaggerError{ - body: localVarBody, - error: localVarHttpResponse.Status, - } + if !isSuccessfulStatus(localVarHttpResponse.StatusCode) { + newErr := NewGenericSwaggerError(localVarBody, localVarHttpResponse.Status, nil, localVarHttpResponse.StatusCode) return localVarHttpResponse, newErr } @@ -286,11 +277,8 @@ func (a *IntegrationResourceApiService) DeleteTagForIntegration(ctx context.Cont return localVarHttpResponse, err } - if localVarHttpResponse.StatusCode >= 300 { - newErr := GenericSwaggerError{ - body: localVarBody, - error: localVarHttpResponse.Status, - } + if !isSuccessfulStatus(localVarHttpResponse.StatusCode) { + newErr := NewGenericSwaggerError(localVarBody, localVarHttpResponse.Status, nil, localVarHttpResponse.StatusCode) return localVarHttpResponse, newErr } @@ -354,11 +342,8 @@ func (a *IntegrationResourceApiService) DeleteTagForIntegrationProvider(ctx cont return localVarHttpResponse, err } - if localVarHttpResponse.StatusCode >= 300 { - newErr := GenericSwaggerError{ - body: localVarBody, - error: localVarHttpResponse.Status, - } + if !isSuccessfulStatus(localVarHttpResponse.StatusCode) { + newErr := NewGenericSwaggerError(localVarBody, localVarHttpResponse.Status, nil, localVarHttpResponse.StatusCode) return localVarHttpResponse, newErr } @@ -424,33 +409,14 @@ func (a *IntegrationResourceApiService) GetIntegrationApi(ctx context.Context, n return localVarReturnValue, localVarHttpResponse, err } - if localVarHttpResponse.StatusCode < 300 { - // If we succeed, return the data, otherwise pass on to decode error. + if isSuccessfulStatus(localVarHttpResponse.StatusCode) { err = a.decode(&localVarReturnValue, localVarBody, localVarHttpResponse.Header.Get("Content-Type")) - if err == nil { - return localVarReturnValue, localVarHttpResponse, err - } - } - - if localVarHttpResponse.StatusCode >= 300 { - newErr := GenericSwaggerError{ - body: localVarBody, - error: localVarHttpResponse.Status, - } - if localVarHttpResponse.StatusCode == 200 { - var v integration.IntegrationApi - err = a.decode(&v, localVarBody, localVarHttpResponse.Header.Get("Content-Type")) - if err != nil { - newErr.error = err.Error() - return localVarReturnValue, localVarHttpResponse, newErr - } - newErr.model = v - return localVarReturnValue, localVarHttpResponse, newErr - } + } else { + newErr := NewGenericSwaggerError(localVarBody, localVarHttpResponse.Status, nil, localVarHttpResponse.StatusCode) return localVarReturnValue, localVarHttpResponse, newErr } - return localVarReturnValue, localVarHttpResponse, nil + return localVarReturnValue, localVarHttpResponse, err } /* @@ -515,33 +481,14 @@ func (a *IntegrationResourceApiService) GetIntegrationApis(ctx context.Context, return localVarReturnValue, localVarHttpResponse, err } - if localVarHttpResponse.StatusCode < 300 { - // If we succeed, return the data, otherwise pass on to decode error. + if isSuccessfulStatus(localVarHttpResponse.StatusCode) { err = a.decode(&localVarReturnValue, localVarBody, localVarHttpResponse.Header.Get("Content-Type")) - if err == nil { - return localVarReturnValue, localVarHttpResponse, err - } - } - - if localVarHttpResponse.StatusCode >= 300 { - newErr := GenericSwaggerError{ - body: localVarBody, - error: localVarHttpResponse.Status, - } - if localVarHttpResponse.StatusCode == 200 { - var v []integration.IntegrationApi - err = a.decode(&v, localVarBody, localVarHttpResponse.Header.Get("Content-Type")) - if err != nil { - newErr.error = err.Error() - return localVarReturnValue, localVarHttpResponse, newErr - } - newErr.model = v - return localVarReturnValue, localVarHttpResponse, newErr - } + } else { + newErr := NewGenericSwaggerError(localVarBody, localVarHttpResponse.Status, nil, localVarHttpResponse.StatusCode) return localVarReturnValue, localVarHttpResponse, newErr } - return localVarReturnValue, localVarHttpResponse, nil + return localVarReturnValue, localVarHttpResponse, err } /* @@ -601,33 +548,14 @@ func (a *IntegrationResourceApiService) GetIntegrationAvailableApis(ctx context. return localVarReturnValue, localVarHttpResponse, err } - if localVarHttpResponse.StatusCode < 300 { - // If we succeed, return the data, otherwise pass on to decode error. + if isSuccessfulStatus(localVarHttpResponse.StatusCode) { err = a.decode(&localVarReturnValue, localVarBody, localVarHttpResponse.Header.Get("Content-Type")) - if err == nil { - return localVarReturnValue, localVarHttpResponse, err - } - } - - if localVarHttpResponse.StatusCode >= 300 { - newErr := GenericSwaggerError{ - body: localVarBody, - error: localVarHttpResponse.Status, - } - if localVarHttpResponse.StatusCode == 200 { - var v []string - err = a.decode(&v, localVarBody, localVarHttpResponse.Header.Get("Content-Type")) - if err != nil { - newErr.error = err.Error() - return localVarReturnValue, localVarHttpResponse, newErr - } - newErr.model = v - return localVarReturnValue, localVarHttpResponse, newErr - } + } else { + newErr := NewGenericSwaggerError(localVarBody, localVarHttpResponse.Status, nil, localVarHttpResponse.StatusCode) return localVarReturnValue, localVarHttpResponse, newErr } - return localVarReturnValue, localVarHttpResponse, nil + return localVarReturnValue, localVarHttpResponse, err } /* @@ -687,33 +615,14 @@ func (a *IntegrationResourceApiService) GetIntegrationProvider(ctx context.Conte return localVarReturnValue, localVarHttpResponse, err } - if localVarHttpResponse.StatusCode < 300 { - // If we succeed, return the data, otherwise pass on to decode error. + if isSuccessfulStatus(localVarHttpResponse.StatusCode) { err = a.decode(&localVarReturnValue, localVarBody, localVarHttpResponse.Header.Get("Content-Type")) - if err == nil { - return localVarReturnValue, localVarHttpResponse, err - } - } - - if localVarHttpResponse.StatusCode >= 300 { - newErr := GenericSwaggerError{ - body: localVarBody, - error: localVarHttpResponse.Status, - } - if localVarHttpResponse.StatusCode == 200 { - var v integration.Integration - err = a.decode(&v, localVarBody, localVarHttpResponse.Header.Get("Content-Type")) - if err != nil { - newErr.error = err.Error() - return localVarReturnValue, localVarHttpResponse, newErr - } - newErr.model = v - return localVarReturnValue, localVarHttpResponse, newErr - } + } else { + newErr := NewGenericSwaggerError(localVarBody, localVarHttpResponse.Status, nil, localVarHttpResponse.StatusCode) return localVarReturnValue, localVarHttpResponse, newErr } - return localVarReturnValue, localVarHttpResponse, nil + return localVarReturnValue, localVarHttpResponse, err } /* @@ -780,33 +689,14 @@ func (a *IntegrationResourceApiService) GetIntegrationProviders(ctx context.Cont return localVarReturnValue, localVarHttpResponse, err } - if localVarHttpResponse.StatusCode < 300 { - // If we succeed, return the data, otherwise pass on to decode error. + if isSuccessfulStatus(localVarHttpResponse.StatusCode) { err = a.decode(&localVarReturnValue, localVarBody, localVarHttpResponse.Header.Get("Content-Type")) - if err == nil { - return localVarReturnValue, localVarHttpResponse, err - } - } - - if localVarHttpResponse.StatusCode >= 300 { - newErr := GenericSwaggerError{ - body: localVarBody, - error: localVarHttpResponse.Status, - } - if localVarHttpResponse.StatusCode == 200 { - var v []integration.Integration - err = a.decode(&v, localVarBody, localVarHttpResponse.Header.Get("Content-Type")) - if err != nil { - newErr.error = err.Error() - return localVarReturnValue, localVarHttpResponse, newErr - } - newErr.model = v - return localVarReturnValue, localVarHttpResponse, newErr - } + } else { + newErr := NewGenericSwaggerError(localVarBody, localVarHttpResponse.Status, nil, localVarHttpResponse.StatusCode) return localVarReturnValue, localVarHttpResponse, newErr } - return localVarReturnValue, localVarHttpResponse, nil + return localVarReturnValue, localVarHttpResponse, err } /* @@ -868,33 +758,14 @@ func (a *IntegrationResourceApiService) GetPromptsWithIntegration(ctx context.Co return localVarReturnValue, localVarHttpResponse, err } - if localVarHttpResponse.StatusCode < 300 { - // If we succeed, return the data, otherwise pass on to decode error. + if isSuccessfulStatus(localVarHttpResponse.StatusCode) { err = a.decode(&localVarReturnValue, localVarBody, localVarHttpResponse.Header.Get("Content-Type")) - if err == nil { - return localVarReturnValue, localVarHttpResponse, err - } - } - - if localVarHttpResponse.StatusCode >= 300 { - newErr := GenericSwaggerError{ - body: localVarBody, - error: localVarHttpResponse.Status, - } - if localVarHttpResponse.StatusCode == 200 { - var v []integration.PromptTemplate - err = a.decode(&v, localVarBody, localVarHttpResponse.Header.Get("Content-Type")) - if err != nil { - newErr.error = err.Error() - return localVarReturnValue, localVarHttpResponse, newErr - } - newErr.model = v - return localVarReturnValue, localVarHttpResponse, newErr - } + } else { + newErr := NewGenericSwaggerError(localVarBody, localVarHttpResponse.Status, nil, localVarHttpResponse.StatusCode) return localVarReturnValue, localVarHttpResponse, newErr } - return localVarReturnValue, localVarHttpResponse, nil + return localVarReturnValue, localVarHttpResponse, err } /* @@ -966,33 +837,14 @@ func (a *IntegrationResourceApiService) GetProvidersAndIntegrations(ctx context. return localVarReturnValue, localVarHttpResponse, err } - if localVarHttpResponse.StatusCode < 300 { - // If we succeed, return the data, otherwise pass on to decode error. + if isSuccessfulStatus(localVarHttpResponse.StatusCode) { err = a.decode(&localVarReturnValue, localVarBody, localVarHttpResponse.Header.Get("Content-Type")) - if err == nil { - return localVarReturnValue, localVarHttpResponse, err - } - } - - if localVarHttpResponse.StatusCode >= 300 { - newErr := GenericSwaggerError{ - body: localVarBody, - error: localVarHttpResponse.Status, - } - if localVarHttpResponse.StatusCode == 200 { - var v []string - err = a.decode(&v, localVarBody, localVarHttpResponse.Header.Get("Content-Type")) - if err != nil { - newErr.error = err.Error() - return localVarReturnValue, localVarHttpResponse, newErr - } - newErr.model = v - return localVarReturnValue, localVarHttpResponse, newErr - } + } else { + newErr := NewGenericSwaggerError(localVarBody, localVarHttpResponse.Status, nil, localVarHttpResponse.StatusCode) return localVarReturnValue, localVarHttpResponse, newErr } - return localVarReturnValue, localVarHttpResponse, nil + return localVarReturnValue, localVarHttpResponse, err } /* @@ -1054,33 +906,14 @@ func (a *IntegrationResourceApiService) GetTagsForIntegration(ctx context.Contex return localVarReturnValue, localVarHttpResponse, err } - if localVarHttpResponse.StatusCode < 300 { - // If we succeed, return the data, otherwise pass on to decode error. + if isSuccessfulStatus(localVarHttpResponse.StatusCode) { err = a.decode(&localVarReturnValue, localVarBody, localVarHttpResponse.Header.Get("Content-Type")) - if err == nil { - return localVarReturnValue, localVarHttpResponse, err - } - } - - if localVarHttpResponse.StatusCode >= 300 { - newErr := GenericSwaggerError{ - body: localVarBody, - error: localVarHttpResponse.Status, - } - if localVarHttpResponse.StatusCode == 200 { - var v []model.TagObject - err = a.decode(&v, localVarBody, localVarHttpResponse.Header.Get("Content-Type")) - if err != nil { - newErr.error = err.Error() - return localVarReturnValue, localVarHttpResponse, newErr - } - newErr.model = v - return localVarReturnValue, localVarHttpResponse, newErr - } + } else { + newErr := NewGenericSwaggerError(localVarBody, localVarHttpResponse.Status, nil, localVarHttpResponse.StatusCode) return localVarReturnValue, localVarHttpResponse, newErr } - return localVarReturnValue, localVarHttpResponse, nil + return localVarReturnValue, localVarHttpResponse, err } /* @@ -1140,33 +973,14 @@ func (a *IntegrationResourceApiService) GetTagsForIntegrationProvider(ctx contex return localVarReturnValue, localVarHttpResponse, err } - if localVarHttpResponse.StatusCode < 300 { - // If we succeed, return the data, otherwise pass on to decode error. + if isSuccessfulStatus(localVarHttpResponse.StatusCode) { err = a.decode(&localVarReturnValue, localVarBody, localVarHttpResponse.Header.Get("Content-Type")) - if err == nil { - return localVarReturnValue, localVarHttpResponse, err - } - } - - if localVarHttpResponse.StatusCode >= 300 { - newErr := GenericSwaggerError{ - body: localVarBody, - error: localVarHttpResponse.Status, - } - if localVarHttpResponse.StatusCode == 200 { - var v []model.TagObject - err = a.decode(&v, localVarBody, localVarHttpResponse.Header.Get("Content-Type")) - if err != nil { - newErr.error = err.Error() - return localVarReturnValue, localVarHttpResponse, newErr - } - newErr.model = v - return localVarReturnValue, localVarHttpResponse, newErr - } + } else { + newErr := NewGenericSwaggerError(localVarBody, localVarHttpResponse.Status, nil, localVarHttpResponse.StatusCode) return localVarReturnValue, localVarHttpResponse, newErr } - return localVarReturnValue, localVarHttpResponse, nil + return localVarReturnValue, localVarHttpResponse, err } /* @@ -1228,33 +1042,14 @@ func (a *IntegrationResourceApiService) GetTokenUsageForIntegration(ctx context. return localVarReturnValue, localVarHttpResponse, err } - if localVarHttpResponse.StatusCode < 300 { - // If we succeed, return the data, otherwise pass on to decode error. + if isSuccessfulStatus(localVarHttpResponse.StatusCode) { err = a.decode(&localVarReturnValue, localVarBody, localVarHttpResponse.Header.Get("Content-Type")) - if err == nil { - return localVarReturnValue, localVarHttpResponse, err - } - } - - if localVarHttpResponse.StatusCode >= 300 { - newErr := GenericSwaggerError{ - body: localVarBody, - error: localVarHttpResponse.Status, - } - if localVarHttpResponse.StatusCode == 200 { - var v int32 - err = a.decode(&v, localVarBody, localVarHttpResponse.Header.Get("Content-Type")) - if err != nil { - newErr.error = err.Error() - return localVarReturnValue, localVarHttpResponse, newErr - } - newErr.model = v - return localVarReturnValue, localVarHttpResponse, newErr - } + } else { + newErr := NewGenericSwaggerError(localVarBody, localVarHttpResponse.Status, nil, localVarHttpResponse.StatusCode) return localVarReturnValue, localVarHttpResponse, newErr } - return localVarReturnValue, localVarHttpResponse, nil + return localVarReturnValue, localVarHttpResponse, err } /* @@ -1314,33 +1109,14 @@ func (a *IntegrationResourceApiService) GetTokenUsageForIntegrationProvider(ctx return localVarReturnValue, localVarHttpResponse, err } - if localVarHttpResponse.StatusCode < 300 { - // If we succeed, return the data, otherwise pass on to decode error. + if isSuccessfulStatus(localVarHttpResponse.StatusCode) { err = a.decode(&localVarReturnValue, localVarBody, localVarHttpResponse.Header.Get("Content-Type")) - if err == nil { - return localVarReturnValue, localVarHttpResponse, err - } - } - - if localVarHttpResponse.StatusCode >= 300 { - newErr := GenericSwaggerError{ - body: localVarBody, - error: localVarHttpResponse.Status, - } - if localVarHttpResponse.StatusCode == 200 { - var v map[string]string - err = a.decode(&v, localVarBody, localVarHttpResponse.Header.Get("Content-Type")) - if err != nil { - newErr.error = err.Error() - return localVarReturnValue, localVarHttpResponse, newErr - } - newErr.model = v - return localVarReturnValue, localVarHttpResponse, newErr - } + } else { + newErr := NewGenericSwaggerError(localVarBody, localVarHttpResponse.Status, nil, localVarHttpResponse.StatusCode) return localVarReturnValue, localVarHttpResponse, newErr } - return localVarReturnValue, localVarHttpResponse, nil + return localVarReturnValue, localVarHttpResponse, err } /* @@ -1402,11 +1178,8 @@ func (a *IntegrationResourceApiService) UpdateTagForIntegration(ctx context.Cont return localVarHttpResponse, err } - if localVarHttpResponse.StatusCode >= 300 { - newErr := GenericSwaggerError{ - body: localVarBody, - error: localVarHttpResponse.Status, - } + if !isSuccessfulStatus(localVarHttpResponse.StatusCode) { + newErr := NewGenericSwaggerError(localVarBody, localVarHttpResponse.Status, nil, localVarHttpResponse.StatusCode) return localVarHttpResponse, newErr } @@ -1470,11 +1243,8 @@ func (a *IntegrationResourceApiService) UpdateTagForIntegrationProvider(ctx cont return localVarHttpResponse, err } - if localVarHttpResponse.StatusCode >= 300 { - newErr := GenericSwaggerError{ - body: localVarBody, - error: localVarHttpResponse.Status, - } + if !isSuccessfulStatus(localVarHttpResponse.StatusCode) { + newErr := NewGenericSwaggerError(localVarBody, localVarHttpResponse.Status, nil, localVarHttpResponse.StatusCode) return localVarHttpResponse, newErr } @@ -1540,11 +1310,8 @@ func (a *IntegrationResourceApiService) SaveIntegrationApi(ctx context.Context, return localVarHttpResponse, err } - if localVarHttpResponse.StatusCode >= 300 { - newErr := GenericSwaggerError{ - body: localVarBody, - error: localVarHttpResponse.Status, - } + if !isSuccessfulStatus(localVarHttpResponse.StatusCode) { + newErr := NewGenericSwaggerError(localVarBody, localVarHttpResponse.Status, nil, localVarHttpResponse.StatusCode) return localVarHttpResponse, newErr } @@ -1608,11 +1375,8 @@ func (a *IntegrationResourceApiService) SaveIntegrationProvider(ctx context.Cont return localVarHttpResponse, err } - if localVarHttpResponse.StatusCode >= 300 { - newErr := GenericSwaggerError{ - body: localVarBody, - error: localVarHttpResponse.Status, - } + if !isSuccessfulStatus(localVarHttpResponse.StatusCode) { + newErr := NewGenericSwaggerError(localVarBody, localVarHttpResponse.Status, nil, localVarHttpResponse.StatusCode) return localVarHttpResponse, newErr } diff --git a/sdk/client/api_metadata_resource.go b/sdk/client/api_metadata_resource.go index 4c70f2e2..63e3f0fc 100644 --- a/sdk/client/api_metadata_resource.go +++ b/sdk/client/api_metadata_resource.go @@ -87,7 +87,7 @@ func (a *MetadataResourceApiService) RegisterWorkflowDef(ctx context.Context, ov return localVarHttpResponse, err } - if localVarHttpResponse.StatusCode >= 300 { + if !isSuccessfulStatus(localVarHttpResponse.StatusCode) { newErr := NewGenericSwaggerError(localVarBody, string(localVarBody), nil, localVarHttpResponse.StatusCode) return localVarHttpResponse, newErr } @@ -161,7 +161,7 @@ func (a *MetadataResourceApiService) RegisterWorkflowDefWithTags(ctx context.Con return localVarHttpResponse, err } - if localVarHttpResponse.StatusCode >= 300 { + if !isSuccessfulStatus(localVarHttpResponse.StatusCode) { newErr := NewGenericSwaggerError(localVarBody, string(localVarBody), nil, localVarHttpResponse.StatusCode) return localVarHttpResponse, newErr } @@ -235,20 +235,14 @@ func (a *MetadataResourceApiService) Get(ctx context.Context, name string, local return localVarReturnValue, localVarHttpResponse, err } - if localVarHttpResponse.StatusCode < 300 { - // If we succeed, return the data, otherwise pass on to decode error. + if isSuccessfulStatus(localVarHttpResponse.StatusCode) { err = a.decode(&localVarReturnValue, localVarBody, localVarHttpResponse.Header.Get("Content-Type")) - if err == nil { - return localVarReturnValue, localVarHttpResponse, err - } - } - - if localVarHttpResponse.StatusCode >= 300 { + } else { newErr := NewGenericSwaggerError(localVarBody, string(localVarBody), nil, localVarHttpResponse.StatusCode) return localVarReturnValue, localVarHttpResponse, newErr } - return localVarReturnValue, localVarHttpResponse, nil + return localVarReturnValue, localVarHttpResponse, err } /* @@ -306,20 +300,14 @@ func (a *MetadataResourceApiService) GetAll(ctx context.Context) ([]model.Workfl return localVarReturnValue, localVarHttpResponse, err } - if localVarHttpResponse.StatusCode < 300 { - // If we succeed, return the data, otherwise pass on to decode error. + if isSuccessfulStatus(localVarHttpResponse.StatusCode) { err = a.decode(&localVarReturnValue, localVarBody, localVarHttpResponse.Header.Get("Content-Type")) - if err == nil { - return localVarReturnValue, localVarHttpResponse, err - } - } - - if localVarHttpResponse.StatusCode >= 300 { + } else { newErr := NewGenericSwaggerError(localVarBody, string(localVarBody), nil, localVarHttpResponse.StatusCode) return localVarReturnValue, localVarHttpResponse, newErr } - return localVarReturnValue, localVarHttpResponse, nil + return localVarReturnValue, localVarHttpResponse, err } /* @@ -379,20 +367,14 @@ func (a *MetadataResourceApiService) GetTaskDef(ctx context.Context, tasktype st return localVarReturnValue, localVarHttpResponse, err } - if localVarHttpResponse.StatusCode < 300 { - // If we succeed, return the data, otherwise pass on to decode error. + if isSuccessfulStatus(localVarHttpResponse.StatusCode) { err = a.decode(&localVarReturnValue, localVarBody, localVarHttpResponse.Header.Get("Content-Type")) - if err == nil { - return localVarReturnValue, localVarHttpResponse, err - } - } - - if localVarHttpResponse.StatusCode >= 300 { + } else { newErr := NewGenericSwaggerError(localVarBody, string(localVarBody), nil, localVarHttpResponse.StatusCode) return localVarReturnValue, localVarHttpResponse, newErr } - return localVarReturnValue, localVarHttpResponse, nil + return localVarReturnValue, localVarHttpResponse, err } /* @@ -450,20 +432,14 @@ func (a *MetadataResourceApiService) GetTaskDefs(ctx context.Context) ([]model.T return localVarReturnValue, localVarHttpResponse, err } - if localVarHttpResponse.StatusCode < 300 { - // If we succeed, return the data, otherwise pass on to decode error. + if isSuccessfulStatus(localVarHttpResponse.StatusCode) { err = a.decode(&localVarReturnValue, localVarBody, localVarHttpResponse.Header.Get("Content-Type")) - if err == nil { - return localVarReturnValue, localVarHttpResponse, err - } - } - - if localVarHttpResponse.StatusCode >= 300 { + } else { newErr := NewGenericSwaggerError(localVarBody, string(localVarBody), nil, localVarHttpResponse.StatusCode) return localVarReturnValue, localVarHttpResponse, newErr } - return localVarReturnValue, localVarHttpResponse, nil + return localVarReturnValue, localVarHttpResponse, err } /* @@ -521,7 +497,7 @@ func (a *MetadataResourceApiService) UpdateTaskDef(ctx context.Context, body mod return localVarHttpResponse, err } - if localVarHttpResponse.StatusCode >= 300 { + if !isSuccessfulStatus(localVarHttpResponse.StatusCode) { newErr := NewGenericSwaggerError(localVarBody, string(localVarBody), nil, localVarHttpResponse.StatusCode) return localVarHttpResponse, newErr } @@ -593,7 +569,7 @@ func (a *MetadataResourceApiService) UpdateTaskDefWithTags(ctx context.Context, return localVarHttpResponse, err } - if localVarHttpResponse.StatusCode >= 300 { + if !isSuccessfulStatus(localVarHttpResponse.StatusCode) { newErr := NewGenericSwaggerError(localVarBody, string(localVarBody), nil, localVarHttpResponse.StatusCode) return localVarHttpResponse, newErr } @@ -656,7 +632,7 @@ func (a *MetadataResourceApiService) RegisterTaskDef(ctx context.Context, body [ return localVarHttpResponse, err } - if localVarHttpResponse.StatusCode >= 300 { + if !isSuccessfulStatus(localVarHttpResponse.StatusCode) { newErr := NewGenericSwaggerError(localVarBody, string(localVarBody), nil, localVarHttpResponse.StatusCode) return localVarHttpResponse, newErr } @@ -729,7 +705,7 @@ func (a *MetadataResourceApiService) RegisterTaskDefWithTags(ctx context.Context return localVarHttpResponse, err } - if localVarHttpResponse.StatusCode >= 300 { + if !isSuccessfulStatus(localVarHttpResponse.StatusCode) { newErr := NewGenericSwaggerError(localVarBody, string(localVarBody), nil, localVarHttpResponse.StatusCode) return localVarHttpResponse, newErr } @@ -791,7 +767,7 @@ func (a *MetadataResourceApiService) UnregisterTaskDef(ctx context.Context, task return localVarHttpResponse, err } - if localVarHttpResponse.StatusCode >= 300 { + if !isSuccessfulStatus(localVarHttpResponse.StatusCode) { newErr := NewGenericSwaggerError(localVarBody, string(localVarBody), nil, localVarHttpResponse.StatusCode) return localVarHttpResponse, newErr } @@ -855,7 +831,7 @@ func (a *MetadataResourceApiService) UnregisterWorkflowDef(ctx context.Context, return localVarHttpResponse, err } - if localVarHttpResponse.StatusCode >= 300 { + if !isSuccessfulStatus(localVarHttpResponse.StatusCode) { newErr := NewGenericSwaggerError(localVarBody, string(localVarBody), nil, localVarHttpResponse.StatusCode) return localVarHttpResponse, newErr } @@ -918,7 +894,7 @@ func (a *MetadataResourceApiService) Update(ctx context.Context, body []model.Wo return localVarHttpResponse, err } - if localVarHttpResponse.StatusCode >= 300 { + if !isSuccessfulStatus(localVarHttpResponse.StatusCode) { newErr := NewGenericSwaggerError(localVarBody, string(localVarBody), nil, localVarHttpResponse.StatusCode) return localVarHttpResponse, newErr } @@ -991,7 +967,7 @@ func (a *MetadataResourceApiService) UpdateWorkflowDefWithTags(ctx context.Conte return localVarHttpResponse, err } - if localVarHttpResponse.StatusCode >= 300 { + if !isSuccessfulStatus(localVarHttpResponse.StatusCode) { newErr := NewGenericSwaggerError(localVarBody, string(localVarBody), nil, localVarHttpResponse.StatusCode) return localVarHttpResponse, newErr } @@ -1049,9 +1025,8 @@ func (a *MetadataResourceApiService) GetTagsForWorkflowDef(ctx context.Context, return localVarReturnValue, err } - if localVarHttpResponse.StatusCode < 300 { + if isSuccessfulStatus(localVarHttpResponse.StatusCode) { extendedWorkflowDef := model.WorkflowDef{} - // If we succeed, return the data, otherwise pass on to decode error. err = a.decode(&extendedWorkflowDef, localVarBody, localVarHttpResponse.Header.Get("Content-Type")) if err == nil { for i := 0; i < len(extendedWorkflowDef.Tags); i++ { @@ -1062,16 +1037,13 @@ func (a *MetadataResourceApiService) GetTagsForWorkflowDef(ctx context.Context, } localVarReturnValue = append(localVarReturnValue, tag) } - return localVarReturnValue, err } - } - - if localVarHttpResponse.StatusCode >= 300 { + } else { newErr := NewGenericSwaggerError(localVarBody, string(localVarBody), nil, localVarHttpResponse.StatusCode) return localVarReturnValue, newErr } - return localVarReturnValue, nil + return localVarReturnValue, err } func (a *MetadataResourceApiService) GetTagsForTaskDef(ctx context.Context, tasktype string) ([]model.MetadataTag, error) { @@ -1122,11 +1094,9 @@ func (a *MetadataResourceApiService) GetTagsForTaskDef(ctx context.Context, task return localVarReturnValue, err } - if localVarHttpResponse.StatusCode < 300 { + if isSuccessfulStatus(localVarHttpResponse.StatusCode) { extendedTaskDef := model.WorkflowDef{} - // If we succeed, return the data, otherwise pass on to decode error. err = a.decode(&extendedTaskDef, localVarBody, localVarHttpResponse.Header.Get("Content-Type")) - if err == nil { for i := 0; i < len(extendedTaskDef.Tags); i++ { value := fmt.Sprintf("%v", extendedTaskDef.Tags[i].Value) @@ -1136,14 +1106,10 @@ func (a *MetadataResourceApiService) GetTagsForTaskDef(ctx context.Context, task } localVarReturnValue = append(localVarReturnValue, tag) } - return localVarReturnValue, err } + } else { + return localVarReturnValue, NewGenericSwaggerError(localVarBody, string(localVarBody), nil, localVarHttpResponse.StatusCode) } - if localVarHttpResponse.StatusCode >= 300 { - newErr := NewGenericSwaggerError(localVarBody, string(localVarBody), nil, localVarHttpResponse.StatusCode) - return localVarReturnValue, newErr - } - - return localVarReturnValue, nil + return localVarReturnValue, err } diff --git a/sdk/client/api_prompt_resource.go b/sdk/client/api_prompt_resource.go index 8fa8ae3f..3ab0cf7b 100644 --- a/sdk/client/api_prompt_resource.go +++ b/sdk/client/api_prompt_resource.go @@ -78,9 +78,8 @@ func (a *PromptResourceApiService) DeleteMessageTemplate(ctx context.Context, na return httpResponse, err } - if httpResponse.StatusCode >= 300 { - newErr := NewGenericSwaggerError(responseBody, httpResponse.Status, nil, httpResponse.StatusCode) - return httpResponse, newErr + if !isSuccessfulStatus(httpResponse.StatusCode) { + return httpResponse, NewGenericSwaggerError(responseBody, httpResponse.Status, nil, httpResponse.StatusCode) } return httpResponse, nil @@ -143,9 +142,8 @@ func (a *PromptResourceApiService) DeleteTagForPromptTemplate(ctx context.Contex return httpResponse, err } - if httpResponse.StatusCode >= 300 { - newErr := NewGenericSwaggerError(responseBody, httpResponse.Status, nil, httpResponse.StatusCode) - return httpResponse, newErr + if !isSuccessfulStatus(httpResponse.StatusCode) { + return httpResponse, NewGenericSwaggerError(responseBody, httpResponse.Status, nil, httpResponse.StatusCode) } return httpResponse, nil @@ -207,21 +205,14 @@ func (a *PromptResourceApiService) GetMessageTemplate(ctx context.Context, name return nil, httpResponse, err } - if httpResponse.StatusCode < 300 { - // If we succeed, return the data, otherwise pass on to decode error. + if isSuccessfulStatus(httpResponse.StatusCode) { err = a.decode(&returnValue, responseBody, httpResponse.Header.Get("Content-Type")) - if err == nil { - return &returnValue, httpResponse, err - } - } - - if httpResponse.StatusCode >= 300 { + } else { newErr := NewGenericSwaggerError(responseBody, httpResponse.Status, nil, httpResponse.StatusCode) - return nil, httpResponse, newErr } - return nil, httpResponse, nil + return &returnValue, httpResponse, err } /* @@ -278,20 +269,14 @@ func (a *PromptResourceApiService) GetMessageTemplates(ctx context.Context) ([]i return returnValue, httpResponse, err } - if httpResponse.StatusCode < 300 { - // If we succeed, return the data, otherwise pass on to decode error. + if isSuccessfulStatus(httpResponse.StatusCode) { err = a.decode(&returnValue, responseBody, httpResponse.Header.Get("Content-Type")) - if err == nil { - return returnValue, httpResponse, err - } - } - - if httpResponse.StatusCode >= 300 { + } else { newErr := NewGenericSwaggerError(responseBody, httpResponse.Status, nil, httpResponse.StatusCode) return returnValue, httpResponse, newErr } - return returnValue, httpResponse, nil + return returnValue, httpResponse, err } /* @@ -350,20 +335,14 @@ func (a *PromptResourceApiService) GetTagsForPromptTemplate(ctx context.Context, return returnValue, httpResponse, err } - if httpResponse.StatusCode < 300 { - // If we succeed, return the data, otherwise pass on to decode error. + if isSuccessfulStatus(httpResponse.StatusCode) { err = a.decode(&returnValue, responseBody, httpResponse.Header.Get("Content-Type")) - if err == nil { - return returnValue, httpResponse, err - } - } - - if httpResponse.StatusCode >= 300 { + } else { newErr := NewGenericSwaggerError(responseBody, httpResponse.Status, nil, httpResponse.StatusCode) return returnValue, httpResponse, newErr } - return returnValue, httpResponse, nil + return returnValue, httpResponse, err } /* @@ -423,9 +402,8 @@ func (a *PromptResourceApiService) PutTagForPromptTemplate(ctx context.Context, return httpResponse, err } - if httpResponse.StatusCode >= 300 { - newErr := NewGenericSwaggerError(responseBody, httpResponse.Status, nil, httpResponse.StatusCode) - return httpResponse, newErr + if !isSuccessfulStatus(httpResponse.StatusCode) { + return httpResponse, NewGenericSwaggerError(responseBody, httpResponse.Status, nil, httpResponse.StatusCode) } return httpResponse, nil @@ -501,9 +479,8 @@ func (a *PromptResourceApiService) SaveMessageTemplate(ctx context.Context, body return httpResponse, err } - if httpResponse.StatusCode >= 300 { - newErr := NewGenericSwaggerError(responseBody, httpResponse.Status, nil, httpResponse.StatusCode) - return httpResponse, newErr + if !isSuccessfulStatus(httpResponse.StatusCode) { + return httpResponse, NewGenericSwaggerError(responseBody, httpResponse.Status, nil, httpResponse.StatusCode) } return httpResponse, nil @@ -566,18 +543,12 @@ func (a *PromptResourceApiService) TestMessageTemplate(ctx context.Context, body return returnValue, httpResponse, err } - if httpResponse.StatusCode < 300 { - // If we succeed, return the data, otherwise pass on to decode error. + if isSuccessfulStatus(httpResponse.StatusCode) { err = a.decode(&returnValue, responseBody, httpResponse.Header.Get("Content-Type")) - if err == nil { - return returnValue, httpResponse, err - } - } - - if httpResponse.StatusCode >= 300 { + } else { newErr := NewGenericSwaggerError(responseBody, httpResponse.Status, nil, httpResponse.StatusCode) return returnValue, httpResponse, newErr } - return returnValue, httpResponse, nil + return returnValue, httpResponse, err } diff --git a/sdk/client/api_scheduler_resource.go b/sdk/client/api_scheduler_resource.go index 00419705..22999b73 100644 --- a/sdk/client/api_scheduler_resource.go +++ b/sdk/client/api_scheduler_resource.go @@ -80,20 +80,14 @@ func (a *SchedulerResourceApiService) DeleteSchedule(ctx context.Context, name s return returnValue, httpResponse, err } - if httpResponse.StatusCode < 300 { - // If we succeed, return the data, otherwise pass on to decode error. + if isSuccessfulStatus(httpResponse.StatusCode) { err = a.decode(&returnValue, responseBody, httpResponse.Header.Get("Content-Type")) - if err == nil { - return returnValue, httpResponse, err - } - } - - if httpResponse.StatusCode >= 300 { + } else { newErr := NewGenericSwaggerError(responseBody, httpResponse.Status, nil, httpResponse.StatusCode) return returnValue, httpResponse, newErr } - return returnValue, httpResponse, nil + return returnValue, httpResponse, err } /* @@ -153,9 +147,8 @@ func (a *SchedulerResourceApiService) DeleteTagForSchedule(ctx context.Context, return httpResponse, err } - if httpResponse.StatusCode >= 300 { - newErr := NewGenericSwaggerError(responseBody, httpResponse.Status, nil, httpResponse.StatusCode) - return httpResponse, newErr + if !isSuccessfulStatus(httpResponse.StatusCode) { + return httpResponse, NewGenericSwaggerError(responseBody, httpResponse.Status, nil, httpResponse.StatusCode) } return httpResponse, nil @@ -225,20 +218,14 @@ func (a *SchedulerResourceApiService) GetAllSchedules(ctx context.Context, optio return returnValue, httpResponse, err } - if httpResponse.StatusCode < 300 { - // If we succeed, return the data, otherwise pass on to decode error. + if isSuccessfulStatus(httpResponse.StatusCode) { err = a.decode(&returnValue, responseBody, httpResponse.Header.Get("Content-Type")) - if err == nil { - return returnValue, httpResponse, err - } - } - - if httpResponse.StatusCode >= 300 { + } else { newErr := NewGenericSwaggerError(responseBody, httpResponse.Status, nil, httpResponse.StatusCode) return returnValue, httpResponse, newErr } - return returnValue, httpResponse, nil + return returnValue, httpResponse, err } /* @@ -317,20 +304,14 @@ func (a *SchedulerResourceApiService) GetNextFewSchedules(ctx context.Context, c return returnValue, httpResponse, err } - if httpResponse.StatusCode < 300 { - // If we succeed, return the data, otherwise pass on to decode error. + if isSuccessfulStatus(httpResponse.StatusCode) { err = a.decode(&returnValue, responseBody, httpResponse.Header.Get("Content-Type")) - if err == nil { - return returnValue, httpResponse, err - } - } - - if httpResponse.StatusCode >= 300 { + } else { newErr := NewGenericSwaggerError(responseBody, httpResponse.Status, nil, httpResponse.StatusCode) return returnValue, httpResponse, newErr } - return returnValue, httpResponse, nil + return returnValue, httpResponse, err } /* @@ -389,20 +370,14 @@ func (a *SchedulerResourceApiService) GetSchedule(ctx context.Context, name stri return returnValue, httpResponse, err } - if httpResponse.StatusCode < 300 { - // If we succeed, return the data, otherwise pass on to decode error. + if isSuccessfulStatus(httpResponse.StatusCode) { err = a.decode(&returnValue, responseBody, httpResponse.Header.Get("Content-Type")) - if err == nil { - return returnValue, httpResponse, err - } - } - - if httpResponse.StatusCode >= 300 { + } else { newErr := NewGenericSwaggerError(responseBody, httpResponse.Status, nil, httpResponse.StatusCode) return returnValue, httpResponse, newErr } - return returnValue, httpResponse, nil + return returnValue, httpResponse, err } /* @@ -461,20 +436,14 @@ func (a *SchedulerResourceApiService) GetTagsForSchedule(ctx context.Context, na return returnValue, httpResponse, err } - if httpResponse.StatusCode < 300 { - // If we succeed, return the data, otherwise pass on to decode error. + if isSuccessfulStatus(httpResponse.StatusCode) { err = a.decode(&returnValue, responseBody, httpResponse.Header.Get("Content-Type")) - if err == nil { - return returnValue, httpResponse, err - } - } - - if httpResponse.StatusCode >= 300 { + } else { newErr := NewGenericSwaggerError(responseBody, httpResponse.Status, nil, httpResponse.StatusCode) return returnValue, httpResponse, newErr } - return returnValue, httpResponse, nil + return returnValue, httpResponse, err } /* @@ -531,20 +500,14 @@ func (a *SchedulerResourceApiService) PauseAllSchedules(ctx context.Context) (ma return returnValue, httpResponse, err } - if httpResponse.StatusCode < 300 { - // If we succeed, return the data, otherwise pass on to decode error. + if isSuccessfulStatus(httpResponse.StatusCode) { err = a.decode(&returnValue, responseBody, httpResponse.Header.Get("Content-Type")) - if err == nil { - return returnValue, httpResponse, err - } - } - - if httpResponse.StatusCode >= 300 { + } else { newErr := NewGenericSwaggerError(responseBody, httpResponse.Status, nil, httpResponse.StatusCode) return returnValue, httpResponse, newErr } - return returnValue, httpResponse, nil + return returnValue, httpResponse, err } /* @@ -603,20 +566,14 @@ func (a *SchedulerResourceApiService) PauseSchedule(ctx context.Context, name st return returnValue, httpResponse, err } - if httpResponse.StatusCode < 300 { - // If we succeed, return the data, otherwise pass on to decode error. + if isSuccessfulStatus(httpResponse.StatusCode) { err = a.decode(&returnValue, responseBody, httpResponse.Header.Get("Content-Type")) - if err == nil { - return returnValue, httpResponse, err - } - } - - if httpResponse.StatusCode >= 300 { + } else { newErr := NewGenericSwaggerError(responseBody, httpResponse.Status, nil, httpResponse.StatusCode) return returnValue, httpResponse, newErr } - return returnValue, httpResponse, nil + return returnValue, httpResponse, err } /* @@ -676,9 +633,8 @@ func (a *SchedulerResourceApiService) PutTagForSchedule(ctx context.Context, bod return httpResponse, err } - if httpResponse.StatusCode >= 300 { - newErr := NewGenericSwaggerError(responseBody, httpResponse.Status, nil, httpResponse.StatusCode) - return httpResponse, newErr + if !isSuccessfulStatus(httpResponse.StatusCode) { + return httpResponse, NewGenericSwaggerError(responseBody, httpResponse.Status, nil, httpResponse.StatusCode) } return httpResponse, nil @@ -738,20 +694,14 @@ func (a *SchedulerResourceApiService) RequeueAllExecutionRecords(ctx context.Con return returnValue, httpResponse, err } - if httpResponse.StatusCode < 300 { - // If we succeed, return the data, otherwise pass on to decode error. + if isSuccessfulStatus(httpResponse.StatusCode) { err = a.decode(&returnValue, responseBody, httpResponse.Header.Get("Content-Type")) - if err == nil { - return returnValue, httpResponse, err - } - } - - if httpResponse.StatusCode >= 300 { + } else { newErr := NewGenericSwaggerError(responseBody, httpResponse.Status, nil, httpResponse.StatusCode) return returnValue, httpResponse, newErr } - return returnValue, httpResponse, nil + return returnValue, httpResponse, err } /* @@ -808,20 +758,14 @@ func (a *SchedulerResourceApiService) ResumeAllSchedules(ctx context.Context) (m return returnValue, httpResponse, err } - if httpResponse.StatusCode < 300 { - // If we succeed, return the data, otherwise pass on to decode error. + if isSuccessfulStatus(httpResponse.StatusCode) { err = a.decode(&returnValue, responseBody, httpResponse.Header.Get("Content-Type")) - if err == nil { - return returnValue, httpResponse, err - } - } - - if httpResponse.StatusCode >= 300 { + } else { newErr := NewGenericSwaggerError(responseBody, httpResponse.Status, nil, httpResponse.StatusCode) return returnValue, httpResponse, newErr } - return returnValue, httpResponse, nil + return returnValue, httpResponse, err } /* @@ -880,20 +824,14 @@ func (a *SchedulerResourceApiService) ResumeSchedule(ctx context.Context, name s return returnValue, httpResponse, err } - if httpResponse.StatusCode < 300 { - // If we succeed, return the data, otherwise pass on to decode error. + if isSuccessfulStatus(httpResponse.StatusCode) { err = a.decode(&returnValue, responseBody, httpResponse.Header.Get("Content-Type")) - if err == nil { - return returnValue, httpResponse, err - } - } - - if httpResponse.StatusCode >= 300 { + } else { newErr := NewGenericSwaggerError(responseBody, httpResponse.Status, nil, httpResponse.StatusCode) return returnValue, httpResponse, newErr } - return returnValue, httpResponse, nil + return returnValue, httpResponse, err } /* @@ -953,20 +891,14 @@ func (a *SchedulerResourceApiService) SaveSchedule(ctx context.Context, body mod return returnValue, httpResponse, err } - if httpResponse.StatusCode < 300 { - // If we succeed, return the data, otherwise pass on to decode error. + if isSuccessfulStatus(httpResponse.StatusCode) { err = a.decode(&returnValue, responseBody, httpResponse.Header.Get("Content-Type")) - if err == nil { - return returnValue, httpResponse, err - } - } - - if httpResponse.StatusCode >= 300 { + } else { newErr := NewGenericSwaggerError(responseBody, httpResponse.Status, nil, httpResponse.StatusCode) return returnValue, httpResponse, newErr } - return returnValue, httpResponse, nil + return returnValue, httpResponse, err } /* @@ -1054,18 +986,12 @@ func (a *SchedulerResourceApiService) SearchV2(ctx context.Context, optionals *S return returnValue, httpResponse, err } - if httpResponse.StatusCode < 300 { - // If we succeed, return the data, otherwise pass on to decode error. + if isSuccessfulStatus(httpResponse.StatusCode) { err = a.decode(&returnValue, responseBody, httpResponse.Header.Get("Content-Type")) - if err == nil { - return returnValue, httpResponse, err - } - } - - if httpResponse.StatusCode >= 300 { + } else { newErr := NewGenericSwaggerError(responseBody, httpResponse.Status, nil, httpResponse.StatusCode) return returnValue, httpResponse, newErr } - return returnValue, httpResponse, nil + return returnValue, httpResponse, err } diff --git a/sdk/client/api_secret_resource.go b/sdk/client/api_secret_resource.go index 99564f11..c51d2818 100644 --- a/sdk/client/api_secret_resource.go +++ b/sdk/client/api_secret_resource.go @@ -77,20 +77,14 @@ func (a *SecretResourceApiService) ClearLocalCache(ctx context.Context) (map[str return returnValue, httpResponse, err } - if httpResponse.StatusCode < 300 { - // If we succeed, return the data, otherwise pass on to decode error. + if isSuccessfulStatus(httpResponse.StatusCode) { err = a.decode(&returnValue, responseBody, httpResponse.Header.Get("Content-Type")) - if err == nil { - return returnValue, httpResponse, err - } - } - - if httpResponse.StatusCode >= 300 { + } else { newErr := NewGenericSwaggerError(responseBody, httpResponse.Status, nil, httpResponse.StatusCode) return returnValue, httpResponse, newErr } - return returnValue, httpResponse, nil + return returnValue, httpResponse, err } /* @@ -147,20 +141,14 @@ func (a *SecretResourceApiService) ClearRedisCache(ctx context.Context) (map[str return returnValue, httpResponse, err } - if httpResponse.StatusCode < 300 { - // If we succeed, return the data, otherwise pass on to decode error. + if isSuccessfulStatus(httpResponse.StatusCode) { err = a.decode(&returnValue, responseBody, httpResponse.Header.Get("Content-Type")) - if err == nil { - return returnValue, httpResponse, err - } - } - - if httpResponse.StatusCode >= 300 { + } else { newErr := NewGenericSwaggerError(responseBody, httpResponse.Status, nil, httpResponse.StatusCode) return returnValue, httpResponse, newErr } - return returnValue, httpResponse, nil + return returnValue, httpResponse, err } /* @@ -219,20 +207,14 @@ func (a *SecretResourceApiService) DeleteSecret(ctx context.Context, key string) return returnValue, httpResponse, err } - if httpResponse.StatusCode < 300 { - // If we succeed, return the data, otherwise pass on to decode error. + if isSuccessfulStatus(httpResponse.StatusCode) { err = a.decode(&returnValue, responseBody, httpResponse.Header.Get("Content-Type")) - if err == nil { - return returnValue, httpResponse, err - } - } - - if httpResponse.StatusCode >= 300 { + } else { newErr := NewGenericSwaggerError(responseBody, httpResponse.Status, nil, httpResponse.StatusCode) return returnValue, httpResponse, newErr } - return returnValue, httpResponse, nil + return returnValue, httpResponse, err } /* @@ -292,9 +274,8 @@ func (a *SecretResourceApiService) DeleteTagForSecret(ctx context.Context, body return httpResponse, err } - if httpResponse.StatusCode >= 300 { - newErr := NewGenericSwaggerError(responseBody, httpResponse.Status, nil, httpResponse.StatusCode) - return httpResponse, newErr + if !isSuccessfulStatus(httpResponse.StatusCode) { + return httpResponse, NewGenericSwaggerError(responseBody, httpResponse.Status, nil, httpResponse.StatusCode) } return httpResponse, nil @@ -356,20 +337,14 @@ func (a *SecretResourceApiService) GetSecret(ctx context.Context, key string) (s return returnValue, httpResponse, err } - if httpResponse.StatusCode < 300 { - // If we succeed, return the data, otherwise pass on to decode error. + if isSuccessfulStatus(httpResponse.StatusCode) { err = a.decode(&returnValue, responseBody, httpResponse.Header.Get("Content-Type")) - if err == nil { - return returnValue, httpResponse, err - } - } - - if httpResponse.StatusCode >= 300 { + } else { newErr := NewGenericSwaggerError(responseBody, httpResponse.Status, nil, httpResponse.StatusCode) return returnValue, httpResponse, newErr } - return returnValue, httpResponse, nil + return returnValue, httpResponse, err } /* @@ -428,20 +403,14 @@ func (a *SecretResourceApiService) GetTags(ctx context.Context, key string) ([]m return returnValue, httpResponse, err } - if httpResponse.StatusCode < 300 { - // If we succeed, return the data, otherwise pass on to decode error. + if isSuccessfulStatus(httpResponse.StatusCode) { err = a.decode(&returnValue, responseBody, httpResponse.Header.Get("Content-Type")) - if err == nil { - return returnValue, httpResponse, err - } - } - - if httpResponse.StatusCode >= 300 { + } else { newErr := NewGenericSwaggerError(responseBody, httpResponse.Status, nil, httpResponse.StatusCode) return returnValue, httpResponse, newErr } - return returnValue, httpResponse, nil + return returnValue, httpResponse, err } /* @@ -498,20 +467,14 @@ func (a *SecretResourceApiService) ListAllSecretNames(ctx context.Context) ([]st return returnValue, httpResponse, err } - if httpResponse.StatusCode < 300 { - // If we succeed, return the data, otherwise pass on to decode error. + if isSuccessfulStatus(httpResponse.StatusCode) { err = a.decode(&returnValue, responseBody, httpResponse.Header.Get("Content-Type")) - if err == nil { - return returnValue, httpResponse, err - } - } - - if httpResponse.StatusCode >= 300 { + } else { newErr := NewGenericSwaggerError(responseBody, httpResponse.Status, nil, httpResponse.StatusCode) return returnValue, httpResponse, newErr } - return returnValue, httpResponse, nil + return returnValue, httpResponse, err } /* @@ -568,20 +531,14 @@ func (a *SecretResourceApiService) ListSecretsThatUserCanGrantAccessTo(ctx conte return returnValue, httpResponse, err } - if httpResponse.StatusCode < 300 { - // If we succeed, return the data, otherwise pass on to decode error. + if isSuccessfulStatus(httpResponse.StatusCode) { err = a.decode(&returnValue, responseBody, httpResponse.Header.Get("Content-Type")) - if err == nil { - return returnValue, httpResponse, err - } - } - - if httpResponse.StatusCode >= 300 { + } else { newErr := NewGenericSwaggerError(responseBody, httpResponse.Status, nil, httpResponse.StatusCode) return returnValue, httpResponse, newErr } - return returnValue, httpResponse, nil + return returnValue, httpResponse, err } /* @@ -638,20 +595,14 @@ func (a *SecretResourceApiService) ListSecretsWithTagsThatUserCanGrantAccessTo(c return returnValue, httpResponse, err } - if httpResponse.StatusCode < 300 { - // If we succeed, return the data, otherwise pass on to decode error. + if isSuccessfulStatus(httpResponse.StatusCode) { err = a.decode(&returnValue, responseBody, httpResponse.Header.Get("Content-Type")) - if err == nil { - return returnValue, httpResponse, err - } - } - - if httpResponse.StatusCode >= 300 { + } else { newErr := NewGenericSwaggerError(responseBody, httpResponse.Status, nil, httpResponse.StatusCode) return returnValue, httpResponse, newErr } - return returnValue, httpResponse, nil + return returnValue, httpResponse, err } /* @@ -713,20 +664,14 @@ func (a *SecretResourceApiService) PutSecret(ctx context.Context, body string, k return returnValue, httpResponse, err } - if httpResponse.StatusCode < 300 { - // If we succeed, return the data, otherwise pass on to decode error. + if isSuccessfulStatus(httpResponse.StatusCode) { err = a.decode(&returnValue, responseBody, httpResponse.Header.Get("Content-Type")) - if err == nil { - return returnValue, httpResponse, err - } - } - - if httpResponse.StatusCode >= 300 { + } else { newErr := NewGenericSwaggerError(responseBody, httpResponse.Status, nil, httpResponse.StatusCode) return returnValue, httpResponse, newErr } - return returnValue, httpResponse, nil + return returnValue, httpResponse, err } /* @@ -786,9 +731,8 @@ func (a *SecretResourceApiService) PutTagForSecret(ctx context.Context, body []m return httpResponse, err } - if httpResponse.StatusCode >= 300 { - newErr := NewGenericSwaggerError(responseBody, httpResponse.Status, nil, httpResponse.StatusCode) - return httpResponse, newErr + if !isSuccessfulStatus(httpResponse.StatusCode) { + return httpResponse, NewGenericSwaggerError(responseBody, httpResponse.Status, nil, httpResponse.StatusCode) } return httpResponse, nil @@ -850,18 +794,12 @@ func (a *SecretResourceApiService) SecretExists(ctx context.Context, key string) return returnValue, httpResponse, err } - if httpResponse.StatusCode < 300 { - // If we succeed, return the data, otherwise pass on to decode error. + if isSuccessfulStatus(httpResponse.StatusCode) { err = a.decode(&returnValue, responseBody, httpResponse.Header.Get("Content-Type")) - if err == nil { - return returnValue, httpResponse, err - } - } - - if httpResponse.StatusCode >= 300 { + } else { newErr := NewGenericSwaggerError(responseBody, httpResponse.Status, nil, httpResponse.StatusCode) return returnValue, httpResponse, newErr } - return returnValue, httpResponse, nil + return returnValue, httpResponse, err } diff --git a/sdk/client/api_tags_resource.go b/sdk/client/api_tags_resource.go index 97f7bace..8b0e6acd 100644 --- a/sdk/client/api_tags_resource.go +++ b/sdk/client/api_tags_resource.go @@ -83,33 +83,14 @@ func (a *TagsApiService) AddTaskTag(ctx context.Context, body model.TagObject, t return localVarReturnValue, localVarHttpResponse, err } - if localVarHttpResponse.StatusCode < 300 { - // If we succeed, return the data, otherwise pass on to decode error. + if isSuccessfulStatus(localVarHttpResponse.StatusCode) { err = a.decode(&localVarReturnValue, localVarBody, localVarHttpResponse.Header.Get("Content-Type")) - if err == nil { - return localVarReturnValue, localVarHttpResponse, err - } - } - - if localVarHttpResponse.StatusCode >= 300 { - newErr := GenericSwaggerError{ - body: localVarBody, - error: localVarHttpResponse.Status, - } - if localVarHttpResponse.StatusCode == 200 { - var v interface{} - err = a.decode(&v, localVarBody, localVarHttpResponse.Header.Get("Content-Type")) - if err != nil { - newErr.error = err.Error() - return localVarReturnValue, localVarHttpResponse, newErr - } - newErr.model = v - return localVarReturnValue, localVarHttpResponse, newErr - } + } else { + newErr := NewGenericSwaggerError(localVarBody, localVarHttpResponse.Status, nil, localVarHttpResponse.StatusCode) return localVarReturnValue, localVarHttpResponse, newErr } - return localVarReturnValue, localVarHttpResponse, nil + return localVarReturnValue, localVarHttpResponse, err } /* @@ -172,33 +153,14 @@ func (a *TagsApiService) AddWorkflowTag(ctx context.Context, body model.TagObjec return localVarReturnValue, localVarHttpResponse, err } - if localVarHttpResponse.StatusCode < 300 { - // If we succeed, return the data, otherwise pass on to decode error. + if isSuccessfulStatus(localVarHttpResponse.StatusCode) { err = a.decode(&localVarReturnValue, localVarBody, localVarHttpResponse.Header.Get("Content-Type")) - if err == nil { - return localVarReturnValue, localVarHttpResponse, err - } - } - - if localVarHttpResponse.StatusCode >= 300 { - newErr := GenericSwaggerError{ - body: localVarBody, - error: localVarHttpResponse.Status, - } - if localVarHttpResponse.StatusCode == 200 { - var v interface{} - err = a.decode(&v, localVarBody, localVarHttpResponse.Header.Get("Content-Type")) - if err != nil { - newErr.error = err.Error() - return localVarReturnValue, localVarHttpResponse, newErr - } - newErr.model = v - return localVarReturnValue, localVarHttpResponse, newErr - } + } else { + newErr := NewGenericSwaggerError(localVarBody, localVarHttpResponse.Status, nil, localVarHttpResponse.StatusCode) return localVarReturnValue, localVarHttpResponse, newErr } - return localVarReturnValue, localVarHttpResponse, nil + return localVarReturnValue, localVarHttpResponse, err } /* @@ -261,33 +223,14 @@ func (a *TagsApiService) DeleteTaskTag(ctx context.Context, body model.TagString return localVarReturnValue, localVarHttpResponse, err } - if localVarHttpResponse.StatusCode < 300 { - // If we succeed, return the data, otherwise pass on to decode error. + if isSuccessfulStatus(localVarHttpResponse.StatusCode) { err = a.decode(&localVarReturnValue, localVarBody, localVarHttpResponse.Header.Get("Content-Type")) - if err == nil { - return localVarReturnValue, localVarHttpResponse, err - } - } - - if localVarHttpResponse.StatusCode >= 300 { - newErr := GenericSwaggerError{ - body: localVarBody, - error: localVarHttpResponse.Status, - } - if localVarHttpResponse.StatusCode == 200 { - var v interface{} - err = a.decode(&v, localVarBody, localVarHttpResponse.Header.Get("Content-Type")) - if err != nil { - newErr.error = err.Error() - return localVarReturnValue, localVarHttpResponse, newErr - } - newErr.model = v - return localVarReturnValue, localVarHttpResponse, newErr - } + } else { + newErr := NewGenericSwaggerError(localVarBody, localVarHttpResponse.Status, nil, localVarHttpResponse.StatusCode) return localVarReturnValue, localVarHttpResponse, newErr } - return localVarReturnValue, localVarHttpResponse, nil + return localVarReturnValue, localVarHttpResponse, err } /* @@ -350,33 +293,14 @@ func (a *TagsApiService) DeleteWorkflowTag(ctx context.Context, body model.TagOb return localVarReturnValue, localVarHttpResponse, err } - if localVarHttpResponse.StatusCode < 300 { - // If we succeed, return the data, otherwise pass on to decode error. + if isSuccessfulStatus(localVarHttpResponse.StatusCode) { err = a.decode(&localVarReturnValue, localVarBody, localVarHttpResponse.Header.Get("Content-Type")) - if err == nil { - return localVarReturnValue, localVarHttpResponse, err - } - } - - if localVarHttpResponse.StatusCode >= 300 { - newErr := GenericSwaggerError{ - body: localVarBody, - error: localVarHttpResponse.Status, - } - if localVarHttpResponse.StatusCode == 200 { - var v interface{} - err = a.decode(&v, localVarBody, localVarHttpResponse.Header.Get("Content-Type")) - if err != nil { - newErr.error = err.Error() - return localVarReturnValue, localVarHttpResponse, newErr - } - newErr.model = v - return localVarReturnValue, localVarHttpResponse, newErr - } + } else { + newErr := NewGenericSwaggerError(localVarBody, localVarHttpResponse.Status, nil, localVarHttpResponse.StatusCode) return localVarReturnValue, localVarHttpResponse, newErr } - return localVarReturnValue, localVarHttpResponse, nil + return localVarReturnValue, localVarHttpResponse, err } /* @@ -434,33 +358,14 @@ func (a *TagsApiService) GetTags1(ctx context.Context) ([]model.TagObject, *http return localVarReturnValue, localVarHttpResponse, err } - if localVarHttpResponse.StatusCode < 300 { - // If we succeed, return the data, otherwise pass on to decode error. + if isSuccessfulStatus(localVarHttpResponse.StatusCode) { err = a.decode(&localVarReturnValue, localVarBody, localVarHttpResponse.Header.Get("Content-Type")) - if err == nil { - return localVarReturnValue, localVarHttpResponse, err - } - } - - if localVarHttpResponse.StatusCode >= 300 { - newErr := GenericSwaggerError{ - body: localVarBody, - error: localVarHttpResponse.Status, - } - if localVarHttpResponse.StatusCode == 200 { - var v []model.TagObject - err = a.decode(&v, localVarBody, localVarHttpResponse.Header.Get("Content-Type")) - if err != nil { - newErr.error = err.Error() - return localVarReturnValue, localVarHttpResponse, newErr - } - newErr.model = v - return localVarReturnValue, localVarHttpResponse, newErr - } + } else { + newErr := NewGenericSwaggerError(localVarBody, localVarHttpResponse.Status, nil, localVarHttpResponse.StatusCode) return localVarReturnValue, localVarHttpResponse, newErr } - return localVarReturnValue, localVarHttpResponse, nil + return localVarReturnValue, localVarHttpResponse, err } /* @@ -520,33 +425,14 @@ func (a *TagsApiService) GetTaskTags(ctx context.Context, taskName string) ([]mo return localVarReturnValue, localVarHttpResponse, err } - if localVarHttpResponse.StatusCode < 300 { - // If we succeed, return the data, otherwise pass on to decode error. + if isSuccessfulStatus(localVarHttpResponse.StatusCode) { err = a.decode(&localVarReturnValue, localVarBody, localVarHttpResponse.Header.Get("Content-Type")) - if err == nil { - return localVarReturnValue, localVarHttpResponse, err - } - } - - if localVarHttpResponse.StatusCode >= 300 { - newErr := GenericSwaggerError{ - body: localVarBody, - error: localVarHttpResponse.Status, - } - if localVarHttpResponse.StatusCode == 200 { - var v []model.TagObject - err = a.decode(&v, localVarBody, localVarHttpResponse.Header.Get("Content-Type")) - if err != nil { - newErr.error = err.Error() - return localVarReturnValue, localVarHttpResponse, newErr - } - newErr.model = v - return localVarReturnValue, localVarHttpResponse, newErr - } + } else { + newErr := NewGenericSwaggerError(localVarBody, localVarHttpResponse.Status, nil, localVarHttpResponse.StatusCode) return localVarReturnValue, localVarHttpResponse, newErr } - return localVarReturnValue, localVarHttpResponse, nil + return localVarReturnValue, localVarHttpResponse, err } /* @@ -608,33 +494,14 @@ func (a *TagsApiService) GetWorkflowTags(ctx context.Context, name string) ([]mo return localVarReturnValue, localVarHttpResponse, err } - if localVarHttpResponse.StatusCode < 300 { - // If we succeed, return the data, otherwise pass on to decode error. + if isSuccessfulStatus(localVarHttpResponse.StatusCode) { err = a.decode(&localVarReturnValue, localVarBody, localVarHttpResponse.Header.Get("Content-Type")) - if err == nil { - return localVarReturnValue, localVarHttpResponse, err - } - } - - if localVarHttpResponse.StatusCode >= 300 { - newErr := GenericSwaggerError{ - body: localVarBody, - error: localVarHttpResponse.Status, - } - if localVarHttpResponse.StatusCode == 200 { - var v []model.TagObject - err = a.decode(&v, localVarBody, localVarHttpResponse.Header.Get("Content-Type")) - if err != nil { - newErr.error = err.Error() - return localVarReturnValue, localVarHttpResponse, newErr - } - newErr.model = v - return localVarReturnValue, localVarHttpResponse, newErr - } + } else { + newErr := NewGenericSwaggerError(localVarBody, localVarHttpResponse.Status, nil, localVarHttpResponse.StatusCode) return localVarReturnValue, localVarHttpResponse, newErr } - return localVarReturnValue, localVarHttpResponse, nil + return localVarReturnValue, localVarHttpResponse, err } /* @@ -697,33 +564,14 @@ func (a *TagsApiService) SetTaskTags(ctx context.Context, body []model.TagObject return localVarReturnValue, localVarHttpResponse, err } - if localVarHttpResponse.StatusCode < 300 { - // If we succeed, return the data, otherwise pass on to decode error. + if isSuccessfulStatus(localVarHttpResponse.StatusCode) { err = a.decode(&localVarReturnValue, localVarBody, localVarHttpResponse.Header.Get("Content-Type")) - if err == nil { - return localVarReturnValue, localVarHttpResponse, err - } - } - - if localVarHttpResponse.StatusCode >= 300 { - newErr := GenericSwaggerError{ - body: localVarBody, - error: localVarHttpResponse.Status, - } - if localVarHttpResponse.StatusCode == 200 { - var v interface{} - err = a.decode(&v, localVarBody, localVarHttpResponse.Header.Get("Content-Type")) - if err != nil { - newErr.error = err.Error() - return localVarReturnValue, localVarHttpResponse, newErr - } - newErr.model = v - return localVarReturnValue, localVarHttpResponse, newErr - } + } else { + newErr := NewGenericSwaggerError(localVarBody, localVarHttpResponse.Status, nil, localVarHttpResponse.StatusCode) return localVarReturnValue, localVarHttpResponse, newErr } - return localVarReturnValue, localVarHttpResponse, nil + return localVarReturnValue, localVarHttpResponse, err } /* @@ -786,31 +634,12 @@ func (a *TagsApiService) SetWorkflowTags(ctx context.Context, body []model.TagOb return localVarReturnValue, localVarHttpResponse, err } - if localVarHttpResponse.StatusCode < 300 { - // If we succeed, return the data, otherwise pass on to decode error. + if isSuccessfulStatus(localVarHttpResponse.StatusCode) { err = a.decode(&localVarReturnValue, localVarBody, localVarHttpResponse.Header.Get("Content-Type")) - if err == nil { - return localVarReturnValue, localVarHttpResponse, err - } - } - - if localVarHttpResponse.StatusCode >= 300 { - newErr := GenericSwaggerError{ - body: localVarBody, - error: localVarHttpResponse.Status, - } - if localVarHttpResponse.StatusCode == 200 { - var v interface{} - err = a.decode(&v, localVarBody, localVarHttpResponse.Header.Get("Content-Type")) - if err != nil { - newErr.error = err.Error() - return localVarReturnValue, localVarHttpResponse, newErr - } - newErr.model = v - return localVarReturnValue, localVarHttpResponse, newErr - } + } else { + newErr := NewGenericSwaggerError(localVarBody, localVarHttpResponse.Status, nil, localVarHttpResponse.StatusCode) return localVarReturnValue, localVarHttpResponse, newErr } - return localVarReturnValue, localVarHttpResponse, nil + return localVarReturnValue, localVarHttpResponse, err } diff --git a/sdk/client/api_task_resource.go b/sdk/client/api_task_resource.go index ffc78f9d..a5d5badc 100644 --- a/sdk/client/api_task_resource.go +++ b/sdk/client/api_task_resource.go @@ -89,20 +89,14 @@ func (a *TaskResourceApiService) All(ctx context.Context) (map[string]int64, *ht return localVarReturnValue, localVarHttpResponse, err } - if localVarHttpResponse.StatusCode < 300 { - // If we succeed, return the data, otherwise pass on to decode error. + if isSuccessfulStatus(localVarHttpResponse.StatusCode) { err = a.decode(&localVarReturnValue, localVarBody, localVarHttpResponse.Header.Get("Content-Type")) - if err == nil { - return localVarReturnValue, localVarHttpResponse, err - } - } - - if localVarHttpResponse.StatusCode >= 300 { + } else { newErr := NewGenericSwaggerError(localVarBody, string(localVarBody), nil, localVarHttpResponse.StatusCode) return localVarReturnValue, localVarHttpResponse, newErr } - return localVarReturnValue, localVarHttpResponse, nil + return localVarReturnValue, localVarHttpResponse, err } /* @@ -160,20 +154,14 @@ func (a *TaskResourceApiService) AllVerbose(ctx context.Context) (map[string]map return localVarReturnValue, localVarHttpResponse, err } - if localVarHttpResponse.StatusCode < 300 { - // If we succeed, return the data, otherwise pass on to decode error. + if isSuccessfulStatus(localVarHttpResponse.StatusCode) { err = a.decode(&localVarReturnValue, localVarBody, localVarHttpResponse.Header.Get("Content-Type")) - if err == nil { - return localVarReturnValue, localVarHttpResponse, err - } - } - - if localVarHttpResponse.StatusCode >= 300 { + } else { newErr := NewGenericSwaggerError(localVarBody, string(localVarBody), nil, localVarHttpResponse.StatusCode) return localVarReturnValue, localVarHttpResponse, newErr } - return localVarReturnValue, localVarHttpResponse, nil + return localVarReturnValue, localVarHttpResponse, err } /* @@ -257,20 +245,14 @@ func (a *TaskResourceApiService) BatchPoll(ctx context.Context, tasktype string, return localVarReturnValue, localVarHttpResponse, err } - if localVarHttpResponse.StatusCode < 300 { - // If we succeed, return the data, otherwise pass on to decode error. + if isSuccessfulStatus(localVarHttpResponse.StatusCode) { err = a.decode(&localVarReturnValue, localVarBody, localVarHttpResponse.Header.Get("Content-Type")) - if err == nil { - return localVarReturnValue, localVarHttpResponse, err - } - } - - if localVarHttpResponse.StatusCode >= 300 { + } else { newErr := NewGenericSwaggerError(localVarBody, string(localVarBody), nil, localVarHttpResponse.StatusCode) return localVarReturnValue, localVarHttpResponse, newErr } - return localVarReturnValue, localVarHttpResponse, nil + return localVarReturnValue, localVarHttpResponse, err } /* @@ -328,20 +310,14 @@ func (a *TaskResourceApiService) GetAllPollData(ctx context.Context) ([]model.Po return localVarReturnValue, localVarHttpResponse, err } - if localVarHttpResponse.StatusCode < 300 { - // If we succeed, return the data, otherwise pass on to decode error. + if isSuccessfulStatus(localVarHttpResponse.StatusCode) { err = a.decode(&localVarReturnValue, localVarBody, localVarHttpResponse.Header.Get("Content-Type")) - if err == nil { - return localVarReturnValue, localVarHttpResponse, err - } - } - - if localVarHttpResponse.StatusCode >= 300 { + } else { newErr := NewGenericSwaggerError(localVarBody, string(localVarBody), nil, localVarHttpResponse.StatusCode) return localVarReturnValue, localVarHttpResponse, newErr } - return localVarReturnValue, localVarHttpResponse, nil + return localVarReturnValue, localVarHttpResponse, err } /* @@ -405,20 +381,14 @@ func (a *TaskResourceApiService) GetExternalStorageLocation1(ctx context.Context return localVarReturnValue, localVarHttpResponse, err } - if localVarHttpResponse.StatusCode < 300 { - // If we succeed, return the data, otherwise pass on to decode error. + if isSuccessfulStatus(localVarHttpResponse.StatusCode) { err = a.decode(&localVarReturnValue, localVarBody, localVarHttpResponse.Header.Get("Content-Type")) - if err == nil { - return localVarReturnValue, localVarHttpResponse, err - } - } - - if localVarHttpResponse.StatusCode >= 300 { + } else { newErr := NewGenericSwaggerError(localVarBody, string(localVarBody), nil, localVarHttpResponse.StatusCode) return localVarReturnValue, localVarHttpResponse, newErr } - return localVarReturnValue, localVarHttpResponse, nil + return localVarReturnValue, localVarHttpResponse, err } /* @@ -478,20 +448,14 @@ func (a *TaskResourceApiService) GetPollData(ctx context.Context, taskType strin return localVarReturnValue, localVarHttpResponse, err } - if localVarHttpResponse.StatusCode < 300 { - // If we succeed, return the data, otherwise pass on to decode error. + if isSuccessfulStatus(localVarHttpResponse.StatusCode) { err = a.decode(&localVarReturnValue, localVarBody, localVarHttpResponse.Header.Get("Content-Type")) - if err == nil { - return localVarReturnValue, localVarHttpResponse, err - } - } - - if localVarHttpResponse.StatusCode >= 300 { + } else { newErr := NewGenericSwaggerError(localVarBody, string(localVarBody), nil, localVarHttpResponse.StatusCode) return localVarReturnValue, localVarHttpResponse, newErr } - return localVarReturnValue, localVarHttpResponse, nil + return localVarReturnValue, localVarHttpResponse, err } /* @@ -551,20 +515,14 @@ func (a *TaskResourceApiService) GetTask(ctx context.Context, taskId string) (mo return localVarReturnValue, localVarHttpResponse, err } - if localVarHttpResponse.StatusCode < 300 { - // If we succeed, return the data, otherwise pass on to decode error. + if isSuccessfulStatus(localVarHttpResponse.StatusCode) { err = a.decode(&localVarReturnValue, localVarBody, localVarHttpResponse.Header.Get("Content-Type")) - if err == nil { - return localVarReturnValue, localVarHttpResponse, err - } - } - - if localVarHttpResponse.StatusCode >= 300 { + } else { newErr := NewGenericSwaggerError(localVarBody, string(localVarBody), nil, localVarHttpResponse.StatusCode) return localVarReturnValue, localVarHttpResponse, newErr } - return localVarReturnValue, localVarHttpResponse, nil + return localVarReturnValue, localVarHttpResponse, err } /* @@ -624,20 +582,14 @@ func (a *TaskResourceApiService) GetTaskLogs(ctx context.Context, taskId string) return localVarReturnValue, localVarHttpResponse, err } - if localVarHttpResponse.StatusCode < 300 { - // If we succeed, return the data, otherwise pass on to decode error. + if isSuccessfulStatus(localVarHttpResponse.StatusCode) { err = a.decode(&localVarReturnValue, localVarBody, localVarHttpResponse.Header.Get("Content-Type")) - if err == nil { - return localVarReturnValue, localVarHttpResponse, err - } - } - - if localVarHttpResponse.StatusCode >= 300 { + } else { newErr := NewGenericSwaggerError(localVarBody, string(localVarBody), nil, localVarHttpResponse.StatusCode) return localVarReturnValue, localVarHttpResponse, newErr } - return localVarReturnValue, localVarHttpResponse, nil + return localVarReturnValue, localVarHttpResponse, err } /* @@ -697,7 +649,7 @@ func (a *TaskResourceApiService) Log(ctx context.Context, body string, taskId st return localVarHttpResponse, err } - if localVarHttpResponse.StatusCode >= 300 { + if !isSuccessfulStatus(localVarHttpResponse.StatusCode) { newErr := NewGenericSwaggerError(localVarBody, string(localVarBody), nil, localVarHttpResponse.StatusCode) return localVarHttpResponse, newErr } @@ -776,20 +728,14 @@ func (a *TaskResourceApiService) Poll(ctx context.Context, tasktype string, loca return localVarReturnValue, localVarHttpResponse, err } - if localVarHttpResponse.StatusCode < 300 { - // If we succeed, return the data, otherwise pass on to decode error. + if isSuccessfulStatus(localVarHttpResponse.StatusCode) { err = a.decode(&localVarReturnValue, localVarBody, localVarHttpResponse.Header.Get("Content-Type")) - if err == nil { - return localVarReturnValue, localVarHttpResponse, err - } - } - - if localVarHttpResponse.StatusCode >= 300 { + } else { newErr := NewGenericSwaggerError(localVarBody, string(localVarBody), nil, localVarHttpResponse.StatusCode) return localVarReturnValue, localVarHttpResponse, newErr } - return localVarReturnValue, localVarHttpResponse, nil + return localVarReturnValue, localVarHttpResponse, err } /* @@ -849,20 +795,14 @@ func (a *TaskResourceApiService) RequeuePendingTask(ctx context.Context, taskTyp return localVarReturnValue, localVarHttpResponse, err } - if localVarHttpResponse.StatusCode < 300 { - // If we succeed, return the data, otherwise pass on to decode error. + if isSuccessfulStatus(localVarHttpResponse.StatusCode) { err = a.decode(&localVarReturnValue, localVarBody, localVarHttpResponse.Header.Get("Content-Type")) - if err == nil { - return localVarReturnValue, localVarHttpResponse, err - } - } - - if localVarHttpResponse.StatusCode >= 300 { + } else { newErr := NewGenericSwaggerError(localVarBody, string(localVarBody), nil, localVarHttpResponse.StatusCode) return localVarReturnValue, localVarHttpResponse, newErr } - return localVarReturnValue, localVarHttpResponse, nil + return localVarReturnValue, localVarHttpResponse, err } /* @@ -950,20 +890,14 @@ func (a *TaskResourceApiService) Search(ctx context.Context, localVarOptionals * return localVarReturnValue, localVarHttpResponse, err } - if localVarHttpResponse.StatusCode < 300 { - // If we succeed, return the data, otherwise pass on to decode error. + if isSuccessfulStatus(localVarHttpResponse.StatusCode) { err = a.decode(&localVarReturnValue, localVarBody, localVarHttpResponse.Header.Get("Content-Type")) - if err == nil { - return localVarReturnValue, localVarHttpResponse, err - } - } - - if localVarHttpResponse.StatusCode >= 300 { + } else { newErr := NewGenericSwaggerError(localVarBody, string(localVarBody), nil, localVarHttpResponse.StatusCode) return localVarReturnValue, localVarHttpResponse, newErr } - return localVarReturnValue, localVarHttpResponse, nil + return localVarReturnValue, localVarHttpResponse, err } /* @@ -1051,20 +985,14 @@ func (a *TaskResourceApiService) SearchV2(ctx context.Context, localVarOptionals return localVarReturnValue, localVarHttpResponse, err } - if localVarHttpResponse.StatusCode < 300 { - // If we succeed, return the data, otherwise pass on to decode error. + if isSuccessfulStatus(localVarHttpResponse.StatusCode) { err = a.decode(&localVarReturnValue, localVarBody, localVarHttpResponse.Header.Get("Content-Type")) - if err == nil { - return localVarReturnValue, localVarHttpResponse, err - } - } - - if localVarHttpResponse.StatusCode >= 300 { + } else { newErr := NewGenericSwaggerError(localVarBody, string(localVarBody), nil, localVarHttpResponse.StatusCode) return localVarReturnValue, localVarHttpResponse, newErr } - return localVarReturnValue, localVarHttpResponse, nil + return localVarReturnValue, localVarHttpResponse, err } /* @@ -1131,20 +1059,14 @@ func (a *TaskResourceApiService) Size(ctx context.Context, localVarOptionals *Ta return localVarReturnValue, localVarHttpResponse, err } - if localVarHttpResponse.StatusCode < 300 { - // If we succeed, return the data, otherwise pass on to decode error. + if isSuccessfulStatus(localVarHttpResponse.StatusCode) { err = a.decode(&localVarReturnValue, localVarBody, localVarHttpResponse.Header.Get("Content-Type")) - if err == nil { - return localVarReturnValue, localVarHttpResponse, err - } - } - - if localVarHttpResponse.StatusCode >= 300 { + } else { newErr := NewGenericSwaggerError(localVarBody, string(localVarBody), nil, localVarHttpResponse.StatusCode) return localVarReturnValue, localVarHttpResponse, newErr } - return localVarReturnValue, localVarHttpResponse, nil + return localVarReturnValue, localVarHttpResponse, err } /* @@ -1206,20 +1128,14 @@ func (a *TaskResourceApiService) UpdateTask(ctx context.Context, taskResult *mod return localVarReturnValue, localVarHttpResponse, err } - if localVarHttpResponse.StatusCode < 300 { - // If we succeed, return the data, otherwise pass on to decode error. + if isSuccessfulStatus(localVarHttpResponse.StatusCode) { err = a.decode(&localVarReturnValue, localVarBody, localVarHttpResponse.Header.Get("Content-Type")) - if err == nil { - return localVarReturnValue, localVarHttpResponse, err - } - } - - if localVarHttpResponse.StatusCode >= 300 { + } else { newErr := NewGenericSwaggerError(localVarBody, string(localVarBody), nil, localVarHttpResponse.StatusCode) return localVarReturnValue, localVarHttpResponse, newErr } - return localVarReturnValue, localVarHttpResponse, nil + return localVarReturnValue, localVarHttpResponse, err } /* @@ -1312,20 +1228,14 @@ func (a *TaskResourceApiService) updateTaskByRefName(ctx context.Context, body m return localVarReturnValue, localVarHttpResponse, err } - if localVarHttpResponse.StatusCode < 300 { - // If we succeed, return the data, otherwise pass on to decode error. + if isSuccessfulStatus(localVarHttpResponse.StatusCode) { err = a.decode(&localVarReturnValue, localVarBody, localVarHttpResponse.Header.Get("Content-Type")) - if err == nil { - return localVarReturnValue, localVarHttpResponse, err - } - } - - if localVarHttpResponse.StatusCode >= 300 { + } else { newErr := NewGenericSwaggerError(localVarBody, string(localVarBody), nil, localVarHttpResponse.StatusCode) return localVarReturnValue, localVarHttpResponse, newErr } - return localVarReturnValue, localVarHttpResponse, nil + return localVarReturnValue, localVarHttpResponse, err } func getHostname() string { diff --git a/sdk/client/api_user_resource.go b/sdk/client/api_user_resource.go index 4dac2adc..558d9822 100644 --- a/sdk/client/api_user_resource.go +++ b/sdk/client/api_user_resource.go @@ -88,20 +88,14 @@ func (a *UserResourceApiService) CheckPermissions(ctx context.Context, userId st return returnValue, httpResponse, err } - if httpResponse.StatusCode < 300 { - // If we succeed, return the data, otherwise pass on to decode error. + if isSuccessfulStatus(httpResponse.StatusCode) { err = a.decode(&returnValue, responseBody, httpResponse.Header.Get("Content-Type")) - if err == nil { - return returnValue, httpResponse, err - } - } - - if httpResponse.StatusCode >= 300 { + } else { newErr := NewGenericSwaggerError(responseBody, httpResponse.Status, nil, httpResponse.StatusCode) return returnValue, httpResponse, newErr } - return returnValue, httpResponse, nil + return returnValue, httpResponse, err } /* @@ -159,14 +153,8 @@ func (a *UserResourceApiService) DeleteUser(ctx context.Context, id string) (*ht return httpResponse, err } - if httpResponse.StatusCode < 300 { - // If we succeed, return the data, otherwise pass on to decode error. - return httpResponse, err - } - - if httpResponse.StatusCode >= 300 { - newErr := NewGenericSwaggerError(responseBody, httpResponse.Status, nil, httpResponse.StatusCode) - return httpResponse, newErr + if !isSuccessfulStatus(httpResponse.StatusCode) { + return httpResponse, NewGenericSwaggerError(responseBody, httpResponse.Status, nil, httpResponse.StatusCode) } return httpResponse, nil @@ -178,13 +166,13 @@ UserResourceApiService Get the permissions this user has over workflows and task - @param userId @return interface{} */ -func (a *UserResourceApiService) GetGrantedPermissions(ctx context.Context, userId string) ([]rbac.GrantedAccess, *http.Response, error) { +func (a *UserResourceApiService) GetGrantedPermissions(ctx context.Context, userId string) (rbac.GrantedAccessResponse, *http.Response, error) { var ( httpMethod = strings.ToUpper("Get") postBody interface{} fileName string fileBytes []byte - returnValue []rbac.GrantedAccess + returnValue rbac.GrantedAccessResponse ) // create path and map variables @@ -228,20 +216,14 @@ func (a *UserResourceApiService) GetGrantedPermissions(ctx context.Context, user return returnValue, httpResponse, err } - if httpResponse.StatusCode < 300 { - // If we succeed, return the data, otherwise pass on to decode error. + if isSuccessfulStatus(httpResponse.StatusCode) { err = a.decode(&returnValue, responseBody, httpResponse.Header.Get("Content-Type")) - if err == nil { - return returnValue, httpResponse, err - } - } - - if httpResponse.StatusCode >= 300 { + } else { newErr := NewGenericSwaggerError(responseBody, httpResponse.Status, nil, httpResponse.StatusCode) return returnValue, httpResponse, newErr } - return returnValue, httpResponse, nil + return returnValue, httpResponse, err } /* @@ -300,20 +282,14 @@ func (a *UserResourceApiService) GetUser(ctx context.Context, id string) (*rbac. return nil, httpResponse, err } - if httpResponse.StatusCode < 300 { - // If we succeed, return the data, otherwise pass on to decode error. + if isSuccessfulStatus(httpResponse.StatusCode) { err = a.decode(&returnValue, responseBody, httpResponse.Header.Get("Content-Type")) - if err == nil { - return &returnValue, httpResponse, err - } - } - - if httpResponse.StatusCode >= 300 { + } else { newErr := NewGenericSwaggerError(responseBody, httpResponse.Status, nil, httpResponse.StatusCode) return nil, httpResponse, newErr } - return nil, httpResponse, nil + return &returnValue, httpResponse, err } /* @@ -376,20 +352,14 @@ func (a *UserResourceApiService) ListUsers(ctx context.Context, optionals *UserR return returnValue, httpResponse, err } - if httpResponse.StatusCode < 300 { - // If we succeed, return the data, otherwise pass on to decode error. + if isSuccessfulStatus(httpResponse.StatusCode) { err = a.decode(&returnValue, responseBody, httpResponse.Header.Get("Content-Type")) - if err == nil { - return returnValue, httpResponse, err - } - } - - if httpResponse.StatusCode >= 300 { + } else { newErr := NewGenericSwaggerError(responseBody, httpResponse.Status, nil, httpResponse.StatusCode) return returnValue, httpResponse, newErr } - return returnValue, httpResponse, nil + return returnValue, httpResponse, err } /* @@ -451,18 +421,12 @@ func (a *UserResourceApiService) UpsertUser(ctx context.Context, body rbac.Upser return nil, httpResponse, err } - if httpResponse.StatusCode < 300 { - // If we succeed, return the data, otherwise pass on to decode error. + if isSuccessfulStatus(httpResponse.StatusCode) { err = a.decode(&returnValue, responseBody, httpResponse.Header.Get("Content-Type")) - if err == nil { - return &returnValue, httpResponse, err - } - } - - if httpResponse.StatusCode >= 300 { + } else { newErr := NewGenericSwaggerError(responseBody, httpResponse.Status, nil, httpResponse.StatusCode) return nil, httpResponse, newErr } - return nil, httpResponse, nil + return &returnValue, httpResponse, err } diff --git a/sdk/client/api_webhooks_config_resource.go b/sdk/client/api_webhooks_config_resource.go index a27704bc..30cf39e0 100644 --- a/sdk/client/api_webhooks_config_resource.go +++ b/sdk/client/api_webhooks_config_resource.go @@ -81,20 +81,14 @@ func (a *WebhooksConfigResourceApiService) CreateWebhook(ctx context.Context, bo return returnValue, httpResponse, err } - if httpResponse.StatusCode < 300 { - // If we succeed, return the data, otherwise pass on to decode error. + if isSuccessfulStatus(httpResponse.StatusCode) { err = a.decode(&returnValue, responseBody, httpResponse.Header.Get("Content-Type")) - if err == nil { - return returnValue, httpResponse, err - } - } - - if httpResponse.StatusCode >= 300 { + } else { newErr := NewGenericSwaggerError(responseBody, httpResponse.Status, nil, httpResponse.StatusCode) return returnValue, httpResponse, newErr } - return returnValue, httpResponse, nil + return returnValue, httpResponse, err } /* @@ -152,9 +146,8 @@ func (a *WebhooksConfigResourceApiService) DeleteTagForWebhook(ctx context.Conte return httpResponse, err } - if httpResponse.StatusCode >= 300 { - newErr := NewGenericSwaggerError(responseBody, httpResponse.Status, nil, httpResponse.StatusCode) - return httpResponse, newErr + if !isSuccessfulStatus(httpResponse.StatusCode) { + return httpResponse, NewGenericSwaggerError(responseBody, httpResponse.Status, nil, httpResponse.StatusCode) } return httpResponse, nil @@ -214,9 +207,8 @@ func (a *WebhooksConfigResourceApiService) DeleteWebhook(ctx context.Context, id return httpResponse, err } - if httpResponse.StatusCode >= 300 { - newErr := NewGenericSwaggerError(responseBody, httpResponse.Status, nil, httpResponse.StatusCode) - return httpResponse, newErr + if !isSuccessfulStatus(httpResponse.StatusCode) { + return httpResponse, NewGenericSwaggerError(responseBody, httpResponse.Status, nil, httpResponse.StatusCode) } return httpResponse, nil @@ -276,20 +268,14 @@ func (a *WebhooksConfigResourceApiService) GetAllWebhook(ctx context.Context) ([ return returnValue, httpResponse, err } - if httpResponse.StatusCode < 300 { - // If we succeed, return the data, otherwise pass on to decode error. + if isSuccessfulStatus(httpResponse.StatusCode) { err = a.decode(&returnValue, responseBody, httpResponse.Header.Get("Content-Type")) - if err == nil { - return returnValue, httpResponse, err - } - } - - if httpResponse.StatusCode >= 300 { + } else { newErr := NewGenericSwaggerError(responseBody, httpResponse.Status, nil, httpResponse.StatusCode) return returnValue, httpResponse, newErr } - return returnValue, httpResponse, nil + return returnValue, httpResponse, err } /* @@ -348,20 +334,14 @@ func (a *WebhooksConfigResourceApiService) GetTagsForWebhook(ctx context.Context return returnValue, httpResponse, err } - if httpResponse.StatusCode < 300 { - // If we succeed, return the data, otherwise pass on to decode error. + if isSuccessfulStatus(httpResponse.StatusCode) { err = a.decode(&returnValue, responseBody, httpResponse.Header.Get("Content-Type")) - if err == nil { - return returnValue, httpResponse, err - } - } - - if httpResponse.StatusCode >= 300 { + } else { newErr := NewGenericSwaggerError(responseBody, httpResponse.Status, nil, httpResponse.StatusCode) return returnValue, httpResponse, newErr } - return returnValue, httpResponse, nil + return returnValue, httpResponse, err } /* @@ -420,20 +400,14 @@ func (a *WebhooksConfigResourceApiService) GetWebhook(ctx context.Context, id st return returnValue, httpResponse, err } - if httpResponse.StatusCode < 300 { - // If we succeed, return the data, otherwise pass on to decode error. + if isSuccessfulStatus(httpResponse.StatusCode) { err = a.decode(&returnValue, responseBody, httpResponse.Header.Get("Content-Type")) - if err == nil { - return returnValue, httpResponse, err - } - } - - if httpResponse.StatusCode >= 300 { + } else { newErr := NewGenericSwaggerError(responseBody, httpResponse.Status, nil, httpResponse.StatusCode) return returnValue, httpResponse, newErr } - return returnValue, httpResponse, nil + return returnValue, httpResponse, err } /* @@ -493,9 +467,8 @@ func (a *WebhooksConfigResourceApiService) PutTagForWebhook(ctx context.Context, return httpResponse, err } - if httpResponse.StatusCode >= 300 { - newErr := NewGenericSwaggerError(responseBody, httpResponse.Status, nil, httpResponse.StatusCode) - return httpResponse, newErr + if !isSuccessfulStatus(httpResponse.StatusCode) { + return httpResponse, NewGenericSwaggerError(responseBody, httpResponse.Status, nil, httpResponse.StatusCode) } return httpResponse, nil @@ -560,18 +533,12 @@ func (a *WebhooksConfigResourceApiService) UpdateWebhook(ctx context.Context, bo return returnValue, httpResponse, err } - if httpResponse.StatusCode < 300 { - // If we succeed, return the data, otherwise pass on to decode error. + if isSuccessfulStatus(httpResponse.StatusCode) { err = a.decode(&returnValue, responseBody, httpResponse.Header.Get("Content-Type")) - if err == nil { - return returnValue, httpResponse, err - } - } - - if httpResponse.StatusCode >= 300 { + } else { newErr := NewGenericSwaggerError(responseBody, httpResponse.Status, nil, httpResponse.StatusCode) return returnValue, httpResponse, newErr } - return returnValue, httpResponse, nil + return returnValue, httpResponse, err } diff --git a/sdk/client/api_workflow_bulk_resource.go b/sdk/client/api_workflow_bulk_resource.go index 5bc0a83b..4ee96df9 100644 --- a/sdk/client/api_workflow_bulk_resource.go +++ b/sdk/client/api_workflow_bulk_resource.go @@ -86,30 +86,14 @@ func (a *WorkflowBulkResourceApiService) PauseWorkflow1(ctx context.Context, bod return localVarReturnValue, localVarHttpResponse, err } - if localVarHttpResponse.StatusCode < 300 { - // If we succeed, return the data, otherwise pass on to decode error. + if isSuccessfulStatus(localVarHttpResponse.StatusCode) { err = a.decode(&localVarReturnValue, localVarBody, localVarHttpResponse.Header.Get("Content-Type")) - if err == nil { - return localVarReturnValue, localVarHttpResponse, err - } - } - - if localVarHttpResponse.StatusCode >= 300 { + } else { newErr := NewGenericSwaggerError(localVarBody, string(localVarBody), nil, localVarHttpResponse.StatusCode) - if localVarHttpResponse.StatusCode == 200 { - var v model.BulkResponse - err = a.decode(&v, localVarBody, localVarHttpResponse.Header.Get("Content-Type")) - if err != nil { - newErr.error = err.Error() - return localVarReturnValue, localVarHttpResponse, newErr - } - newErr.model = v - return localVarReturnValue, localVarHttpResponse, newErr - } return localVarReturnValue, localVarHttpResponse, newErr } - return localVarReturnValue, localVarHttpResponse, nil + return localVarReturnValue, localVarHttpResponse, err } /* @@ -182,30 +166,14 @@ func (a *WorkflowBulkResourceApiService) Restart1(ctx context.Context, body []st return localVarReturnValue, localVarHttpResponse, err } - if localVarHttpResponse.StatusCode < 300 { - // If we succeed, return the data, otherwise pass on to decode error. + if isSuccessfulStatus(localVarHttpResponse.StatusCode) { err = a.decode(&localVarReturnValue, localVarBody, localVarHttpResponse.Header.Get("Content-Type")) - if err == nil { - return localVarReturnValue, localVarHttpResponse, err - } - } - - if localVarHttpResponse.StatusCode >= 300 { + } else { newErr := NewGenericSwaggerError(localVarBody, string(localVarBody), nil, localVarHttpResponse.StatusCode) - if localVarHttpResponse.StatusCode == 200 { - var v model.BulkResponse - err = a.decode(&v, localVarBody, localVarHttpResponse.Header.Get("Content-Type")) - if err != nil { - newErr.error = err.Error() - return localVarReturnValue, localVarHttpResponse, newErr - } - newErr.model = v - return localVarReturnValue, localVarHttpResponse, newErr - } return localVarReturnValue, localVarHttpResponse, newErr } - return localVarReturnValue, localVarHttpResponse, nil + return localVarReturnValue, localVarHttpResponse, err } func (a *WorkflowBulkResourceApiService) ResumeWorkflow(ctx context.Context, body []string) (model.BulkResponse, *http.Response, error) { @@ -270,30 +238,14 @@ func (a *WorkflowBulkResourceApiService) ResumeWorkflow1(ctx context.Context, bo return localVarReturnValue, localVarHttpResponse, err } - if localVarHttpResponse.StatusCode < 300 { - // If we succeed, return the data, otherwise pass on to decode error. + if isSuccessfulStatus(localVarHttpResponse.StatusCode) { err = a.decode(&localVarReturnValue, localVarBody, localVarHttpResponse.Header.Get("Content-Type")) - if err == nil { - return localVarReturnValue, localVarHttpResponse, err - } - } - - if localVarHttpResponse.StatusCode >= 300 { + } else { newErr := NewGenericSwaggerError(localVarBody, string(localVarBody), nil, localVarHttpResponse.StatusCode) - if localVarHttpResponse.StatusCode == 200 { - var v model.BulkResponse - err = a.decode(&v, localVarBody, localVarHttpResponse.Header.Get("Content-Type")) - if err != nil { - newErr.error = err.Error() - return localVarReturnValue, localVarHttpResponse, newErr - } - newErr.model = v - return localVarReturnValue, localVarHttpResponse, newErr - } return localVarReturnValue, localVarHttpResponse, newErr } - return localVarReturnValue, localVarHttpResponse, nil + return localVarReturnValue, localVarHttpResponse, err } /* @@ -357,30 +309,14 @@ func (a *WorkflowBulkResourceApiService) Retry1(ctx context.Context, body []stri return localVarReturnValue, localVarHttpResponse, err } - if localVarHttpResponse.StatusCode < 300 { - // If we succeed, return the data, otherwise pass on to decode error. + if isSuccessfulStatus(localVarHttpResponse.StatusCode) { err = a.decode(&localVarReturnValue, localVarBody, localVarHttpResponse.Header.Get("Content-Type")) - if err == nil { - return localVarReturnValue, localVarHttpResponse, err - } - } - - if localVarHttpResponse.StatusCode >= 300 { + } else { newErr := NewGenericSwaggerError(localVarBody, string(localVarBody), nil, localVarHttpResponse.StatusCode) - if localVarHttpResponse.StatusCode == 200 { - var v model.BulkResponse - err = a.decode(&v, localVarBody, localVarHttpResponse.Header.Get("Content-Type")) - if err != nil { - newErr.error = err.Error() - return localVarReturnValue, localVarHttpResponse, newErr - } - newErr.model = v - return localVarReturnValue, localVarHttpResponse, newErr - } return localVarReturnValue, localVarHttpResponse, newErr } - return localVarReturnValue, localVarHttpResponse, nil + return localVarReturnValue, localVarHttpResponse, err } /* @@ -455,28 +391,12 @@ func (a *WorkflowBulkResourceApiService) Terminate(ctx context.Context, body []s return localVarReturnValue, localVarHttpResponse, err } - if localVarHttpResponse.StatusCode < 300 { - // If we succeed, return the data, otherwise pass on to decode error. + if isSuccessfulStatus(localVarHttpResponse.StatusCode) { err = a.decode(&localVarReturnValue, localVarBody, localVarHttpResponse.Header.Get("Content-Type")) - if err == nil { - return localVarReturnValue, localVarHttpResponse, err - } - } - - if localVarHttpResponse.StatusCode >= 300 { + } else { newErr := NewGenericSwaggerError(localVarBody, string(localVarBody), nil, localVarHttpResponse.StatusCode) - if localVarHttpResponse.StatusCode == 200 { - var v model.BulkResponse - err = a.decode(&v, localVarBody, localVarHttpResponse.Header.Get("Content-Type")) - if err != nil { - newErr.error = err.Error() - return localVarReturnValue, localVarHttpResponse, newErr - } - newErr.model = v - return localVarReturnValue, localVarHttpResponse, newErr - } return localVarReturnValue, localVarHttpResponse, newErr } - return localVarReturnValue, localVarHttpResponse, nil + return localVarReturnValue, localVarHttpResponse, err } diff --git a/sdk/client/api_workflow_resource.go b/sdk/client/api_workflow_resource.go index 3f5d69eb..6ff60a34 100644 --- a/sdk/client/api_workflow_resource.go +++ b/sdk/client/api_workflow_resource.go @@ -83,7 +83,7 @@ func (a *WorkflowResourceApiService) Decide(ctx context.Context, workflowId stri return localVarHttpResponse, err } - if localVarHttpResponse.StatusCode >= 300 { + if !isSuccessfulStatus(localVarHttpResponse.StatusCode) { newErr := NewGenericSwaggerError(localVarBody, string(localVarBody), nil, localVarHttpResponse.StatusCode) return localVarHttpResponse, newErr } @@ -156,7 +156,7 @@ func (a *WorkflowResourceApiService) Delete(ctx context.Context, workflowId stri return localVarHttpResponse, err } - if localVarHttpResponse.StatusCode >= 300 { + if !isSuccessfulStatus(localVarHttpResponse.StatusCode) { newErr := NewGenericSwaggerError(localVarBody, string(localVarBody), nil, localVarHttpResponse.StatusCode) return localVarHttpResponse, newErr } @@ -230,20 +230,14 @@ func (a *WorkflowResourceApiService) GetExecutionStatus(ctx context.Context, wor return localVarReturnValue, localVarHttpResponse, err } - if localVarHttpResponse.StatusCode < 300 { - // If we succeed, return the data, otherwise pass on to decode error. + if isSuccessfulStatus(localVarHttpResponse.StatusCode) { err = a.decode(&localVarReturnValue, localVarBody, localVarHttpResponse.Header.Get("Content-Type")) - if err == nil { - return localVarReturnValue, localVarHttpResponse, err - } - } - - if localVarHttpResponse.StatusCode >= 300 { + } else { newErr := NewGenericSwaggerError(localVarBody, string(localVarBody), nil, localVarHttpResponse.StatusCode) return localVarReturnValue, localVarHttpResponse, newErr } - return localVarReturnValue, localVarHttpResponse, nil + return localVarReturnValue, localVarHttpResponse, err } func (a *WorkflowResourceApiService) GetWorkflowState(ctx context.Context, workflowId string, includeOutput bool, includeVariables bool) (model.WorkflowState, *http.Response, error) { @@ -298,20 +292,14 @@ func (a *WorkflowResourceApiService) GetWorkflowState(ctx context.Context, workf return localVarReturnValue, localVarHttpResponse, err } - if localVarHttpResponse.StatusCode < 300 { - // If we succeed, return the data, otherwise pass on to decode error. + if isSuccessfulStatus(localVarHttpResponse.StatusCode) { err = a.decode(&localVarReturnValue, localVarBody, localVarHttpResponse.Header.Get("Content-Type")) - if err == nil { - return localVarReturnValue, localVarHttpResponse, err - } - } - - if localVarHttpResponse.StatusCode >= 300 { + } else { newErr := NewGenericSwaggerError(localVarBody, string(localVarBody), nil, localVarHttpResponse.StatusCode) return localVarReturnValue, localVarHttpResponse, newErr } - return localVarReturnValue, localVarHttpResponse, nil + return localVarReturnValue, localVarHttpResponse, err } /* @@ -375,20 +363,14 @@ func (a *WorkflowResourceApiService) GetExternalStorageLocation(ctx context.Cont return localVarReturnValue, localVarHttpResponse, err } - if localVarHttpResponse.StatusCode < 300 { - // If we succeed, return the data, otherwise pass on to decode error. + if isSuccessfulStatus(localVarHttpResponse.StatusCode) { err = a.decode(&localVarReturnValue, localVarBody, localVarHttpResponse.Header.Get("Content-Type")) - if err == nil { - return localVarReturnValue, localVarHttpResponse, err - } - } - - if localVarHttpResponse.StatusCode >= 300 { + } else { newErr := NewGenericSwaggerError(localVarBody, string(localVarBody), nil, localVarHttpResponse.StatusCode) return localVarReturnValue, localVarHttpResponse, newErr } - return localVarReturnValue, localVarHttpResponse, nil + return localVarReturnValue, localVarHttpResponse, err } /* @@ -467,20 +449,14 @@ func (a *WorkflowResourceApiService) GetRunningWorkflow(ctx context.Context, nam return localVarReturnValue, localVarHttpResponse, err } - if localVarHttpResponse.StatusCode < 300 { - // If we succeed, return the data, otherwise pass on to decode error. + if isSuccessfulStatus(localVarHttpResponse.StatusCode) { err = a.decode(&localVarReturnValue, localVarBody, localVarHttpResponse.Header.Get("Content-Type")) - if err == nil { - return localVarReturnValue, localVarHttpResponse, err - } - } - - if localVarHttpResponse.StatusCode >= 300 { + } else { newErr := NewGenericSwaggerError(localVarBody, string(localVarBody), nil, localVarHttpResponse.StatusCode) return localVarReturnValue, localVarHttpResponse, newErr } - return localVarReturnValue, localVarHttpResponse, nil + return localVarReturnValue, localVarHttpResponse, err } /* @@ -552,20 +528,14 @@ func (a *WorkflowResourceApiService) GetWorkflows(ctx context.Context, body []st return localVarReturnValue, localVarHttpResponse, err } - if localVarHttpResponse.StatusCode < 300 { - // If we succeed, return the data, otherwise pass on to decode error. + if isSuccessfulStatus(localVarHttpResponse.StatusCode) { err = a.decode(&localVarReturnValue, localVarBody, localVarHttpResponse.Header.Get("Content-Type")) - if err == nil { - return localVarReturnValue, localVarHttpResponse, err - } - } - - if localVarHttpResponse.StatusCode >= 300 { + } else { newErr := NewGenericSwaggerError(localVarBody, string(localVarBody), nil, localVarHttpResponse.StatusCode) return localVarReturnValue, localVarHttpResponse, newErr } - return localVarReturnValue, localVarHttpResponse, nil + return localVarReturnValue, localVarHttpResponse, err } func (a *WorkflowResourceApiService) GetWorkflowsBatch(ctx context.Context, body map[string][]string, localVarOptionals *WorkflowResourceApiGetWorkflowsOpts) (map[string][]model.Workflow, *http.Response, error) { @@ -624,20 +594,14 @@ func (a *WorkflowResourceApiService) GetWorkflowsBatch(ctx context.Context, body return localVarReturnValue, localVarHttpResponse, err } - if localVarHttpResponse.StatusCode < 300 { - // If we succeed, return the data, otherwise pass on to decode error. + if isSuccessfulStatus(localVarHttpResponse.StatusCode) { err = a.decode(&localVarReturnValue, localVarBody, localVarHttpResponse.Header.Get("Content-Type")) - if err == nil { - return localVarReturnValue, localVarHttpResponse, err - } - } - - if localVarHttpResponse.StatusCode >= 300 { + } else { newErr := NewGenericSwaggerError(localVarBody, string(localVarBody), nil, localVarHttpResponse.StatusCode) return localVarReturnValue, localVarHttpResponse, newErr } - return localVarReturnValue, localVarHttpResponse, nil + return localVarReturnValue, localVarHttpResponse, err } /* @@ -716,20 +680,14 @@ func (a *WorkflowResourceApiService) GetWorkflows1(ctx context.Context, name str return localVarReturnValue, localVarHttpResponse, err } - if localVarHttpResponse.StatusCode < 300 { - // If we succeed, return the data, otherwise pass on to decode error. + if isSuccessfulStatus(localVarHttpResponse.StatusCode) { err = a.decode(&localVarReturnValue, localVarBody, localVarHttpResponse.Header.Get("Content-Type")) - if err == nil { - return localVarReturnValue, localVarHttpResponse, err - } - } - - if localVarHttpResponse.StatusCode >= 300 { + } else { newErr := NewGenericSwaggerError(localVarBody, string(localVarBody), nil, localVarHttpResponse.StatusCode) return localVarReturnValue, localVarHttpResponse, newErr } - return localVarReturnValue, localVarHttpResponse, nil + return localVarReturnValue, localVarHttpResponse, err } /* @@ -786,7 +744,7 @@ func (a *WorkflowResourceApiService) PauseWorkflow(ctx context.Context, workflow return localVarHttpResponse, err } - if localVarHttpResponse.StatusCode >= 300 { + if !isSuccessfulStatus(localVarHttpResponse.StatusCode) { newErr := NewGenericSwaggerError(localVarBody, string(localVarBody), nil, localVarHttpResponse.StatusCode) return localVarHttpResponse, newErr } @@ -854,20 +812,14 @@ func (a *WorkflowResourceApiService) Rerun(ctx context.Context, body model.Rerun return localVarReturnValue, localVarHttpResponse, err } - if localVarHttpResponse.StatusCode < 300 { - // If we succeed, return the data, otherwise pass on to decode error. + if isSuccessfulStatus(localVarHttpResponse.StatusCode) { err = a.decode(&localVarReturnValue, localVarBody, localVarHttpResponse.Header.Get("Content-Type")) - if err == nil { - return localVarReturnValue, localVarHttpResponse, err - } - } - - if localVarHttpResponse.StatusCode >= 300 { + } else { newErr := NewGenericSwaggerError(localVarBody, string(localVarBody), nil, localVarHttpResponse.StatusCode) return localVarReturnValue, localVarHttpResponse, newErr } - return localVarReturnValue, localVarHttpResponse, nil + return localVarReturnValue, localVarHttpResponse, err } /* @@ -924,7 +876,7 @@ func (a *WorkflowResourceApiService) ResetWorkflow(ctx context.Context, workflow return localVarHttpResponse, err } - if localVarHttpResponse.StatusCode >= 300 { + if !isSuccessfulStatus(localVarHttpResponse.StatusCode) { newErr := NewGenericSwaggerError(localVarBody, string(localVarBody), nil, localVarHttpResponse.StatusCode) return localVarHttpResponse, newErr } @@ -997,7 +949,7 @@ func (a *WorkflowResourceApiService) Restart(ctx context.Context, workflowId str return localVarHttpResponse, err } - if localVarHttpResponse.StatusCode >= 300 { + if !isSuccessfulStatus(localVarHttpResponse.StatusCode) { newErr := NewGenericSwaggerError(localVarBody, string(localVarBody), nil, localVarHttpResponse.StatusCode) return localVarHttpResponse, newErr } @@ -1059,7 +1011,7 @@ func (a *WorkflowResourceApiService) ResumeWorkflow(ctx context.Context, workflo return localVarHttpResponse, err } - if localVarHttpResponse.StatusCode >= 300 { + if !isSuccessfulStatus(localVarHttpResponse.StatusCode) { newErr := NewGenericSwaggerError(localVarBody, string(localVarBody), nil, localVarHttpResponse.StatusCode) return localVarHttpResponse, newErr } @@ -1132,7 +1084,7 @@ func (a *WorkflowResourceApiService) Retry(ctx context.Context, workflowId strin return localVarHttpResponse, err } - if localVarHttpResponse.StatusCode >= 300 { + if !isSuccessfulStatus(localVarHttpResponse.StatusCode) { newErr := NewGenericSwaggerError(localVarBody, string(localVarBody), nil, localVarHttpResponse.StatusCode) return localVarHttpResponse, newErr } @@ -1225,20 +1177,14 @@ func (a *WorkflowResourceApiService) Search(ctx context.Context, localVarOptiona return localVarReturnValue, localVarHttpResponse, err } - if localVarHttpResponse.StatusCode < 300 { - // If we succeed, return the data, otherwise pass on to decode error. + if isSuccessfulStatus(localVarHttpResponse.StatusCode) { err = a.decode(&localVarReturnValue, localVarBody, localVarHttpResponse.Header.Get("Content-Type")) - if err == nil { - return localVarReturnValue, localVarHttpResponse, err - } - } - - if localVarHttpResponse.StatusCode >= 300 { + } else { newErr := NewGenericSwaggerError(localVarBody, string(localVarBody), nil, localVarHttpResponse.StatusCode) return localVarReturnValue, localVarHttpResponse, newErr } - return localVarReturnValue, localVarHttpResponse, nil + return localVarReturnValue, localVarHttpResponse, err } /* @@ -1326,20 +1272,14 @@ func (a *WorkflowResourceApiService) SearchV2(ctx context.Context, localVarOptio return localVarReturnValue, localVarHttpResponse, err } - if localVarHttpResponse.StatusCode < 300 { - // If we succeed, return the data, otherwise pass on to decode error. + if isSuccessfulStatus(localVarHttpResponse.StatusCode) { err = a.decode(&localVarReturnValue, localVarBody, localVarHttpResponse.Header.Get("Content-Type")) - if err == nil { - return localVarReturnValue, localVarHttpResponse, err - } - } - - if localVarHttpResponse.StatusCode >= 300 { + } else { newErr := NewGenericSwaggerError(localVarBody, string(localVarBody), nil, localVarHttpResponse.StatusCode) return localVarReturnValue, localVarHttpResponse, newErr } - return localVarReturnValue, localVarHttpResponse, nil + return localVarReturnValue, localVarHttpResponse, err } /* @@ -1427,20 +1367,14 @@ func (a *WorkflowResourceApiService) SearchWorkflowsByTasks(ctx context.Context, return localVarReturnValue, localVarHttpResponse, err } - if localVarHttpResponse.StatusCode < 300 { - // If we succeed, return the data, otherwise pass on to decode error. + if isSuccessfulStatus(localVarHttpResponse.StatusCode) { err = a.decode(&localVarReturnValue, localVarBody, localVarHttpResponse.Header.Get("Content-Type")) - if err == nil { - return localVarReturnValue, localVarHttpResponse, err - } - } - - if localVarHttpResponse.StatusCode >= 300 { + } else { newErr := NewGenericSwaggerError(localVarBody, string(localVarBody), nil, localVarHttpResponse.StatusCode) return localVarReturnValue, localVarHttpResponse, newErr } - return localVarReturnValue, localVarHttpResponse, nil + return localVarReturnValue, localVarHttpResponse, err } /* @@ -1528,30 +1462,14 @@ func (a *WorkflowResourceApiService) SearchWorkflowsByTasksV2(ctx context.Contex return localVarReturnValue, localVarHttpResponse, err } - if localVarHttpResponse.StatusCode < 300 { - // If we succeed, return the data, otherwise pass on to decode error. + if isSuccessfulStatus(localVarHttpResponse.StatusCode) { err = a.decode(&localVarReturnValue, localVarBody, localVarHttpResponse.Header.Get("Content-Type")) - if err == nil { - return localVarReturnValue, localVarHttpResponse, err - } - } - - if localVarHttpResponse.StatusCode >= 300 { + } else { newErr := NewGenericSwaggerError(localVarBody, string(localVarBody), nil, localVarHttpResponse.StatusCode) - if localVarHttpResponse.StatusCode == 200 { - var v model.SearchResultWorkflow - err = a.decode(&v, localVarBody, localVarHttpResponse.Header.Get("Content-Type")) - if err != nil { - newErr.error = err.Error() - return localVarReturnValue, localVarHttpResponse, newErr - } - newErr.model = v - return localVarReturnValue, localVarHttpResponse, newErr - } return localVarReturnValue, localVarHttpResponse, newErr } - return localVarReturnValue, localVarHttpResponse, nil + return localVarReturnValue, localVarHttpResponse, err } /* @@ -1612,7 +1530,7 @@ func (a *WorkflowResourceApiService) SkipTaskFromWorkflow(ctx context.Context, w return localVarHttpResponse, err } - if localVarHttpResponse.StatusCode >= 300 { + if !isSuccessfulStatus(localVarHttpResponse.StatusCode) { newErr := NewGenericSwaggerError(localVarBody, string(localVarBody), nil, localVarHttpResponse.StatusCode) return localVarHttpResponse, newErr } @@ -1699,20 +1617,14 @@ func (a *WorkflowResourceApiService) StartWorkflow(ctx context.Context, body map return localVarReturnValue, localVarHttpResponse, err } - if localVarHttpResponse.StatusCode < 300 { - // If we succeed, return the data, otherwise pass on to decode error. + if isSuccessfulStatus(localVarHttpResponse.StatusCode) { err = a.decode(&localVarReturnValue, localVarBody, localVarHttpResponse.Header.Get("Content-Type")) - if err == nil { - return localVarReturnValue, localVarHttpResponse, err - } - } - - if localVarHttpResponse.StatusCode >= 300 { + } else { newErr := NewGenericSwaggerError(localVarBody, string(localVarBody), nil, localVarHttpResponse.StatusCode) return localVarReturnValue, localVarHttpResponse, newErr } - return localVarReturnValue, localVarHttpResponse, nil + return localVarReturnValue, localVarHttpResponse, err } func (a *WorkflowResourceApiService) ExecuteWorkflow(ctx context.Context, body model.StartWorkflowRequest, requestId string, name string, version int32, waitUntilTask string) (model.WorkflowRun, *http.Response, error) { @@ -1773,33 +1685,14 @@ func (a *WorkflowResourceApiService) ExecuteWorkflow(ctx context.Context, body m return localVarReturnValue, localVarHttpResponse, err } - if localVarHttpResponse.StatusCode < 300 { - // If we succeed, return the data, otherwise pass on to decode error. + if isSuccessfulStatus(localVarHttpResponse.StatusCode) { err = a.decode(&localVarReturnValue, localVarBody, localVarHttpResponse.Header.Get("Content-Type")) - if err == nil { - return localVarReturnValue, localVarHttpResponse, err - } - } - - if localVarHttpResponse.StatusCode >= 300 { - newErr := GenericSwaggerError{ - body: localVarBody, - error: localVarHttpResponse.Status, - } - if localVarHttpResponse.StatusCode == 200 { - var v model.WorkflowRun - err = a.decode(&v, localVarBody, localVarHttpResponse.Header.Get("Content-Type")) - if err != nil { - newErr.error = err.Error() - return localVarReturnValue, localVarHttpResponse, newErr - } - newErr.model = v - return localVarReturnValue, localVarHttpResponse, newErr - } + } else { + newErr := NewGenericSwaggerError(localVarBody, localVarHttpResponse.Status, nil, localVarHttpResponse.StatusCode) return localVarReturnValue, localVarHttpResponse, newErr } - return localVarReturnValue, localVarHttpResponse, nil + return localVarReturnValue, localVarHttpResponse, err } /* @@ -1860,20 +1753,14 @@ func (a *WorkflowResourceApiService) StartWorkflowWithRequest(ctx context.Contex return localVarReturnValue, localVarHttpResponse, err } - if localVarHttpResponse.StatusCode < 300 { - // If we succeed, return the data, otherwise pass on to decode error. + if isSuccessfulStatus(localVarHttpResponse.StatusCode) { err = a.decode(&localVarReturnValue, localVarBody, localVarHttpResponse.Header.Get("Content-Type")) - if err == nil { - return localVarReturnValue, localVarHttpResponse, err - } - } - - if localVarHttpResponse.StatusCode >= 300 { + } else { newErr := NewGenericSwaggerError(localVarBody, string(localVarBody), nil, localVarHttpResponse.StatusCode) return localVarReturnValue, localVarHttpResponse, newErr } - return localVarReturnValue, localVarHttpResponse, nil + return localVarReturnValue, localVarHttpResponse, err } /* @@ -1946,7 +1833,7 @@ func (a *WorkflowResourceApiService) Terminate(ctx context.Context, workflowId s return localVarHttpResponse, err } - if localVarHttpResponse.StatusCode >= 300 { + if !isSuccessfulStatus(localVarHttpResponse.StatusCode) { newErr := NewGenericSwaggerError(localVarBody, string(localVarBody), nil, localVarHttpResponse.StatusCode) return localVarHttpResponse, newErr } diff --git a/sdk/client/generic_swagger_error.go b/sdk/client/generic_swagger_error.go index 162b59df..0fbfbfc0 100644 --- a/sdk/client/generic_swagger_error.go +++ b/sdk/client/generic_swagger_error.go @@ -46,3 +46,12 @@ func NewGenericSwaggerError(body []byte, errorMsg string, model interface{}, sta statusCode: statusCode, } } + +func WrapErrorMessage(err GenericSwaggerError, errorMsg string) GenericSwaggerError { + return GenericSwaggerError{ + body: err.body, + error: errorMsg, + model: err.model, + statusCode: err.statusCode, + } +} diff --git a/sdk/client/health_check_resource.go b/sdk/client/health_check_resource.go index 1e253ef1..405ada7d 100644 --- a/sdk/client/health_check_resource.go +++ b/sdk/client/health_check_resource.go @@ -82,18 +82,12 @@ func (a *HealthCheckResourceApiService) DoCheck(ctx context.Context) (model.Heal return localVarReturnValue, localVarHttpResponse, err } - if localVarHttpResponse.StatusCode < 300 { - // If we succeed, return the data, otherwise pass on to decode error. + if isSuccessfulStatus(localVarHttpResponse.StatusCode) { err = a.decode(&localVarReturnValue, localVarBody, localVarHttpResponse.Header.Get("Content-Type")) - if err == nil { - return localVarReturnValue, localVarHttpResponse, err - } - } - - if localVarHttpResponse.StatusCode >= 300 { + } else { newErr := NewGenericSwaggerError(localVarBody, string(localVarBody), nil, localVarHttpResponse.StatusCode) return localVarReturnValue, localVarHttpResponse, newErr } - return localVarReturnValue, localVarHttpResponse, nil + return localVarReturnValue, localVarHttpResponse, err } diff --git a/sdk/client/user_client.go b/sdk/client/user_client.go index 007210cb..d9ce8bc5 100644 --- a/sdk/client/user_client.go +++ b/sdk/client/user_client.go @@ -2,14 +2,15 @@ package client import ( "context" - "github.com/conductor-sdk/conductor-go/sdk/model/rbac" "net/http" + + "github.com/conductor-sdk/conductor-go/sdk/model/rbac" ) type UserClient interface { CheckPermissions(ctx context.Context, userId string, type_ string, id string) (map[string]interface{}, *http.Response, error) DeleteUser(ctx context.Context, id string) (*http.Response, error) - GetGrantedPermissions(ctx context.Context, userId string) ([]rbac.GrantedAccess, *http.Response, error) + GetGrantedPermissions(ctx context.Context, userId string) (rbac.GrantedAccessResponse, *http.Response, error) GetUser(ctx context.Context, id string) (*rbac.ConductorUser, *http.Response, error) ListUsers(ctx context.Context, optionals *UserResourceApiListUsersOpts) ([]rbac.ConductorUser, *http.Response, error) UpsertUser(ctx context.Context, body rbac.UpsertUserRequest, id string) (*rbac.ConductorUser, *http.Response, error) diff --git a/sdk/workflow/executor/executor.go b/sdk/workflow/executor/executor.go index 1bd982bf..63da6851 100644 --- a/sdk/workflow/executor/executor.go +++ b/sdk/workflow/executor/executor.go @@ -83,7 +83,7 @@ func NewWorkflowExecutor(apiClient *client.APIClient) *WorkflowExecutor { // RegisterWorkflow Registers the workflow on the server. Overwrites if the flag is set. If the 'overwrite' flag is not set // and the workflow definition differs from the one on the server, the call will fail with response code 409 func (e *WorkflowExecutor) RegisterWorkflow(overwrite bool, workflow *model.WorkflowDef) error { - response, err := e.metadataClient.RegisterWorkflowDef( + _, err := e.metadataClient.RegisterWorkflowDef( context.Background(), overwrite, *workflow, @@ -91,9 +91,7 @@ func (e *WorkflowExecutor) RegisterWorkflow(overwrite bool, workflow *model.Work if err != nil { return err } - if response.StatusCode > 299 { - return fmt.Errorf(response.Status) - } + return nil } @@ -120,7 +118,7 @@ func (e *WorkflowExecutor) ExecuteWorkflow(startWorkflowRequest *model.StartWork if error != nil { return nil, error } - return &workflowRun, err + return &workflowRun, nil } // MonitorExecution monitors the workflow execution @@ -140,7 +138,7 @@ func (e *WorkflowExecutor) StartWorkflow(startWorkflowRequest *model.StartWorkfl if err != nil { return "", err } - return id, err + return id, nil } // StartWorkflows Start workflows in bulk @@ -210,14 +208,18 @@ func (e *WorkflowExecutor) getWorkflow(retry int, workflowId string, includeTask &client.WorkflowResourceApiGetExecutionStatusOpts{ IncludeTasks: optional.NewBool(includeTasks)}, ) + + // 404s in GetWorkflow are a bit inconsistent with other errors since + // it's using fmt.Errorf("..."). Keeping it this way to avoid breaking changes if response.StatusCode == 404 { return nil, fmt.Errorf("no such workflow by Id %s", workflowId) } + if response.StatusCode > 399 && response.StatusCode < 500 && response.StatusCode != 429 { return nil, err } - if err != nil { + if err != nil { if retry < 0 { return nil, err } else { @@ -225,10 +227,9 @@ func (e *WorkflowExecutor) getWorkflow(retry int, workflowId string, includeTask retry = retry - 1 return e.getWorkflow(retry, workflowId, includeTasks) } - } - return &workflow, err + return &workflow, nil } // GetWorkflowStatus Get the status of the workflow execution. @@ -401,7 +402,7 @@ func (e *WorkflowExecutor) ReRun(workflowId string, reRunRequest model.RerunWork if err != nil { return "", err } - return id, err + return id, nil } // SkipTasksFromWorkflow Skips a given task execution from a current running workflow. @@ -426,7 +427,10 @@ func (e *WorkflowExecutor) UpdateTask(taskId string, workflowInstanceId string, return err } taskResult.Status = status - e.taskClient.UpdateTask(context.Background(), taskResult) + _, _, err = e.taskClient.UpdateTask(context.Background(), taskResult) + if err != nil { + return err + } return nil } @@ -436,38 +440,33 @@ func (e *WorkflowExecutor) UpdateTaskByRefName(taskRefName string, workflowInsta if err != nil { return err } - _, response, err := e.taskClient.UpdateTaskByRefName(context.Background(), outputData, workflowInstanceId, taskRefName, string(status)) + + _, _, err = e.taskClient.UpdateTaskByRefName(context.Background(), outputData, workflowInstanceId, taskRefName, string(status)) if err != nil { return err } - if response.StatusCode == 404 { - return fmt.Errorf(response.Status) - } + return nil } // GetTask by task Id returns nil if no such task is found by the id func (e *WorkflowExecutor) GetTask(taskId string) (task *model.Task, err error) { - t, response, err := e.taskClient.GetTask(context.Background(), taskId) + t, _, err := e.taskClient.GetTask(context.Background(), taskId) if err != nil { return nil, err } - if response.StatusCode == 404 { - return nil, nil - } + return &t, nil } // RemoveWorkflow Remove workflow execution permanently from the system // Returns nil if no workflow is found by the id func (e *WorkflowExecutor) RemoveWorkflow(workflowId string) error { - response, err := e.workflowClient.Delete(context.Background(), workflowId, &client.WorkflowResourceApiDeleteOpts{ArchiveWorkflow: optional.NewBool(false)}) + _, err := e.workflowClient.Delete(context.Background(), workflowId, &client.WorkflowResourceApiDeleteOpts{ArchiveWorkflow: optional.NewBool(false)}) if err != nil { return err } - if response.StatusCode != 200 { - return fmt.Errorf(response.Status) - } + return nil } diff --git a/test/integration_tests/application_client_test.go b/test/integration_tests/application_client_test.go index 10f28ae9..f225390c 100644 --- a/test/integration_tests/application_client_test.go +++ b/test/integration_tests/application_client_test.go @@ -200,7 +200,7 @@ func TestApplicationClientErrorHandling(t *testing.T) { // Assume error handling in client converts HTTP 404 to specific Go error; response might not be accessible directly // Try to get tags for a non-existent application - _, resp, err = appClient.GetTagsForApplication(ctx, invalidAppId) + _, _, err = appClient.GetTagsForApplication(ctx, invalidAppId) assert.Nil(t, err) // Try to delete a tag from a non-existent application diff --git a/test/integration_tests/env_client_test.go b/test/integration_tests/env_client_test.go index ea1bdf08..bb1c2d5e 100644 --- a/test/integration_tests/env_client_test.go +++ b/test/integration_tests/env_client_test.go @@ -33,7 +33,7 @@ func TestDeleteEnvVariable(t *testing.T) { message, resp, err := envClient.DeleteEnvVariable(ctx, "testKey") assert.NoError(t, err) assert.NotNil(t, resp) - assert.Equal(t, "", message) + assert.Equal(t, "test value", message) } diff --git a/test/integration_tests/executor_test.go b/test/integration_tests/executor_test.go index 4f527b92..19610a97 100644 --- a/test/integration_tests/executor_test.go +++ b/test/integration_tests/executor_test.go @@ -1,25 +1,92 @@ package integration_tests import ( + "fmt" "testing" "github.com/conductor-sdk/conductor-go/sdk/client" + "github.com/conductor-sdk/conductor-go/sdk/model" + "github.com/conductor-sdk/conductor-go/sdk/workflow" "github.com/conductor-sdk/conductor-go/test/testdata" "github.com/stretchr/testify/assert" ) +const ( + // IDs are hardcoded on purpose. It should not be found. + notFoundWorkflowId = "2b3ea839-9aeb-11ef-9ac5-ce590b39fb93" + notFoundTaskId = "75c6875d-9ba8-11ef-82ba-0680bfba1f84" +) + func TestRetryNotFound(t *testing.T) { executor := testdata.WorkflowExecutor - // Workflow id is hardcoded on purpose. It should not be found. - err := executor.Retry("2b3ea839-9aeb-11ef-9ac5-ce590b39fb93", true) + + err := executor.Retry(notFoundWorkflowId, true) assert.Error(t, err, "Retry is expected to return an error") if swaggerErr, ok := err.(client.GenericSwaggerError); ok { - // hmm... this should be a 404 or 400, but it's a 500 right now. - assert.Error(t, err, "GetWorkflow was expected to return a 500 error") - assert.Equal(t, 500, swaggerErr.StatusCode()) + assert.Equal(t, 500, swaggerErr.StatusCode()) //TODO on test server update, should be a 404 + } else { + assert.Fail(t, "err is not of type GenericSwaggerError") + } +} + +func TestRegisterWorkflow(t *testing.T) { + executor := testdata.WorkflowExecutor + + wf := workflow.ConductorWorkflow{} + wf.Name("registration_test_wf"). + Description("E2E test - Workflow Registration test"). + Version(1). + Add(workflow.NewSimpleTask( + "SIMPLE", "simple_ref", + )) + + // register the workflow + err := executor.RegisterWorkflow(true, wf.ToWorkflowDef()) + assert.Nil(t, err) + + // modify the workflow and register with overwrite: false, to force a 409 + wf.Add(workflow.NewSimpleTask( + "SIMPLE", "simple_ref_2", + )) + err = executor.RegisterWorkflow(false, wf.ToWorkflowDef()) + assert.Error(t, err, "Retry is expected to return an error") + + if swaggerErr, ok := err.(client.GenericSwaggerError); ok { + assert.Equal(t, 409, swaggerErr.StatusCode()) + } else { + assert.Fail(t, "err is not of type GenericSwaggerError") + } +} + +func TestGetWorkflow(t *testing.T) { + executor := testdata.WorkflowExecutor + + wf, err := executor.GetWorkflow(notFoundWorkflowId, false) + + assert.Nil(t, wf) + assert.Error(t, err, "GetWorkflow is expected to return an error") + assert.Equal(t, fmt.Sprintf("no such workflow by Id %s", notFoundWorkflowId), err.Error()) +} + +func TestUpdateTaskByRefName(t *testing.T) { + executor := testdata.WorkflowExecutor + err := executor.UpdateTaskByRefName("task_ref", notFoundWorkflowId, model.CompletedTask, map[string]interface{}{}) + assert.Error(t, err, "UpdateTaskByRefName is expected to return an error") + if swaggerErr, ok := err.(client.GenericSwaggerError); ok { + assert.Equal(t, 500, swaggerErr.StatusCode()) //TODO on test server update, should be a 404 } else { - assert.Fail(t, "err is not of type GenericSwaggerError ") + assert.Fail(t, "err is not of type GenericSwaggerError") } +} +func TestUpdate(t *testing.T) { + executor := testdata.WorkflowExecutor + err := executor.UpdateTask(notFoundTaskId, notFoundWorkflowId, model.CompletedTask, map[string]interface{}{}) + assert.Error(t, err, "UpdateTask is expected to return an error") + if swaggerErr, ok := err.(client.GenericSwaggerError); ok { + assert.Equal(t, 404, swaggerErr.StatusCode()) + } else { + assert.Fail(t, "err is not of type GenericSwaggerError") + } } diff --git a/test/integration_tests/integration_client_test.go b/test/integration_tests/integration_client_test.go index 128d0ebb..d2ae54b5 100644 --- a/test/integration_tests/integration_client_test.go +++ b/test/integration_tests/integration_client_test.go @@ -108,7 +108,7 @@ func TestIntegrationClient(t *testing.T) { require.NoError(t, err, "Failed to delete integration API") // Cleanup: Deleting providers to clean the test environment - for i, _ := range integrationEntries { + for i := range integrationEntries { resp, err = integrationClient.DeleteIntegrationProvider(ctx, names[i]) require.NoError(t, err, "error deleting integration provider") require.NotNil(t, resp, "response should not be nil for DeleteIntegrationProvider") diff --git a/test/integration_tests/user_client_test.go b/test/integration_tests/user_client_test.go index 62dad7fd..9e2d927d 100644 --- a/test/integration_tests/user_client_test.go +++ b/test/integration_tests/user_client_test.go @@ -63,8 +63,8 @@ func TestGetGrantedPermissions(t *testing.T) { if resp.StatusCode != http.StatusOK { t.Errorf("Expected status OK, got %v", resp.Status) } - if len(permissions) != 0 { - t.Errorf("Expected non-empty permissions %d", len(permissions)) + if len(permissions.GrantedAccess) != 0 { + t.Errorf("Expected non-empty permissions %d", len(permissions.GrantedAccess)) } }