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

Masked input test #56

Draft
wants to merge 352 commits into
base: audit-2024
Choose a base branch
from
Draft

Masked input test #56

wants to merge 352 commits into from

Conversation

sahilSingh-sapient
Copy link
Contributor

Added the following:

  • Functional testing of the masked inputs. (Static & Dynamic)
  • Functional testing with keyboard interactions (focus, type, backspace, arrowLeft)
  • Focus flow is tested already in visibleFocusStates.test.js

);
expect(maskedTelephone).toBe("123-456-781");
});
});
Copy link
Collaborator

Choose a reason for hiding this comment

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

These are great so far! Thank you.

I would also like to ensure we test the mouse functionality as well:

  1. Can we have a test that will click in the middle of a filled out masked input field? I want the test to be able to detect if the visual cursor and the input cursor match up.
  2. Another similar test to see if, when selecting a string in the middle of the input field, the visual selected and the input selected area match up.
  3. Checking what happens if invalid input occurs. e.g. letters in phone numbers.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Copy, will work on getting these in.

Alison Hall and others added 29 commits May 24, 2024 14:43
Add README info about contributing and PRs
Use the automatically changed CSS file formatting
file_put_contents does not create a folder structure when invoked and need to call mkdir to get that taken care of.
Fixed #67: Needed to make folder for cache if not there.
Alison Hall and others added 30 commits July 3, 2024 15:39
Co-authored-by: Alison Hall <alisonh0101@gmail.com>
Signed-off-by: AKSHAY PANCHAL <akshaypanchal67@gmail.com>
Co-authored-by: Alison Hall <alisonh0101@gmail.com>
Signed-off-by: AKSHAY PANCHAL <akshaypanchal67@gmail.com>
Co-authored-by: Alison Hall <alisonh0101@gmail.com>
Signed-off-by: AKSHAY PANCHAL <akshaypanchal67@gmail.com>
Co-authored-by: Alison Hall <alisonh0101@gmail.com>
Signed-off-by: AKSHAY PANCHAL <akshaypanchal67@gmail.com>
Co-authored-by: Alison Hall <alisonh0101@gmail.com>
Signed-off-by: AKSHAY PANCHAL <akshaypanchal67@gmail.com>
Add additional terms for components
[Issue# 49] - Enable Content Grammar / Spelling Issue Fix
…r-character-counter

ENABLE-145 Unit Tests for Character Counter
…idget

Signed-off-by: Jadan Ou <17204721+ocjadan@users.noreply.github.com>
feat:  Added a new webpage for infographics
…-character-counter-widget

Enable-8 create a better character counter widget
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.