This change looks good to me. The test suite addition is particularly welcome. That code is absolutely critical judging by the number and detail of the comments, so the additional test coverage counts many many times over. Big +1 for that.
On line 352 of the diff you have a comment that says '# this should not raise an exception', and then on the next line you call self.assertRaises(). The comment probably needs to be deleted.
Hi Benji,
This change looks good to me. The test suite addition is particularly welcome. That code is absolutely critical judging by the number and detail of the comments, so the additional test coverage counts many many times over. Big +1 for that.
On line 352 of the diff you have a comment that says '# this should not raise an exception', and then on the next line you call self.assertRais es(). The comment probably needs to be deleted.
Otherwise, this change looks good. r=mars
Maris