Skip to content

test: [DBAAS1-1268] Cypress tests for Premium Plans and Horizontal Resizing#12854

Merged
jdamore-linode merged 8 commits intolinode:developfrom
stayal712:DBAAS1-1268-premium-plans-test
Sep 12, 2025
Merged

test: [DBAAS1-1268] Cypress tests for Premium Plans and Horizontal Resizing#12854
jdamore-linode merged 8 commits intolinode:developfrom
stayal712:DBAAS1-1268-premium-plans-test

Conversation

@stayal712
Copy link
Copy Markdown
Contributor

Description 📝

  • Test coverage for dbaas v2 resize
  • Test coverage for dbaas premium plans in available region

Changes 🔄

List any change(s) relevant to the reviewer.

  • Add constants for premium plans database clusters
  • Add common test case for scaling vertically (up/down)
  • Add test case for horizontal scaling

Target release date 🗓️

N/A

How to test 🧪

pnpm cy:run -s "cypress/e2e/core/databases/resize-database.spec.ts"
pnpm cy:run -s "cypress/e2e/core/databases/create-database.spec.ts"
pnpm cy:run -s "cypress/e2e/core/databases/delete-database.spec.ts"
pnpm cy:run -s "cypress/e2e/core/databases/update-database.spec.ts"
pnpm cy:run -s "cypress/e2e/core/databases/advanced-configuration.spec.ts"

  • Run the tests or some subset of tests and confirm all actions can be performed on a premium plan cluster.
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

@stayal712 stayal712 requested review from a team as code owners September 10, 2025 15:12
@stayal712 stayal712 requested review from bnussman-akamai, dmcintyr-akamai and hana-akamai and removed request for a team September 10, 2025 15:12
Copy link
Copy Markdown
Contributor

@jdamore-linode jdamore-linode left a comment

Choose a reason for hiding this comment

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

Thanks @stayal712! Confirmed that the tests pass 🚀

I posted a couple quick asks before approving. Let me know your thoughts, and feel free to reach out if you have any questions!

Copy link
Copy Markdown
Contributor

@jdamore-linode jdamore-linode left a comment

Choose a reason for hiding this comment

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

Thanks for the super quick turnaround @stayal712, looks good! Approved pending CI

@jdamore-linode
Copy link
Copy Markdown
Contributor

@stayal712 I think pulling in the latest changes from develop should resolve those vm-host-maintenance-linode.spec.ts failures, but you're welcome to disregard all of those failures: confirming they're not related to your changes.

One last thing: would you mind running pnpm changeset to generate a changeset doc so that these additions get documented in our changelog entry for the next release?

@linode-gh-bot
Copy link
Copy Markdown
Collaborator

Cloud Manager UI test results

🎉 804 passing tests on test run #6 ↗︎

❌ Failing✅ Passing↪️ Skipped🕐 Duration
0 Failing804 Passing9 Skipped37m 34s

@github-project-automation github-project-automation bot moved this from Review to Approved in Cloud Manager Sep 11, 2025
@bnussman-akamai bnussman-akamai added Approved Multiple approvals and ready to merge! and removed Add'tl Approval Needed Waiting on another approval! labels Sep 11, 2025
@jdamore-linode jdamore-linode merged commit ef51eec into linode:develop Sep 12, 2025
35 checks passed
@github-project-automation github-project-automation bot moved this from Approved to Merged in Cloud Manager Sep 12, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Approved Multiple approvals and ready to merge!

Projects

Archived in project

Development

Successfully merging this pull request may close these issues.

4 participants