node.jsunit-testingmocha.jssinonshould.js

Catching thrown errors with SinonJS


I've got a method that may throw an Error, but I'm having trouble writing a SinonJS/Mocha/Should unit test case for this condition.

Sample function under test:

function testError(value) {
  if (!value) {
    throw new Error('No value');
    return false;
  }
};

Sample test:

describe('#testError', function() {
  it('throws an error', function() {
    var spy = sinon.spy(testError);
    testError(false);
    spy.threw().should.be.true();
  });
});

This outputs:

  #testError
    1) throws an error


  0 passing (11ms)
  1 failing

  1) #testError throws an error:
     Error: No value
      at testError (tests/unit/js/test-error.js:6:14)
      at Context.<anonymous> (tests/unit/js/test-error.js:14:6)

I was expecting Sinon to catch the Error and allow me to spy on the throw, but it seems to fail the test instead. Any ideas?

I referred to Don't sinon.js spys catch errors? but the only solution there is to use expect. I'd prefer to keep with a single assertion library if possible.


Solution

  • It appears that this works inside a try/catch:

    function foo() { throw new Error("hey!"); }
    var fooSpy = sinon.spy(foo);
    try {
      fooSpy();
    } catch (e) {
      // pass
    }
    assert(fooSpy.threw());
    

    Note that you have to call fooSpy, not foo itself.

    But also note that .should.be.true() is not part of Sinon, so you're probably already using Chai or a similar library, in which case the expect(foo).to.have.thrown() or assert.throws(foo, someError) syntax seems much nicer.

    Update: If you're using ShouldJS, looks like you can use should.throws. I still think this is nicer than using the Sinon version for this purpose.