should there be an assertion test here?