Amaury Forgeot d'Arc <[email protected]> added the comment:
Here is my review of issue7092_syntax_imports_v3.diff:
- test_itertools.py: please replace
[tuple([arg[i] if i < len(arg) else None for arg in args])
for i in range(max(map(len, args)))]
by something more readable (nested for loops for example)
- test_mailbox.py: why doesn't test_support.import_module('rfc822')
specify deprecated=True? maybe the module can be imported normally...
- test_pyclbr.py: replace self.assertTrue(key in obj) with assertIn()
- test_wsgiref.py: replace self.assertTrue(key in h.environ)
- test_queue.py: test could be identical to the one in py3k.
----------
nosy: +amaury.forgeotdarc
_______________________________________
Python tracker <[email protected]>
<http://bugs.python.org/issue7092>
_______________________________________
_______________________________________________
Python-bugs-list mailing list
Unsubscribe:
http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
