132 cypress test for contact us form#148
132 cypress test for contact us form#148Tausifqureshi786 wants to merge 3 commits intoTechIsHiring:devfrom
Conversation
Add 'data-cy' attribute value to the inputs, error spans, headers and the send message button to safely fetch them in cypress tests
Add test suite for contact us component for the given test cases in the issue TechIsHiring#132
|
Someone is attempting to deploy a commit to a Personal Account owned by @TechIsHiring on Vercel. @TechIsHiring first needs to authorize it. |
|
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
chadstewart
left a comment
There was a problem hiding this comment.
Thanks so much for your contribution. So just wanted to request some changes.
Outside of what I added, could you test the input fields separately please? Also could you add tests where you test where the input fields have have invalid characters or is in an invalid format and not just empty please?
Thanks much and looking forward to your update!
|
Hey @Tausifqureshi786, just wanted to reach out and ask if you were able to make any further progress on this PR? |
|
@chadstewart @Tausifqureshi786 Let me know if you'd like me to finish this up. In terms of consistency with https://github.com/TechIsHiring/techishiring-website/blob/dev/src/components/organisms/footer/footer.cy.tsx, should the test file here be moved to be in same folder as the component? |
Description
Edited the current contact-form.tsx file to add some helper attributes to elements and added the test suite for the isolated component using cypress.
-- No name, email and message is provided
-- All three proper inputs are provided.
-- Only valid email is provided
Related Issue
Resolves: #132
Motivation and Context
n/a
How Has This Been Tested?
Screenshots (if appropriate):
Case 1:

Case 2:

Case 3:
