-
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
[e2e] [Complete user profile - unassociated] Ensure first name / last name have char limits #971
[e2e] [Complete user profile - unassociated] Ensure first name / last name have char limits #971
Conversation
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.
Looking good!
Could we make a ticket for future work, and then link it here in the code so we don't lose track of it?
I could see in that ticket:
- adding max lengths to inputs in coordination with the backend team (after the DevChat on Monday)
- a util function that checks a given input field to see if it passes the check
- using that util function to check our existing inputs
Future work:
- right now first name and last name restrict just based on maxLength, but we could make future tests to check that trying to submit data with restricted char sets (after some discussion) throws an error as well
e2e/pages/shared-lending-platform/NonAssociatedUserProfile.spec.ts
Outdated
Show resolved
Hide resolved
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.
Looks great @tanner-ricks! Thanks for adding the TODO comment. Looking forward to chatting about this in DevChat!
A great first PR of many!
…-profile-ensure-char-limits
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.
Giving you the approval, but I'm hoping some more congrats gifs are coming @tanner-ricks! 🎉
Welcome to the team!
Characters limits and unicode character input tolerance need to be validated in an e2e test.
Changes
How to test this PR