Skip to content

Commit ab40f17

Browse files
committed
build/util: remove unnecessary Classname attribute
This was added back in, I think accidentally as a result of a bad copy-paste, in `8e6b3935749df40aac835aad15c2b8337426f9f4`. This restores the previous behavior. Note that the comment talks about how `Classname` is missing, so we expect it not to be there. Release justification: Non-production code changes Part of: DEVINF-1658 Epic: DEVINF-1582 Release note: none
1 parent 2f2800e commit ab40f17

File tree

1 file changed

+5
-6
lines changed

1 file changed

+5
-6
lines changed

pkg/build/util/util.go

Lines changed: 5 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -45,12 +45,11 @@ type testCase struct {
4545
// this isn't Java so there isn't a classname) and excluding it causes
4646
// the TeamCity UI to display the same data in a slightly more coherent
4747
// and usable way.
48-
Classname string `xml:"classname,attr"`
49-
Name string `xml:"name,attr"`
50-
Time string `xml:"time,attr"`
51-
Failure *XMLMessage `xml:"failure,omitempty"`
52-
Error *XMLMessage `xml:"error,omitempty"`
53-
Skipped *XMLMessage `xml:"skipped,omitempty"`
48+
Name string `xml:"name,attr"`
49+
Time string `xml:"time,attr"`
50+
Failure *XMLMessage `xml:"failure,omitempty"`
51+
Error *XMLMessage `xml:"error,omitempty"`
52+
Skipped *XMLMessage `xml:"skipped,omitempty"`
5453
}
5554

5655
// XMLMessage is a catch-all structure containing details about a test

0 commit comments

Comments
 (0)