use the name from the chart as default if none is specified

Signed-off-by: Christian Richter <[email protected]>

cmd/hypper/install_test.go

@@ -61,10 +61,9 @@ func TestInstallCmd(t *testing.T) {
 		},
 		// Install, no name or annotations specified
 		{
-			name:      "install, with no name or annot specified",
-			cmd:       "install testdata/testcharts/vanilla-helm",
-			golden:    "output/install-no-name-or-annot.txt",
-			wantError: true,
+			name:   "install, with no name or annot specified",
+			cmd:    "install testdata/testcharts/vanilla-helm",
+			golden: "output/install-no-name-or-annot.txt",
 		},
 	}
 	runTestActionCmd(t, tests)

cmd/hypper/testdata/output/install-no-name-or-annot.txt

@@ -1 +1,2 @@
-Error: must either provide a name, set the correct chart annotations, or specify --generate-name
+Installing chart "empty" in namespace "default"…
+Done! 👏 

pkg/action/install.go

@@ -124,7 +124,7 @@ func (i *Install) Name(chart *chart.Chart, args []string) (string, error) {
 	}
 
 	if !i.GenerateName {
-		return "", errors.New("must either provide a name, set the correct chart annotations, or specify --generate-name")
+		return chart.Name(), nil
 	}
 
 	base := filepath.Base(args[0])

pkg/action/install_test.go

@@ -119,10 +119,10 @@ func TestName(t *testing.T) {
 	instAction.ReleaseName = ""
 	chart = buildChart()
 	_, err = instAction.Name(chart, []string{"chart-uri"})
-	if err == nil {
-		t.Fatal("expected an error")
+	if err != nil {
+		t.Fatal(err)
 	}
-	is.Equal("must either provide a name, set the correct chart annotations, or specify --generate-name", err.Error())
+	is.Equal("fleet", name)
 }
 
 func TestNameGenerateName(t *testing.T) {