[issue5609] Create Unit Tests for nturl2path module
Roundup Robot added the comment: New changeset 5f8fe382f9db by Senthil Kumaran in branch '3.3': #5609 - test_urllib coverage for url2pathname and pathname2url. Patch http://hg.python.org/cpython/rev/5f8fe382f9db New changeset 7b3f1c6a67d9 by Senthil Kumaran in branch 'default': merge from 3.3 http://hg.python.org/cpython/rev/7b3f1c6a67d9 -- nosy: +python-dev ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue5609 ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue5609] Create Unit Tests for nturl2path module
Changes by Senthil Kumaran sent...@uthcode.com: -- resolution: - fixed stage: patch review - committed/rejected status: open - closed type: - behavior ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue5609 ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue5609] Create Unit Tests for nturl2path module
Thomas Fenzl added the comment: The authoritative implementation of this functionality would be Microsoft's PathCreateFromUrl (http://msdn.microsoft.com/en-us/library/windows/desktop/bb773581%28v=vs.85%29.aspx) and UrlCreateFromPath (http://msdn.microsoft.com/en-us/library/windows/desktop/bb773773%28v=vs.85%29.aspx), right? If so I'll run a few more complex conversions and round-trips through those and add the tests -- ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue5609 ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue5609] Create Unit Tests for nturl2path module
Senthil Kumaran added the comment: Thanks for the patch and test coverage. LGTM. I shall verify further and commit it. -- assignee: - orsenthil nosy: +orsenthil ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue5609 ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue5609] Create Unit Tests for nturl2path module
Thomas Fenzl added the comment: The tests work on linux, so I removed the check for nt. Also I adapted them to the latest implementation using ':' instead of '|' as drive letter separator. -- keywords: +patch nosy: +Thomas Fenzl Added file: http://bugs.python.org/file29516/test_urllib.diff ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue5609 ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue5609] Create Unit Tests for nturl2path module
Mark Lawrence breamore...@yahoo.co.uk added the comment: Maksim could you please respond to the comments on Rietveld, thanks. -- nosy: +BreamoreBoy ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue5609 ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue5609] Create Unit Tests for nturl2path module
Changes by Ezio Melotti ezio.melo...@gmail.com: -- nosy: +ezio.melotti priority: - normal stage: - patch review ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue5609 ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue5609] Create Unit Tests for nturl2path module
New submission from Maksim Kozyarchuk maksim_kozyarc...@yahoo.com: Added Unit Tests for nturl2path module. http://codereview.appspot.com/32072/show -- components: Tests messages: 84590 nosy: Kozyarchuk severity: normal status: open title: Create Unit Tests for nturl2path module versions: Python 3.1 ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue5609 ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue5609] Create Unit Tests for nturl2path module
Maksim Kozyarchuk maksim_kozyarc...@yahoo.com added the comment: Patch is available on Appshot, please review/approve. http://codereview.appspot.com/32072/show -- ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue5609 ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com