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

chore: fix network name selection in E2E #2267

Merged
merged 2 commits into from
Jul 11, 2023
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
6 changes: 4 additions & 2 deletions cypress/e2e/smoke/create_safe_simple.cy.js
Original file line number Diff line number Diff line change
Expand Up @@ -11,7 +11,7 @@ describe('Create Safe form', () => {
cy.contains('button', 'Accept selection').click()

// Ensure wallet is connected to correct chain via header
cy.contains('E2E Wallet @ Görli')
cy.contains(/E2E Wallet @ G(ö|oe)rli/)

cy.contains('Create new Account').click()
})
Expand All @@ -33,7 +33,9 @@ describe('Create Safe form', () => {

// Switch back to Görli
cy.get('[data-cy="create-safe-select-network"]').click()
cy.contains('li span', 'Görli').click()

// Prevent Base Mainnet Goerli from being selected
cy.contains('li span', /^G(ö|oe)rli$/).click()

cy.contains('button', 'Next').click()
})
Expand Down
10 changes: 6 additions & 4 deletions cypress/e2e/smoke/nfts.cy.js
Original file line number Diff line number Diff line change
Expand Up @@ -6,7 +6,7 @@ describe('Assets > NFTs', () => {

cy.visit(`/balances/nfts?safe=${TEST_SAFE}`)
cy.contains('button', 'Accept selection').click()
cy.contains('E2E Wallet @ Görli')
cy.contains(/E2E Wallet @ G(ö|oe)rli/)
})

describe('should have NFTs', () => {
Expand All @@ -18,7 +18,7 @@ describe('Assets > NFTs', () => {
cy.get('tbody tr:first-child').contains('td:first-child', 'BillyNFT721')
cy.get('tbody tr:first-child').contains('td:first-child', '0x0000...816D')

cy.get('tbody tr:first-child').contains('td:nth-child(2)', 'Kitaro #261')
cy.get('tbody tr:first-child').contains('td:nth-child(2)', 'Kitaro World #261')

cy.get(
'tbody tr:first-child td:nth-child(3) a[href="https://testnets.opensea.io/assets/0x000000000faE8c6069596c9C805A1975C657816D/443"]',
Expand All @@ -30,8 +30,10 @@ describe('Assets > NFTs', () => {
cy.get('tbody tr:first-child td:nth-child(2)').click()

// Modal
cy.get('div[role="dialog"]').contains('Kitaro #261')
cy.get('div[role="dialog"]').contains('Görli')
cy.get('div[role="dialog"]').contains('Kitaro World #261')

// Prevent Base Mainnet Goerli from being selected
cy.get('div[role="dialog"]').contains(/^G(ö|oe)rli$/)
cy.get('div[role="dialog"]').contains(
'a[href="https://testnets.opensea.io/assets/0x000000000faE8c6069596c9C805A1975C657816D/443"]',
'View on OpenSea',
Expand Down
Loading