Skip to content
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

Open
wants to merge 3 commits into
base: master
Choose a base branch
from
Open

Testing #92

wants to merge 3 commits into from

Conversation

TrefilovAnd
Copy link
Contributor

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

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);
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