Skip to content

Commit 23f5a01

Browse files
committed
Clean up reporting
Signed-off-by: Hamish Hutchings <hamish@drybrough.nl>
1 parent 3467370 commit 23f5a01

File tree

1 file changed

+10
-2
lines changed

1 file changed

+10
-2
lines changed

output/standard.go

Lines changed: 10 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -51,6 +51,7 @@ func (s *Standard) Output(results []CheckResult) error {
5151

5252
var totalFailures int
5353
var totalExceptions int
54+
var totalExclusions int
5455
var totalWarnings int
5556
var totalSuccesses int
5657
var totalSkipped int
@@ -96,11 +97,12 @@ func (s *Standard) Output(results []CheckResult) error {
9697
totalFailures += len(result.Failures)
9798
totalExceptions += len(result.Exceptions)
9899
totalWarnings += len(result.Warnings)
100+
totalExclusions += len(result.Excludes)
99101
totalSkipped += len(result.Skipped)
100102
totalSuccesses += result.Successes
101103
}
102104

103-
totalTests := totalFailures + totalExceptions + totalWarnings + totalSuccesses + totalSkipped
105+
totalTests := totalFailures + totalExceptions + totalWarnings + totalSuccesses + totalExclusions + totalSkipped
104106

105107
var pluralSuffixTests string
106108
if totalTests != 1 {
@@ -122,12 +124,18 @@ func (s *Standard) Output(results []CheckResult) error {
122124
pluralSuffixExceptions = "s"
123125
}
124126

125-
outputText := fmt.Sprintf("%v test%s, %v passed, %v warning%s, %v failure%s, %v exception%s",
127+
var pluralSuffixExclusions string
128+
if totalExclusions != 1 {
129+
pluralSuffixExclusions = "s"
130+
}
131+
132+
outputText := fmt.Sprintf("%v test%s, %v passed, %v warning%s, %v failure%s, %v exception%s, %v exclusion%s",
126133
totalTests, pluralSuffixTests,
127134
totalSuccesses,
128135
totalWarnings, pluralSuffixWarnings,
129136
totalFailures, pluralSuffixFailures,
130137
totalExceptions, pluralSuffixExceptions,
138+
totalExclusions, pluralSuffixExclusions,
131139
)
132140

133141
if s.ShowSkipped {

0 commit comments

Comments
 (0)