Skip to content

test: [M3-10492] - Prepare Cypress tests for IAM navigation feature flag flip#12723

Merged
jdamore-linode merged 7 commits intolinode:developfrom
jdamore-linode:M3-10492-cypress-tests-iam-navigation-feature-flag
Aug 20, 2025
Merged

test: [M3-10492] - Prepare Cypress tests for IAM navigation feature flag flip#12723
jdamore-linode merged 7 commits intolinode:developfrom
jdamore-linode:M3-10492-cypress-tests-iam-navigation-feature-flag

Conversation

@jdamore-linode
Copy link
Copy Markdown
Contributor

Description 📝

This updates our Cypress tests that had previously been mocking the IAM feature flag to be disabled so that they now pass with the flag enabled.

Changes 🔄

  • Mocks the iamRbacPrimaryNavChanges feature flag to be enabled in tests that had previously been mocking it to be disabled
  • Adjust tests as needed so that they pass when iamRbacPrimaryNavChanges is enabled
  • Removes the iamRbacPrimaryNavChanges mock in tests that already pass when the feature flag is enabled and disabled
  • Skip a couple tests that are currently failing due to bugs
  • Add documentation, comments, etc. to help with clean up/bug fixes

Scope 🚢

Upon production release, changes in this PR will be visible to:

  • All customers
  • Some customers (e.g. in Beta or Limited Availability)
  • No customers / Not applicable

Target release date 🗓️

8/26/2026

How to test 🧪

We can rely on CI for this. Alternatively, you can use the following command to run all of the tests that were touched by this PR:

pnpm cy:run -s "cypress/e2e/core/account/account-maintenance.spec.ts,cypress/e2e/core/account/quotas-nav.spec.ts,cypress/e2e/core/account/restricted-user-details-pages.spec.ts,cypress/e2e/core/account/service-transfer.spec.ts,cypress/e2e/core/billing/billing-contact.spec.ts,cypress/e2e/core/billing/billing-invoices.spec.ts,cypress/e2e/core/billing/credit-card-expired-banner.spec.ts,cypress/e2e/core/billing/google-pay.spec.ts,cypress/e2e/core/billing/restricted-user-billing.spec.ts,cypress/e2e/core/billing/smoke-billing-activity.spec.ts,cypress/e2e/core/general/smoke-deep-link.spec.ts,cypress/e2e/core/linodes/rebuild-linode.spec.ts,cypress/e2e/core/linodes/smoke-delete-linode.spec.ts,cypress/e2e/core/linodes/smoke-linode-landing-table.spec.ts,cypress/e2e/core/notificationsAndEvents/qemu-reboot-upgrade-notice.spec.ts,cypress/e2e/core/parentChild/account-switching.spec.ts"
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 tests and CI checks are passing
  • TypeScript compilation succeeded without errors
  • Code passes all linting rules

@jdamore-linode jdamore-linode requested a review from a team as a code owner August 19, 2025 14:28
@jdamore-linode jdamore-linode requested review from dwiley-akamai and hasyed-akamai and removed request for a team August 19, 2025 14:28
@github-project-automation github-project-automation bot moved this from Review to Approved in Cloud Manager Aug 19, 2025
@jdamore-linode jdamore-linode added Approved Multiple approvals and ready to merge! and removed Ready for Review labels Aug 19, 2025
Copy link
Copy Markdown
Contributor

@dwiley-akamai dwiley-akamai left a comment

Choose a reason for hiding this comment

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

CI is in good shape 🚀

@linode-gh-bot
Copy link
Copy Markdown
Collaborator

Cloud Manager UI test results

🎉 731 passing tests on test run #13 ↗︎

❌ Failing✅ Passing↪️ Skipped🕐 Duration
0 Failing731 Passing4 Skipped115m 29s

@jdamore-linode jdamore-linode merged commit 7283132 into linode:develop Aug 20, 2025
35 checks passed
@github-project-automation github-project-automation bot moved this from Approved to Merged in Cloud Manager Aug 20, 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! 🚨 Urgent

Projects

Archived in project

Development

Successfully merging this pull request may close these issues.

6 participants