-
Notifications
You must be signed in to change notification settings - Fork 1.4k
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
feat: added services page playwright tests #3928
Conversation
We would appreciate it if you could provide us with more info about this issue/pr! |
775715f
to
3ff7156
Compare
cc79351
to
c8edc18
Compare
this is base setup this test would be extended to cover happy / edge case flows |
await expect(p99Latency).toEqual('P99 latency (in ms)'); | ||
const errorRate = await page | ||
.locator( | ||
`th[aria-label*="this column's title is Error Rate (% of total)"] .ant-table-column-title`, |
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.
would recommend to add data test id / constant hard code thing would not scale
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.
yes will add data-test-id's when completing the flow
Test Case added :-
No Services Test ->
With Services Data ->