Skip to content

Commit 1939d75

Browse files
- Fixed Unit tests
1 parent 6805875 commit 1939d75

2 files changed

Lines changed: 13 additions & 1 deletion

File tree

internal/commands/util/import_test.go

Lines changed: 8 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -20,6 +20,7 @@ func TestImport_ImportSarifFileWithCorrectFlags_CreateImportSuccessfully(t *test
2020
&mock.ByorMockWrapper{},
2121
mock.ApplicationsMockWrapper{},
2222
&mock.FeatureFlagsMockWrapper{},
23+
&mock.TenantConfigurationMockWrapper{},
2324
)
2425
cmd.SetArgs([]string{"utils", "import", "--project-name", "my-project", "--import-file-path", "my-path.sarif"})
2526
err := cmd.Execute()
@@ -36,6 +37,7 @@ func TestImport_ImportSarifFileProjectDoesntExist_CreateImportWithProvidedNewNam
3637
&mock.ByorMockWrapper{},
3738
mock.ApplicationsMockWrapper{},
3839
&mock.FeatureFlagsMockWrapper{},
40+
&mock.TenantConfigurationMockWrapper{},
3941
)
4042
cmd.SetArgs([]string{"utils", "import", "--project-name", "MOCK-PROJECT-NOT-EXIST", "--import-file-path", "my-path.sarif"})
4143
err := cmd.Execute()
@@ -52,6 +54,7 @@ func TestImport_ImportSarifFileMissingImportFilePath_CreateImportReturnsErrorWit
5254
&mock.ByorMockWrapper{},
5355
mock.ApplicationsMockWrapper{},
5456
&mock.FeatureFlagsMockWrapper{},
57+
&mock.TenantConfigurationMockWrapper{},
5558
)
5659
cmd.SetArgs([]string{"utils", "import", "--project-name", "my-project"})
5760
err := cmd.Execute()
@@ -67,6 +70,7 @@ func TestImport_ImportSarifFileEmptyImportFilePathValue_CreateImportReturnsError
6770
&mock.ByorMockWrapper{},
6871
mock.ApplicationsMockWrapper{},
6972
&mock.FeatureFlagsMockWrapper{},
73+
&mock.TenantConfigurationMockWrapper{},
7074
)
7175
cmd.SetArgs([]string{"utils", "import", "--project-name", "my-project", "--import-file-path", ""})
7276
err := cmd.Execute()
@@ -82,6 +86,7 @@ func TestImport_ImportSarifFileMissingImportProjectName_CreateImportReturnsError
8286
&mock.ByorMockWrapper{},
8387
mock.ApplicationsMockWrapper{},
8488
&mock.FeatureFlagsMockWrapper{},
89+
&mock.TenantConfigurationMockWrapper{},
8590
)
8691
cmd.SetArgs([]string{"utils", "import", "--import-file-path", "my-path.zip"})
8792
err := cmd.Execute()
@@ -97,6 +102,7 @@ func TestImport_ImportSarifFileProjectNameNotProvided_CreateImportWithProvidedNe
97102
&mock.ByorMockWrapper{},
98103
mock.ApplicationsMockWrapper{},
99104
&mock.FeatureFlagsMockWrapper{},
105+
&mock.TenantConfigurationMockWrapper{},
100106
)
101107
cmd.SetArgs([]string{"utils", "import", "--project-name", "", "--import-file-path", "my-path.sarif"})
102108
err := cmd.Execute()
@@ -112,6 +118,7 @@ func TestImport_ImportSarifFileUnacceptedFileExtension_CreateImportReturnsErrorW
112118
&mock.ByorMockWrapper{},
113119
mock.ApplicationsMockWrapper{},
114120
&mock.FeatureFlagsMockWrapper{},
121+
&mock.TenantConfigurationMockWrapper{},
115122
)
116123
cmd.SetArgs([]string{"utils", "import", "--project-name", "MOCK-PROJECT-NOT-EXIST", "--import-file-path", "my-path.txt"})
117124
err := cmd.Execute()
@@ -127,6 +134,7 @@ func TestImport_ImportSarifFileMissingExtension_CreateImportReturnsErrorWithCorr
127134
&mock.ByorMockWrapper{},
128135
mock.ApplicationsMockWrapper{},
129136
&mock.FeatureFlagsMockWrapper{},
137+
&mock.TenantConfigurationMockWrapper{},
130138
)
131139
cmd.SetArgs([]string{"utils", "import", "--project-name", "MOCK-PROJECT-NOT-EXIST", "--import-file-path", "some/path/no/extension/my-path"})
132140
err := cmd.Execute()

internal/services/projects_test.go

Lines changed: 5 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -19,6 +19,7 @@ func TestFindProject(t *testing.T) {
1919
accessManagementWrapper wrappers.AccessManagementWrapper
2020
applicationsWrapper wrappers.ApplicationsWrapper
2121
featureFlagsWrapper wrappers.FeatureFlagsWrapper
22+
tenantWrapper wrappers.TenantConfigurationWrapper
2223
}
2324
tests := []struct {
2425
name string
@@ -37,6 +38,7 @@ func TestFindProject(t *testing.T) {
3738
accessManagementWrapper: &mock.AccessManagementMockWrapper{},
3839
applicationsWrapper: &mock.ApplicationsMockWrapper{},
3940
featureFlagsWrapper: &mock.FeatureFlagsMockWrapper{},
41+
tenantWrapper: &mock.TenantConfigurationMockWrapper{},
4042
},
4143
want: "MOCK",
4244
wantErr: false,
@@ -51,6 +53,7 @@ func TestFindProject(t *testing.T) {
5153
accessManagementWrapper: &mock.AccessManagementMockWrapper{},
5254
applicationsWrapper: &mock.ApplicationsMockWrapper{},
5355
featureFlagsWrapper: &mock.FeatureFlagsMockWrapper{},
56+
tenantWrapper: &mock.TenantConfigurationMockWrapper{},
5457
},
5558
want: "ID-new-MOCK",
5659
wantErr: false,
@@ -66,7 +69,8 @@ func TestFindProject(t *testing.T) {
6669
ttt.args.groupsWrapper,
6770
ttt.args.accessManagementWrapper,
6871
ttt.args.applicationsWrapper,
69-
ttt.args.featureFlagsWrapper)
72+
ttt.args.featureFlagsWrapper,
73+
ttt.args.tenantWrapper)
7074
if (err != nil) != ttt.wantErr {
7175
t.Errorf("FindProject() error = %v, wantErr %v", err, ttt.wantErr)
7276
return

0 commit comments

Comments
 (0)