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-9030] - Object Storage Gen2 Warning Notice Missing Region Names for Unavailable Regions #11530

Merged

Conversation

dmcintyr-akamai
Copy link
Contributor

@dmcintyr-akamai dmcintyr-akamai commented Jan 16, 2025

Description 📝

Added 2 integration tests to test for presence of warning notice.

Changes 🔄

List any change(s) relevant to the reviewer.

  • test contents of warning notice when there is error for 1 region
  • test contents of warning notice when there is error for >1 region

How to test 🧪

Prerequisites

(How to setup test environment)

Run tests in packages/manager/cypress/e2e/core/objectStorageGen2/bucket-object-gen2.spec.ts. 2 additional tests will be present.

  • 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

@dmcintyr-akamai dmcintyr-akamai requested a review from a team as a code owner January 16, 2025 17:42
@dmcintyr-akamai dmcintyr-akamai requested review from jdamore-linode and removed request for a team January 16, 2025 17:42
@dmcintyr-akamai dmcintyr-akamai requested a review from a team as a code owner January 17, 2025 16:47
@dmcintyr-akamai dmcintyr-akamai requested review from hkhalil-akamai and pmakode-akamai and removed request for a team January 17, 2025 16:47
Copy link

Coverage Report:
Base Coverage: 87.17%
Current Coverage: 87.17%

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 @dmcintyr-akamai, these are solid! Posted a couple pretty trivial suggestions, but functionally the tests are in great shape as-is and are passing reliably for me.

Let me know if you have any questions or thoughts about anything below 👍

Copy link
Contributor

@pmakode-akamai pmakode-akamai left a comment

Choose a reason for hiding this comment

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

All tests pass ✅

Screenshot 2025-01-21 at 11 43 30 AM

Approving pending Joe's comments.

thank you!

@bnussman-akamai bnussman-akamai changed the title test: [MS-9030] Gen2 Warning Notice Missing Region Names for Unavailable Regions test: [MS-9030] Object Storage Gen2 Warning Notice Missing Region Names for Unavailable Regions Jan 22, 2025
@bnussman-akamai bnussman-akamai added the Object Storage deals with Object Storage label Jan 22, 2025
@bnussman-akamai bnussman-akamai changed the title test: [MS-9030] Object Storage Gen2 Warning Notice Missing Region Names for Unavailable Regions test: [MS-9030] - Object Storage Gen2 Warning Notice Missing Region Names for Unavailable Regions Jan 22, 2025
@mjac0bs mjac0bs changed the title test: [MS-9030] - Object Storage Gen2 Warning Notice Missing Region Names for Unavailable Regions test: [M3-9030] - Object Storage Gen2 Warning Notice Missing Region Names for Unavailable Regions Jan 22, 2025
@hkhalil-akamai hkhalil-akamai added Approved Multiple approvals and ready to merge! and removed Add'tl Approval Needed Waiting on another approval! labels Jan 22, 2025
Copy link
Contributor

@hkhalil-akamai hkhalil-akamai left a comment

Choose a reason for hiding this comment

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

Thanks @dmcintyr-akamai!

Aside: fill in or remove unnecessary sections from the PR description to make it more digestible.

@linode-gh-bot
Copy link
Collaborator

Cloud Manager UI test results

🎉 476 passing tests on test run #3 ↗︎

❌ Failing✅ Passing↪️ Skipped🕐 Duration
0 Failing476 Passing2 Skipped86m 51s

@jaalah-akamai jaalah-akamai merged commit 4d0f60e into linode:develop Jan 24, 2025
22 of 23 checks passed
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! Object Storage deals with Object Storage
Projects
None yet
Development

Successfully merging this pull request may close these issues.

7 participants