shouldn't the return value be checked as well in the tests?