Closed yrik closed 9 years ago
Could you please add some unittests for this? Just put some known good values and confirm the right thing come out. Thanks
Merged! Thanks. I added some unitests. There was a merge conflict, I think it's OK. Let me know if anything has broken.
Coverage remained the same when pulling 12aa22144c679c5da1a264b80d8944f2de7ad824 on yrik:patch-2 into 1593ea12c006e871564a64901c3189adbfefe6a3 on rory:master.