test: [DI-26399]- Add Cypress verification tests for CloudPulse NodeBalancer widget #12568
Conversation
|
Temporarily skipping a flaky test case in the time range picker spec due to inconsistent behavior. The test will be re-enabled once the root cause is addressed. |
mjac0bs
changed the title
mjac0bs
changed the title
agorthi-akamai
requested review from
coliu-akamai and
hana-akamai
and removed request for
a team
…balancer-widget-verification_linode_July_24
nikhagra-akamai
added
Add'tl Approval Needed
coliu-akamai
left a comment
coliu-akamai
left a comment
There was a problem hiding this comment.
✅ confirmed tests pass
thank you!
| }); | ||
| it('should allow users to select the desired aggregation and view the latest data from the API displayed in the graph', () => { |
There was a problem hiding this comment.
nit (non-blocking): could we add a line of space between each test/before each for better readability?
There was a problem hiding this comment.
Done — spacing added between tests and before each block as suggested.
| }); | ||
|
|
||
| it('should implement and validate custom date/time picker for a specific date and time range', () => { | ||
| it.skip('should implement and validate custom date/time picker for a specific date and time range', () => { |
There was a problem hiding this comment.
question: I might be out of the loop - why are we skipping this test?
There was a problem hiding this comment.
Temporarily skipping a flaky test case in the time range picker spec due to inconsistent behavior. The test will be re-enabled once the root cause is addressed.
…ps://github.com/agorthi-akamai/manager into nodebalancer-widget-verification_linode_July_24
…balancer-widget-verification_linode_July_24
Cloud Manager UI test results🎉 695 passing tests on test run #10 ↗︎
|
6 participants
Description 📝
This commit introduces a comprehensive integration test suite for validating the CloudPulse Nodebalancer Dashboard UI and its interaction with backend APIs. The tests ensure the accuracy, functionality, and responsiveness of the metrics widgets under various user interactions.
Test cases include:
Mocks added:
These tests ensure that future changes to the CloudPulse Nodebalancer integration do not break core functionality or visual consistency.
Changes 🔄
List any change(s) relevant to the reviewer.
Target release date 🗓️: 26/07/2025
Please specify a release date (and environment, if applicable) to guarantee timely review of this PR. If exact date is not known, please approximate and update it as needed.
Preview 📷
Include a screenshot or screen recording of the change.
Uploading nodebalancer-widget-verification.spec.ts.mp4…
🔒 Use the Mask Sensitive Data setting for security.
💡 Use
<video src="" />tag when including recordings in table.How to test 🧪
pnpm cypress run --spec cypress/e2e/core/cloudpulse/nodebalancer-widget-verification.spec.ts
As an Author, to speed up the review process, I considered 🤔
👀 Doing a self review
❔ Our contribution guidelines
🤏 Splitting feature into small PRs
➕ Adding a changeset
🧪 Providing/improving test coverage
🔐 Removing all sensitive information from the code and PR description
🚩 Using a feature flag to protect the release
👣 Providing comprehensive reproduction steps
📑 Providing or updating our documentation
🕛 Scheduling a pair reviewing session
📱 Providing mobile support
♿ Providing accessibility support
As an Author, before moving this PR from Draft to Open, I confirmed ✅