Enhance test results summary by adding rerun details and modifying short results table headers

Author: lukaszcl

tools/flakeguard/cmd/run.go

@@ -204,6 +204,10 @@ var RunTestsCmd = &cobra.Command{
 				flushSummaryAndExit(ErrorExitCode)
 			}
 
+			fmt.Fprint(&summaryBuffer, "\nFailed Tests That Were Rerun:\n\n")
+			reports.PrintTestResultsTable(&summaryBuffer, rerunResults, false, false, true)
+			fmt.Fprintln(&summaryBuffer)
+
 			// Save the rerun test report to file
 			if rerunResultsPath != "" && len(rerunResults) > 0 {
 				if err := reports.SaveTestResultsToFile(rerunResults, rerunResultsPath); err != nil {
@@ -219,10 +223,6 @@ var RunTestsCmd = &cobra.Command{
 			})
 
 			if len(failedAfterRerun) > 0 {
-				fmt.Fprint(&summaryBuffer, "\nTests That Have 0 Success Rate After Reruns:\n\n")
-				reports.PrintTestResultsTable(&summaryBuffer, failedAfterRerun, false, false, true)
-				fmt.Fprintln(&summaryBuffer)
-
 				fmt.Fprint(&summaryBuffer, "\nLogs:\n\n")
 				err := rerunReport.PrintGotestsumOutput(&summaryBuffer, "pkgname")
 				if err != nil {

tools/flakeguard/reports/presentation.go

@@ -82,8 +82,7 @@ func generateShortTestResultsTable(
 ) [][]string {
 	p := message.NewPrinter(language.English)
 
-	// Define the columns you want
-	headers := []string{"Name", "Path", "Runs", "Failures", "Code Owners"}
+	headers := []string{"Name", "Ever Passed", "Runs", "Successes", "Code Owners", "Path"}
 
 	// Optionally format the headers for Markdown
 	if markdown {
@@ -101,18 +100,25 @@ func generateShortTestResultsTable(
 			continue
 		}
 
+		// Determine whether the test has passed at least once
+		passed := "NO"
+		if r.Successes > 0 {
+			passed = "YES"
+		}
+
 		// Format the Code Owners
 		owners := "Unknown"
 		if len(r.CodeOwners) > 0 {
 			owners = strings.Join(r.CodeOwners, ", ")
 		}
 
 		row := []string{
-			r.TestName,
-			r.TestPath,
-			p.Sprintf("%d", r.Runs),
-			p.Sprintf("%d", r.Failures),
-			owners,
+			r.TestName,                   // "Name"
+			passed,                       // "Passed"
+			p.Sprintf("%d", r.Runs),      // "Runs"
+			p.Sprintf("%d", r.Successes), // "Passes"
+			owners,                       // "Code Owners"
+			r.TestPath,                   // "Path"
 		}
 
 		table = append(table, row)