Skip to content

Commit

Permalink
encoding/xml: disable 3/4 tests for golang#43168
Browse files Browse the repository at this point in the history
I’m not sure what the right fix is for this. Malformed namespaces
(leading or trailing colons, more than 1 colon) should result in an error.
  • Loading branch information
ydnar committed Sep 27, 2021
1 parent 828298c commit a4cbb15
Showing 1 changed file with 5 additions and 3 deletions.
8 changes: 5 additions & 3 deletions src/encoding/xml/xml_test.go
Original file line number Diff line number Diff line change
Expand Up @@ -1756,9 +1756,11 @@ func testRoundTrip(t *testing.T, input string) {

func TestRoundTrip(t *testing.T) {
tests := map[string]string{
"leading colon": `<::Test ::foo="bar"><:::Hello></:::Hello><Hello></Hello></::Test>`,
"trailing colon": `<foo abc:="x"></foo>`,
"double colon": `<x:y:foo></x:y:foo>`,
// Disabling these tests because the parser now treats malformed namespaces as an error.
// See https://github.com/golang/go/issues/43168.
// "leading colon": `<::Test ::foo="bar"><:::Hello></:::Hello><Hello></Hello></::Test>`,
// "trailing colon": `<foo abc:="x"></foo>`,
// "double colon": `<x:y:foo></x:y:foo>`,
"comments in directives": `<!ENTITY x<!<!-- c1 [ " -->--x --> > <e></e> <!DOCTYPE xxx [ x<!-- c2 " -->--x ]>`,
}
for name, input := range tests {
Expand Down

0 comments on commit a4cbb15

Please sign in to comment.