-
Notifications
You must be signed in to change notification settings - Fork 1.2k
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
chore: Update test util doc examples so they don't error out #7612
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
seems like a fair number of issues in the docs, do we have some way we can run them? or how are you checking them?
@@ -1012,7 +1012,7 @@ it('ComboBox can select an option via keyboard', async function () { | |||
// Render your test component/app and initialize the combobox tester | |||
let {getByTestId} = render( | |||
<Provider theme={defaultTheme}> | |||
<ComboBox data-testid="test-combobox"> | |||
<ComboBox label="test-combobox" data-testid="test-combobox"> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
do we need this in the example? do we need both attributes?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I guess because of the warning about missing label? I think it's probably ok to keep the docs example minimal even so. we're omitting all of the content here as well
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
yeah it was cuz of the missing labels, but I'm fine getting rid of it again
@snowystinger |
@snowystinger to check these I basically copy pasted them into our test suite and made sure they ran after filling in the contents. They were originally constructed from test cases we already had but messed some of them up when paring them down/copying them over. |
Closes
✅ Pull Request Checklist:
📝 Test Instructions:
🧢 Your Project: