Files
05cab32f0e tests(web): connect wallet via seeded storage in e2e (#8350)
* tests(web): connect wallet via seeded storage in e2e

Replace the UI-driven connectSigner flow with storage-seeded auto-reconnect
across the Cypress suite. Adds connectSignerViaStorage (seed on visit, or seed
+ reload) and migrates 139 entry connects; keeps UI connectSigner for the 27
connect/switch-flow sites that act on in-page state.

* tests(web): seed batch via single page load in batch e2e

Add optional extraStorage to connectSignerViaStorage so localStorage is
seeded in onBeforeLoad on the initial visit. Split batch_tx into
build-via-UI and pre-seeded describes, dropping addToLocalStorage +
cy.reload() double-loads.

* tests(web): seed state on a single page load across e2e specs

Replace visit-then-write-then-reload double-loads with a single seeded
load (extraStorage on connectSignerViaStorage, or onBeforeLoad on
cy.visit), and drop the arbitrary post-reload settle waits. Covers the
multichain, nested-safes, batch, address-book, safe-selector and
create-safe specs (regression, smoke, visual).

* tests(web): seed state on a single page load in more e2e specs

Fold pre-seeded localStorage into the initial visit (extraStorage /
onBeforeLoad) and drop the follow-up reloads across the address-book,
messages-popup, CF-safe, import/export and load-safe specs. Split specs
whose shared beforeEach served both seeding and non-seeding tests into
nested describes so no test double-loads.

* tests(web): remove redundant wallet reconnect reloads in e2e specs

Pass the URL to connectSignerViaStorage so the wallet is seeded on the
initial visit instead of visiting then reconnecting (which reloaded).
Where a shared beforeEach served both connected and disconnected (or
differently-seeded) tests, split into nested describes so no test
double-loads. Covers proposers, swaps, twaps, assets, swaps_tokens,
sidebar, spending_limits, bulk_execution, proposers_2, safe_selector.

* tests(web): single-load owner-management e2e specs

Split add/remove/replace-owner specs into connected / disconnected /
non-owner / pre-seeded describes so the wallet is seeded on the initial
visit (no reconnect reload) without forcing a connect onto tests that
assert disconnected or non-owner state.

* tests(web): wait for wallet connection after seeded connect

The last wallet reconnects asynchronously after the page loads, so
connectSignerViaStorage now waits for the connected-wallet header chip
(open-account-center) before resolving, instead of letting tests proceed
while still disconnected. Opt out via waitForConnection:false for pages
without that chip (welcome/accounts in safe_selector).

* tests(web): address review on seeded-connect PR

- Drop the explicit 30s timeout on the connected-wallet wait; the repo
  default (10s) is plenty for the post-load reconnect and fails faster.
- Make recovery.cy.js consistent: the remaining connectSigner calls now
  use connectSignerViaStorage like the rest of the file. All connects are
  at test start (none mid-flow), matching the already-converted tests.

* fix: wait for the wallet connection to establish to avoid failing tests due to disconnected wallet

---------

Co-authored-by: vmosharova <[email protected]>
2026-07-31 17:00:56 +02:00

62 lines
2.2 KiB
JavaScript

import * as constants from '../../support/constants'
import * as sideBar from '../pages/sidebar.pages'
import * as navigation from '../pages/navigation.page'
import * as swaps from '../pages/swaps.pages.js'
import { exchangeStr, clickOnBridgeOption } from '../pages/bridge.pages.js'
import { getSafes, CATEGORIES } from '../../support/safes/safesHandler.js'
import * as wallet from '../../support/utils/wallet.js'
let staticSafes = []
const walletCredentials = JSON.parse(Cypress.env('CYPRESS_WALLET_CREDENTIALS'))
const signer = walletCredentials.OWNER_4_PRIVATE_KEY
describe('Sidebar tests', () => {
before(async () => {
staticSafes = await getSafes(CATEGORIES.static)
})
describe('Disconnected', () => {
beforeEach(() => {
cy.visit(constants.homeUrl + staticSafes.SEP_STATIC_SAFE_9)
})
it('Verify New Transaction button disabled for non-owners', () => {
sideBar.verifyNewTxBtnStatus(constants.enabledStates.disabled)
})
it('Verify the side menu buttons exist', () => {
sideBar.verifySideListItemsNew()
})
it('Verify counter in the "Transaction" menu item if there are tx in the queue tab', () => {
sideBar.verifyTxCounterNew(1)
})
it('Verify that clicking on Bridge in the sidebar opens the exchange iframe', () => {
const iframeSelector = `iframe[src*="${constants.bridgeWidget}"]`
clickOnBridgeOption()
swaps.acceptLegalDisclaimer()
cy.get(iframeSelector).should('be.visible')
cy.get(iframeSelector).then(($iframe) => {
const $body = $iframe.contents().find('body')
cy.wrap($body).should('exist')
cy.wrap($body).contains(exchangeStr).should('be.visible')
})
})
})
describe('Connected', () => {
it('Verify New transaction button enabled for owners', () => {
wallet.connectSignerViaStorage(signer, constants.homeUrl + staticSafes.SEP_STATIC_SAFE_9)
sideBar.verifyNewTxBtnStatus(constants.enabledStates.enabled)
})
it('Verify New transaction button enabled for beneficiaries who are non-owners', () => {
wallet.connectSignerViaStorage(signer, constants.homeUrl + staticSafes.SEP_STATIC_SAFE_11)
sideBar.verifyNewTxBtnStatus(constants.enabledStates.enabled)
})
})
})