Message140048
I have made a patch based on your suggestions made to higery, Éric. I have two questions:
1) In Distributions.get_reinitialized_command should the reinitialization of the subcommands also get passed the kwargs? Unfortunately my understanding of the (sub)command flow is not rock solid.
2) What are your thoughts on an effective test for this? This command does not currently have one. I believe I know a simple way to test the new kwargs functionality but if one were to write a test for get_reinitialized_command I think it should probably test the rest of the function as well.
Thoughts? |
|
Date |
User |
Action |
Args |
2011-07-09 03:21:04 | thomas.holmes | set | recipients:
+ thomas.holmes, tarek, eric.araujo, alexis, higery |
2011-07-09 03:21:04 | thomas.holmes | set | messageid: <1310181664.29.0.359737758353.issue12344@psf.upfronthosting.co.za> |
2011-07-09 03:21:03 | thomas.holmes | link | issue12344 messages |
2011-07-09 03:21:03 | thomas.holmes | create | |
|