Bricolage

A unit testing success story

This summer I was accepted to Google Summer of Code to build a memetracker application for Drupal.

I want the Memetracker module to facilitate online conversations amongst distributed communities. For my dream to become reality, Memetracker will need help from many talented developers. This summer I’ll be setting a solid foundation for future development of the memetracker module. One important thing I’ll be doing is writing unit tests for all the functions of the memetracker. Unit tests, I’m convinced, is one thing Memetracker needs in order to ensure its long term success.

Unit tests are at the heart of Test Driven Development (TDD). TDD and unit tests when used have been shown to improve the quality of code and reduce bugs and regressions (see this excellent post by Robert Douglass for a full introduction to unit testing).

My brother Ben is a software engineer and has long preached to me the benefits of unit tests and TDD. When I started programming Memetracker, I agreed in principle with the benefits of unit tests but as I wanted to see something working more than write boring test code, I just dived into the coding without writing any unit tests. But as the lines of code accumulated and the complexity grew, weird bugs started cropping up and having a solid coverage of unit tests as a safety net started sounding better and better.

So I stopped coding and spent a day studying unit testing and SimpleTest (the testing framework Drupal uses). There was a painful learning curve to wrap my head around the concepts but just a few days of coding with unit tests has made me a convert.

A quick example of how SimpleTest has helped me improve the quality of my code.

Memetracker tries to present to readers the most interesting memes (conversations around a topic) to readers on a web page. To learn which memes are interesting, Memetracker records the articles people click on from the memebrowsing page. By counting clicks, memetracker can calculate an interestingness score for the different memes and from that score, determine which memes to display and in what order.

When a person clicks on a article, that information is sent from the browser to the memetracker module. In memetracker.module is a function, memetrackersetclick(), which saves the click information to a table in the database.

I wanted to make sure this function works correctly so I wrote a unit test which sent a fake click to the memetrackersetclick function and then tested that the click was actually saved to the database.

So I wrote the test… and it worked!

click_test

But wait a second — I thought — it shouldn’t of worked. To save the click data a memetracker object has to be loaded which then does the actual saving of the click information. But when I wrote the unit test, I had made up an id for the memetracker object. All information for different memetracker objects are stored in the database (one website can have multiple memetrackers). If a memetracker didn’t exist in the database, it shouldn’t be loaded.

I had already written some code to validate that a memetracker object existed before loading but somehow that validation wasn’t working. I looked at the validation code and quickly found the problem. Problem solved, and to ensure this didn’t happen again I wrote a new unit test to ensure my validation code works correctly in the future.

Unit tests are valuable for any coding project longer than a few lines. Once you get past the learning curve, writing new tests are easy. And it’s a great feeling after making changes to run my tests and know in seconds everything is still working as it should.

To learn more about unit testing, read Angie Bryon’s (webchick) great blog post about how unit tests let you to refactor without fear. The Lullabot crew had a recent podcast on the wonders of unit testing. And lastly, there’s some great documentation being collected here.

Happy unit testing!

Tagged with drupal | google summer of code | unit testing

Posted July 01, 2008


Kyle Mathews lives and works in San Francisco building useful things. You should follow him on Twitter