-
Notifications
You must be signed in to change notification settings - Fork 2.7k
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
Added basic test cases to donut chart #33127
base: charting/web-components
Are you sure you want to change the base?
Added basic test cases to donut chart #33127
Conversation
}); | ||
|
||
test('Should render chart properly', async ({ page }) => { | ||
const element = page.locator('fluent-donut-chart'); |
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.
Does the page render all donut-chart stories by default? We should have separate examples for testing to avoid modifying the test file whenever the stories change.
See this:
fluentui/packages/web-components/src/accordion/accordion.spec.ts
Lines 21 to 32 in e32acb8
await page.setContent(/* html */ ` | |
<fluent-accordion expand-mode="multi"> | |
<fluent-accordion-item> | |
<span slot="heading">Heading 1</span> | |
<div>Content 1</div> | |
</fluent-accordion-item> | |
<fluent-accordion-item> | |
<span slot="heading">Heading 2</span> | |
<div>Content 2</div> | |
</fluent-accordion-item> | |
</fluent-accordion> | |
`); |
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.
For now added test cases on existing example which is there in the stories.
Added basic test cases to donut chart and horizontalbar chart