[issue24579] Additional tests for urllib module

2015-07-06 Thread Craig Northway
New submission from Craig Northway: Patch containing additional tests for urllib module to increase code coverage for url parsing. Current coverage: Lib/urllib/parse 51320261% Updated coverage: Lib/urllib/parse 51313973% --

[issue24579] Additional tests for urllib module

2015-07-06 Thread Craig Northway
Changes by Craig Northway craig.north...@gmail.com: -- keywords: +patch Added file: http://bugs.python.org/file39877/urllib.patch ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue24579 ___

[issue24579] Additional tests for urllib module

2015-07-06 Thread Martin Panter
Martin Panter added the comment: Have you seen the existing test cases in /Lib/test/test_urlparse.py? If not, I don’t blame you, because of the odd test file naming and structuring. :) But I suspect some of your tests may be redundant. But you might be in a good position to suggest some

[issue24579] Additional tests for urllib module

2015-07-06 Thread Craig Northway
Changes by Craig Northway craig.north...@gmail.com: -- versions: +Python 3.6 ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue24579 ___ ___

[issue24579] Additional tests for urllib module

2015-07-06 Thread Craig Northway
Craig Northway added the comment: Whoops, looks like there is definitely some redundancy. I didn't notice those tests at the time I wrote these (PyCon AU last year) and while I have been grappling with getting permission to make the contributions it looks like it's been worked on a bit. I'll