feat: add extended-junit-xml format

golangci · Aug 16, 2024 · 977b49c · 977b49c
1 parent adbdfdb
commit 977b49c
Show file tree

Hide file tree

Showing 6 changed files with 72 additions and 13 deletions.
diff --git a/.golangci.next.reference.yml b/.golangci.next.reference.yml
@@ -70,6 +70,7 @@ output:
   # - `checkstyle`
   # - `code-climate`
   # - `junit-xml`
+  # - `extended-junit-xml`
   # - `github-actions`
   # - `teamcity`
   # - `sarif`

diff --git a/jsonschema/golangci.next.jsonschema.json b/jsonschema/golangci.next.jsonschema.json
@@ -514,6 +514,7 @@
                   "checkstyle",
                   "code-climate",
                   "junit-xml",
+                  "extended-junit-xml",
                   "github-actions",
                   "teamcity",
                   "sarif"

diff --git a/pkg/config/output.go b/pkg/config/output.go
@@ -17,6 +17,7 @@ const (
 	OutFormatCodeClimate       = "code-climate"
 	OutFormatHTML              = "html"
 	OutFormatJunitXML          = "junit-xml"
+	OutFormatExtendedJunitXML  = "extended-junit-xml"
 	OutFormatGithubActions     = "github-actions" // Deprecated
 	OutFormatTeamCity          = "teamcity"
 	OutFormatSarif             = "sarif"
@@ -32,6 +33,7 @@ var AllOutputFormats = []string{
 	OutFormatCodeClimate,
 	OutFormatHTML,
 	OutFormatJunitXML,
+	OutFormatExtendedJunitXML,
 	OutFormatGithubActions,
 	OutFormatTeamCity,
 	OutFormatSarif,

diff --git a/pkg/printers/junitxml.go b/pkg/printers/junitxml.go
@@ -30,6 +30,8 @@ type testCaseXML struct {
 	Name      string     `xml:"name,attr"`
 	ClassName string     `xml:"classname,attr"`
 	Failure   failureXML `xml:"failure"`
+	File      string     `xml:"file,attr,omitempty"`
+	Line      int        `xml:"line,attr,omitempty"`
 }
 
 type failureXML struct {
@@ -39,11 +41,15 @@ type failureXML struct {
 }
 
 type JunitXML struct {
-	w io.Writer
+	extended bool
+	w        io.Writer
 }
 
-func NewJunitXML(w io.Writer) *JunitXML {
-	return &JunitXML{w: w}
+func NewJunitXML(extended bool, w io.Writer) *JunitXML {
+	return &JunitXML{
+		extended: extended,
+		w:        w,
+	}
 }
 
 func (p JunitXML) Print(issues []result.Issue) error {
@@ -68,6 +74,11 @@ func (p JunitXML) Print(issues []result.Issue) error {
 			},
 		}
 
+		if p.extended {
+			tc.File = i.Pos.Filename
+			tc.Line = i.Pos.Line
+		}
+
 		testSuite.TestCases = append(testSuite.TestCases, tc)
 		suites[suiteName] = testSuite
 	}

diff --git a/pkg/printers/junitxml_test.go b/pkg/printers/junitxml_test.go
@@ -42,13 +42,14 @@ func TestJunitXML_Print(t *testing.T) {
 		},
 	}
 
-	buf := new(bytes.Buffer)
-	printer := NewJunitXML(buf)
-
-	err := printer.Print(issues)
-	require.NoError(t, err)
-
-	expected := `<testsuites>
+	testCases := []struct {
+		desc     string
+		extended bool
+		expected string
+	}{
+		{
+			desc: "basic",
+			expected: `<testsuites>
   <testsuite name="path/to/filea.go" tests="1" errors="0" failures="1">
     <testcase name="linter-a" classname="path/to/filea.go:10:4">
       <failure message="path/to/filea.go:10:4: some issue" type="warning"><![CDATA[warning: some issue
@@ -69,7 +70,48 @@ Details: func foo() {
 }]]></failure>
     </testcase>
   </testsuite>
-</testsuites>`
+</testsuites>`,
+		},
+		{
+			desc:     "extended/complete",
+			extended: true,
+			expected: `<testsuites>
+  <testsuite name="path/to/filea.go" tests="1" errors="0" failures="1">
+    <testcase name="linter-a" classname="path/to/filea.go:10:4" file="path/to/filea.go" line="10">
+      <failure message="path/to/filea.go:10:4: some issue" type="warning"><![CDATA[warning: some issue
+Category: linter-a
+File: path/to/filea.go
+Line: 10
+Details: ]]></failure>
+    </testcase>
+  </testsuite>
+  <testsuite name="path/to/fileb.go" tests="1" errors="0" failures="1">
+    <testcase name="linter-b" classname="path/to/fileb.go:300:9" file="path/to/fileb.go" line="300">
+      <failure message="path/to/fileb.go:300:9: another issue" type="error"><![CDATA[error: another issue
+Category: linter-b
+File: path/to/fileb.go
+Line: 300
+Details: func foo() {
+	fmt.Println("bar")
+}]]></failure>
+    </testcase>
+  </testsuite>
+</testsuites>`,
+		},
+	}
+
+	for _, test := range testCases {
+		test := test
+		t.Run(test.desc, func(t *testing.T) {
+			t.Parallel()
 
-	assert.Equal(t, expected, buf.String())
+			buf := new(bytes.Buffer)
+			printer := NewJunitXML(test.extended, buf)
+
+			err := printer.Print(issues)
+			require.NoError(t, err)
+
+			assert.Equal(t, test.expected, buf.String())
+		})
+	}
 }
diff --git a/pkg/printers/printer.go b/pkg/printers/printer.go
@@ -130,7 +130,9 @@ func (c *Printer) createPrinter(format string, w io.Writer) (issuePrinter, error
 	case config.OutFormatHTML:
 		p = NewHTML(w)
 	case config.OutFormatJunitXML:
-		p = NewJunitXML(w)
+		p = NewJunitXML(false, w)
+	case config.OutFormatExtendedJunitXML:
+		p = NewJunitXML(true, w)
 	case config.OutFormatGithubActions:
 		p = NewGitHubAction(w)
 	case config.OutFormatTeamCity: