Both!!! One of the things I liked about Sinon is it would alert you of stupid mistakes like missing function or argument mis matches...

On Mar 2, 2013, at 1:09 PM, Julien Wajsberg <notifications@github.com> wrote:

> In apps/calendar/js/controllers/sync.js:
> 
> > @@ -44,9 +44,14 @@ Calendar.ns('Controllers').Sync = (function() {
> >       *    controller.once('syncComplete', cb);
> >       *
> >       */
> > -    all: function() {
> > +    all: function(callback) {
> > +      if (callback) {
> less mocking in tests required.
> or better mocking ? :)
> 
> —
> Reply to this email directly or view it on GitHub.