After the mailnews/ and mail/ source trees reopened yesterday, I checked in a whole bunch of patches that had been queued up during the freeze. Some of them had been in my development tree, some of them hadn’t.
A little while later the SeaMonkey unit test tinderbox was orange. This surprised me, as I was fairly sure I had run all the mailnews tests with the various patches.
It turns out one of the patches (a follow up to bug 424570) had caused a regression where deleting address book cards wouldn’t work properly. This was one of those cases where before we had unit tests I wouldn’t have been surprised if we hadn’t detected the regression for a while.
So as Dan said, score 1 for MailNews automated tests. Although we’ve already seen regressions caught many times for the core tests, its good to see that the MailNews ones are now starting to catch regressions as well.
2 responses so far ↓
1 Wayne // May 11, 2008 at 0:53
awesome
2 Ted Mielczarek // May 12, 2008 at 13:01
That is excellent! I’ve seen so many test failures in the Firefox 3 release cycle that I know would not have been caught until a beta release, that it gives me great confidence in this release. Hopefully it will do the same for the Thunderbird release!
Leave a Comment