wondering if we should use a "done" callback and pass the "error" as parameter to done. (which in mocha's language means that there is an error).

I guess this work because the marionette js framework is really synchronous behind the scene, but this looks strange when we read the code.

That said I won't block for this so do it if you think it's better, otherwise it's fine like this.