Skip to content

test: [M3-8668] - Disallow legacy regions from being selected via chooseRegion - #11892

Merged
jdamore-linode merged 3 commits into
linode:developfrom
jdamore-linode:M3-8668-disallow-legacy-regions-chooseregions
Mar 24, 2025
Merged

test: [M3-8668] - Disallow legacy regions from being selected via chooseRegion#11892
jdamore-linode merged 3 commits into
linode:developfrom
jdamore-linode:M3-8668-disallow-legacy-regions-chooseregions

Conversation

@jdamore-linode

@jdamore-linode jdamore-linode commented Mar 20, 2025

Copy link
Copy Markdown
Contributor

Description 📝

Disallow legacy regions from being chosen when using the chooseRegion util within our Cypress tests.

Changes 🔄

  • Add legacy regions to list of disallowed regions

Target release date 🗓️

N/A.

How to test 🧪

  • We can rely on CI for this.
  • Alternatively, run the tests or some subset of tests and confirm no resources get created in disallowed regions
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

@jdamore-linode
jdamore-linode marked this pull request as ready for review March 21, 2025 13:38
@jdamore-linode
jdamore-linode requested a review from a team as a code owner March 21, 2025 13:38
@jdamore-linode
jdamore-linode requested review from dmcintyr-akamai and removed request for a team March 21, 2025 13:38
@jdamore-linode
jdamore-linode requested a review from a team as a code owner March 21, 2025 13:42
@jdamore-linode
jdamore-linode requested review from bill-akamai and cpathipa and removed request for a team March 21, 2025 13:42
@linode-gh-bot

Copy link
Copy Markdown

Cloud Manager UI test results

🎉 533 passing tests on test run #3 ↗︎

❌ Failing✅ Passing↪️ Skipped🕐 Duration
0 Failing533 Passing4 Skipped108m 24s

@github-project-automation github-project-automation Bot moved this from Review to Approved in Cloud Manager Mar 24, 2025
@cpathipa cpathipa added the Approved Multiple approvals and ready to merge! label Mar 24, 2025
@jdamore-linode
jdamore-linode merged commit 4d1476e into linode:develop Mar 24, 2025
@github-project-automation github-project-automation Bot moved this from Approved to Merged in Cloud Manager Mar 24, 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! Ready for Review

Projects

Archived in project

Development

Successfully merging this pull request may close these issues.

4 participants