Add test for controller Consultor etc. #11
Conversation
| } | ||
|
|
||
| let r = Math.floor(Math.random() * (n <= 0 ? 0 : n - 1)); | ||
| var r = Math.floor(Math.random() * (n <= 0 ? 0 : n - 1)); |
There was a problem hiding this comment.
выдавалась ошибка при npm start, скорее всего связано с моим локальныи окружением
There was a problem hiding this comment.
поставьте 6ю ноду, она поддерживает let и значения по умолчанию в функциях, а эти изменения отмените
|
а чего файл |
|
|
||
| User.createByVKId(usersModel, VK_USER_ID) | ||
| .then(function () { | ||
| User.getByVKId(usersModel, VK_USER_ID) |
There was a problem hiding this comment.
нам не нужно создавать нового пользователя, можно подставить любой произвольный id, userId = 'a12345678901' например
There was a problem hiding this comment.
возможно, мне нужно перепроверить, но по моему там вопросы выбираются по реальным user_id
There was a problem hiding this comment.
в тестовых данных нет ничего реального, тут все изолировано надо проверять
There was a problem hiding this comment.
я могу ошибаться, но если на вход дать фейковый id юзера, то мы не получим вопросов, выборка идёт с учётом id юзера(из users)
There was a problem hiding this comment.
да, точно. Это Consultor.getAllQuestions криво написан, что завязан на пользователей :(
вопрос снимается, надо будет порефакторить.
No description provided.