Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

x/net/webdav: test failure due to xml name space changes #9978

Closed
davecheney opened this issue Feb 24, 2015 · 3 comments
Closed

x/net/webdav: test failure due to xml name space changes #9978

davecheney opened this issue Feb 24, 2015 · 3 comments

Comments

@davecheney
Copy link
Contributor

The net/webdav tests are failing, I suspect because of a map ordering assumption in the test.

        xml_test.go:355: section 9.1.3: XML body
                got  "<?xml version=\"1.0\" encoding=\"UTF-8\"?><D:multistatus xmlns:D=\"DAV:\"><response xmlns=\"DAV:\"><href>http://example.com/foo</href><propstat><prop xmlns=\"\"><bigbox xmlns=\"http://ns.example.com/boxschema/\"><BoxType xmlns=\"http://ns.example.com/boxschema/\">Box type A</BoxType></bigbox><author xmlns=\"http://ns.example.com/boxschema/\"><Name xmlns=\"http://ns.example.com/boxschema/\">J.J. Johnson</Name></author></prop><status>HTTP/1.1 200 OK</status></propstat><propstat><prop xmlns=\"\"><DingALing xmlns=\"http://ns.example.com/boxschema/\"></DingALing><Random xmlns=\"http://ns.example.com/boxschema/\"></Random></prop><status>HTTP/1.1 403 Forbidden</status><responsedescription> The user does not have access to the DingALing property.</responsedescription></propstat></response><D:responsedescription> There has been an access violation error.</D:responsedescription></D:multistatus>"
                want "<?xml version=\"1.0\" encoding=\"UTF-8\"?><D:multistatus xmlns:D=\"DAV:\"><response xmlns=\"DAV:\"><href xmlns=\"DAV:\">http://example.com/foo</href><propstat xmlns=\"DAV:\"><prop><bigbox xmlns=\"http://ns.example.com/boxschema/\"><BoxType xmlns=\"http://ns.example.com/boxschema/\">Box type A</BoxType></bigbox><author xmlns=\"http://ns.example.com/boxschema/\"><Name xmlns=\"http://ns.example.com/boxschema/\">J.J. Johnson</Name></author></prop><status xmlns=\"DAV:\">HTTP/1.1 200 OK</status></propstat><propstat xmlns=\"DAV:\"><prop><DingALing xmlns=\"http://ns.example.com/boxschema/\"></DingALing><Random xmlns=\"http://ns.example.com/boxschema/\"></Random></prop><status xmlns=\"DAV:\">HTTP/1.1 403 Forbidden</status><responsedescription xmlns=\"DAV:\"> The user does not have access to the DingALing property.</responsedescription></propstat></response><D:responsedescription> There has been an access violation error.</D:responsedescription></D:multistatus>"
FAIL
FAIL    golang.org/x/net/webdav 0.426s

/cc @nigeltao

@davecheney davecheney changed the title x/net/webdav: test failure due to map ordering x/net/webdav: test failure due to xml name space changes Feb 24, 2015
@davecheney
Copy link
Contributor Author

oh, no, i was mistaken, this is because of @rogpeppe 's fix for #7535

@nigeltao
Copy link
Contributor

Please review https://go-review.googlesource.com/5714

@mikioh mikioh changed the title x/net/webdav: test failure due to xml name space changes webdav: test failure due to xml name space changes Feb 24, 2015
@mikioh
Copy link
Contributor

mikioh commented Mar 12, 2015

Closed by golang/net@6dc0abc.

@mikioh mikioh closed this as completed Mar 12, 2015
@mikioh mikioh changed the title webdav: test failure due to xml name space changes x/net/webdav: test failure due to xml name space changes Jul 30, 2015
@mikioh mikioh modified the milestone: Unreleased Jul 30, 2015
@golang golang locked and limited conversation to collaborators Aug 5, 2016
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

4 participants