Added logging and added support for valgrind strategy.

When tests passed, but valgrind failed and valgrind_strategy is fail
![screen shot 2014-09-23 at 20 22 14](https://cloud.githubusercontent.com/assets/1354151/4379461/ddc03232-4360-11e4-8619-7e313237f5ed.jpg)

Yep, not a big fan of string comparisons either, but its used elsewhere too, lets refactor it another time.

We want to update the error message only when tests have passed but valgrind failed. Thus this logic