should be checking the return values in that test