Skip to content

test: [DI-30016] - Add spec for Edit notification channel#13431

Open
shnagend-akamai wants to merge 6 commits intolinode:developfrom
shnagend-akamai:edit_notification_channel_automation_linodePR_linode_February_24
Open

test: [DI-30016] - Add spec for Edit notification channel#13431
shnagend-akamai wants to merge 6 commits intolinode:developfrom
shnagend-akamai:edit_notification_channel_automation_linodePR_linode_February_24

Conversation

@shnagend-akamai
Copy link
Contributor

@shnagend-akamai shnagend-akamai commented Feb 24, 2026

Description 📝

This PR introduces a new test specification for editing a notification channel in the Linode Manager application. It covers updating the channel name and modifying the list of recipients.

Changes 🔄

  • Adds a test spec file to cover the "Edit notification channel" functionality.
  • Includes UI test scripts to automate the workflow of editing a notification channel via the application interface.
  • Corresponding API intercepts have been added to mock scenarios.
  • The new specs ensure that notifications channels can be edited successfully, and that UI and backend behaviors are as expected.

Target release date 🗓️

Upcoming release

Preview 📷

alert-notification-channel-edit.spec.ts.mp4

How to test 🧪

pnpm cy:run -s "cypress/e2e/core/cloudpulse/alert-notification-channel-edit.spec.ts"

Author Checklists

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


  • I have read and considered all applicable items listed above.

As an Author, before moving this PR from Draft to Open, I confirmed ✅

  • All tests and CI checks are passing
  • TypeScript compilation succeeded without errors
  • Code passes all linting rules

@shnagend-akamai shnagend-akamai marked this pull request as ready for review February 24, 2026 14:34
@shnagend-akamai shnagend-akamai requested a review from a team as a code owner February 24, 2026 14:34
Copy link
Contributor

@dwiley-akamai dwiley-akamai left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Test passes locally & in CI ✅

@shnagend-akamai
Copy link
Contributor Author

@dmcintyr-akamai Can you please review this and approve.

@santoshp210-akamai santoshp210-akamai requested review from dmcintyr-akamai and jdamore-linode and removed request for dmcintyr-akamai March 2, 2026 09:46
@github-project-automation github-project-automation bot moved this from Review to Approved in Cloud Manager Mar 4, 2026
@santoshp210-akamai
Copy link
Contributor

The failing cypress tests are unrelated the PR changes, synced again with the base branch. Once the jobs are done, will be merging this PR

@linode-gh-bot
Copy link
Collaborator

Cloud Manager UI test results

🔺 3 failing tests on test run #3 ↗︎

❌ Failing✅ Passing↪️ Skipped🕐 Duration
3 Failing868 Passing11 Skipped36m 56s

Details

Failing Tests
SpecTest
object-storage.e2e.spec.tsCloud Manager Cypress Tests→object storage end-to-end tests » can create and delete object storage buckets
account-switching.spec.tsCloud Manager Cypress Tests→Parent/Child account switching→From Child to Parent » can switch from Proxy user back to Parent account user from Billing page
account-switching.spec.tsCloud Manager Cypress Tests→Parent/Child account switching→From Child to Child » can switch to another Child account as a Proxy user

Troubleshooting

Use this command to re-run the failing tests:

pnpm cy:run -s "cypress/e2e/core/objectStorage/object-storage.e2e.spec.ts,cypress/e2e/core/parentChild/account-switching.spec.ts"

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

Status: Approved

Development

Successfully merging this pull request may close these issues.

6 participants