Only use the new test results when writing the output summary (#89)

rwx-research · Nov 25, 2024 · db3e97a · db3e97a
1 parent aad1913
commit db3e97a
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 2 deletions.
diff --git a/internal/cli/run.go b/internal/cli/run.go
@@ -809,7 +809,7 @@ func (s Service) reportTestResults(
 	}
 
 	if cfg.PrintSummary {
-		if err := reporting.WriteTextSummary(os.Stdout, testResults, reportingConfiguration); err != nil {
+		if err := reporting.WriteTextSummary(os.Stdout, newlyExecutedTestResults, reportingConfiguration); err != nil {
 			s.Log.Warnf("Unable to write text summary to stdout: %s", err.Error())
 		} else {
 			// Append an empty line to make output more readable

diff --git a/internal/reporting/text.go b/internal/reporting/text.go
@@ -27,7 +27,12 @@ func WriteTextSummary(file fs.File, testResults v1.TestResults, _ Configuration)
 		statuses[test.Attempt.Status.Kind] = tests
 	}
 
-	_, err := file.Write([]byte(fmt.Sprintf("\nCaptain detected a total of %d tests.\n", totalTests)))
+	pluralizeTests := "tests"
+	if totalTests == 1 {
+		pluralizeTests = "test"
+	}
+
+	_, err := file.Write([]byte(fmt.Sprintf("\nCaptain detected a total of %d %s.\n", totalTests, pluralizeTests)))
 	if err != nil {
 		return errors.WithStack(err)
 	}