omit LICENSE file when licence is set to none (spf13#937)

Creating a project without a license should not create an empty LICENSE
file.

cobra/cmd/init_test.go

@@ -5,6 +5,9 @@ import (
 	"io/ioutil"
 	"os"
 	"path/filepath"
+	"reflect"
+	"sort"
+	"strings"
 	"testing"
 )
 
@@ -45,3 +48,46 @@ func TestGoldenInitCmd(t *testing.T) {
 		}
 	}
 }
+
+func TestInitNoLicense(t *testing.T) {
+	project := getProject()
+	project.Legal = noLicense
+	defer os.RemoveAll(project.AbsolutePath)
+
+	err := project.Create()
+	if err != nil {
+		t.Fatal(err)
+	}
+
+	root := project.AbsolutePath
+
+	want := []string{"main.go", "cmd/root.go"}
+	var got []string
+	err = filepath.Walk(root, func(path string, info os.FileInfo, err error) error {
+		if err != nil {
+			return err
+		}
+		if info.IsDir() {
+			return nil
+		}
+		relpath, err := filepath.Rel(root, path)
+		if err != nil {
+			return err
+		}
+		got = append(got, relpath)
+		return nil
+	})
+	if err != nil {
+		t.Fatalf("walking path %q: %v", root, err)
+	}
+	sort.StringSlice(got).Sort()
+	sort.StringSlice(want).Sort()
+	if !reflect.DeepEqual(got, want) {
+		t.Fatalf(
+			"In %s, found:\n  %s\nwant:\n  %s",
+			root,
+			strings.Join(got, ", "),
+			strings.Join(want, ", "),
+		)
+	}
+}

cobra/cmd/licenses.go

@@ -36,9 +36,11 @@ type License struct {
 	Header          string   // License header for source files
 }
 
+var noLicense = License{"None", []string{"none", "false"}, "", ""}
+
 func init() {
 	// Allows a user to not use a license.
-	Licenses["none"] = License{"None", []string{"none", "false"}, "", ""}
+	Licenses["none"] = noLicense
 
 	initApache2()
 	initMit()

cobra/cmd/project.go

@@ -64,6 +64,9 @@ func (p *Project) Create() error {
 	}
 
 	// create license
+	if p.Legal.Name == noLicense.Name {
+		return nil
+	}
 	return p.createLicenseFile()
 }