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: convert TableHeader to RTL #13204

Merged
merged 17 commits into from
Feb 23, 2023
Merged

test: convert TableHeader to RTL #13204

merged 17 commits into from
Feb 23, 2023

Conversation

andreancardona
Copy link
Contributor

@andreancardona andreancardona commented Feb 21, 2023

Closes #12668

Changed
Screenshot 2023-02-22 at 12 05 06

  • Updates and expands test coverage for TableHeader
  • Update snapshot

Testing / Reviewing

Ensure all tests pass and all props are covered by tests

yarn test packages/react/src/components/DataTable/__tests__/TableHeader-test.js

@netlify
Copy link

netlify bot commented Feb 21, 2023

Deploy Preview for carbon-components-react ready!

Built without sensitive environment variables

Name Link
🔨 Latest commit 4af4485
🔍 Latest deploy log https://app.netlify.com/sites/carbon-components-react/deploys/63f546b5398fa50008f2b26e
😎 Deploy Preview https://deploy-preview-13204--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 Feb 21, 2023

Deploy Preview for carbon-components-react ready!

Built without sensitive environment variables

Name Link
🔨 Latest commit dc0bcca
🔍 Latest deploy log https://app.netlify.com/sites/carbon-components-react/deploys/63f547a67992c9000888a01b
😎 Deploy Preview https://deploy-preview-13204--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 Feb 21, 2023

Deploy Preview for carbon-components-react ready!

Built without sensitive environment variables

Name Link
🔨 Latest commit c430e5f
🔍 Latest deploy log https://app.netlify.com/sites/carbon-components-react/deploys/63f7c238d4990200085403fc
😎 Deploy Preview https://deploy-preview-13204--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.

@andreancardona andreancardona changed the title Update table header rtl test: convert TableHeader to RTL Feb 21, 2023
Copy link
Collaborator

@francinelucca francinelucca left a comment

Choose a reason for hiding this comment

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

I see some other props in the component that don't seem to have tests:
-className
-colSpan
-id
-isSortable
-onClick
-translateWithId
-scope
-children

@andreancardona andreancardona marked this pull request as ready for review February 22, 2023 19:04
Copy link
Collaborator

@francinelucca francinelucca left a comment

Choose a reason for hiding this comment

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

LGTM, just one minor detail 🙏🏻

…test.js

Co-authored-by: Francine Lucca <40550942+francinelucca@users.noreply.github.com>
@andreancardona andreancardona enabled auto-merge (squash) February 23, 2023 18:25
@andreancardona andreancardona merged commit 9e4f0fa into carbon-design-system:main Feb 23, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Use RTL for testing in DataTable/__tests__/TableHeader-test.js
3 participants