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

Test user page #192

Merged
merged 2 commits into from
Jul 27, 2023
Merged

Test user page #192

merged 2 commits into from
Jul 27, 2023

Conversation

dadukhankevin
Copy link
Contributor

@dadukhankevin dadukhankevin commented Jul 21, 2023

Some basic tests for /user
CanUpdateAccountInfo

  • changes the email address and display name and ensures a success notification is shown
  • it might be good to change this to check if the email verification shows up but I'm out of time

DisplaysFormErrorsOnInvalidData

  • If the email is blank there should be an error

CanResetPassword

  • Simply ensures that clicking on the reset password link works

** I'm having some trouble running any tests so this is a draft, this is mostly just me trying to figure out tests and hopefully, it can actually help

@github-actions
Copy link

UI unit Tests

1 tests   1 ✔️  0s ⏱️
1 suites  0 💤
1 files    0

Results for commit a588280.

@github-actions
Copy link

github-actions bot commented Jul 21, 2023

Unit Tests

8 tests   8 ✔️  9s ⏱️
3 suites  0 💤
1 files    0

Results for commit 685d60b.

♻️ This comment has been updated with latest results.

@dadukhankevin dadukhankevin added the engineering Non user-facing change label Jul 21, 2023
@hahn-kev hahn-kev marked this pull request as ready for review July 27, 2023 05:59
@hahn-kev hahn-kev merged commit 45927f1 into develop Jul 27, 2023
4 checks passed
@myieye myieye deleted the testUserPage branch July 27, 2023 08:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
engineering Non user-facing change
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants