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

gtr: Refactor ReportBuilder

Browse Source
This commit is contained in:
Joël Stemmer 2019-10-06 23:18:41 +01:00
parent 3f9d5b62db
commit 52a5393fd3
1 changed files with 10 additions and 12 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

22
pkg/gtr/builder.go
View File

@ -30,6 +30,13 @@ func NewReportBuilder(packageName string) *ReportBuilder {
} }
} }
func (b *ReportBuilder) newId() int {
id := b.nextId
b.lastId = id
b.nextId += 1
return id
}
func (b *ReportBuilder) flush() { func (b *ReportBuilder) flush() {
if len(b.tests) > 0 { if len(b.tests) > 0 {
b.CreatePackage(b.packageName, 0) b.CreatePackage(b.packageName, 0)
@ -42,11 +49,7 @@ func (b *ReportBuilder) Build() Report {
} }
func (b *ReportBuilder) CreateTest(name string) { func (b *ReportBuilder) CreateTest(name string) {
id := b.nextId b.tests[b.newId()] = Test{Name: name}
b.lastId = id
b.nextId += 1
b.tests[id] = Test{Name: name}
} }
func (b *ReportBuilder) EndTest(name, result string, duration time.Duration) { func (b *ReportBuilder) EndTest(name, result string, duration time.Duration) {
@ -60,11 +63,7 @@ func (b *ReportBuilder) EndTest(name, result string, duration time.Duration) {
} }
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) {
id := b.nextId b.benchmarks[b.newId()] = Benchmark{
b.lastId = id
b.nextId += 1
b.benchmarks[id] = Benchmark{
Name: name, Name: name,
Result: Pass, Result: Pass,
Iterations: iterations, Iterations: iterations,
@ -76,10 +75,9 @@ func (b *ReportBuilder) Benchmark(name string, iterations int64, nsPerOp, mbPerS
} }
func (b *ReportBuilder) CreatePackage(name string, duration time.Duration) { func (b *ReportBuilder) CreatePackage(name string, duration time.Duration) {
// Collect tests and benchmarks for this package, maintaining insertion order.
var tests []Test var tests []Test
var benchmarks []Benchmark var benchmarks []Benchmark
// Iterate by id to maintain original test order
for id := 1; id < b.nextId; id++ { for id := 1; id < b.nextId; id++ {
if t, ok := b.tests[id]; ok { if t, ok := b.tests[id]; ok {
tests = append(tests, t) tests = append(tests, t)