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(ComboBox): add vrt #9497

Merged
merged 4 commits into from
Aug 26, 2021
Merged

Conversation

jnm2377
Copy link
Contributor

@jnm2377 jnm2377 commented Aug 18, 2021

Closes #9268

Adds percy snapshots for ComboBox

Testing / Reviewing

  • view Percy test details, check ComboBox snapshot, ensure styles are applied correctly, approve snapshot if visuals are correct :)
  • ensure all tests are passing

@jnm2377 jnm2377 requested a review from a team as a code owner August 18, 2021 03:15
@netlify
Copy link

netlify bot commented Aug 18, 2021

✔️ Deploy Preview for carbon-react-next ready!

🔨 Explore the source changes: 2d1558a

🔍 Inspect the deploy log: https://app.netlify.com/sites/carbon-react-next/deploys/6127744af78cbb000a9f79fc

😎 Browse the preview: https://deploy-preview-9497--carbon-react-next.netlify.app

@jnm2377
Copy link
Contributor Author

jnm2377 commented Aug 18, 2021

@tay1orjones I tried to add a click event on this like Accordion tests, and check to see that the list items are showing, but I'm running into issues with it. I get an error when I try to check for Option 1 text. Any idea what I'm doing wrong?

@netlify
Copy link

netlify bot commented Aug 18, 2021

✔️ Deploy Preview for carbon-elements ready!

🔨 Explore the source changes: 2d1558a

🔍 Inspect the deploy log: https://app.netlify.com/sites/carbon-elements/deploys/6127744a9cd2b90008b5ebc4

😎 Browse the preview: https://deploy-preview-9497--carbon-elements.netlify.app

@netlify
Copy link

netlify bot commented Aug 18, 2021

✔️ Deploy Preview for carbon-components-react ready!
Built without sensitive environment variables

🔨 Explore the source changes: 2d1558a

🔍 Inspect the deploy log: https://app.netlify.com/sites/carbon-components-react/deploys/6127744a03302e00080995a7

😎 Browse the preview: https://deploy-preview-9497--carbon-components-react.netlify.app

@jnm2377
Copy link
Contributor Author

jnm2377 commented Aug 18, 2021

Ok, I think I fixed the error I was running into. Should be ready for review.

Comment on lines +18 to +23
const items = [
{
id: 'option-0',
text:
'An example option that is really long to show what should be done to handle long text',
},
Copy link
Member

Choose a reason for hiding this comment

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

Forgot to mention something I found useful for Dropdown was the generateItems and generateGenericItem helpers from ../ListBox/test-helpers. It helps cut down the boilerplate a bit

Copy link
Member

@tw15egan tw15egan left a comment

Choose a reason for hiding this comment

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

Sorry for the late review! Nice work

@kodiakhq kodiakhq bot merged commit b5628f4 into carbon-design-system:main Aug 26, 2021
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.

Add VRT for ComboBox
3 participants