as a rule I try to avoir putting assertions inside a stub... because if the stub is not called, the assertion is not checked either. Moreover, because it's not wrapped in the `done` call, an assertion error won't be caught and reported properly to mocha, and we'd have probably no stack trace (although here there is no asynchronicity so this could work).