-
Notifications
You must be signed in to change notification settings - Fork 363
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-6611] - Add new assertions for linode backup tests #11326
Conversation
Coverage Report: ✅ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Clean! 🧼 Thanks Azure!
Edit: You're welcome to disregard the test failures but I think all of those are fixed in develop now -- shout out to @bnussman-akamai for the machine images fixes!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
"@linode/manager": Tests | ||
--- | ||
|
||
Add new assertions for linode backup tests ([#11326](https://github.com/linode/manager/pull/11326)) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Add new assertions for linode backup tests ([#11326](https://github.com/linode/manager/pull/11326)) | |
Add new assertions for linode backup Cypress tests ([#11326](https://github.com/linode/manager/pull/11326)) |
Optional nit for clarity
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Cloud Manager UI test results🔺 4 failing tests on test run #3 ↗︎
Details
TroubleshootingUse this command to re-run the failing tests: yarn cy:run -s "cypress/e2e/core/linodes/update-linode-labels.spec.ts,cypress/e2e/core/linodes/linode-storage.spec.ts,cypress/e2e/core/linodes/linode-storage.spec.ts,cypress/e2e/core/linodes/switch-linode-state.spec.ts" |
Cloud Manager E2E Run #6910
Run Properties:
|
Project |
Cloud Manager E2E
|
Branch Review |
develop
|
Run status |
Failed #6910
|
Run duration | 28m 50s |
Commit |
8dfa028491: test: [M3-6611] - Add new assertions for linode backup tests (#11326)
|
Committer | Azure-akamai |
View all properties for this run ↗︎ |
Test results | |
---|---|
Failures |
1
|
Flaky |
3
|
Pending |
2
|
Skipped |
3
|
Passing |
461
|
View all changes introduced in this branch ↗︎ |
Tests for review
cypress/e2e/core/linodes/linode-storage.spec.ts • 1 failed test
Test | Artifacts | |
---|---|---|
linode storage tab > try to delete in use disk |
Screenshots
Video
|
resize-linode.spec.ts • 1 flaky test
Test | Artifacts | |
---|---|---|
resize linode > resizes a linode by increasing size when offline: cold migration |
Screenshots
Video
|
linode-config.spec.ts • 1 flaky test
Test | Artifacts | |
---|---|---|
Linode Config management > End-to-End > Clones a config |
Screenshots
Video
|
update-linode-labels.spec.ts • 1 flaky test
Test | Artifacts | |
---|---|---|
update linode label > updates a linode label from the "Settings" tab |
Screenshots
Video
|
Description 📝
The cypress/e2e/core/linodes/backup-linode.spec.ts have a few shortcomings that should be addressed:
Changes 🔄
List any change(s) relevant to the reviewer.
How to test 🧪
As an Author, I have considered 🤔