-
Notifications
You must be signed in to change notification settings - Fork 5
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Testing #92
base: master
Are you sure you want to change the base?
Testing #92
Conversation
app.js
Outdated
@@ -25,7 +25,7 @@ if (process.env.NODE_ENV && process.env.NODE_ENV === 'prod') { | |||
|
|||
// app.use(require('connect-livereload')()); | |||
app.set('view engine', '.hbs'); | |||
app.set('port', process.env.PORT || 8080); | |||
app.set('port', process.env.PORT || 8081); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
зачем?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
эт случайно. ща уберу
|
||
const validData = { | ||
password: '1', | ||
email: 'bugiMail' |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
email не валидный
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
просто он в базе лежит, а я других паролей не помню)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
погоди, а ты прям к реальной базе подключаешься?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
честно... сам не понял. почему то только 2 прошли туда
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
по сути нужно замокать БД, это как то через proxy делается
Просто я вот щас удалю того юзера и все повалится, так не должно быть.
В целом хорошо, но эт касяк)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
тут и паспорт надо заменять получается... дикость) будет сложна)))
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
ага :D
|
||
const validData = { | ||
password: '1', | ||
email: 'bugiMail' |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
по сути нужно замокать БД, это как то через proxy делается
Просто я вот щас удалю того юзера и все повалится, так не должно быть.
В целом хорошо, но эт касяк)
Тестирование SignUp и SignIn