Skip to content

test: [M3-8138] - Add Cypress test for login redirect upon API unauthorized response - #10655

Merged
cliu-akamai merged 3 commits into
linode:developfrom
cliu-akamai:feature/M3-8138
Jul 19, 2024
Merged

test: [M3-8138] - Add Cypress test for login redirect upon API unauthorized response#10655
cliu-akamai merged 3 commits into
linode:developfrom
cliu-akamai:feature/M3-8138

Conversation

@cliu-akamai

Copy link
Copy Markdown
Contributor

Description 📝

Add integration test for login redirect upon API unauthorized response.

Major Changes 🔄

  • Check users will be redirected to login page when the API is not authorize.

How to test 🧪

yarn cy:run -s "cypress/e2e/core/general/account-login-redirect.spec.ts"

@cliu-akamai
cliu-akamai requested a review from a team as a code owner July 8, 2024 20:08
@cliu-akamai
cliu-akamai requested review from AzureLatte and removed request for a team July 8, 2024 20:08
@github-actions

github-actions Bot commented Jul 8, 2024

Copy link
Copy Markdown

Coverage Report:
Base Coverage: 82.42%
Current Coverage: 82.42%

@AzureLatte AzureLatte 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

@cliu-akamai
cliu-akamai requested a review from a team as a code owner July 18, 2024 19:16
@cliu-akamai
cliu-akamai requested review from hkhalil-akamai and mjac0bs and removed request for a team July 18, 2024 19:16

@mjac0bs mjac0bs 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.

Thanks for adding the changeset.

Screenshot 2024-07-19 at 10 29 37 AM

The one CI test failure is unrelated flake, so this is good to go!

@mjac0bs mjac0bs added the Approved Multiple approvals and ready to merge! label Jul 19, 2024
@cliu-akamai
cliu-akamai merged commit 36e8ad7 into linode:develop Jul 19, 2024
@cliu-akamai
cliu-akamai deleted the feature/M3-8138 branch July 19, 2024 19:08
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!

Projects

Archived in project

Development

Successfully merging this pull request may close these issues.

4 participants