Поймать заброшенные ошибки с SinonJS

Question

У меня есть метод, который может вызвать ошибку, но у меня возникли проблемы с написанием тестового примера SinonJS/Mocha/Should для этого условия.

функция Образец испытываемого: тест

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

Пример:

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

Этот выход:

#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) 

Я ожидал Sinon поймать ошибку и позвольте мне шпионить бросок, но, похоже, он не прошел тест. Есть идеи?

Я ссылаюсь на Don't sinon.js spys catch errors?, но единственным решением является использование expect. Я предпочел бы, если это возможно, придерживаться единой библиотеки утверждений.

Answer 1

Оказывается, что это работает внутри try/catch:

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

Обратите внимание, что вы должны вызвать fooSpy, неfoo сам.

Но также отметить, что .should.be.true() является не части Синона, так что вы, вероятно, уже используете Chai или аналогичную библиотеку, в этом случае expect(foo).to.have.thrown() или assert.throws(foo, someError) синтаксис кажется намного лучше.

Обновление: Если вы используете ShouldJS, похоже, вы можете использовать should.throws. Я все еще думаю, что это лучше, чем использование версии Sinon для этой цели.

Answer 2

Пересмотренный

После полезные советы @ nrabinowitz, вот это решение, которое использует should.throws. Это позволяет избежать использования всего Sinon.spy.

describe('#testError', function() { 
    it('throws an error', function() { 
    should.throws(function() { 
     testError(false); 
    }); 
    }); 
});