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: Reset active test when encountering a status line

Browse Source 
This is to ensure that we don't append output lines that follow after a
status line to the last active test.
This commit is contained in:
Joël Stemmer 2019-10-07 00:24:42 +01:00
parent f7ae0905a2
commit 5007397e33
2 changed files with 7 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
pkg/gtr/builder.go
View File

@ -76,6 +76,10 @@ func (b *ReportBuilder) EndTest(name, result string, duration time.Duration) {
b.tests[id] = t b.tests[id] = t
} }
func (b *ReportBuilder) End() {
b.lastId = 0
}
func (b *ReportBuilder) Benchmark(name string, iterations int64, nsPerOp, mbPerSec float64, bytesPerOp, allocsPerOp int64) { func (b *ReportBuilder) Benchmark(name string, iterations int64, nsPerOp, mbPerSec float64, bytesPerOp, allocsPerOp int64) {
b.benchmarks[b.newId()] = Benchmark{ b.benchmarks[b.newId()] = Benchmark{
Name: name, Name: name,

4
pkg/gtr/gtr.go
View File

@ -84,7 +84,8 @@ func FromEvents(events []Event, packageName string) Report {
report.EndTest(ev.Name, ev.Result, ev.Duration) report.EndTest(ev.Name, ev.Result, ev.Duration)
case "benchmark": case "benchmark":
report.Benchmark(ev.Name, ev.Iterations, ev.NsPerOp, ev.MBPerSec, ev.BytesPerOp, ev.AllocsPerOp) report.Benchmark(ev.Name, ev.Iterations, ev.NsPerOp, ev.MBPerSec, ev.BytesPerOp, ev.AllocsPerOp)
case "status": // ignore for now case "status":
report.End()
case "summary": case "summary":
report.CreatePackage(ev.Name, ev.Result, ev.Duration, ev.Data) report.CreatePackage(ev.Name, ev.Result, ev.Duration, ev.Data)
case "coverage": case "coverage":
@ -198,6 +199,7 @@ func JUnit(report Report) junit.Testsuites {
func formatOutput(output []string) string { func formatOutput(output []string) string {
var lines []string var lines []string
for _, line := range output { for _, line := range output {
// TODO: should this change depending on subtest level?
line = strings.TrimPrefix(line, " ") line = strings.TrimPrefix(line, " ")
line = strings.TrimPrefix(line, "\t") line = strings.TrimPrefix(line, "\t")
lines = append(lines, line) lines = append(lines, line)