-
Notifications
You must be signed in to change notification settings - Fork 8
Add test for dynamically added inputs #98
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
Add test for dynamically added inputs #98
Conversation
152beba to
62fba3e
Compare
| \\"nextId\\": 22, | ||
| \\"node\\": { | ||
| \\"type\\": 3, | ||
| \\"textContent\\": \\"textarea should be masked\\", |
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.
bug here
| \\"tagName\\": \\"input\\", | ||
| \\"attributes\\": { | ||
| \\"id\\": \\"input\\", | ||
| \\"value\\": \\"input should not be masked\\" |
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.
lgtm
| \\"type\\": 3, | ||
| \\"data\\": { | ||
| \\"source\\": 5, | ||
| \\"text\\": \\"input should not be maskedmoo\\", |
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.
lgtm
| \\"attributes\\": { | ||
| \\"id\\": \\"input-unmasked\\", | ||
| \\"class\\": \\"rr-unmask\\", | ||
| \\"value\\": \\"************************\\" |
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.
this should be unmasked
| \\"type\\": 3, | ||
| \\"data\\": { | ||
| \\"source\\": 5, | ||
| \\"text\\": \\"************************\\", |
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.
unmasked
| \\"data\\": { | ||
| \\"source\\": 5, | ||
| \\"text\\": \\"textarea test\\", | ||
| \\"text\\": \\"**************************\\", |
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.
should be unmasked
| \\"nextId\\": 22, | ||
| \\"node\\": { | ||
| \\"type\\": 3, | ||
| \\"textContent\\": \\"textarea should not be masked\\", |
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.
lgtm
| \\"type\\": 3, | ||
| \\"data\\": { | ||
| \\"source\\": 5, | ||
| \\"text\\": \\"motextarea should not be masked\\", |
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.
lgtm
016ce19 to
02ab226
Compare
62fba3e to
a0cc839
Compare
…ded inputs (mutation vs snapshot)
a0cc839 to
a48b390
Compare
add tests for different maskAllInput configurations on dynamically added inputs (mutation vs snapshot)