Friendly error message for non-existent file/directory

Signed-off-by: Anshul Verma <anshulvermapatel@gmail.com>

Friendly error message for non-existent file/directory

Signed-off-by: Anshul Verma <anshulvermapatel@gmail.com>

Friendly error message for non-existent file/directory

Signed-off-by: Anshul Verma <anshulvermapatel@gmail.com>

pkg/chart/metadata.go

@@ -67,7 +67,8 @@ type Metadata struct {
 // Validate checks the metadata for known issues, returning an error if metadata is not correct
 func (md *Metadata) Validate() error {
 	if md == nil {
-		return ValidationError("chart.metadata is required")
+		// if md == nil, it mean 'Chart.yaml' file is not present in the directory.
+		return ValidationError("Chart.yaml file is missing")
 	}
 	if md.APIVersion == "" {
 		return ValidationError("chart.metadata.apiVersion is required")

pkg/chart/metadata_test.go

@@ -0,0 +1,28 @@
+/*
+Copyright The Helm Authors.
+
+Licensed under the Apache License, Version 2.0 (the "License");
+you may not use this file except in compliance with the License.
+You may obtain a copy of the License at
+
+    http://www.apache.org/licenses/LICENSE-2.0
+
+Unless required by applicable law or agreed to in writing, software
+distributed under the License is distributed on an "AS IS" BASIS,
+WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+See the License for the specific language governing permissions and
+limitations under the License.
+*/
+
+package chart
+
+import "testing"
+
+func TestValidate(t *testing.T) {
+	var metadata *Metadata = nil
+	error := metadata.Validate()
+	errorMessage := "validation: Chart.yaml file is missing"
+	if error.Error() != errorMessage {
+		t.Errorf("The error message is other than '%s'", errorMessage)
+	}
+}

pkg/chartutil/chartfile.go

@@ -74,7 +74,7 @@ func IsChartDir(dirName string) (bool, error) {
 		return false, err
 	}
 	if chartContent == nil {
-		return false, errors.Errorf("chart metadata (%s) missing", ChartfileName)
+		return false, errors.Errorf("(%s) : No such file or directory", ChartfileName)
 	}
 	if chartContent.Name == "" {
 		return false, errors.Errorf("invalid chart (%s): name must not be empty", ChartfileName)