Skip to content

test: [M3-7434]: Add a Cypress test for OBJ Multicluster Bucket create flow - #10211

Merged
jdamore-linode merged 4 commits into
linode:developfrom
jdamore-linode:M3-7434-obj-multicluster-create-bucket
Feb 22, 2024
Merged

jdamore-linode merged 4 commits into
linode:developfrom
jdamore-linode:M3-7434-obj-multicluster-create-bucket

Conversation

@jdamore-linode

Copy link
Copy Markdown
Contributor

Description 📝

Adds a Cypress test to cover the OBJ Bucket creation flow when OBJ Multicluster is enabled. Using mock API data, it confirms that a bucket can be created, that Cloud uses a region ID rather than a cluster ID in its payload, and that the landing page updates to reflect the new bucket.

Changes 🔄

  • Add a test for Bucket creation when OBJ Multicluster feature is enabled
  • Update OBJ create mock util to be more consistent with other utils
  • Add UI helper to find an autocomplete popper
  • Add mock util for OBJ create error

How to test 🧪

We can rely on CI for this, or yarn cy:debug and search for object-storage.smoke.spec.ts

As an Author I have considered 🤔

Check all that apply

  • 👀 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

@jdamore-linode
jdamore-linode requested a review from a team as a code owner February 20, 2024 22:07
@jdamore-linode
jdamore-linode requested review from hana-akamai and jaalah-akamai and removed request for a team February 20, 2024 22:07
@github-actions

github-actions Bot commented Feb 20, 2024 •

Copy link
Copy Markdown

Coverage Report: ✅
Base Coverage: 81.33%
Current Coverage: 81.33%

@cpathipa cpathipa left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM!

@dwiley-akamai dwiley-akamai left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Test passed remotely and locally ✅

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants