Fix comment

internal/wrappers/client.go

@@ -70,7 +70,7 @@ var cachedAccessToken string
 var cachedAccessTime time.Time
 var Domains = make(map[string]struct{})
 
-func retryHTTPRequest(requestFunc func() (*http.Response, error), retries int, baseDelayInMillis time.Duration) (*http.Response, error) {
+func retryHTTPRequest(requestFunc func() (*http.Response, error), retries int, baseDelayInMilliSec time.Duration) (*http.Response, error) {
 
 	var resp *http.Response
 	var err error
@@ -84,7 +84,7 @@ func retryHTTPRequest(requestFunc func() (*http.Response, error), retries int, b
 			return resp, nil
 		}
 		_ = resp.Body.Close()
-		time.Sleep(baseDelayInMillis * (1 << attempt))
+		time.Sleep(baseDelayInMilliSec * (1 << attempt))
 	}
 	return resp, nil
 }

internal/wrappers/client_test.go

@@ -5,6 +5,7 @@ import (
 	"github.com/stretchr/testify/assert"
 	"net/http"
 	"testing"
+	"time"
 )
 
 type mockReadCloser struct{}
@@ -25,7 +26,7 @@ func TestRetryHTTPRequest_Success(t *testing.T) {
 		}, nil
 	}
 
-	resp, err := retryHTTPRequest(fn, retryAttempts, retryDelay)
+	resp, err := retryHTTPRequest(fn, retryAttempts, retryDelay*time.Millisecond)
 	assert.NoError(t, err)
 	assert.NotNil(t, resp)
 	assert.Equal(t, http.StatusOK, resp.StatusCode)
@@ -47,7 +48,7 @@ func TestRetryHTTPRequest_RetryOnBadGateway(t *testing.T) {
 		}, nil
 	}
 
-	resp, err := retryHTTPRequest(fn, retryAttempts, retryDelay)
+	resp, err := retryHTTPRequest(fn, retryAttempts, retryDelay*time.Millisecond)
 	assert.NoError(t, err)
 	assert.NotNil(t, resp)
 	assert.Equal(t, http.StatusOK, resp.StatusCode)
@@ -59,7 +60,7 @@ func TestRetryHTTPRequest_Fail(t *testing.T) {
 		return nil, errors.New("network error")
 	}
 
-	resp, err := retryHTTPRequest(fn, retryAttempts, retryDelay)
+	resp, err := retryHTTPRequest(fn, retryAttempts, retryDelay*time.Millisecond)
 	assert.Error(t, err)
 	assert.Nil(t, resp)
 }
@@ -72,7 +73,7 @@ func TestRetryHTTPRequest_EndWithBadGateway(t *testing.T) {
 		}, nil
 	}
 
-	resp, err := retryHTTPRequest(fn, retryAttempts, retryDelay)
+	resp, err := retryHTTPRequest(fn, retryAttempts, retryDelay*time.Millisecond)
 	assert.NoError(t, err)
 	assert.NotNil(t, resp)
 	assert.Equal(t, http.StatusBadGateway, resp.StatusCode)

internal/wrappers/wrapper-constants.go → internal/wrappers/constants.go

@@ -1,9 +1,7 @@
 package wrappers
 
-import "time"
-
 const (
 	limitValue    = "10000"
 	retryAttempts = 4
-	retryDelay    = 500 * time.Millisecond
+	retryDelay    = 500
 )

internal/wrappers/projects-http.go

@@ -7,6 +7,7 @@ import (
 	"github.com/pkg/errors"
 	"github.com/spf13/viper"
 	"net/http"
+	"time"
 
 	errorConstants "github.com/checkmarx/ast-cli/internal/constants/errors"
 	commonParams "github.com/checkmarx/ast-cli/internal/params"
@@ -32,7 +33,7 @@ func (p *ProjectsHTTPWrapper) Create(model *Project) (*ProjectResponseModel, *Er
 	fn := func() (*http.Response, error) {
 		return SendHTTPRequest(http.MethodPost, p.path, bytes.NewBuffer(jsonBytes), true, clientTimeout)
 	}
-	resp, err := retryHTTPRequest(fn, retryAttempts, retryDelay)
+	resp, err := retryHTTPRequest(fn, retryAttempts, retryDelay*time.Millisecond)
 	if err != nil {
 		return nil, nil, err
 	}
@@ -54,7 +55,7 @@ func (p *ProjectsHTTPWrapper) Update(projectID string, model *Project) error {
 	fn := func() (*http.Response, error) {
 		return SendHTTPRequest(http.MethodPut, fmt.Sprintf("%s/%s", p.path, projectID), bytes.NewBuffer(jsonBytes), true, clientTimeout)
 	}
-	resp, err := retryHTTPRequest(fn, retryAttempts, retryDelay)
+	resp, err := retryHTTPRequest(fn, retryAttempts, retryDelay*time.Millisecond)
 	if err != nil {
 		return err
 	}
@@ -87,7 +88,7 @@ func (p *ProjectsHTTPWrapper) UpdateConfiguration(projectID string, configuratio
 	fn := func() (*http.Response, error) {
 		return SendHTTPRequestWithQueryParams(http.MethodPatch, "api/configuration/project", params, bytes.NewBuffer(jsonBytes), clientTimeout)
 	}
-	resp, err := retryHTTPRequest(fn, retryAttempts, retryDelay)
+	resp, err := retryHTTPRequest(fn, retryAttempts, retryDelay*time.Millisecond)
 	if err != nil {
 		return nil, err
 	}
@@ -111,7 +112,7 @@ func (p *ProjectsHTTPWrapper) Get(params map[string]string) (
 	fn := func() (*http.Response, error) {
 		return SendHTTPRequestWithQueryParams(http.MethodGet, p.path, params, nil, clientTimeout)
 	}
-	resp, err := retryHTTPRequest(fn, retryAttempts, retryDelay)
+	resp, err := retryHTTPRequest(fn, retryAttempts, retryDelay*time.Millisecond)
 	if err != nil {
 		return nil, nil, err
 	}
@@ -152,7 +153,7 @@ func (p *ProjectsHTTPWrapper) GetByID(projectID string) (
 	fn := func() (*http.Response, error) {
 		return SendHTTPRequest(http.MethodGet, p.path+"/"+projectID, http.NoBody, true, clientTimeout)
 	}
-	resp, err := retryHTTPRequest(fn, retryAttempts, retryDelay)
+	resp, err := retryHTTPRequest(fn, retryAttempts, retryDelay*time.Millisecond)
 	if err != nil {
 		return nil, nil, err
 	}
@@ -199,7 +200,7 @@ func (p *ProjectsHTTPWrapper) GetBranchesByID(projectID string, params map[strin
 	fn := func() (*http.Response, error) {
 		return SendHTTPRequestWithQueryParams(http.MethodGet, p.path+request, params, nil, clientTimeout)
 	}
-	resp, err := retryHTTPRequest(fn, retryAttempts, retryDelay)
+	resp, err := retryHTTPRequest(fn, retryAttempts, retryDelay*time.Millisecond)
 	if err != nil {
 		return nil, nil, err
 	}
@@ -237,7 +238,7 @@ func (p *ProjectsHTTPWrapper) Delete(projectID string) (*ErrorModel, error) {
 	fn := func() (*http.Response, error) {
 		return SendHTTPRequest(http.MethodDelete, p.path+"/"+projectID, http.NoBody, true, clientTimeout)
 	}
-	resp, err := retryHTTPRequest(fn, retryAttempts, retryDelay)
+	resp, err := retryHTTPRequest(fn, retryAttempts, retryDelay*time.Millisecond)
 	if err != nil {
 		return nil, err
 	}
@@ -258,7 +259,7 @@ func (p *ProjectsHTTPWrapper) Tags() (
 	fn := func() (*http.Response, error) {
 		return SendHTTPRequest(http.MethodGet, p.path+"/tags", http.NoBody, true, clientTimeout)
 	}
-	resp, err := retryHTTPRequest(fn, retryAttempts, retryDelay)
+	resp, err := retryHTTPRequest(fn, retryAttempts, retryDelay*time.Millisecond)
 	if err != nil {
 		return nil, nil, err
 	}