Question

I’m currently consulting on an existing system, and I suspect the right next step is to add unit tests because of the types of exceptions that are occurring (null pointers, null lists, invalid return data). However, an employee who has a “personal investment” in the application insists on integration tests, even though the problems being reported are not related to specific use cases failing. In this case is it better to start with unit or integration tests?

Answer 1

Typically, it is very difficult to retrofit an untested codebase to have unit tests. There will be a high degree of coupling and getting unit tests to run will be a bigger time sink than the returns you’ll get. I recommend the following:

• Get at least one copy of Working Effectively With Legacy Code by Michael Feathers and go through it together with people on the team. It deals with this exact issue.
• Enforce a rigorous unit testing (preferably TDD) policy on all new code that gets written. This will ensure new code doesn’t become legacy code and getting new code to be tested will drive refactoring of the old code for testability.
• If you have the time (which you probably won’t), write a few key focused integration tests over critical paths of your system. This is a good sanity check that the refactoring you’re doing in step #2 isn’t breaking core functionality.