-
Notifications
You must be signed in to change notification settings - Fork 357
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-8319] - Add CM Logout test #10733
Conversation
Coverage Report: ✅ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks great @AzureLatte! Posted a couple little suggestions, but the one about the URL assertion is the only one that matters that much -- approved pending that!
cy.findByTestId('nav-group-profile').click(); | ||
cy.findByTestId('menu-item-Log Out') | ||
.should('be.visible') | ||
.should('be.enabled') | ||
.click(); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
nit: we could make this a little more idiomatic using the userMenu
and userMenuButton
UI helpers
cy.findByTestId('nav-group-profile').click(); | |
cy.findByTestId('menu-item-Log Out') | |
.should('be.visible') | |
.should('be.enabled') | |
.click(); | |
ui.userMenuButton | |
.find() | |
.click(); | |
ui.userMenu | |
.find() | |
.should('be.visible') | |
.within(() => { | |
cy.findByText('Log Out').should('be.visible').click(); | |
}); |
cy.visit('/login'); | ||
cy.url().should('endWith', `/login`); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
cy.visit('/login'); | |
cy.url().should('endWith', `/login`); | |
cy.visit('/'); | |
cy.url().should('startWith', `${LOGIN_ROOT}/login`); |
Otherwise the URL assertion will immediately pass without waiting for Cloud to actually redirect to the Login application since both pages end with /login
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Updated, thank you!
Volumn failure has been fixed in #10728 |
Description 📝
Create a Cloud Manager end-to-end test which tests the Cloud Manager logout flow. This will confirm that Cloud Manager log out functionality works as expected and implicitly confirm that the login application is up, which is useful from a synthetic testing perspective.
Changes 🔄
How to test 🧪
As an Author I have considered 🤔
Check all that apply