Skip to content

Commit 0e91714

Browse files
alenonalenon
committed
Added tests
1 parent a5a2b92 commit 0e91714

25 files changed

+680
-186
lines changed

‎.github/scripts/gotest.sh‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -74,7 +74,7 @@ exitCode="$(cat "${exitCodeFile}")"
7474
echoDebug "Tests Exit Code: $exitCode"
7575

7676
if [[ -n "${JSON_OUTFILE}" ]]; then
77-
echoDebug "Gernerating JSON test report at: ${JSON_OUTFILE}"
77+
echoDebug "Generating JSON test report at: ${JSON_OUTFILE}"
7878
go tool test2json < "${OUTFILE}" > "${JSON_OUTFILE}"
7979
fi
8080

‎.github/workflows/analysis.yml‎

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -18,6 +18,9 @@ jobs:
1818
with:
1919
go-version-file: go.mod
2020

21+
- name: Generate Mocks
22+
run: make generate-mock
23+
2124
- name: Static Code Analysis
2225
uses: golangci/golangci-lint-action@v6
2326
with:

‎.github/workflows/tests.yml‎

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -36,8 +36,7 @@ jobs:
3636
env:
3737
XUNIT_OUTFILE: test-reports/unit.xml
3838
run: |
39-
test -d "$(dirname $XUNIT_OUTFILE)" || mkdir -p "$(dirname $XUNIT_OUTFILE)"
40-
.github/scripts/gotest.sh ./...
39+
make test
4140
4241
- name: Publish Results
4342
uses: EnricoMi/publish-unit-test-result-action@v2

‎.golangci.yml‎

Lines changed: 38 additions & 38 deletions
Original file line numberDiff line numberDiff line change
@@ -1,48 +1,48 @@
11
# Options for analysis running
22
# More info could be found at https://golangci-lint.run/usage/configuration/
33
run:
4-
# timeout for analysis, e.g. 30s, 5m, default is 1m
5-
timeout: 5m
6-
modules-download-mode: readonly
4+
# timeout for analysis, e.g. 30s, 5m, default is 1m
5+
timeout: 5m
6+
modules-download-mode: readonly
77

88
# List of useful linters could be found at https://github.com/golangci/awesome-go-linters
99
linters:
10-
disable-all: true
11-
enable:
12-
- errcheck
13-
- copyloopvar
14-
- gofumpt
15-
- goimports
16-
- gosimple
17-
- govet
18-
- ineffassign
19-
- makezero
20-
- misspell
21-
- noctx
22-
- nolintlint
23-
- rowserrcheck
24-
- sqlclosecheck
25-
- staticcheck
26-
- unconvert
27-
- unused
28-
- wastedassign
29-
- gosec
10+
disable-all: true
11+
enable:
12+
- errcheck
13+
- copyloopvar
14+
- gofumpt
15+
- goimports
16+
- gosimple
17+
- govet
18+
- ineffassign
19+
- makezero
20+
- misspell
21+
- noctx
22+
- nolintlint
23+
- rowserrcheck
24+
- sqlclosecheck
25+
- staticcheck
26+
- unconvert
27+
- unused
28+
- wastedassign
29+
- gosec
3030

3131
linters-settings:
32-
staticcheck:
33-
# https://staticcheck.io/docs/options#checks
34-
checks: [ "all","-SA1019","-SA1029" ]
35-
gosec:
36-
excludes: ["G204", "G301", "G302", "G304", "G306", "G601", "G101", "G407"]
37-
exclude-generated: true
38-
exclude-test-files: true
39-
config:
40-
global:
41-
nosec: true
32+
staticcheck:
33+
# https://staticcheck.io/docs/options#checks
34+
checks: [ "all","-SA1019","-SA1029" ]
35+
gosec:
36+
excludes: [ "G204", "G301", "G302", "G304", "G306", "G601", "G101", "G407" ]
37+
exclude-generated: true
38+
exclude-test-files: true
39+
config:
40+
global:
41+
nosec: true
4242

4343
issues:
44-
exclude-use-default: false
45-
# Maximum issues count per one linter. Set to 0 to disable. Default is 50.
46-
max-issues-per-linter: 0
47-
# Maximum count of issues with the same text. Set to 0 to disable. Default is 3.
48-
max-same-issues: 0
44+
exclude-use-default: false
45+
# Maximum issues count per one linter. Set to 0 to disable. Default is 50.
46+
max-issues-per-linter: 0
47+
# Maximum count of issues with the same text. Set to 0 to disable. Default is 3.
48+
max-same-issues: 0

‎Makefile‎

Lines changed: 7 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -34,7 +34,7 @@ GOIMPORTS:
3434
########## ANALYSE ##########
3535

3636
GOLANGCI_LINT = ${TOOLS_DIR}/golangci-lint
37-
GOLANGCI_LINT_VERSION = 1.62.2
37+
GOLANGCI_LINT_VERSION = 1.63.4
3838

3939
verify: GOLANGCI_LINT
4040
echo $(GO_SOURCES)
@@ -63,7 +63,12 @@ build-install:: build
6363
.PHONY: clean-mock
6464
clean-mock:
6565
@echo Cleaning generated mock files
66-
find . -path "*/mocks/*.go" -delete
66+
@find . -name "*_mock.go" -delete
67+
68+
.PHONY: clean
69+
clean:: clean-mock
70+
@echo Cleaning generated files
71+
@rm -rf ${BINARY_CLI}
6772

6873
.PHONY: generate-mock
6974
generate-mock: clean-mock

‎application/app/auth/app_details.go‎

Lines changed: 0 additions & 15 deletions
This file was deleted.

‎application/app/context.go‎

Lines changed: 10 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -1,32 +1,33 @@
11
package app
22

33
import (
4-
"github.com/jfrog/jfrog-cli-application/application/service"
4+
"github.com/jfrog/jfrog-cli-application/application/service/systems"
5+
"github.com/jfrog/jfrog-cli-application/application/service/versions"
56
)
67

78
type Context interface {
8-
GetVersionService() service.VersionService
9-
GetSystemService() service.SystemService
9+
GetVersionService() versions.VersionService
10+
GetSystemService() systems.SystemService
1011
GetConfig() interface{}
1112
}
1213

1314
type context struct {
14-
versionService service.VersionService
15-
systemService service.SystemService
15+
versionService versions.VersionService
16+
systemService systems.SystemService
1617
}
1718

1819
func NewAppContext() Context {
1920
return &context{
20-
versionService: service.NewVersionService(),
21-
systemService: service.NewSystemService(),
21+
versionService: versions.NewVersionService(),
22+
systemService: systems.NewSystemService(),
2223
}
2324
}
2425

25-
func (c *context) GetVersionService() service.VersionService {
26+
func (c *context) GetVersionService() versions.VersionService {
2627
return c.versionService
2728
}
2829

29-
func (c *context) GetSystemService() service.SystemService {
30+
func (c *context) GetSystemService() systems.SystemService {
3031
return c.systemService
3132
}
3233

‎application/app/context_test.go‎

Lines changed: 38 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,38 @@
1+
package app
2+
3+
import (
4+
"testing"
5+
6+
mocksystems "github.com/jfrog/jfrog-cli-application/application/service/systems/mocks"
7+
mockversions "github.com/jfrog/jfrog-cli-application/application/service/versions/mocks"
8+
9+
"github.com/stretchr/testify/assert"
10+
)
11+
12+
func TestNewAppContext(t *testing.T) {
13+
ctx := NewAppContext()
14+
assert.NotNil(t, ctx)
15+
assert.NotNil(t, ctx.GetVersionService())
16+
assert.NotNil(t, ctx.GetSystemService())
17+
}
18+
19+
func TestGetVersionService(t *testing.T) {
20+
mockVersionService := &mockversions.MockVersionService{}
21+
ctx := &context{
22+
versionService: mockVersionService,
23+
}
24+
assert.Equal(t, mockVersionService, ctx.GetVersionService())
25+
}
26+
27+
func TestGetSystemService(t *testing.T) {
28+
mockSystemService := &mocksystems.MockSystemService{}
29+
ctx := &context{
30+
systemService: mockSystemService,
31+
}
32+
assert.Equal(t, mockSystemService, ctx.GetSystemService())
33+
}
34+
35+
func TestGetConfig(t *testing.T) {
36+
ctx := &context{}
37+
assert.Nil(t, ctx.GetConfig())
38+
}

‎application/commands/system/ping_cmd.go‎

Lines changed: 9 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1,23 +1,30 @@
11
package system
22

3+
//go:generate ${PROJECT_DIR}/scripts/mockgen.sh ${GOFILE}
4+
35
import (
46
"github.com/jfrog/jfrog-cli-application/application/app"
57
"github.com/jfrog/jfrog-cli-application/application/commands"
68
"github.com/jfrog/jfrog-cli-application/application/commands/utils"
79
"github.com/jfrog/jfrog-cli-application/application/common"
810
"github.com/jfrog/jfrog-cli-application/application/service"
11+
"github.com/jfrog/jfrog-cli-application/application/service/systems"
912
commonCLiCommands "github.com/jfrog/jfrog-cli-core/v2/common/commands"
1013
"github.com/jfrog/jfrog-cli-core/v2/plugins/components"
1114
coreConfig "github.com/jfrog/jfrog-cli-core/v2/utils/config"
1215
)
1316

1417
type pingCommand struct {
15-
systemService service.SystemService
18+
systemService systems.SystemService
1619
serverDetails *coreConfig.ServerDetails
1720
}
1821

1922
func (pc *pingCommand) Run() error {
20-
ctx := &service.Context{ServerDetails: pc.serverDetails}
23+
ctx, err := service.NewContext(*pc.serverDetails)
24+
if err != nil {
25+
return err
26+
}
27+
2128
return pc.systemService.Ping(ctx)
2229
}
2330

‎application/commands/version/create_app_version_cmd.go‎

Lines changed: 8 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -3,6 +3,8 @@ package version
33
import (
44
"encoding/json"
55

6+
"github.com/jfrog/jfrog-cli-application/application/service/versions"
7+
68
"github.com/jfrog/jfrog-cli-application/application/app"
79
"github.com/jfrog/jfrog-cli-application/application/commands"
810
"github.com/jfrog/jfrog-cli-application/application/commands/utils"
@@ -19,7 +21,7 @@ import (
1921
)
2022

2123
type createAppVersionCommand struct {
22-
versionService service.VersionService
24+
versionService versions.VersionService
2325
serverDetails *coreConfig.ServerDetails
2426
requestPayload *model.CreateAppVersionRequest
2527
}
@@ -29,7 +31,11 @@ type createVersionSpec struct {
2931
}
3032

3133
func (cv *createAppVersionCommand) Run() error {
32-
ctx := &service.Context{ServerDetails: cv.serverDetails}
34+
ctx, err := service.NewContext(*cv.serverDetails)
35+
if err != nil {
36+
return err
37+
}
38+
3339
return cv.versionService.CreateAppVersion(ctx, cv.requestPayload)
3440
}
3541

0 commit comments

Comments
 (0)