InfoQ has a piece on how the Adium (a kick-ass IM application for OS X) team does code reviews by going through SVN commit mails and looking for anything suspicious. That’s all good, and certainly a practice that’s been widely adopted where I work as well. But the thing that caught my eye was this quote by Peter Hosey:
Some of them also made me think “man, I wish we had some tests, since those would have caught this”.
Manual code reviews are certainly beneficial, but I couldn’t imagine doing them without having good, automated tests for my code first. Just think of the regressions issues…
One Comment
Yup. We actually do have a few automated unit tests, since a couple months before the interview. The coverage is still woefully inadequate (it’s just a couple of our utility additions to built-in Cocoa classes), but it’s progress.
Post a Comment