-
Notifications
You must be signed in to change notification settings - Fork 1
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
added user validation stuff #11
Conversation
add username to seed data file
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.
like model file not the latest version, did you pull latest?
I made one small change to the seed data file (didn't have a username). Happy to approve but think there's room for some more unit tests;
- trying to create a user with an existing email address
- trying to create a user with an existing username
- password validations (too short, white spaces, etc.)
- unable to create a user with an invalid username/password/email address
We can make this a separate ticket though.
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.
Happy to approve but think there’s room for some more unit tests;
- trying to create a user with an existing email address
- trying to create a user with an existing username
- password validations (too short, white spaces, etc.)
- unable to create a user with an invalid username/password/emaill address
We can make this a separate ticket though.
No description provided.