template: validate post-processor only/except

template/template.go

@@ -109,6 +109,19 @@ func (t *Template) Validate() error {
 		}
 	}
 
+	// Verify post-processors
+	for i, chain := range t.PostProcessors {
+		for j, p := range chain {
+			// Validate only/except
+			if verr := p.OnlyExcept.Validate(t); verr != nil {
+				for _, e := range multierror.Append(verr).Errors {
+					err = multierror.Append(err, fmt.Errorf(
+						"post-processor %d.%d: %s", i+1, j+1, e))
+				}
+			}
+		}
+	}
+
 	return err
 }
 

template/template_test.go

@@ -52,6 +52,26 @@ func TestTemplateValidate(t *testing.T) {
 			"validate-good-prov-except.json",
 			false,
 		},
+
+		{
+			"validate-bad-pp-only.json",
+			true,
+		},
+
+		{
+			"validate-good-pp-only.json",
+			false,
+		},
+
+		{
+			"validate-bad-pp-except.json",
+			true,
+		},
+
+		{
+			"validate-good-pp-except.json",
+			false,
+		},
 	}
 
 	for _, tc := range cases {

template/test-fixtures/validate-bad-pp-except.json

+{
+    "builders": [{
+        "type": "foo"
+    }],
+
+    "post-processors": [{
+        "type": "bar",
+        "except": ["bar"]
+    }]
+}

template/test-fixtures/validate-bad-pp-only.json

+{
+    "builders": [{
+        "type": "foo"
+    }],
+
+    "post-processors": [{
+        "type": "bar",
+        "only": ["bar"]
+    }]
+}

template/test-fixtures/validate-good-pp-except.json

+{
+    "builders": [{
+        "type": "foo"
+    }],
+
+    "post-processors": [{
+        "type": "bar",
+        "except": ["foo"]
+    }]
+}

template/test-fixtures/validate-good-pp-only.json

+{
+    "builders": [{
+        "type": "foo"
+    }],
+
+    "post-processors": [{
+        "type": "bar",
+        "only": ["foo"]
+    }]
+}