Вопрос №55640 от пользователя Аcя Калинина в уроке «Модификация», курс «JS: Массивы»

Maksim Litvinov

Добрый день, Анастасия! Чтобы разобраться с ошибкой, нужно изучить вывод тестов:

xpect(received).toEqual(expected) // deep equality

    - Expected  - 1
    + Received  + 3

    - Array []
    + Array [
    +   undefined,
    + ]

      4 |
      5 | test('get', () => {
    > 6 |   expect(swap([])).toEqual([]);
        |                    ^

Тесты ожидают, что если передать в функцию пустой массив, вернется пустой массив, но фактически вернулся массив с undefined. Значит, функция работает некорректно с пустым массивом. Я вижу, что такая проверка есть у вас в коде. Нужно проверить, срабатывает ли она. Попробуйте поставить внутри console.log("!!!!!"). Если сообщение не выведется на экран, значит эта проверка не отрабатывает. В этом случае, нужно будет проверить условие в if, все ли там корректно.

Аcя Калинина

Я изменила код, теперь работает с пустым массивом. Но возникла проблема, когда в массиве стало 2 элемента. https://ru.hexlet.io/code_reviews/420003

Maksim Litvinov

На самом деле, все гораздо проще :) И первая и вторая версия кода из ревью будут работать, если поправите опечатку в свойстве length

Аcя Калинина

Да уж, спасибо) Не думала, что могу такую ошибку допустить. Надеюсь, в дальнейшем глаз будет заточен.