Author chris.jerdonek
Recipients anton.barkovsky, chris.jerdonek, georg.brandl, r.david.murray
Date 2012-08-11.17:43:48
SpamBayes Score -1.0
Marked as misclassified Yes
Message-id <>
It seems like these tests can be made more DRY.  For example, the line `args = popen.cmd_line` appears in every test.  You could make an assert_args() helper method to address this.  There also seems to be a lot of overlap between tests for each browser.  A DRY approach would let one see more easily how the tests differ across browsers.

Do you also need to include the test boilerplate at the bottom?  See, for example--
Date User Action Args
2012-08-11 17:43:49chris.jerdoneksetrecipients: + chris.jerdonek, georg.brandl, r.david.murray, anton.barkovsky
2012-08-11 17:43:49chris.jerdoneksetmessageid: <>
2012-08-11 17:43:49chris.jerdoneklinkissue15557 messages
2012-08-11 17:43:48chris.jerdonekcreate