Issue 19383: nturl2path test coverage (original) (raw)

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.