This issue tracker has been migrated to GitHub, and is currently read-only.
For more information, see the GitHub FAQs in the Python's Developer Guide.

Author todddeluca
Recipients Sharif.Nassar, alexis, eric.araujo, tarek, todddeluca, tshepang
Date 2012-05-27.10:03:07
SpamBayes Score -1.0
Marked as misclassified Yes
Message-id <1338112989.82.0.747040724199.issue14858@psf.upfronthosting.co.za>
In-reply-to
Content
This patch fixes the problem where the user would select a maturity status when runnning 'pysetup create' and the resulting setup.cfq would have a maturity status one less than the user selected.

It also fixes the behavior where a user can select '0' as a maturity status, which is not a valid selection, and setup.cfg will contain maturity status 'Development Status :: 7 - Inactive'.  This behavior is related to the implementation of the off-by-one error above, since '0' is translated into a list lookup using the index -1.  

Finally, the patch addresses behavior where a user cannot enter a blank maturity status (e.g. by just pressing the 'return' key) in order to skip the question.

The unit tests added to 'test_create.py' cover expected behavior when entering and blank selection, an invalid selection, and a valid selection.  

These tests were run using Python 2.7.3 on my Mac OS X 10.6.8 (Snow Leopard).
History
Date User Action Args
2012-05-27 10:03:10todddelucasetrecipients: + todddeluca, tarek, eric.araujo, alexis, tshepang, Sharif.Nassar
2012-05-27 10:03:09todddelucasetmessageid: <1338112989.82.0.747040724199.issue14858@psf.upfronthosting.co.za>
2012-05-27 10:03:09todddelucalinkissue14858 messages
2012-05-27 10:03:08todddelucacreate