fix: don't add findings to result if none were generated

Signed-off-by: George Vauter <gvauter@redhat.com>

Author: gvauter

framework/reporter.go

@@ -261,18 +261,22 @@ func (r *Reporter) GenerateAssessmentResults(ctx context.Context, planHref strin
 	}
 	reviewedConrols := r.findControls(*implementationSettings)
 
-	oscalResults := []oscalTypes.Result{
-		{
-			UUID:             uuid.NewUUID(),
-			Title:            "Automated Assessment Result",
-			Description:      "Assessment Results Automatically Genererated from PVP Results",
-			Start:            time.Now(),
-			ReviewedControls: reviewedConrols,
-			Observations:     &oscalObservations,
-			Findings:         &oscalFindings,
-		},
+	oscalResult := oscalTypes.Result{
+		UUID:             uuid.NewUUID(),
+		Title:            "Automated Assessment Result",
+		Description:      "Assessment Results Automatically Genererated from PVP Results",
+		Start:            time.Now(),
+		ReviewedControls: reviewedConrols,
+		Observations:     &oscalObservations,
+	}
+
+	if len(oscalFindings) > 0 {
+		oscalResult.Findings = &oscalFindings
+	}
+
+	assessmentResults.Results = []oscalTypes.Result{
+		oscalResult,
 	}
-	assessmentResults.Results = oscalResults
 
 	return assessmentResults, nil