Author barry
Recipients barry, chris.jerdonek, elopio, michael.foord, r.david.murray, rbcollins, vila, zach.ware
Date 2014-09-08.15:33:10
SpamBayes Score -1.0
Marked as misclassified Yes
Message-id <1410190391.03.0.18868985922.issue16662@psf.upfronthosting.co.za>
In-reply-to
Content
One thing I really do not like about Rob's last patch is that it exacerbates the documentation discrepancy for loadTestsFromModule().  As previously mentioned, use_load_tests arg was already not documented, and now the patch adds another undocumented pattern default arg.  Undocumented "unofficial" APIs are really a fib - we treat them as official APIs for backward compatibility reasons anyway, so I think it behooves us to document them.

In the same vein, the load_tests Protocol really should tell the truth about its third argument - i.e. it will not always be None.

As Michael suggests in http://bugs.python.org/issue16662#msg200274 I think we should just remove use_load_tests.  We'll still need to document the new pattern=None, unless there's a better way to handle that.
History
Date User Action Args
2014-09-08 15:33:11barrysetrecipients: + barry, rbcollins, vila, r.david.murray, michael.foord, chris.jerdonek, zach.ware, elopio
2014-09-08 15:33:11barrysetmessageid: <1410190391.03.0.18868985922.issue16662@psf.upfronthosting.co.za>
2014-09-08 15:33:11barrylinkissue16662 messages
2014-09-08 15:33:10barrycreate