nturl2path test coverage

[ColinWilliams]

BPO	19383
Nosy	@orsenthil, @ezio-melotti
Files	nturl2path.patch

^{Note: these values reflect the state of the issue at the time it was migrated and might not reflect the current state.}

Show more details

GitHub fields:

assignee = None
closed_at = None
created_at = <Date 2013-10-24.23:33:34.720>
labels = ['type-feature', 'tests']
title = 'nturl2path test coverage'
updated_at = <Date 2013-11-16.23:45:11.596>
user = 'https://bugs.python.org/ColinWilliams'

bugs.python.org fields:

activity = <Date 2013-11-16.23:45:11.596>
actor = 'ezio.melotti'
assignee = 'none'
closed = False
closed_date = None
closer = None
components = ['Tests']
creation = <Date 2013-10-24.23:33:34.720>
creator = 'Colin.Williams'
dependencies = []
files = ['32345']
hgrepos = []
issue_num = 19383
keywords = ['patch']
message_count = 2.0
messages = ['201197', '203107']
nosy_count = 3.0
nosy_names = ['orsenthil', 'ezio.melotti', 'Colin.Williams']
pr_nums = []
priority = 'normal'
resolution = None
stage = 'patch review'
status = 'open'
superseder = None
type = 'enhancement'
url = 'https://bugs.python.org/issue19383'
versions = ['Python 3.4']

[ColinWilliams]

Full coverage on this module

[ezio-melotti]

The assertions have their arguments in the wrong order, it should be (actual, expected).
In the code of the module there are also some examples in the comments, e.g.:
# path is something like ////host/path/on/remote/host
# convert this to \\host\path\on\remote\host
Even though the path is covered, there's no similar test in this patch.
I would suggest you to include all the examples from the module in the tests.

Also note that this module is already indirectly tested in test_urllib.