fix: address review comments

#101 (review)

cmd/chisel/cmd_info.go

@@ -134,5 +134,5 @@ func selectPackageSlices(release *setup.Release, queries []string) (packages []*
 		}
 		packages = append(packages, pkg)
 	}
-	return
+	return packages, notFound
 }

cmd/chisel/cmd_info_test.go

@@ -99,7 +99,7 @@ var infoTests = []infoTest{{
 					- mypkg2_myslice
 	`,
 }, {
-	summary: "All kinds of path",
+	summary: "All kinds of paths",
 	input:   infoRelease,
 	query:   []string{"mypkg3"},
 	stdout: `
@@ -172,6 +172,7 @@ var infoTests = []infoTest{{
 }}
 
 var testKey = testutil.PGPKeys["key1"]
+
 var defaultChiselYaml = `
 	format: chisel-v1
 	archives:
@@ -182,8 +183,7 @@ var defaultChiselYaml = `
 	v1-public-keys:
 		test-key:
 			id: ` + testKey.ID + `
-			armor: |` + "\n" + testutil.PrefixEachLine(testKey.PubKeyArmor, "\t\t\t\t\t\t") + `
-`
+			armor: |` + "\n" + testutil.PrefixEachLine(testKey.PubKeyArmor, "\t\t\t\t\t\t")
 
 var infoRelease = map[string]string{
 	"chisel.yaml": string(defaultChiselYaml),
@@ -255,6 +255,6 @@ func (s *ChiselSuite) TestInfoCommand(c *C) {
 		}
 		c.Assert(err, IsNil)
 		test.stdout = string(testutil.Reindent(test.stdout))
-		c.Assert(strings.TrimSpace(s.Stdout()), Equals, strings.TrimSpace(test.stdout))
+		c.Assert(s.Stdout(), Equals, strings.TrimSpace(test.stdout)+"\n")
 	}
 }

internal/setup/setup.go

@@ -357,15 +357,14 @@ type yamlPackage struct {
 }
 
 type yamlPath struct {
-	Dir     bool    `yaml:"make,omitempty"`
-	Copy    string  `yaml:"copy,omitempty"`
-	Text    *string `yaml:"text,omitempty"`
-	Symlink string  `yaml:"symlink,omitempty"`
-	Mutable bool    `yaml:"mutable,omitempty"`
-
-	Until PathUntil `yaml:"until,omitempty"`
-	Arch  yamlArch  `yaml:"arch,omitempty"`
-	Mode  yamlMode  `yaml:"mode,omitempty"`
+	Dir     bool      `yaml:"make,omitempty"`
+	Copy    string    `yaml:"copy,omitempty"`
+	Text    *string   `yaml:"text,omitempty"`
+	Symlink string    `yaml:"symlink,omitempty"`
+	Mutable bool      `yaml:"mutable,omitempty"`
+	Until   PathUntil `yaml:"until,omitempty"`
+	Arch    yamlArch  `yaml:"arch,omitempty"`
+	Mode    yamlMode  `yaml:"mode,omitempty"`
 }
 
 func (yp *yamlPath) MarshalYAML() (interface{}, error) {
@@ -419,6 +418,8 @@ func (ya yamlArch) MarshalYAML() (interface{}, error) {
 	return ya.List, nil
 }
 
+var _ yaml.Marshaler = yamlArch{}
+
 type yamlMode uint
 
 func (ym yamlMode) MarshalYAML() (interface{}, error) {
@@ -433,6 +434,8 @@ func (ym yamlMode) MarshalYAML() (interface{}, error) {
 	return node, nil
 }
 
+var _ yaml.Marshaler = yamlMode(0)
+
 type yamlSlice struct {
 	Essential []string             `yaml:"essential,omitempty"`
 	Contents  map[string]*yamlPath `yaml:"contents,omitempty"`

tests/info/task.yaml

@@ -1,4 +1,4 @@
-summary: Chisel can show detailed info about slices
+summary: Chisel can show detailed information about slices
 
 execute: |
   # Install dependencies.