As for the tests, as long as they follow all the code paths, that's good enough (currently, I don't think there is 100% coverage--i.e. difference values for parameters like closing etc.)