Skip to content
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

Update cypress to latest version to support execution on latest #988

Merged
merged 1 commit into from
Feb 14, 2024

Conversation

ikethecoder
Copy link
Member

No description provided.

…ome and edge browser

2)Update cypress tests to latest changes in aps portal and gwa cli
3)Update cypress tests to latest methods in latest cypress version to clear cookies and session storage
@ikethecoder ikethecoder merged commit 2282d60 into dev Feb 14, 2024
1 check passed
@ikethecoder ikethecoder deleted the gwa-cypress-version-update branch February 14, 2024 22:48
Copy link
Contributor

@rustyjux rustyjux left a comment

Choose a reason for hiding this comment

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

Worked for me in test, except for repeated hangs on 02/05 and 14/02. Curious to see what happens when running in CI/CD

@@ -24,7 +24,7 @@ class ApiDirectoryPage {
cy.contains('a', product.name, { timeout: 10000 }).should('be.visible');
cy.contains(product.name).click()
if (elevatedAccess) {
cy.contains('For elevated access, please Request Access').should('be.visible');
cy.contains('For elevated access, please request acces').should('be.visible');
Copy link
Contributor

Choose a reason for hiding this comment

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

typo

Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
cy.contains('For elevated access, please request acces').should('be.visible');
cy.contains('For elevated access, please request access').should('be.visible');

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants