Merge lp:~stefanor/wadllib/datetime-924240 into lp:wadllib
Proposed by
Stefano Rivera
Status: | Merged |
---|---|
Approved by: | Richard Harding |
Approved revision: | 28 |
Merged at revision: | 28 |
Proposed branch: | lp:~stefanor/wadllib/datetime-924240 |
Merge into: | lp:wadllib |
Diff against target: |
19 lines (+2/-7) 1 file modified
src/wadllib/application.py (+2/-7) |
To merge this branch: | bzr merge lp:~stefanor/wadllib/datetime-924240 |
Related bugs: |
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
j.c.sackett (community) | Approve | ||
Richard Harding | code* | Approve | |
Review via email: mp+97099@code.launchpad.net |
Description of the change
I don't understand the purpose of the roundtrippnig through ET in _from_string. More to the point, it claims to track namespaces, but clearly doesn't correctly, causing LP: #924240.
My not understanding, is entirely possibly due to me not knowing this code at all, but the change fixes the issue for me, and the tests still pass under python 2 & 3.
To post a comment you must log in.
So I'm working on getting a test case for this and so I understand how this stuff works. I've got the start of a test case, but it passes with both the old code and your update.
So I'm wondering if maybe the test data in the wadllib isn't up to date. The JSON file that houses the tested data has date strings in the format
"date_created": "2005-06- 06T08:59: 51.561685+ 00:00"
And I wonder if that's not true for what you're getting.
Can you please get a copy of the json you're testing against so I can try to create a test that breaks as you're seeing it to make sure this doesn't become an issue in the future.
This is a paste of the test I've started to try to fire up: /pastebin. canonical. com/62637/
https:/