Include LintMetadata in the LintResult (#729)

* Include LintMetadata in the LintResult

* Don't include LintMetadata in LintResult's JSON output

v3/lint/result.go

@@ -62,8 +62,9 @@ var (
 // LintResult contains a LintStatus, and an optional human-readable description.
 // The output of a lint is a LintResult.
 type LintResult struct {
-	Status  LintStatus `json:"result"`
-	Details string     `json:"details,omitempty"`
+	Status       LintStatus   `json:"result"`
+	Details      string       `json:"details,omitempty"`
+	LintMetadata LintMetadata `json:"-"`
 }
 
 // MarshalJSON implements the json.Marshaler interface.

v3/resultset.go

@@ -39,6 +39,7 @@ func (z *ResultSet) executeCertificate(o *x509.Certificate, registry lint.Regist
 	// Run each lint from the registry.
 	for _, lint := range registry.CertificateLints().Lints() {
 		res := lint.Execute(o, registry.GetConfiguration())
+		res.LintMetadata = lint.LintMetadata
 		z.Results[lint.Name] = res
 		z.updateErrorStatePresent(res)
 	}
@@ -52,6 +53,7 @@ func (z *ResultSet) executeRevocationList(o *x509.RevocationList, registry lint.
 	// Run each lints from the registry.
 	for _, lint := range registry.RevocationListLints().Lints() {
 		res := lint.Execute(o, registry.GetConfiguration())
+		res.LintMetadata = lint.LintMetadata
 		z.Results[lint.Name] = res
 		z.updateErrorStatePresent(res)
 	}

v3/zlint_test.go

@@ -145,4 +145,7 @@ anything = "at all"
 	if result.Status != lint.Pass {
 		t.Fatalf("expected lint to pass, got %v (%s)", result.Status, result.Details)
 	}
+	if result.LintMetadata.Name == "" {
+		t.Fatal("expected lint metadata to have a name, got empty")
+	}
 }