Merge "[#308] Removed code style issue on Error string"

cmd/config/get_manifest_test.go

@@ -52,7 +52,7 @@ func TestGetManifestConfigCmd(t *testing.T) {
 			Name:    "missing",
 			CmdLine: "manifestMissing",
 			Cmd:     cmd.NewGetManifestCommand(settings),
-			Error:   fmt.Errorf("Missing configuration: Manifest with name 'manifestMissing'"),
+			Error:   fmt.Errorf("missing configuration: Manifest with name 'manifestMissing'"),
 		},
 		{
 			Name:    "no-manifests",

pkg/bootstrap/isogen/command_test.go

@@ -66,7 +66,7 @@ func TestBootstrapIso(t *testing.T) {
 	defer cleanup(t)
 
 	volBind := tempVol + ":/dst"
-	testErr := fmt.Errorf("TestErr")
+	testErr := fmt.Errorf("testErr")
 	testCfg := &config.Bootstrap{
 		Container: &config.Container{
 			Volume:           volBind,

pkg/config/options.go

@@ -177,7 +177,7 @@ func (o *ManifestOptions) Validate() error {
 		return fmt.Errorf("you must specify a non-empty Manifest name")
 	}
 	if o.RemoteRef != "" {
-		return fmt.Errorf("Repository checkout by RemoteRef is not yet implemented\n%w", errors.ErrNotImplemented{})
+		return fmt.Errorf("repository checkout by RemoteRef is not yet implemented\n%w", errors.ErrNotImplemented{})
 	}
 	if o.IsPrimary && o.RepoName == "" {
 		return ErrMissingRepositoryName{}

pkg/k8s/utils/utils_test.go

@@ -48,16 +48,16 @@ func TestManifestBundleReader(t *testing.T) {
 		},
 		{
 			name:      "Read error",
-			errString: "Failed to read from bundle",
+			errString: "failed to read from bundle",
 			reader: fakeReaderWriter{
-				readErr: fmt.Errorf("Failed to read from bundle"),
+				readErr: fmt.Errorf("failed to read from bundle"),
 			},
 		},
 		{
 			name:      "Write error",
-			errString: "Failed to write bundle",
+			errString: "failed to write bundle",
 			writer: fakeReaderWriter{
-				writeErr: fmt.Errorf("Failed to write bundle"),
+				writeErr: fmt.Errorf("failed to write bundle"),
 			},
 		},
 	}