Creating testable code

Answer 1

How about taking the existing functions, move the code inside into a new class and call the new methods from the existing functions and then override this class during tests?

Like so:

 static DBAccessClass dac = new DBAccessClass ();

 void origFunction() { dac.origFunction(); }

And in the test:

 dac = new DBAccessMockup();

Answer 2

You can try to override at the linker level. Have two different .cpp files that implement the functions in the header that describes the DB interface - one calling the real DB, one calling a fake interface. When you link for unit test, replace one with the other (Target Specific Variables in GNU make may help).

Answer 3

You may also want to look at Michael Feathers' book Working Effectively with Legacy Code. Not only does he discuss exactly these types of problems, but the book includes numerous examples in C++ (in addition to Java, C, and C#). Feathers is also the original creator of CppUnit.

Answer 4

1. Make file into class and override methods. [...]
2. Wrap the whole source file with an #ifdef [...]

If possible, go with 1, as you will have a centralized way of referring to the database code (a class pointer instead of X functions). That means modularity, ease in replacing the implementation (with stubs or with another DB back-end) and more encapsulated code.

If you go with 2, consider replacing the implementation of the functions. That is, inside the original functions, engage the testing code (based on an if).

Your tested code is completely agnostic if running in a test environment or not, the performance loss is negligible (if(booleanFlagHere) is a negligible cost in most cases) and you don't have to modify the tested code at all).