__path__ attribute of modules loaded by zipimporter is untested

[exarkun]

BPO	8942
Nosy	@ncoghlan, @bitdancer, @ericsnowcurrently
Files	issue8942_test_for_zipimporter.patch: Improved test so the path of a package loaded from a zip file, using zipimporter, is checked.

^{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 = <Date 2012-03-13.01:20:24.528>
created_at = <Date 2010-06-08.16:15:30.936>
labels = ['easy', 'type-feature', 'tests']
title = '__path__ attribute of modules loaded by zipimporter is untested'
updated_at = <Date 2012-03-13.01:20:24.511>
user = 'https://bugs.python.org/exarkun'

bugs.python.org fields:

activity = <Date 2012-03-13.01:20:24.511>
actor = 'r.david.murray'
assignee = 'none'
closed = True
closed_date = <Date 2012-03-13.01:20:24.528>
closer = 'r.david.murray'
components = ['Tests']
creation = <Date 2010-06-08.16:15:30.936>
creator = 'exarkun'
dependencies = []
files = ['24796']
hgrepos = []
issue_num = 8942
keywords = ['patch', 'easy']
message_count = 5.0
messages = ['107327', '155453', '155458', '155551', '155552']
nosy_count = 6.0
nosy_names = ['exarkun', 'ncoghlan', 'r.david.murray', 'python-dev', 'eric.snow', 'tati_alchueyr']
pr_nums = []
priority = 'normal'
resolution = 'fixed'
stage = 'resolved'
status = 'closed'
superseder = None
type = 'enhancement'
url = 'https://bugs.python.org/issue8942'
versions = ['Python 3.3']

[exarkun]

Packages loaded from zip files have a __path__ sort of like any other package. The zipimport tests don't verify that this attribute has the correct value, though.

[tatialchueyr]

I'm working on it

[tatialchueyr]

Improved test so the __path__ of a package loaded from a zip file, using zipimporter, is checked.

[python-dev]

New changeset 7e576ad85663 by R David Murray in branch 'default':
bpo-8942: add test for __path__ contents in zipimport tests
http://hg.python.org/cpython/rev/7e576ad85663

[bitdancer]

I committed Tatiana's patch because it looks reasonable to me, but I'm not familiar enough with zipimport to know if additional tests are needed. If you think there are additional tests that should be added, please reopen the issue.