-
Notifications
You must be signed in to change notification settings - Fork 85
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(accordion): migrate component tests to playwright #6255
Conversation
2ba588a
to
e2781af
Compare
This pull request is automatically built and testable in CodeSandbox. To see build info of the built libraries, click here or the icon next to each commit SHA. Latest deployment of this branch, based on commit 4e64dbe:
|
|
||
([ | ||
["white", "rgb(255, 255, 255)", "be.visible"], | ||
["transparent", "rgba(0, 0, 0, 0)", "be.visible"], |
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.
Should third parameter "be.visible" be here?
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.
good spot, thanks. Looks like I forgot to remove this from the Cypress test even though we're not using it.
page, | ||
}) => { | ||
await mount(<AccordionComponent />, { | ||
hooksConfig: { focusRedesignOptOut: true }, |
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.
Should we import HooksConfig from playwright and use it here like: "await mount("
mount, | ||
page, | ||
}) => { | ||
await mount(<AccordionGroupComponent />, { |
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.
same here
page, | ||
}) => { | ||
await mount(<AccordionGroupComponent />, { | ||
hooksConfig: { focusRedesignOptOut: true }, |
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.
same here
page, | ||
}) => { | ||
await mount(<AccordionGroupComponent />, { | ||
hooksConfig: { focusRedesignOptOut: true }, |
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.
same here
Nothing further to add |
e2781af
to
4e64dbe
Compare
🎉 This PR is included in version 119.12.2 🎉 The release is available on: Your semantic-release bot 📦🚀 |
Proposed behaviour
The Accordion component should have in-browser tests using the Playwright framework
Current behaviour
Accordion currently has Cypress tests instead of Playwright tests
Checklist
d.ts
file added or updated if requiredQA
Additional context
Testing instructions
The following CodeSandbox is an example of the broken behaviour.
You can see the new behaviour by looking at the version in the comment by
codesandbox[bot]
.