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

internal: move gotest.Options creation into separate function

Browse Source 
The gotest and gojson parsers use the same options (for now). To avoid
duplication the list of parser options is created in a separate
function.
This commit is contained in:
Joël Stemmer 2022-05-21 22:44:38 +01:00
parent dcbbd9fb22
commit b4847b2e36
1 changed files with 9 additions and 8 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

17
internal/gojunitreport/go-junit-report.go
View File

@ -38,15 +38,9 @@ func (c Config) Run(input io.Reader, output io.Writer) (*gtr.Report, error) {
var p parser
switch c.Parser {
case "gotest":
p = gotest.NewParser(
gotest.PackageName(c.PackageName),
gotest.TimestampFunc(c.timestampFunc),
)
p = gotest.NewParser(c.gotestOptions()...)
case "gojson":
p = gotest.NewJSONParser(
gotest.PackageName(c.PackageName),
gotest.TimestampFunc(c.timestampFunc),
)
p = gotest.NewJSONParser(c.gotestOptions()...)
default:
return nil, fmt.Errorf("invalid parser: %s", c.Parser)
}
@ -98,3 +92,10 @@ func (c Config) writeXML(w io.Writer, report gtr.Report) error {
_, err := fmt.Fprintf(w, "\n")
return err
}
func (c Config) gotestOptions() []gotest.Option {
return []gotest.Option{
gotest.PackageName(c.PackageName),
gotest.TimestampFunc(c.timestampFunc),
}
}