Question

Since a few days ago I’ve started to feel interested in Unit Testing and TDD in C# and VS2010. I’ve read blog posts, watched youtube tutorials, and plenty more stuff that explains why TDD and Unit Testing are so good for your code, and how to do it.

But the biggest problem I find is, that I don’t know what to check in my tests and what not to check.

I understand that I should check all the logical operations, problems with references and dependencies, but for example, should I create an unit test for a string formatting that’s supossed to be user-input? Or is it just wasting my time while I just can check it in the actual code?

Is there any guide to clarify this problem?

Answer 1

In TDD every line of code must be justified by a failing test-case written before the code.

This means that you cannot develop any code without a test-case. If you have a line of code (condition, branch, assignment, expression, constant, etc.) that can be modified or deleted without causing any test to fail, it means this line of code is useless and should be deleted (or you have a missing test to support its existence).

That is a bit extreme, but this is how TDD works. That being said if you have a piece of code and you are wondering whether it should be tested or not, you are not doing TDD correctly. But if you have a string formatting routine or variable incrementation or whatever small piece of code out there, there must be a test case supporting it.

UPDATE (use-case suggested by Ed.):

Like for example, adding an object to a list and creating a test to see if it is really inside or there is a duplicate when the list shouldn’t allow them.

Here is a counterexample, you would be surprised how hard it is to spot copy-paste errors and how common they are:

private Set<String> inclusions = new HashSet<String>();
private Set<String> exclusions = new HashSet<String>();

public void include(String item) {
    inclusions.add(item);
}

public void exclude(String item) {
    inclusions.add(item);
}

On the other hand testing include() and exclude() methods alone is an overkill because they do not represent any use-cases by themselves. However, they are probably part of some business use-case, you should test instead.

Obviously you shouldn’t test whether x in x = 7 is really 7 after assignment. Also testing generated getters/setters is an overkill. But it is the easiest code that often breaks. All too often due to copy&paste errors or typos (especially in dynamic languages).

See also:

• Mutation testing