fix gofmt

Signed-off-by: Stephanie <[email protected]>

Author: yangcao77

pkg/validation/projects.go

@@ -63,7 +63,7 @@ func ValidateProjects(projects []v1alpha2.Project) error {
 				errString += fmt.Sprintf("\nproject %s has more than one remote defined, but has no checkoutfrom remote defined", project.Name)
 				continue
 			}
-				err := validateRemoteMap(gitSource.Remotes, gitSource.CheckoutFrom.Remote, project.Name)
+			err := validateRemoteMap(gitSource.Remotes, gitSource.CheckoutFrom.Remote, project.Name)
 			if err != nil {
 				errString += fmt.Sprintf("\n%s", err.Error())
 			}

pkg/validation/projects_test.go

@@ -13,7 +13,7 @@ func generateDummyGitStarterProject(name string, checkoutRemote *v1alpha2.Checko
 		ProjectSource: v1alpha2.ProjectSource{
 			Git: &v1alpha2.GitProjectSource{
 				GitLikeProjectSource: v1alpha2.GitLikeProjectSource{
-					Remotes: remotes,
+					Remotes:      remotes,
 					CheckoutFrom: checkoutRemote,
 				},
 			},
@@ -27,7 +27,7 @@ func generateDummyGithubStarterProject(name string, checkoutRemote *v1alpha2.Che
 		ProjectSource: v1alpha2.ProjectSource{
 			Github: &v1alpha2.GithubProjectSource{
 				GitLikeProjectSource: v1alpha2.GitLikeProjectSource{
-					Remotes: remotes,
+					Remotes:      remotes,
 					CheckoutFrom: checkoutRemote,
 				},
 			},
@@ -41,7 +41,7 @@ func generateDummyGitProject(name string, checkoutRemote *v1alpha2.CheckoutFrom,
 		ProjectSource: v1alpha2.ProjectSource{
 			Git: &v1alpha2.GitProjectSource{
 				GitLikeProjectSource: v1alpha2.GitLikeProjectSource{
-					Remotes: remotes,
+					Remotes:      remotes,
 					CheckoutFrom: checkoutRemote,
 				},
 			},
@@ -55,7 +55,7 @@ func generateDummyGithubProject(name string, checkoutRemote *v1alpha2.CheckoutFr
 		ProjectSource: v1alpha2.ProjectSource{
 			Github: &v1alpha2.GithubProjectSource{
 				GitLikeProjectSource: v1alpha2.GitLikeProjectSource{
-					Remotes: remotes,
+					Remotes:      remotes,
 					CheckoutFrom: checkoutRemote,
 				},
 			},
@@ -110,8 +110,8 @@ func TestValidateStarterProjects(t *testing.T) {
 		{
 			name: "Invalid Starter Project with empty remotes",
 			starterProjects: []v1alpha2.StarterProject{
-				generateDummyGithubStarterProject("project1",  &v1alpha2.CheckoutFrom{Remote: "origin"}, map[string]string{}),
-				generateDummyGithubStarterProject("project3",  &v1alpha2.CheckoutFrom{Remote: "origin"}, map[string]string{"origin": "originremote", "test": "testremote"}),
+				generateDummyGithubStarterProject("project1", &v1alpha2.CheckoutFrom{Remote: "origin"}, map[string]string{}),
+				generateDummyGithubStarterProject("project3", &v1alpha2.CheckoutFrom{Remote: "origin"}, map[string]string{"origin": "originremote", "test": "testremote"}),
 			},
 			wantErr: &atleastOneRemoteErr,
 		},
@@ -143,44 +143,44 @@ func TestValidateProjects(t *testing.T) {
 		{
 			name: "Valid Project",
 			projects: []v1alpha2.Project{
-				generateDummyGitProject("project1",  &v1alpha2.CheckoutFrom{Remote: "origin"}, map[string]string{"origin": "originremote"}),
-				generateDummyGithubProject("project2",  &v1alpha2.CheckoutFrom{Remote: "origin"}, map[string]string{"origin": "originremote"}),
+				generateDummyGitProject("project1", &v1alpha2.CheckoutFrom{Remote: "origin"}, map[string]string{"origin": "originremote"}),
+				generateDummyGithubProject("project2", &v1alpha2.CheckoutFrom{Remote: "origin"}, map[string]string{"origin": "originremote"}),
 			},
 		},
 		{
 			name: "Invalid Project with multiple remotes but no checkoutfrom",
 			projects: []v1alpha2.Project{
-				generateDummyGithubProject("project2",nil, map[string]string{"origin": "originremote", "test": "testremote"}),
+				generateDummyGithubProject("project2", nil, map[string]string{"origin": "originremote", "test": "testremote"}),
 			},
 			wantErr: &missingCheckOutFromRemoteErr,
 		},
 		{
 			name: "Invalid Project with multiple remote and empty checkout remote",
 			projects: []v1alpha2.Project{
-				generateDummyGitProject("project2",  &v1alpha2.CheckoutFrom{Remote: "origin"}, map[string]string{"origin": "originremote"}),
-				generateDummyGithubProject("project1",  &v1alpha2.CheckoutFrom{Remote: ""}, map[string]string{"origin": "originremote", "test": "testremote"}),
+				generateDummyGitProject("project2", &v1alpha2.CheckoutFrom{Remote: "origin"}, map[string]string{"origin": "originremote"}),
+				generateDummyGithubProject("project1", &v1alpha2.CheckoutFrom{Remote: ""}, map[string]string{"origin": "originremote", "test": "testremote"}),
 			},
 			wantErr: &missingCheckOutFromRemoteErr,
 		},
 		{
 			name: "Invalid Project with wrong checkout",
 			projects: []v1alpha2.Project{
-				generateDummyGithubProject("project1",  &v1alpha2.CheckoutFrom{Remote: "origin1"}, map[string]string{"origin": "originremote", "test": "testremote"}),
-				generateDummyGitProject("project2",  &v1alpha2.CheckoutFrom{Remote: "origin1"}, map[string]string{"origin2": "originremote2"}),
+				generateDummyGithubProject("project1", &v1alpha2.CheckoutFrom{Remote: "origin1"}, map[string]string{"origin": "originremote", "test": "testremote"}),
+				generateDummyGitProject("project2", &v1alpha2.CheckoutFrom{Remote: "origin1"}, map[string]string{"origin2": "originremote2"}),
 			},
 			wantErr: &wrongCheckoutErr,
 		},
 		{
 			name: "Valid Project with empty checkout remote",
 			projects: []v1alpha2.Project{
-				generateDummyGitProject("project1",  &v1alpha2.CheckoutFrom{Remote: ""}, map[string]string{"origin": "originremote"}),
+				generateDummyGitProject("project1", &v1alpha2.CheckoutFrom{Remote: ""}, map[string]string{"origin": "originremote"}),
 			},
 		},
 		{
 			name: "Invalid Project with empty remotes",
 			projects: []v1alpha2.Project{
-				generateDummyGitProject("project1",  &v1alpha2.CheckoutFrom{Remote: "origin"}, map[string]string{}),
-				generateDummyGithubProject("project2",  &v1alpha2.CheckoutFrom{Remote: "origins"}, map[string]string{"origin": "originremote", "test": "testremote"}),
+				generateDummyGitProject("project1", &v1alpha2.CheckoutFrom{Remote: "origin"}, map[string]string{}),
+				generateDummyGithubProject("project2", &v1alpha2.CheckoutFrom{Remote: "origins"}, map[string]string{"origin": "originremote", "test": "testremote"}),
 			},
 			wantErr: &atleastOneRemoteErr,
 		},