Keep empty packages by default in testdox format

And only hide Fuzz test cases, always report the top level test
function.

Author: dnephin

README.md

@@ -48,6 +48,7 @@ Commonly used formats (see `--help` for a full list):
  * `dots` - print a character for each test.
  * `pkgname` (default) - print a line for each package.
  * `testname` - print a line for each test and package.
+ * `testdox` - print a sentence for each test using [gotestdox](https://github.com/bitfield/gotestdox).
  * `standard-quiet` - the standard `go test` format.
  * `standard-verbose` - the standard `go test -v` format.
 

cmd/main.go

@@ -139,8 +139,8 @@ Formats:
     dots-v2                  experimental dots format, one package per line
     pkgname                  print a line for each package
     pkgname-and-test-fails   print a line for each package and failed test output
-    testdox                  TestDox format (print a sentence for each test)
     testname                 print a line for each test and package
+    testdox                  print a sentence for each test using gotestdox
     github-actions           testname format with github actions log grouping
     standard-quiet           standard go test format
     standard-verbose         standard go test -v format

cmd/testdata/gotestsum-help-text

@@ -35,8 +35,8 @@ Formats:
     dots-v2                  experimental dots format, one package per line
     pkgname                  print a line for each package
     pkgname-and-test-fails   print a line for each package and failed test output
-    testdox                  TestDox format (print a sentence for each test)
     testname                 print a line for each test and package
+    testdox                  print a sentence for each test using gotestdox
     github-actions           testname format with github actions log grouping
     standard-quiet           standard go test format
     standard-verbose         standard go test -v format

testjson/format.go

@@ -103,8 +103,7 @@ func testDoxFormat(out io.Writer, opts FormatOptions) EventFormatter {
 			if !event.Action.IsTerminal() {
 				return nil
 			}
-			if len(results[event.Package]) == 0 {
-				// No testdox for you
+			if opts.HideEmptyPackages && len(results[event.Package]) == 0 {
 				return nil
 			}
 			fmt.Fprintf(buf, "%s:\n", event.Package)
@@ -123,7 +122,7 @@ func testDoxFormat(out io.Writer, opts FormatOptions) EventFormatter {
 		case event.Action.IsTerminal():
 			// Fuzz test cases tend not to have interesting names,
 			// so only report these if they're failures
-			if strings.HasPrefix(event.Test, "Fuzz") && event.Action == ActionPass {
+			if isFuzzCase(event) {
 				return nil
 			}
 			results[event.Package] = append(results[event.Package], Result{
@@ -135,6 +134,12 @@ func testDoxFormat(out io.Writer, opts FormatOptions) EventFormatter {
 	})
 }
 
+func isFuzzCase(event TestEvent) bool {
+	return strings.HasPrefix(event.Test, "Fuzz") &&
+		event.Action == ActionPass &&
+		TestName(event.Test).IsSubTest()
+}
+
 func testNameFormat(out io.Writer) EventFormatter {
 	buf := bufio.NewWriter(out)
 	// nolint:errcheck

testjson/testdata/format/testdox-coverage.out

@@ -1,3 +1,5 @@
+gotest.tools/gotestsum/testjson/internal/badmain:
+
 gotest.tools/gotestsum/testjson/internal/good:
  ✓ Nested success (0.00s)
  ✓ Nested success a (0.00s)

testjson/testdata/format/testdox-shuffle.out

@@ -1,3 +1,5 @@
+gotest.tools/gotestsum/testjson/internal/badmain:
+
 gotest.tools/gotestsum/testjson/internal/good:
  ✓ Nested success (0.00s)
  ✓ Nested success a (0.00s)

testjson/testdata/format/testdox.out

@@ -1,3 +1,7 @@
+gotest.tools/gotestsum/testjson/internal/badmain:
+
+gotest.tools/gotestsum/testjson/internal/empty:
+
 gotest.tools/gotestsum/testjson/internal/good:
  ✓ Nested success (0.00s)
  ✓ Nested success a (0.00s)