Message166957
I haven't commented on the content of your patch (I'm not knowledgeable enough in this area), but as for your test, I noticed that you removed the part that asserted something about the argument passed to cb().
Ideally when testing a callback, you want to test whatever is important about it, like: that it gets called at the right times (and as many times as it should), and that it is getting passed the right data.
So you might also want to make whatever assertions are appropriate about the packet(s) passed to cb(). |
|
Date |
User |
Action |
Args |
2012-07-31 03:59:16 | chris.jerdonek | set | recipients:
+ chris.jerdonek, pitrou, tweksteen |
2012-07-31 03:59:16 | chris.jerdonek | set | messageid: <1343707156.08.0.383480705889.issue15464@psf.upfronthosting.co.za> |
2012-07-31 03:59:15 | chris.jerdonek | link | issue15464 messages |
2012-07-31 03:59:15 | chris.jerdonek | create | |
|