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: [M3-8758]- Fixed delete-volume.spec.ts flaky test #11365

Merged

Conversation

harsh-akamai
Copy link
Contributor

Description 📝

The test doesn't wait for the Volume to finish being created before attempting to delete.

Changes 🔄

  • Added polling to the Volumes endpoint to wait for the Volume to be ready before proceeding with the rest of the test.

Target release date 🗓️

12/12

How to test 🧪

  • Run delete-volume.spec.ts test using yarn cy:debug
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 unit tests are passing
  • TypeScript compilation succeeded without errors
  • Code passes all linting rules

@harsh-akamai harsh-akamai added the e2e Indicates that a PR touches Cypress tests in some way label Dec 4, 2024
@harsh-akamai harsh-akamai self-assigned this Dec 4, 2024
@harsh-akamai harsh-akamai changed the title test: [M3-8758]- Fixed delete-volume.spec.ts flaky test test: [M3-8758]- Fixed delete-volume.spec.ts flaky test Dec 4, 2024
@linode-gh-bot
Copy link
Collaborator

Cloud Manager UI test results

🎉 465 passing tests on test run #2 ↗︎

❌ Failing✅ Passing↪️ Skipped🕐 Duration
0 Failing465 Passing2 Skipped93m 40s

Copy link
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 @harsh-akamai! Happy to re-review if you make any changes before taking this out of draft, but this is rock solid as-is. Ran the test 50 times and it passed every time 👍

@harsh-akamai harsh-akamai marked this pull request as ready for review December 5, 2024 06:11
@harsh-akamai harsh-akamai requested review from a team as code owners December 5, 2024 06:11
@harsh-akamai harsh-akamai requested review from AzureLatte, hana-akamai and carrillo-erik and removed request for a team December 5, 2024 06:11
@hana-akamai hana-akamai added the Approved Multiple approvals and ready to merge! label Dec 5, 2024
@harsh-akamai harsh-akamai merged commit 82c8990 into linode:develop Dec 6, 2024
23 checks passed
@harsh-akamai harsh-akamai deleted the M3-8757-volume-delete-E2E-test branch December 6, 2024 06:39
Copy link

cypress bot commented Dec 6, 2024

Cloud Manager E2E    Run #6930

Run Properties:  status check passed Passed #6930  •  git commit 82c8990218: test: [M3-8758]- Fixed `delete-volume.spec.ts` flaky test (#11365)
Project Cloud Manager E2E
Branch Review develop
Run status status check passed Passed #6930
Run duration 28m 16s
Commit git commit 82c8990218: test: [M3-8758]- Fixed `delete-volume.spec.ts` flaky test (#11365)
Committer Harsh Shankar Rao
View all properties for this run ↗︎

Test results
Tests that failed  Failures 0
Tests that were flaky  Flaky 3
Tests that did not run due to a developer annotating a test with .skip  Pending 2
Tests that did not run due to a failure in a mocha hook  Skipped 0
Tests that passed  Passing 466
View all changes introduced in this branch ↗︎

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! e2e Indicates that a PR touches Cypress tests in some way
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants