nhyatt/go-junit-report
1
0
Fork 0
mirror of https://github.com/jstemmer/go-junit-report.git synced 2025-04-05 05:00:15 -05:00
Code Issues Projects Releases Wiki Activity

gtr: Handle failed summary when no tests failed

Browse Source
This commit is contained in:
Joël Stemmer 2019-10-07 01:17:53 +01:00
parent 4fdb6ca564
commit 506a968ade
1 changed files with 33 additions and 16 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

49
pkg/gtr/builder.go
View File

@ -23,6 +23,7 @@ type ReportBuilder struct {
packageName string packageName string
} }
// TODO: move gtr.FromEvents to ReportBuilder so we could have report builders for different parsers.
func NewReportBuilder(packageName string) *ReportBuilder { func NewReportBuilder(packageName string) *ReportBuilder {
return &ReportBuilder{ return &ReportBuilder{
tests: make(map[int]Test), tests: make(map[int]Test),
@ -97,6 +98,11 @@ func (b *ReportBuilder) CreateBuildError(packageName string) {
} }
func (b *ReportBuilder) CreatePackage(name, result string, duration time.Duration, data string) { func (b *ReportBuilder) CreatePackage(name, result string, duration time.Duration, data string) {
pkg := Package{
Name: name,
Duration: duration,
}
// Build errors are treated somewhat differently. Rather than having a // Build errors are treated somewhat differently. Rather than having a
// single package with all build errors collected so far, we only care // single package with all build errors collected so far, we only care
// about the build errors for this particular package. // about the build errors for this particular package.
@ -107,10 +113,9 @@ func (b *ReportBuilder) CreatePackage(name, result string, duration time.Duratio
} }
buildErr.Duration = duration buildErr.Duration = duration
buildErr.Cause = data buildErr.Cause = data
b.packages = append(b.packages, Package{ pkg.BuildError = buildErr
Name: name, b.packages = append(b.packages, pkg)
BuildError: buildErr,
})
delete(b.buildErrors, id) delete(b.buildErrors, id)
// TODO: reset state // TODO: reset state
// TODO: buildErrors shouldn't reset/use nextId/lastId, they're more like a global cache // TODO: buildErrors shouldn't reset/use nextId/lastId, they're more like a global cache
@ -121,10 +126,6 @@ func (b *ReportBuilder) CreatePackage(name, result string, duration time.Duratio
// If we've collected output, but there were no tests or benchmarks then // If we've collected output, but there were no tests or benchmarks then
// either there were no tests, or there was some other non-build error. // either there were no tests, or there was some other non-build error.
if len(b.output) > 0 && len(b.tests) == 0 && len(b.benchmarks) == 0 { if len(b.output) > 0 && len(b.tests) == 0 && len(b.benchmarks) == 0 {
pkg := Package{
Name: name,
Duration: duration,
}
if parseResult(result) == Fail { if parseResult(result) == Fail {
pkg.RunError = Error{ pkg.RunError = Error{
Name: name, Name: name,
@ -138,6 +139,16 @@ func (b *ReportBuilder) CreatePackage(name, result string, duration time.Duratio
return return
} }
// If the summary result says we failed, but there were no failing tests
// then something else must have failed.
if parseResult(result) == Fail && (len(b.tests) > 0 || len(b.benchmarks) > 0) && !b.containsFailingTest() {
pkg.RunError = Error{
Name: name,
Output: b.output,
}
b.output = nil
}
// Collect tests and benchmarks for this package, maintaining insertion order. // Collect tests and benchmarks for this package, maintaining insertion order.
var tests []Test var tests []Test
var benchmarks []Benchmark var benchmarks []Benchmark
@ -150,14 +161,11 @@ func (b *ReportBuilder) CreatePackage(name, result string, duration time.Duratio
} }
} }
b.packages = append(b.packages, Package{ pkg.Coverage = b.coverage
Name: name, pkg.Output = b.output
Duration: duration, pkg.Tests = tests
Coverage: b.coverage, pkg.Benchmarks = benchmarks
Output: b.output, b.packages = append(b.packages, pkg)
Tests: tests,
Benchmarks: benchmarks,
})
b.nextId = 1 b.nextId = 1
b.lastId = 0 b.lastId = 0
@ -218,6 +226,15 @@ func (b *ReportBuilder) findBenchmark(name string) int {
return -1 return -1
} }
func (b *ReportBuilder) containsFailingTest() bool {
for _, test := range b.tests {
if test.Result == Fail {
return true
}
}
return false
}
func parseResult(r string) Result { func parseResult(r string) Result {
switch r { switch r {
case "PASS": case "PASS":