Задание 77/79

[Aleksey-Bor]

Мне представляется в тесте по заданию 77/79 -ошибка в тесте.

Тест не учитывает ситуацию, когда в функцию передается строка, которая начинается с большой буквы.

Например, на это решение тест возвращает следующий ответ.

Решение:

const makeItFunny = (str, nElem) => {
let strLowerCase = str.toLowerCase(); 
let rez = '';
let i = 0;

while (i < str.length) {

  if ((i + 1) % nElem === 0)  {
    rez = rez + strLowerCase[i].toUpperCase();
  } else {
    rez = rez + strLowerCase[i];
  }

  ++i;
}  

return rez;
}

Тест отвечает:

Error: expect(received).toEqual(expected) // deep equality
Expected: "I NevEr LooK bAck"
Received: "i NevEr LooK bAck"
at file:///exercises-javascript/modules/50-loops/50-mutators/test.js:5:22
at test (file:///exercises-javascript/src/tests.js:34:5)

Это принципиально неверно. Согласно условию задачи, от нас требуют написать функцию у которой каждый n-ный элемент переведен в верхний регистр. Почему верным считается возврат, у которого первый символ в верхнем регистре?

Задание

Напишите функцию makeItFunny(), которая принимает на вход строку и возвращает ее копию, у которой каждый n-ный > элемент переведен в верхний регистр. n – задается на входе в функцию.

[ashikov]

Добрый день, Алексей!

Тест не учитывает ситуацию, когда в функцию передается строка, которая начинается с большой буквы.

Почему же? Первый тест:

const text = 'I never look back';
expect(f(text, 3)).toEqual('I NevEr LooK bAck');

Строка text начинается с буквы "I" в верхнем регистре.

В условиях задачи ничего не сказано о том, что нужно перед переводом символа в верхний регистр сначала переводить его в нижний. А функция которую вы привели как раз так и поступает, поэтому тесты и не проходят. Согласно условий какой-то из не n-ных элементов, также может быть в верхнем регистре. Если бы была необходима какая-то нормализация входящих данных, то об этом явно было бы сказано.

[Aleksey-Bor]

Ок, спасибо. Согласен пт, 11 сент. 2020 г. в 12:00, Roman Ashikov <notifications@github.com>:

…

Добрый день, Алексей! Тест не учитывает ситуацию, когда в функцию передается строка, которая начинается с большой буквы. Почему же? Первый тест: const text = 'I never look back'; expect(f(text, 3)).toEqual('I NevEr LooK bAck'); Строка text начинается с буквы "I" в верхнем регистре. В условиях задачи ничего не сказано о том, что нужно перед переводом символа в верхний регистр сначала переводить его в нижний. А функция которую вы привели как раз так и поступает, поэтому тесты и не проходят. Согласно условий какой-то из не n-ных элементов, также может быть в верхнем регистре. Если бы была необходима какая-то нормализация входящих данных, то об этом явно было бы сказано. — You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub <#335 (comment)>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/AQ325HWRLDLF43KUZFMBKI3SFHRJ7ANCNFSM4RE4K3QQ> .

[ashikov]

@mokevnin Думаю, можно закрыть этот issue.