Better valgrind support

Refactored valgrind validations check to work like checkstyle validations for java

In previous implementation users got confisung results of tests passing
and valgrind errors.
This might be read as saying that "none" is required to make valgrind errors visible when that is not the case. Maybe just remove the sentence about "none"?