Skip to content

Comments

Testing#92

Open
TrefilovAnd wants to merge 3 commits intourfu-2016:masterfrom
TrefilovAnd:testing
Open

Testing#92
TrefilovAnd wants to merge 3 commits intourfu-2016:masterfrom
TrefilovAnd:testing

Conversation

@TrefilovAnd
Copy link
Contributor

Тестирование SignUp и SignIn

app.js Outdated
// app.use(require('connect-livereload')());
app.set('view engine', '.hbs');
app.set('port', process.env.PORT || 8080);
app.set('port', process.env.PORT || 8081);
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

зачем?

Copy link
Contributor Author

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'
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

email не валидный

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

просто он в базе лежит, а я других паролей не помню)

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

погоди, а ты прям к реальной базе подключаешься?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

честно... сам не понял. почему то только 2 прошли туда

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

по сути нужно замокать БД, это как то через proxy делается
Просто я вот щас удалю того юзера и все повалится, так не должно быть.
В целом хорошо, но эт касяк)

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

тут и паспорт надо заменять получается... дикость) будет сложна)))

Copy link
Contributor

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'
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

по сути нужно замокать БД, это как то через proxy делается
Просто я вот щас удалю того юзера и все повалится, так не должно быть.
В целом хорошо, но эт касяк)

@TrefilovAnd TrefilovAnd changed the title test auth Testing May 13, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants