Почему проходит тест с разными значениями?

Question

[tostershmoster]

Почему Jest тест проходит с неверными значениями?

import { ISettings } from '../RangeSlider/types';
import { Model } from './Model';

let settings: ISettings;

function getPrivateMethod(methodName: string, settings: ISettings) {
  const modelProto = Model.prototype as any;
  const instance = Object.create(modelProto);
  return () => modelProto[methodName].call(instance, settings);
}

describe('validateSettings function check: ', () => {
  beforeEach(() => {
    settings = {
      min: 0,
      max: 1500,
      valueFrom: 1000,
      valueTo: 1490,
      isScaleVisible: true,
      isTwoRunners: true,
      isVertical: false,
      isTooltipsVisible: true,
      step: 10,
    };
  });

  test('"settings.min >= settings.max" should throw Error', () => {
    // тест не должен проходить при таких значениях, должен только при min > max
    settings.min = 1000;
    settings.max = 1500;

    const minIsBiggerThanMax = getPrivateMethod('validateSettings', settings);

    expect(minIsBiggerThanMax).toThrow();
  });
});

тестируемый код

private static validateSettings(settings: ISettings): void {
  if (settings.min >= settings.max) {
    throw new Error("'max' must be greater than 'min'");
  }
  // if (settings.valueFrom < settings.min) {
  //   throw new Error("'valueFrom' must be greater than 'min'");
  // }
  // if (settings.valueFrom > settings.valueTo) {
  //   throw new Error("'valueFrom' must be less than 'valueTo'");
  // }
  // if (settings.valueTo > settings.max) {
  //   throw new Error("'valueTo' must be less than 'max'");
  // }
  // if (settings.max - settings.min < settings.step) {
  //   throw new Error(`'step' must be less than ${settings.max - settings.min}`);
  // }
}

конструктор Model

constructor(settings: ISettings) {
  Model.validateSettings(settings);
  this.settings = settings;

  // default options
  this.minValue = this.settings.min;
  this.maxValue = this.settings.max;
  this.isTwoRunners = this.settings.isTwoRunners;
  this.isScaleVisible = this.settings.isScaleVisible;
  this.isVertical = this.settings.isVertical;
  this.isTooltipsVisible = this.settings.isTooltipsVisible;

  this.valueFrom = this.getThumbValue(this.settings, 'from');
  this.valueTo = this.getThumbValue(this.settings, 'to');

  this.step = this.getStepInPercents(this.settings.step);
  this.rangePercent = (this.settings.max - this.settings.min) / 100;

  this.getStepInPercents = this.getStepInPercents.bind(this);
}

Как вообще дебажить эти тесты? Не понятно где что отваливается. В браузере нет этих *.test.ts модулей. Инклудить их в код?

Answer 1

[Антон Антон]

Потому что функция не бросает исключение, а сама его перехватывает.
Нужно удалить try-catch обертку.

Comments

[tostershmoster]

если удалить try-catch то получаю ошибку

[Антон Антон]

tostershmoster, похоже, что у тебя getPrivateMethod не возвращает метод, а выполняет его.

[tostershmoster]

ошибка появляется в функции getPrivateMethod на строке model = new Model(settings);

[Антон Антон]

tostershmoster, а конструктор не вызывает validateSettings?

[tostershmoster]

Да этот метод вызывается в конструкторе

constructor(settings: ISettings) {
  this.validateSettings(settings);
  this.settings = settings;
  // ...

[Антон Антон]

tostershmoster, ну вот и ответ

[tostershmoster]

и что делать в таком случае, если мне нужна валидация настроек? создавать в коде экземпляр модел с кривыми настройками и уже после создания вызывать этот метод у созданного объекта model.validateSettings?

[Антон Антон]

tostershmoster, ну, если в конструкторе логики другой нет, то можно тестировать конструктор (создание экземпляра с кривыми настройками). Если есть - то можно вынести функцию из класса, а в классе хранить ссылку на неё (или сделать метод статическим) и тестировать без создания экземпляра.

[tostershmoster]

constructor(settings: ISettings) {
    this.validateSettings(settings);
    this.settings = settings;

    // default options
    this.minValue = this.settings.min;
    this.maxValue = this.settings.max;
    this.isTwoRunners = this.settings.isTwoRunners;
    this.isScaleVisible = this.settings.isScaleVisible;
    this.isVertical = this.settings.isVertical;
    this.isTooltipsVisible = this.settings.isTooltipsVisible;

    this.valueFrom = this.getThumbValue(this.settings, 'from');
    this.valueTo = this.getThumbValue(this.settings, 'to');

    this.step = this.getStepInPercents(this.settings.step);
    this.rangePercent = (this.settings.max - this.settings.min) / 100;

    this.getStepInPercents = this.getStepInPercents.bind(this);
  }

[Антон Антон]

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

Answer 2

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

Во-первых, убрать try-catch из метода
Во-вторых

function getPrivateMethod(methodName: string, settings: ISettings) {
  const modelProto = Model.prototype as any;
  const instance = Object.create(modelProto);
  return () => modelProto[methodName].call(instance, settings);
}

Comments

[tostershmoster]

А почему Model.prototype с типом any?
И чем вариант modelProto[methodName].call(instance, settings) лучше modelProto[methodName](settings)?

[tostershmoster]

нужно добавить constructor для статического метода

function getPrivateMethod(methodName: string, settings: ISettings) {
  const modelProto = Model.prototype as any;
  const instance = Object.create(modelProto);
  return () => modelProto.constructor[methodName].call(instance, settings);
}

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

А почему Model.prototype с типом any?

tostershmoster, потому что у прототипа будет тот же тип, что и у инстанса, а TypeScript не даст взять проперти по произвольной строке и тем более не даст взять приватное поле.

Почему возвращается стрелочная функция?

А что по Вашему должно возвращаться?

И чем вариант modelProto[methodName].call(instance, settings) лучше modelProto[methodName](settings)?

Второй вариант вызовет метод непосредственно с прототипом в качестве this, а значит все изменения this в методе применятся к прототипу и тест сломает Ваш класс, метод call у функции (метод - это та же функция) позволяет задать произвольный this при вызове
https://developer.mozilla.org/ru/docs/Web/JavaScri...

тест также проходит с неверными значениями и при min > max и min < max,
а должен только с min > max

Вы в тесте проверяете любое исключение, скорее всего Ваш код падает еще в каком то месте, в toThrow можно передать объект ошибки (матчится глубоким сравнением), строку (сравнивает с message) или регулярку (матчит message)
https://jestjs.io/docs/expect#tothrowerror

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

нужно добавить constructor для статического метода

tostershmoster, Вы переписали вопрос и идете с претензиями к отвечающим, когда я отвечал никакого статического метода не было...
https://qna.habr.com/questionversion?question_id=1...

[tostershmoster]

Дмитрий Беляев, никаких притензий нет, просто заметки на полях. И в комментариях до вашего ответа статический метод уже обсуждался, поэтому я и подумал что вы отвечали с учетом этих комментов