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(TextInput): update TextInput tests to RTL #11996

Merged
merged 3 commits into from
Aug 25, 2022

Conversation

tw15egan
Copy link
Collaborator

Closes #11995

Updates tests for TextInput to RTL

Changelog

Changed

-Tests should all be following the new RTL structure/format

Removed

  • Old tests

Testing / Reviewing

Ensure all tests pass, cover all props/behaviors, and that the proper structure is being adhered to.

@tw15egan tw15egan requested a review from a team as a code owner August 23, 2022 12:24
@tw15egan tw15egan changed the title test(TextInput): update TextInput tests to RTL test(TextInput): update TextInput tests to RTL Aug 23, 2022
@tw15egan
Copy link
Collaborator Author

@abbeyhrt the test script was a huge help! 🎉

@netlify
Copy link

netlify bot commented Aug 23, 2022

Deploy Preview for carbon-components-react ready!

Built without sensitive environment variables

Name Link
🔨 Latest commit f695dd7
🔍 Latest deploy log https://app.netlify.com/sites/carbon-components-react/deploys/6307609740870700080abbe8
😎 Deploy Preview https://deploy-preview-11996--carbon-components-react.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site settings.

@netlify
Copy link

netlify bot commented Aug 23, 2022

Deploy Preview for carbon-elements ready!

Name Link
🔨 Latest commit f695dd7
🔍 Latest deploy log https://app.netlify.com/sites/carbon-elements/deploys/630760979d8f5f0008083116
😎 Deploy Preview https://deploy-preview-11996--carbon-elements.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site settings.

@kodiakhq kodiakhq bot merged commit 40be69e into carbon-design-system:main Aug 25, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Archived in project
Development

Successfully merging this pull request may close these issues.

Use RTL for testing in TextInput/TextInput-test.js
3 participants