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

e2e tests for changing password feature #92

Merged
merged 7 commits into from
Aug 23, 2018

Conversation

dennis00010011b
Copy link

@dennis00010011b dennis00010011b commented Aug 22, 2018

Relates to #82

Added tests for feature 'Change password':

  • check screen "Settings" -> "Change password"
    • checks if "Change password" button is present and enabled
    • screen contains correct title
    • screen contains correct label
    • clicking the button "No" bring back to "Setting" screen
  • validation of errors:
    • error if new password shorter than 8 digits
    • error if new password doesn't match confirmation
    • error if new password match old password
    • error if old password incorrect
    • no errors if old, new, confirm new passwords are correct; user can change password
  • check if new password is accepted:
    • user can log out
    • accepts new password after lock

@ghost ghost assigned dennis00010011b Aug 22, 2018
@ghost ghost added the in progress label Aug 22, 2018
@dennis00010011b dennis00010011b changed the title Change pass e2etest e2e tests for changing password feature Aug 22, 2018
Copy link
Collaborator

@vbaranov vbaranov left a comment

Choose a reason for hiding this comment

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

lint test failed:

✖ 45 problems (12 errors, 33 warnings)
12 errors, 33 warnings potentially fixable with the --fix option.

@dennis00010011b you can fix it with npm run lint -- --fix

@dennis00010011b
Copy link
Author

@vbaranov
fixed

@vbaranov vbaranov self-requested a review August 23, 2018 13:48
@vbaranov vbaranov merged commit 8ac8405 into poanetwork:develop Aug 23, 2018
@ghost ghost removed the awaiting for review label Aug 23, 2018
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.

None yet

2 participants