Message224793
| Author |
ezio.melotti |
| Recipients |
BreamoreBoy, eric.araujo, ezio.melotti, kristjan.jonsson, michael.foord, pitrou, r.david.murray, serhiy.storchaka, stutzbach, vzhong, zach.ware |
| Date |
2014年08月05日.01:33:52 |
| SpamBayes Score |
-1.0 |
| Marked as misclassified |
Yes |
| Message-id |
<1407202432.19.0.172806984751.issue14534@psf.upfronthosting.co.za> |
| In-reply-to |
| Content |
I left a few comments on rietveld.
The patch should also include documentation.
> Class decorator approach looks less obvious to me. [...]
> But encountering the unittest.base_class decorator, I need to look in
> the documentation and/or sources to understand how it works.
I also agree that seeing @testBaseClass (the current name in the patch) gives very little indication of what's going on, but perhaps that could be fixed with a better name.
The two importants things are that the class is ignored/skipped, and that this doesn't propagate to the subclasses (as one might expect).
Would something like @ignoredBaseClass/@skippedBaseClass be better? Another options could be a more explicit @dontRunBaseClassTests, even though I don't particularly like the name. |
|
History
|
|---|
| Date |
User |
Action |
Args |
| 2014年08月05日 01:33:52 | ezio.melotti | set | recipients:
+ ezio.melotti, pitrou, kristjan.jonsson, stutzbach, eric.araujo, r.david.murray, michael.foord, BreamoreBoy, zach.ware, serhiy.storchaka, vzhong |
| 2014年08月05日 01:33:52 | ezio.melotti | set | messageid: <1407202432.19.0.172806984751.issue14534@psf.upfronthosting.co.za> |
| 2014年08月05日 01:33:52 | ezio.melotti | link | issue14534 messages |
| 2014年08月05日 01:33:52 | ezio.melotti | create |
|