dont change root function

Author: EyalDelarea

apptrust/http/http_client.go

@@ -25,7 +25,7 @@ type ApptrustHttpClient interface {
 	GetHttpClient() *jfroghttpclient.JfrogHttpClient
 	Post(path string, requestBody interface{}, params map[string]string) (resp *http.Response, body []byte, err error)
 	Get(path string) (resp *http.Response, body []byte, err error)
-	Patch(path string, requestBody interface{}, params map[string]string) (resp *http.Response, body []byte, err error)
+	Patch(path string, requestBody interface{}) (resp *http.Response, body []byte, err error)
 	Delete(path string) (resp *http.Response, body []byte, err error)
 }
 
@@ -112,8 +112,8 @@ func (c *apptrustHttpClient) Get(path string) (resp *http.Response, body []byte,
 	return response, body, err
 }
 
-func (c *apptrustHttpClient) Patch(path string, requestBody interface{}, params map[string]string) (resp *http.Response, body []byte, err error) {
-	url, err := utils.BuildUrl(c.serverDetails.Url, apptrustApiPath+path, params)
+func (c *apptrustHttpClient) Patch(path string, requestBody interface{}) (resp *http.Response, body []byte, err error) {
+	url, err := utils.BuildUrl(c.serverDetails.Url, apptrustApiPath+path, nil)
 	if err != nil {
 		return nil, nil, err
 	}

apptrust/service/applications/application_service.go

@@ -42,7 +42,7 @@ func (as *applicationService) CreateApplication(ctx service.Context, requestBody
 
 func (as *applicationService) UpdateApplication(ctx service.Context, requestBody *model.AppDescriptor) error {
 	endpoint := fmt.Sprintf("/v1/applications/%s", requestBody.ApplicationKey)
-	response, responseBody, err := ctx.GetHttpClient().Patch(endpoint, requestBody, nil)
+	response, responseBody, err := ctx.GetHttpClient().Patch(endpoint, requestBody)
 	if err != nil {
 		return err
 	}

apptrust/service/versions/version_service.go

@@ -77,8 +77,8 @@ func (vs *versionService) ReleaseAppVersion(ctx service.Context, applicationKey,
 }
 
 func (vs *versionService) RollbackAppVersion(ctx service.Context, applicationKey, version string, request *model.RollbackAppVersionRequest) error {
-	endpoint := fmt.Sprintf("/v1/applications/%s/versions/%s/rollback", applicationKey, version)
-	response, responseBody, err := ctx.GetHttpClient().Post(endpoint, request, map[string]string{"async": "false"})
+	endpoint := fmt.Sprintf("/v1/applications/%s/versions/%s/rollback?async=false", applicationKey, version)
+	response, responseBody, err := ctx.GetHttpClient().Post(endpoint, request, map[string]string{})
 	if err != nil {
 		return err
 	}
@@ -108,8 +108,8 @@ func (vs *versionService) DeleteAppVersion(ctx service.Context, applicationKey,
 }
 
 func (vs *versionService) UpdateAppVersion(ctx service.Context, applicationKey string, version string, request *model.UpdateAppVersionRequest) error {
-	endpoint := fmt.Sprintf("/v1/applications/%s/versions/%s", applicationKey, version)
-	response, responseBody, err := ctx.GetHttpClient().Patch(endpoint, request, map[string]string{"async": "false"})
+	endpoint := fmt.Sprintf("/v1/applications/%s/versions/%s?async=false", applicationKey, version)
+	response, responseBody, err := ctx.GetHttpClient().Patch(endpoint, request)
 	if err != nil {
 		return err
 	}

apptrust/service/versions/version_service_test.go

@@ -397,7 +397,7 @@ func TestUpdateAppVersion(t *testing.T) {
 	for _, tt := range tests {
 		t.Run(tt.name, func(t *testing.T) {
 			mockHttpClient := mockhttp.NewMockApptrustHttpClient(ctrl)
-			mockHttpClient.EXPECT().Patch("/v1/applications/test-app/versions/1.0.0", tt.request, map[string]string{"async": "false"}).
+			mockHttpClient.EXPECT().Patch("/v1/applications/test-app/versions/1.0.0?async=false", tt.request).
 				Return(tt.mockResponse, []byte(tt.mockResponseBody), tt.mockError).Times(1)
 
 			mockCtx := mockservice.NewMockContext(ctrl)
@@ -466,8 +466,8 @@ func TestRollbackAppVersion(t *testing.T) {
 			mockClient := mockhttp.NewMockApptrustHttpClient(ctrl)
 			mockCtx.EXPECT().GetHttpClient().Return(mockClient)
 
-			expectedEndpoint := "/v1/applications/" + tt.applicationKey + "/versions/" + tt.version + "/rollback"
-			mockClient.EXPECT().Post(expectedEndpoint, tt.payload, map[string]string{"async": "false"}).
+			expectedEndpoint := "/v1/applications/" + tt.applicationKey + "/versions/" + tt.version + "/rollback?async=false"
+			mockClient.EXPECT().Post(expectedEndpoint, tt.payload, map[string]string{}).
 				Return(&http.Response{StatusCode: tt.expectedStatus}, []byte(""), nil)
 
 			service := NewVersionService()