Мокко / Чай ожидают. К броску не ловят брошенные ошибки

У меня проблемы с тем, чтобы Chai's expect.to.throwработал в тесте для моего приложения node.js. Тест продолжает давать сбой на выданную ошибку, но если я оберну тестовый пример в попытку и поймать и утверждать на обнаруженную ошибку, это работает.

Не expect.to.throwработает, как я думаю, что должно или что-то?

it('should throw an error if you try to get an undefined property', function (done) {
  var params = { a: 'test', b: 'test', c: 'test' };
  var model = new TestModel(MOCK_REQUEST, params);

  // neither of these work
  expect(model.get('z')).to.throw('Property does not exist in model schema.');
  expect(model.get('z')).to.throw(new Error('Property does not exist in model schema.'));

  // this works
  try { 
    model.get('z'); 
  }
  catch(err) {
    expect(err).to.eql(new Error('Property does not exist in model schema.'));
  }

  done();
});

Провал:

19 passing (25ms)
  1 failing

  1) Model Base should throw an error if you try to get an undefined property:
     Error: Property does not exist in model schema.

Вы должны передать функцию expect. Как это:

expect(model.get.bind(model, 'z')).to.throw('Property does not exist in model schema.');
expect(model.get.bind(model, 'z')).to.throw(new Error('Property does not exist in model schema.'));

То , как вы делаете это, вы передаете в expectв результате вызова model.get('z'). Но чтобы проверить, выброшено ли что-то, вам нужно передать функцию expect, которая expectбудет вызывать сама себя. bindМетод , используемый выше , создает новую функцию , которая при вызове будет вызывать model.getс thisнабором к значению modelи первому набору аргументов к 'z'.

Хорошее объяснение bindможно найти здесь .

Как говорится в этом ответе , вы также можете просто обернуть свой код в анонимную функцию, например:

expect(function(){
    model.get('z');
}).to.throw('Property does not exist in model schema.');

И если вы уже используете ES6 / ES2015, то вы также можете использовать функцию стрелки. По сути, это то же самое, что и обычная анонимная функция, но короче.

expect(() => model.get('z')).to.throw('Property does not exist in model schema.');

Этот вопрос имеет много-много дубликатов, включая вопросы, не упоминающие библиотеку утверждений Чай. Вот основы, собранные вместе:

Утверждение должно вызывать функцию, а не вычислять ее немедленно.

assert.throws(x.y.z);      
   // FAIL.  x.y.z throws an exception, which immediately exits the
   // enclosing block, so assert.throw() not called.
assert.throws(()=>x.y.z);  
   // assert.throw() is called with a function, which only throws
   // when assert.throw executes the function.
assert.throws(function () { x.y.z });   
   // if you cannot use ES6 at work
function badReference() { x.y.z }; assert.throws(badReference);  
   // for the verbose
assert.throws(()=>model.get(z));  
   // the specific example given.
homegrownAssertThrows(model.get, z);
   //  a style common in Python, but not in JavaScript

Вы можете проверить наличие конкретных ошибок, используя любую библиотеку утверждений:

Узел

  assert.throws(() => x.y.z);
  assert.throws(() => x.y.z, ReferenceError);
  assert.throws(() => x.y.z, ReferenceError, /is not defined/);
  assert.throws(() => x.y.z, /is not defined/);
  assert.doesNotThrow(() => 42);
  assert.throws(() => x.y.z, Error);
  assert.throws(() => model.get.z, /Property does not exist in model schema./)

Должен

  should.throws(() => x.y.z);
  should.throws(() => x.y.z, ReferenceError);
  should.throws(() => x.y.z, ReferenceError, /is not defined/);
  should.throws(() => x.y.z, /is not defined/);
  should.doesNotThrow(() => 42);
  should.throws(() => x.y.z, Error);
  should.throws(() => model.get.z, /Property does not exist in model schema./)

Чай Ожидать

  expect(() => x.y.z).to.throw();
  expect(() => x.y.z).to.throw(ReferenceError);
  expect(() => x.y.z).to.throw(ReferenceError, /is not defined/);
  expect(() => x.y.z).to.throw(/is not defined/);
  expect(() => 42).not.to.throw();
  expect(() => x.y.z).to.throw(Error);
  expect(() => model.get.z).to.throw(/Property does not exist in model schema./);

Вы должны обрабатывать исключения, которые «избегают» теста

it('should handle escaped errors', function () {
  try {
    expect(() => x.y.z).not.to.throw(RangeError);
  } catch (err) {
    expect(err).to.be.a(ReferenceError);
  }
});

На первый взгляд это может показаться странным. Как и при езде на велосипеде, он просто «щелкает» вечно, когда щелкает.

примеры из док ...;)

потому что вы полагаетесь на thisконтекст:

• который теряется, когда функция вызывается .throw
• у него нет возможности узнать, что это должно быть

Вы должны использовать один из следующих вариантов:

• обернуть вызов метода или функции внутри другой функции

• связать контекст

  // wrap the method or function call inside of another function
  expect(function () { cat.meow(); }).to.throw();  // Function expression
  expect(() => cat.meow()).to.throw();             // ES6 arrow function
  
  // bind the context
  expect(cat.meow.bind(cat)).to.throw();           // Bind

Еще одна возможная реализация, более громоздкая, чем решение .bind (), но та, которая помогает понять, что ожидаемо () требует функции, которая предоставляет thisконтекст для покрытой функции, вы можете использовать call(), например,

expect(function() {model.get.call(model, 'z');}).to.throw('...');

Я нашел хороший способ обойти это:

// The test, BDD style
it ("unsupported site", () => {
    The.function(myFunc)
    .with.arguments({url:"https://www.ebay.com/"})
    .should.throw(/unsupported/);
});


// The function that does the magic: (lang:TypeScript)
export const The = {
    'function': (func:Function) => ({
        'with': ({
            'arguments': function (...args:any) {
                return () => func(...args);
            }
        })
    })
};

Это намного более читабельно, чем моя старая версия:

it ("unsupported site", () => {
    const args = {url:"https://www.ebay.com/"}; //Arrange
    function check_unsupported_site() { myFunc(args) } //Act
    check_unsupported_site.should.throw(/unsupported/) //Assert
});