mikhama / core-js-101 Goto Github PK

• Valid passwords will only be alphanumeric characters (+ underscore).
  Знак подчеркивания в пароле не проходит тесты. Либо тесты надо поменять (строку 'Pa__W0rd', перенести в блок допустимых пароле), либо условие.

Tests using assert.equal are misleading.
assert.equal() expects actual value as first parameter, and expected - as second. But tests indicate as expected values something, that is returned from user defined functions, and hardcoded in test files values are indicated as actual.
Node.js assert.equal() documentation - https://nodejs.org/api/assert.html#assert_assert_equal_actual_expected_message.

function getHead in 03-arrays-tasks.js has wrong first example:
[ 1, 3, 4, 5 ], 2 => [ 1, 2 ]
should be: [ 1, 3, 4, 5 ], 2 => [ 1, 3 ]

Я заметил что тут неправильно сделаны тесты.

Если я все так понял - суть промисов в асинхронности, но тесты принимают синхронный код
потому что использутеся

const promises = [ Promise.resolve(1), Promise.resolve(2), Promise.resolve(3) ]

которые резолвятся тут же

Я вроде как понимаю, что из стека вызова они все равно уходят в асинхронное web API, оттуда попадают в Task Queue и уже потом возвращаются в stack.
Но тем не менее просто синхронный код через for проходит тесты. А если добавить реальной задержки промису:

const promises = [ new Promise((res) => { setTimeout(() => res(1), 1000) }), Promise.resolve(2), Promise.resolve(3) ]

вот тогда все сломается как и положено и синхронный код уже не пройдет тест

/**

• Return Promise object that should be resolved with array containing plain values.
• Function receive an array of Promise objects.
• @param {Promise[]} array
• @return {Promise}
• @example
• const promises = [Promise.resolve(1), Promise.resolve(3), Promise.resolve(12)]
• const p = processAllPromises(promises);
• p.then((res) => {

•  console.log(res) // => [1, 2, 3] **Здесь должен быть массив [0, 1, 2]**
  

• })

*/

05-regex-tasks.js - getPasswordValidator

• Valid passwords will only be alphanumeric characters.

AssertionError [ERR_ASSERTION]: Regex matches 'Pa__W0rd'
      + expected - actual

      -false
      +true

Словил ошибку в тестах задачи getPositivesCount и запутался, потому что в выводе + expected - actual перепутаны.

function getFalsyValuesCount. in the parameters of the function, the result was incorrectly specified as an array. receiving data type Number webstorm sends a warning.

1. Алгоритм приведенный по ссылке https://en.wikipedia.org/wiki/Luhn_algorithm некорректен, для прохождения всех тестов отбрасывать последнюю цифру не нужно(либо должна учитываться в окончательном вычислении суммы)
2. Следует придерживаться упрощенного алгоритма: https://ru.wikipedia.org/wiki/%D0%90%D0%BB%D0%B3%D0%BE%D1%80%D0%B8%D1%82%D0%BC_%D0%9B%D1%83%D0%BD%D0%B0