Есть такая функция, которая принимает в себя две даты, заводит пустой счётчик и увеличивает его на один если в массиве дат производственного календаря есть такая дата.
function countNonWorkingDays(fromDate, toDate) {
let counter = 0;
const formatProductionCalendar = productionCalendar.map(element => {
const d = new Date(element);
return `${('0' + d.getDate()).slice(-2)}-${('0' + (d.getMonth() + 1)).slice(
-2,
)}-${d.getFullYear()}`;
});
let currentDate = moment(fromDate, 'DD-MM-YYYY');
const finishDate = moment(toDate, 'DD-MM-YYYY');
while (currentDate <= finishDate) {
if (formatProductionCalendar.includes(currentDate.format('DD-MM-YYYY'))) {
counter++;
}
currentDate = moment(currentDate).add(1, 'days');
}
return counter;
}
Массив дат нерабочих дней в производственном календаре (в самой функции я форматирую productionCalendar под формат 'DD-MM-YYYY' и в итоге получается правильный массив formatProductionCalenar):
[
'2022-01-01',
'2022-01-02',
'2022-01-03',
'2022-01-04',
'2022-01-05',
'2022-01-06',
'2022-01-07',
'2022-01-08',
'2022-01-09',
'2022-01-15',
'2022-01-16',
'2022-01-22',
'2022-01-23',
'2022-01-29',
'2022-01-30',
]
При запуске подобного теста я ожидаю получить 15 нерабочих дней
const fromDate = new Date(2022, 1, 1);
const toDate = new Date(2022, 2, 1);
describe('countNonWorkingDays working correct', () => {
it('countNonWorkingDays for January', () => {
expect(countNonWorkingDays(fromDate, toDate)).to.equal(15);
});
});
Но вместо этого счётчик возвращает 10. Что я делаю неправильно и есть ли варианты упростить такую функцию? А также рассмотреть вариант при котором fromDate > toDate и получать корректную ошибку