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.

Sign up for GitHub

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

Jump to bottom

write login flow tests #5487

Closed
wants to merge 7 commits into from
Closed

write login flow tests #5487

wants to merge 7 commits into from

Conversation

ahmadshaheer
Copy link
Collaborator

@ahmadshaheer ahmadshaheer commented Jul 14, 2024 •
edited
Loading

Summary

Write Unit tests for the login flow cases

  • Login form is rendered correctly
  • Display "Please enter a valid email address" if email is not provided
  • Display "Invalid configuration detected, please contact your administrator" if Invalid email is submitted
  • Display "Login with SSO" if valid email is provided and Next is clicked
  • Display email, password, forgot password if password=Y
  • Display tooltip with text "Ask your admin to reset your password and send you a new invite link" if forgot password is clicked

Related Issues / PR's

close #5486

Screenshots

NA

Affected Areas and Manually Tested Areas

@github-actions github-actions bot added the enhancement New feature or request label Jul 14, 2024
@ahmadshaheer ahmadshaheer changed the title Feat/write login flow tests write login flow tests Jul 14, 2024
@ahmadshaheer ahmadshaheer marked this pull request as ready for review July 15, 2024 15:35
@ahmadshaheer
Copy link
Collaborator Author

closing this PR in favor of #5540 because of the messy commit history

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@YounixM YounixM Awaiting requested review from YounixM YounixM is a code owner

Assignees
No one assigned
Labels
enhancement New feature or request
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Add Tests for the Login Flow
1 participant
@ahmadshaheer