Shouldn't you also test for return values?