remove GoTestSummary

dev-tools/mage/gotest.go

@@ -27,11 +27,9 @@ import (
 	"os/exec"
 	"path"
 	"path/filepath"
-	"sort"
 	"strings"
 	"time"
 
-	"github.com/jstemmer/go-junit-report/parser"
 	"github.com/magefile/mage/mg"
 	"github.com/magefile/mage/sh"
 	"github.com/pkg/errors"
@@ -323,114 +321,6 @@ func makeCommand(ctx context.Context, env map[string]string, cmd string, args ..
 	return c
 }
 
-// GoTestSummary is a summary of test results.
-type GoTestSummary struct {
-	*parser.Report               // Report generated by parsing test output.
-	Pass           int           // Number of passing tests.
-	Fail           int           // Number of failed tests.
-	Skip           int           // Number of skipped tests.
-	Packages       int           // Number of packages tested.
-	Duration       time.Duration // Total go test running duration.
-	Files          map[string]string
-}
-
-// NewGoTestSummary builds a new GoTestSummary. It returns an error if it cannot
-// resolve the absolute paths to the given files.
-func NewGoTestSummary(d time.Duration, r *parser.Report, outputFiles map[string]string) (*GoTestSummary, error) {
-	files := map[string]string{}
-	for name, file := range outputFiles {
-		if file == "" {
-			continue
-		}
-		absFile, err := filepath.Abs(file)
-		if err != nil {
-			return nil, errors.Wrapf(err, "failed resolving absolute path for %v", file)
-		}
-		files[name+":"] = absFile
-	}
-
-	summary := &GoTestSummary{
-		Report:   r,
-		Duration: d,
-		Packages: len(r.Packages),
-		Files:    files,
-	}
-
-	for _, pkg := range r.Packages {
-		for _, t := range pkg.Tests {
-			switch t.Result {
-			case parser.PASS:
-				summary.Pass++
-			case parser.FAIL:
-				summary.Fail++
-			case parser.SKIP:
-				summary.Skip++
-			default:
-				return nil, errors.Errorf("Unknown test result value: %v", t.Result)
-			}
-		}
-	}
-
-	return summary, nil
-}
-
-// Failures returns a string containing the list of failed test cases and their
-// output.
-func (s *GoTestSummary) Failures() string {
-	b := new(strings.Builder)
-
-	if s.Fail > 0 {
-		fmt.Fprintln(b, "FAILURES:")
-		for _, pkg := range s.Report.Packages {
-			for _, t := range pkg.Tests {
-				if t.Result != parser.FAIL {
-					continue
-				}
-				fmt.Fprintln(b, "Package:", pkg.Name)
-				fmt.Fprintln(b, "Test:   ", t.Name)
-				for _, line := range t.Output {
-					if strings.TrimSpace(line) != "" {
-						fmt.Fprintln(b, line)
-					}
-				}
-				fmt.Fprintln(b, "----")
-			}
-		}
-	}
-
-	return strings.TrimRight(b.String(), "\n")
-}
-
-// String returns a summary of the testing results (number of fail/pass/skip,
-// test duration, number packages, output files).
-func (s *GoTestSummary) String() string {
-	b := new(strings.Builder)
-
-	fmt.Fprintln(b, "SUMMARY:")
-	fmt.Fprintln(b, "  Fail:    ", s.Fail)
-	fmt.Fprintln(b, "  Skip:    ", s.Skip)
-	fmt.Fprintln(b, "  Pass:    ", s.Pass)
-	fmt.Fprintln(b, "  Packages:", len(s.Report.Packages))
-	fmt.Fprintln(b, "  Duration:", s.Duration)
-
-	// Sort the list of files and compute the column width.
-	var names []string
-	var nameWidth int
-	for name := range s.Files {
-		if len(name) > nameWidth {
-			nameWidth = len(name)
-		}
-		names = append(names, name)
-	}
-	sort.Strings(names)
-
-	for _, name := range names {
-		fmt.Fprintf(b, "  %-*s %s\n", nameWidth, name, s.Files[name])
-	}
-
-	return strings.TrimRight(b.String(), "\n")
-}
-
 // BuildSystemTestBinary runs BuildSystemTestGoBinary with default values.
 func BuildSystemTestBinary() error {
 	return BuildSystemTestGoBinary(DefaultTestBinaryArgs())