Agreed, fixing up.


On Mon, Jul 22, 2013 at 1:37 PM, Eran Hammer <notifications@github.com>wrote:

> In test/broadcast.js:
>
> > @@ -319,8 +319,7 @@ describe('Broadcast', function () {
> >                          expect(obj.events.length).to.be.greaterThan(0)
> >
> >                          broadcast2 && broadcast2.kill(0);
> > -                        broadcast2 && done();
> > -                        broadcast2 = null;
> > +                        broadcast2 && done() && (broadcast2 = null);
>
> This is too complicated for one line. Test + function call + test on
> return + assignment?!
>
> —
> Reply to this email directly or view it on GitHub<https://github.com/spumko/good/pull/68/files#r5326731>
> .
>