fix: add check for origin and name (#181)

yaml/secret.go

@@ -79,7 +79,7 @@ func (s *SecretSlice) UnmarshalYAML(unmarshal func(interface{}) error) error {
 
 	// iterate through each element in the secret slice
 	for _, secret := range *secretSlice {
-		if len(secret.Name) == 0 {
+		if secret.Origin.Empty() && len(secret.Name) == 0 {
 			continue
 		}
 

yaml/secret_test.go

@@ -265,6 +265,37 @@ func TestYaml_SecretSlice_UnmarshalYAML(t *testing.T) {
 						},
 					},
 				},
+				{
+					Name:   "",
+					Key:    "",
+					Engine: "",
+					Type:   "",
+					Origin: Origin{
+						Environment: map[string]string{"FOO": "bar"},
+						Image:       "target/vela-vault:latest",
+						Parameters: map[string]interface{}{
+							"addr": "vault.company.com",
+						},
+						Pull: "always",
+						Ruleset: Ruleset{
+							If: Rules{
+								Event: []string{"push"},
+							},
+							Operator: "and",
+							Matcher:  "filepath",
+						},
+						Secrets: StepSecretSlice{
+							{
+								Source: "foo",
+								Target: "foo",
+							},
+							{
+								Source: "foobar",
+								Target: "foobar",
+							},
+						},
+					},
+				},
 			},
 		},
 		{

yaml/testdata/secret.yml

@@ -26,3 +26,13 @@
     ruleset:
       event: [ push ]
     secrets: [ foo, foobar ] 
+- origin:
+    environment:
+      FOO: bar
+    image: target/vela-vault:latest
+    pull: true
+    parameters:
+      addr: vault.company.com
+    ruleset:
+      event: [ push ]
+    secrets: [ foo, foobar ]