Debugging? Write more tests!

I see a lot of people on the internet talking about writing unit tests.  While they can provide tons of advice on when you need unit tests, I don't see a lot of people talking about how to start.  I mean, you're not suddenly unit testing on a brand new project.  You're going to likely start with a few thousand lines of spaghetti code.

So where do you start with that?  How do you test the untestable, extremely tightly coupled code that throws errors on every page?  Do you even bother?  Shouldn't you fix it, refactor it, then test it? ABSOLUTELY NOT!*

Unit Test First!
I cannot emphasize enough how important it is to test the code before you change it.  How are you supposed to ensure that existing functionality is kept if you don't find out what the existing functionality is?  Sure, it threw a bug on every page, but they were expected bugs.  They might be accounted for elsewhere in the code, and your change could break everything.

OK, that's out of the way.  You can read lots of websites that try to define what a good unit test is.  The truth though, is that if you're not writing a test in the first place, you're problem isn't defining a good test.  When I get a user story, my first task is to write a test that would verify the functionality expected in the story.  Usually, this test ends up being an integration test, though it's technically called an acceptance test, but it also helps me to define the parameters of my unit tests.

So if you decide to heed my advice, to Unit Test before you refactor, and you start with a massive pile of spaghetti code, the only place you really can start is with some integration tests.  Then you find the units you can test, then you slowly start to decouple, making sure all the tests pass as you do.  This is how you start testing code.

As a wrap up, let's give a realistic coding example.  This is our mess of code in a very generic high-level diagram.
You have an ugly database, with an ugly data access layer, all intermingled with ugly code.  Your code probably has lines in it like this:

var sql = "SELECT * FROM codeRepo INTO #tempCode WHERE codeName = '" + fixName + "'";

And half of the company's software passes over that line of code at some point.  This code is not only not unit testable, it's dangerous to test because of the nasty side-effects it could have.  So hop on over to Dev, and build some integration tests.  You'll start with some standard use cases like: CanSaveFiles() or CanBookClients() and this may take a while (a long while).  But eventually, you'll get to the point where you can create a test called ClientCodeRepoFixTest() and you might not even know what the outcome is going to be.  That's OK!  That's what a trusty debugger is for!.  You will just have to figure out what this ugly bit of undocumented code does.  Not only will your job security go up enormously after you do, but you'll be able to make the code work, decouple it, refactor it, and make the whole company make more money.  Then you have a reason to go talk to your boss about that bonus he forgot to give you.

*This is just my opinion, please disregard it if you're a troll


Popular Posts