I agree Hannes, for _this_ methods test only need to test that it can use some CodeValidator correctly. My main concern is that the actual CodeValidator wasn't tested. I'm wondering if we should include a test for it in this PR, I want us to get away from adding any code without supplying appropriate test for it. What do you think? I don't really mind if we split it out into a separate issue. I just want to make sure in general if we are adding code we supply tests to test that code to.