From 3e15044962c2e3027d7cc799ac9b794c4372c826 Mon Sep 17 00:00:00 2001
From: John Mulligan <jmulligan@redhat.com>
Date: Wed, 20 May 2020 13:14:15 -0400
Subject: [PATCH] implements: add io.Writer argument when creating text report

This io.Writer will be the destination for writes from the report.
This better matches with what the JSON report was doing.

Signed-off-by: John Mulligan <jmulligan@redhat.com>
---
 .../implements/internal/implements/report.go  | 24 +++++++++++++------
 1 file changed, 17 insertions(+), 7 deletions(-)

diff --git a/contrib/implements/internal/implements/report.go b/contrib/implements/internal/implements/report.go
index bd80912..2645c15 100644
--- a/contrib/implements/internal/implements/report.go
+++ b/contrib/implements/internal/implements/report.go
@@ -2,6 +2,7 @@ package implements
 
 import (
 	"fmt"
+	"io"
 	"sort"
 	"strings"
 )
@@ -25,11 +26,20 @@ type Reporter interface {
 // TextReport implements a streaming plain-text output report.
 type TextReport struct {
 	o ReportOptions
+	// io destination
+	dest io.Writer
 }
 
 // NewTextReport creates a new TextReport.
-func NewTextReport(o ReportOptions) *TextReport {
-	return &TextReport{o}
+func NewTextReport(o ReportOptions, dest io.Writer) *TextReport {
+	return &TextReport{o, dest}
+}
+
+func (r *TextReport) printf(f string, v ...interface{}) {
+	_, err := fmt.Fprintf(r.dest, f, v...)
+	if err != nil {
+		panic(err.Error())
+	}
 }
 
 // Report on the given code inspector.
@@ -41,20 +51,20 @@ func (r *TextReport) Report(name string, ii *Inspector) error {
 
 	found := len(ii.found)
 	total := len(ii.expected)
-	fmt.Printf(
+	r.printf(
 		"%s functions covered: %d/%d : %v%%\n",
 		packageLabel,
 		found,
 		total,
 		(100*found)/total)
 	missing := total - found - ii.deprecatedMissing
-	fmt.Printf(
+	r.printf(
 		"%s functions missing: %d/%d : %v%%\n",
 		packageLabel,
 		missing,
 		total,
 		(100*missing)/total)
-	fmt.Printf(
+	r.printf(
 		"  (note missing count does not include deprecated functions in ceph)\n")
 
 	if !o.List {
@@ -78,7 +88,7 @@ func (r *TextReport) Report(name string, ii *Inspector) error {
 					tags = " (" + strings.TrimSpace(tags) + ")"
 				}
 			}
-			fmt.Printf("  Found: %s%s\n", cf.Name, tags)
+			r.printf("  Found: %s%s\n", cf.Name, tags)
 		}
 	}
 	for _, cf := range ii.expected {
@@ -89,7 +99,7 @@ func (r *TextReport) Report(name string, ii *Inspector) error {
 		if o.Annotate && cf.isDeprecated() {
 			d = " (deprecated)"
 		}
-		fmt.Printf("  Missing: %s%s\n", cf.Name, d)
+		r.printf("  Missing: %s%s\n", cf.Name, d)
 	}
 	return nil
 }