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

103 lines
4.0 KiB
JavaScript

const path = require('path')
import { format } from 'date-fns'
import * as constants from '../../support/constants'
import * as addressBook from '../../e2e/pages/address_book.page'
import * as ls from '../../support/localstorage_data.js'
import * as accountsModal from '../pages/accounts_modal.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
const NAME = 'Owner1'
const EDITED_NAME = 'Edited Owner1'
const importedSafe = 'imported-safe'
describe('Address book tests', () => {
before(async () => {
staticSafes = await getSafes(CATEGORIES.static)
})
const seedAddressBook = (value) => (win) =>
win.localStorage.setItem(constants.localStorageKeys.SAFE_v2__addressBook, JSON.stringify(value))
it('Verify owners name can be edited', () => {
cy.visit(constants.addressBookUrl + staticSafes.SEP_STATIC_SAFE_4, {
onBeforeLoad: seedAddressBook(ls.addressBookData.sepoliaAddress1),
})
addressBook.clickOnEditEntryBtn()
addressBook.typeInNameInput(EDITED_NAME)
addressBook.clickOnSaveEntryBtn()
addressBook.verifyNameWasChanged(NAME, EDITED_NAME)
})
it('Verify the address book file can be exported', () => {
cy.visit(constants.addressBookUrl + staticSafes.SEP_STATIC_SAFE_4, {
onBeforeLoad: seedAddressBook(ls.addressBookData.dataSet),
})
cy.contains(ls.addressBookData.dataSet[11155111]['0xf405BC611F4a4c89CCB3E4d083099f9C36D966f8'])
const date = format(new Date(), 'yyyy-MM-dd', { timeZone: 'UTC' })
const fileName = `safe-address-book-${date}.csv`
addressBook.clickOnExportFileBtn()
addressBook.verifyExportMessage(12)
addressBook.confirmExport()
const downloadsFolder = Cypress.config('downloadsFolder')
cy.readFile(path.join(downloadsFolder, fileName), 'utf-8').then((content) => {
const lines = content
.replace(/^\uFEFF/, '')
.trim()
.split('\r\n')
const [header, ...dataLines] = lines
const actualData = dataLines.reduce((acc, line) => {
const [address, name, chainId] = line.split(',')
acc[chainId] = acc[chainId] || {}
acc[chainId][address] = name
return acc
}, {})
Object.keys(ls.addressBookData.dataSet).forEach((chainId) => {
cy.log(`Checking chainId: ${chainId}`)
const actualChainData = actualData[chainId] || {}
const expectedChainData = ls.addressBookData.dataSet[chainId]
Object.keys(expectedChainData).forEach((address) => {
const actualName = actualChainData[address]
const expectedName = expectedChainData[address]
cy.log(`ChainId: ${chainId}, Address: ${address}, Actual Name: ${actualName}, Expected Name: ${expectedName}`)
expect(actualName).to.equal(expectedName)
})
})
})
})
it('Verify that importing a csv file does not alter addresses in the Address book not present in the file', () => {
cy.visit(constants.addressBookUrl + staticSafes.SEP_STATIC_SAFE_4, {
onBeforeLoad: seedAddressBook(ls.addressBookData.sepoliaAddress1),
})
addressBook.clickOnImportFileBtn()
addressBook.importCSVFile(addressBook.validCSVFile)
addressBook.clickOnImportBtn()
addressBook.verifyDataImported([constants.RECIPIENT_ADDRESS])
})
it('Verify Safe name changes after uploading a csv file', () => {
wallet.connectSignerViaStorage(signer, constants.addressBookUrl + staticSafes.SEP_STATIC_SAFE_4, {
extraStorage: {
[constants.localStorageKeys.SAFE_v2__addedSafes]: ls.addedSafes.set2,
[constants.localStorageKeys.SAFE_v2__addressBook]: ls.addressBookData.addedSafesImport,
},
})
addressBook.clickOnImportFileBtn()
addressBook.importCSVFile(addressBook.addedSafesCSVFile)
addressBook.clickOnImportBtn()
accountsModal.openAccountsModal()
accountsModal.verifyAccountsListContains(importedSafe)
})
})