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

test(form-field): Add testing-library/react and cypress tests #482

Merged
merged 13 commits into from
Mar 13, 2020
Merged

test(form-field): Add testing-library/react and cypress tests #482

merged 13 commits into from
Mar 13, 2020

Conversation

mannycarrera4
Copy link
Contributor

@mannycarrera4 mannycarrera4 commented Feb 24, 2020

Summary

Back when we re wrote some test to React Testing Library, we didn't get around to FormField, this PR addresses those test
Fixes FormField test in: #286

Checklist

  • branch has been rebased on the latest master commit
  • tests are changed or added
  • yarn test passes

Additional References

@cypress
Copy link

cypress bot commented Feb 24, 2020



Test summary

197 0 0 0


Run details

Project canvas-kit
Status Passed
Commit 68b563a ℹ️
Started Mar 13, 2020 9:01 PM
Ended Mar 13, 2020 9:04 PM
Duration 03:01 💡
OS Linux Ubuntu Linux - 16.04
Browser Electron 78

View run in Cypress Dashboard ➡️


This comment has been generated by cypress-bot as a result of this project's GitHub integration settings. You can manage this integration in this project's settings in the Cypress Dashboard

@lychyi lychyi added this to In progress (PRs) in Current Sprint (7/20 - 8/9) via automation Feb 26, 2020
@mannycarrera4 mannycarrera4 marked this pull request as ready for review March 4, 2020 16:27
Copy link
Contributor

@anicholls anicholls left a comment

Choose a reason for hiding this comment

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

Can you also add tests for:

modules/form-field/react/spec/FormField.spec.tsx Outdated Show resolved Hide resolved
modules/form-field/react/spec/FormField.spec.tsx Outdated Show resolved Hide resolved
modules/form-field/react/spec/FormField.spec.tsx Outdated Show resolved Hide resolved
modules/form-field/react/stories/stories.tsx Outdated Show resolved Hide resolved
modules/form-field/react/stories/stories.tsx Outdated Show resolved Hide resolved
@anicholls anicholls merged commit 05bb0b7 into Workday:master Mar 13, 2020
Current Sprint (7/20 - 8/9) automation moved this from In Progress to Done Mar 13, 2020
@anicholls anicholls changed the title test(form-field): Add new test for form field test(form-field): Add testing-library/react and cypress tests Mar 13, 2020
@jpante jpante added this to the Component Testing Coverage milestone Apr 3, 2020
@anicholls anicholls mentioned this pull request Apr 13, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
No open projects
Development

Successfully merging this pull request may close these issues.

None yet

3 participants