Как обработать и тестировать ошибку отсутствия обязательных аргументов конструктора при создании объекта?

Question

[Алексей]

Как можно недать конструктору создать экземпляр, если не переданы обязательные аргументы(соответственно надо обозначить эти обязательные аргументы) и проверить всё это тестом.
Правильный вариант, в соответствии с решением, предложенным @mudrenokanton

// код
"use strict";
var Checkbox = function(elementId,target,expression){
    if(elementId == undefined || target == undefined || expression == undefined){
        return {error:true}
    }
    // ...
};
// тест
describe("simple create class object Checkbox", function () {
    it("should throw exception with no arguments", function () {
        var object = new Checkbox();
        assert(object.error)
    });
});

https://jsfiddle.net/Quncore/hxzycx5y/

Answer 1

[Антон Мудренок]

var object = function(){new Checkbox();};
assert.throws(object)

Что тут вообще происходит? Вы передаете тело функции в assert, вы же ничего не вызываете и ни с чем не сравниваете.

Если хотите предотвратить действие, то пишите в блоке try catch.
По хорошему лучше создавать фабричный обьект, а потом вызывать для него initialize(...params) в котором уже делать нужную проверку. Зачем вам эта имитация работы конструкторов.

upd. А еще вы должны врубиться, что никаких конструкоторов нет, это просто вызов функции в котором вы можете делать что захотите:

var Construct = function(a, b) { 
    var error = {}; 
    error.message = "wrong params"; 
    if (true) { return error }; //делаете тут проверку
    // а тут дальше пишите ваш "конструктор"
};
var a = new Construct();  // Object {message: "wrong params"}

Answer 2

[Станислав Макаров]

Эх, что только люди не делают, чтобы пофиксить JS.

Лучший способ тестировать ошибку отсутствия обязательных аргументов - пользоваться TypeScript.

Что касается вашего варианта - я бы делал строгую проверку на undefined, т.к. в качестве значения аргумента иногда вполне может передаваться null - тогда ваша проверка не будет корректно отрабатывать.

Comments

[Алексей]

это всё здорово, но мне надо, чтобы конструктор не возвратил новый объект типа Checkbox

[Станислав Макаров]

Алексей а как он возвратит новый объект, если он кидает исключение?

Answer 3

[Дмитрий Беляев]

function Checkbox(elementId, target, expression) {
  if(arguments.length < 3) {
    throw new Error('Не переданы обязательные аргументы!');
  }
  // Ваш код
}

Comments

[Алексей]

в первоначальной версии так и было, а как потом в тесте проверить это?

[Дмитрий Беляев]

Алексей: перехватить исключение через try-catch

[Алексей]

Дмитрий Беляев: как это будет выглядеть в блоке it теста?

[Дмитрий Беляев]

Алексей:

describe("simple create class object Checkbox", function () {
    it("should throw exception with no arguments", function () {
        try {
            new Checkbox();
        } catch(err) {
            assert(err);
        }
    });
});

[Алексей]

Дмитрий Беляев: в catch(err) будет передан объект new Error, созданный в тестируемом коде?
для подобных проверок лучше писать функцию, но где? Перед всеми describe или в начале блока describe, объединяющего describe, которые заняты подобными проверками?

[Алексей]

Нашёл пост, советуют использовать should.js
следующим образом: .should.throw() и .should.not.throw()
https://github.com/chaijs/chai/issues/71#issuecomm...