You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
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='https://github.com/serhiy-storchaka'closed_at=<Date2014-05-28.15:39:31.309>created_at=<Date2014-05-13.10:12:40.242>labels= ['type-feature', 'tests']
title='Add test for ntpath.expanduser'updated_at=<Date2014-05-28.15:39:31.292>user='https://github.com/PCManticore'
LGTM. But most other tests in this file use tester() to test str and bytes. And it would be good to test multicomponent arguments (e.g. '~test\\foo\\bar' and '~/foo/bar').
Activity
PCManticore commentedon May 13, 2014
Hello. This patch adds a test for ntpath.expanduser, which was missing. Also, it increases the test coverage for ntpath.py from 68% to 75%.
serhiy-storchaka commentedon May 25, 2014
LGTM. But most other tests in this file use tester() to test str and bytes. And it would be good to test multicomponent arguments (e.g. '~test\\foo\\bar' and '~/foo/bar').
PCManticore commentedon May 25, 2014
Here's an updated patch.
python-dev commentedon May 28, 2014
New changeset 5e599b301ebd by Serhiy Storchaka in branch '2.7':
Issue bpo-21493: Added test for ntpath.expanduser(). Original patch by
http://hg.python.org/cpython/rev/5e599b301ebd
New changeset a981a088512c by Serhiy Storchaka in branch '3.4':
Issue bpo-21493: Added test for ntpath.expanduser(). Original patch by
http://hg.python.org/cpython/rev/a981a088512c
New changeset 5a71a7e60fe6 by Serhiy Storchaka in branch 'default':
Issue bpo-21493: Added test for ntpath.expanduser(). Original patch by
http://hg.python.org/cpython/rev/5a71a7e60fe6
serhiy-storchaka commentedon May 28, 2014
Thanks Claudiu.