[issue14183] Test coverage for packaging.install and packaging.pypi.wrapper

Éric Araujo added the comment: distutils2/packaging development has stopped. -- resolution: -> out of date stage: patch review -> committed/rejected status: open -> closed ___ Python tracker __

[issue14183] Test coverage for packaging.install and packaging.pypi.wrapper

Francisco Martín Brugué added the comment: I've updated the patch. Let me know if something has to be changed. -- Added file: http://bugs.python.org/file24727/issue14183_fbb9847b8f43.patch ___ Python tracker _

[issue14183] Test coverage for packaging.install and packaging.pypi.wrapper

Francisco Martín Brugué added the comment: Nadeem: > - Don't use spaces around the '=' sign when used to indicate a > keyword argument or a default parameter value. ok, "code formating" is not working as expected (at least for lambdas...) Éric : > If you grep packaging tests for XXX|TODO|

[issue14183] Test coverage for packaging.install and packaging.pypi.wrapper

Nadeem Vawda added the comment: >> >> -install.install_dists = lambda x, y=None: None >> +install.install_dists = lambda x, y = None: None >> PEP 8: Never put spaces in a function (or lambda) signature. > > I cannot find that exactly in pep8. >From the section "Whitespace in Ex

[issue14183] Test coverage for packaging.install and packaging.pypi.wrapper

Francisco Martín Brugué added the comment: Hi Éric, > - # Test that the isntalled raises an exception if the project does not > + # Test that the installed raises an exception if the project does not > It took me many seconds to find the change :) The editor told me :) > > +def test_ins

[issue14183] Test coverage for packaging.install and packaging.pypi.wrapper

Éric Araujo added the comment: Thanks for the patch. -# Test that the isntalled raises an exception if the project does not +# Test that the installed raises an exception if the project does not It took me many seconds to find the change :) +def test_installation_get_infos_