Skip to content
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

test: [DBAAS1-661] - Test automation for database resize #10461

Merged
merged 19 commits into from
May 28, 2024

Conversation

sujai-git
Copy link
Contributor

@sujai-git sujai-git commented May 13, 2024

Description 📝

Automation test added to cover resize feature for DBAAS. Below are the testcases that are covered in the test:

  1. Verify new tab "Resize" displays for a DBaas cluster
  2. User should be able to select a plan for scaling up current plan.
  3. User should be able to click "Resize Database Cluster" button after plan selection.
  4. Database cluster should update newly selected plan values in UI after updating the plan.
  5. Verify an event is triggered when vertical scaling is initiated and the event is closed when it finishes.
  6. Summary section should display billing related information.
  7. User should not be allowed to select plan smaller than current plan.
  8. User should not be allowed to resize a failed/provisioning database. Verify error message/UI validation.
  9. Verify dbs can be scaled up from dedicated to shared cpu and vice versa.

How to test :

yarn cy:run -s "cypress/e2e/core/databases/resize-database.spec.ts,cypress/e2e/core/notificationsAndEvents/events.spec.ts"

@sujai-git sujai-git requested review from a team as code owners May 13, 2024 20:54
@sujai-git sujai-git requested review from AzureLatte, jdamore-linode and carrillo-erik and removed request for a team May 13, 2024 20:54
@bnussman-akamai bnussman-akamai changed the title test: [DBAAS1-661] - Test automation for database resize. test: [DBAAS1-661] - Test automation for database resize May 15, 2024
@bnussman-akamai bnussman-akamai added e2e Indicates that a PR touches Cypress tests in some way DBaaS Relates to Database as a Service labels May 15, 2024
Copy link
Contributor

@AzureLatte AzureLatte left a comment

Choose a reason for hiding this comment

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

Test passed
Screenshot 2024-05-21 at 11 51 51 AM

Copy link

github-actions bot commented May 21, 2024

Coverage Report:
Base Coverage: 81.74%
Current Coverage: 81.74%

Copy link
Member

Choose a reason for hiding this comment

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

We only need 1 changeset per PR. Can you remove 3 of these and keep 1 of them?

"@linode/manager": Tests
---

Test automation for database resize ([#10461](https://github.com/linode/manager/pull/10461))
Copy link
Contributor

Choose a reason for hiding this comment

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

You only really need one changeset for the manager package. No need to run the yarn script on every commit. Feel free to remove the other changeset files under the manager package, however, since the types in the api-v4 package were changed, that will require it's own changeset.

@bnussman-akamai bnussman-akamai merged commit 49b9a01 into linode:develop May 28, 2024
18 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
DBaaS Relates to Database as a Service e2e Indicates that a PR touches Cypress tests in some way
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants