mirror of
https://github.com/luxfi/safe-wallet.git
synced 2026-10-30 15:19:21 +00:00
05cab32f0e
* 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]>
136 lines
5.2 KiB
JavaScript
136 lines
5.2 KiB
JavaScript
import * as constants from '../../support/constants.js'
|
|
import * as main from '../pages/main.page.js'
|
|
import * as safeNav from '../pages/safe_navigation.pages'
|
|
import * as network from '../pages/network.pages.js'
|
|
import * as dashboard from '../pages/dashboard.pages.js'
|
|
import * as ls from '../../support/localstorage_data.js'
|
|
import { getSafes, CATEGORIES } from '../../support/safes/safesHandler.js'
|
|
import * as wallet from '../../support/utils/wallet.js'
|
|
import * as navigation from '../pages/navigation.page.js'
|
|
import * as create_wallet from '../pages/create_wallet.pages.js'
|
|
import * as owner from '../pages/owners.pages.js'
|
|
|
|
import { suspendOutreachModal } from '../pages/modals.page.js'
|
|
|
|
let staticSafes = []
|
|
|
|
const walletCredentials = JSON.parse(Cypress.env('CYPRESS_WALLET_CREDENTIALS'))
|
|
const signer = walletCredentials.OWNER_4_PRIVATE_KEY
|
|
|
|
const sidebarNavItem = '[data-testid="sidebar-list-item"]'
|
|
|
|
describe('Multichain setup tests', { defaultCommandTimeout: 60000 }, () => {
|
|
before(async () => {
|
|
staticSafes = await getSafes(CATEGORIES.static)
|
|
})
|
|
|
|
beforeEach(() => {
|
|
wallet.connectSignerViaStorage(signer, constants.BALANCE_URL + staticSafes.MATIC_STATIC_SAFE_28, {
|
|
extraStorage: {
|
|
[constants.localStorageKeys.SAFE_v2__addedSafes]: ls.addedSafes.set5,
|
|
[constants.localStorageKeys.SAFE_v2__addressBook]: ls.addressBookData.multichain,
|
|
},
|
|
})
|
|
})
|
|
|
|
// Renamed from: 'Verify that batch tx with safe activation is not allowed for the CF safes'
|
|
it('Verify that CF safes block transaction creation and signer management', () => {
|
|
const safe = main.changeSafeChainName(staticSafes.MATIC_STATIC_SAFE_28, 'eth')
|
|
|
|
network.addNetwork(constants.networks.ethereum)
|
|
cy.contains(network.createSafeMsg(constants.networks.ethereum))
|
|
|
|
safeNav.openSelector()
|
|
safeNav.expandMultichainRowByAddress(staticSafes.MATIC_STATIC_SAFE_28.split(':')[1].slice(0, 6))
|
|
safeNav.clickNotActivatedSubAccount()
|
|
|
|
main.verifyElementsCount(navigation.newTxBtn, 0)
|
|
main.verifyElementsCount(create_wallet.activateAccountBtn, 2)
|
|
|
|
cy.visit(constants.setupUrl + safe)
|
|
owner.verifyManageSignersBtnIsDisabled()
|
|
cy.contains(sidebarNavItem, 'Apps').should('be.disabled')
|
|
})
|
|
|
|
it('Verify notification if the owner set up was changed in original safe', () => {
|
|
const safeAddress = staticSafes.MATIC_STATIC_SAFE_28.split(':')[1]
|
|
// Mock /v2/safes to return deviating owners across chains — triggers InconsistentSignerSetupWarning
|
|
cy.intercept('GET', '**/v2/safes**', [
|
|
{
|
|
address: { value: safeAddress },
|
|
chainId: '137',
|
|
threshold: 1,
|
|
owners: [{ value: constants.DEFAULT_OWNER_ADDRESS }],
|
|
fiatTotal: '0',
|
|
queued: 0,
|
|
},
|
|
{
|
|
address: { value: safeAddress },
|
|
chainId: '11155111',
|
|
threshold: 1,
|
|
owners: [{ value: constants.SEPOLIA_OWNER_2 }],
|
|
fiatTotal: '0',
|
|
queued: 0,
|
|
},
|
|
])
|
|
|
|
cy.visit(constants.homeUrl + staticSafes.MATIC_STATIC_SAFE_28)
|
|
dashboard.expandActionRequiredPanel()
|
|
dashboard.checkInconsistentSignersMsgDisplayed()
|
|
dashboard.verifySafeInUrl(staticSafes.MATIC_STATIC_SAFE_28)
|
|
dashboard.clickActionInPanel(dashboard.reviewSignersTestId)
|
|
cy.url().should('include', '/settings/setup').and('include', staticSafes.MATIC_STATIC_SAFE_28)
|
|
})
|
|
|
|
it('Verify warning on add owner for one safe in the group', () => {
|
|
cy.visit(constants.setupUrl + staticSafes.MATIC_STATIC_SAFE_28)
|
|
owner.openManageSignersWindow()
|
|
owner.clickOnAddSignerBtn()
|
|
owner.typeOwnerAddressManage(4, constants.SEPOLIA_OWNER_2)
|
|
owner.clickOnNextBtnManage()
|
|
|
|
owner.verifyInconsistentSignersWarning(constants.networks.polygon)
|
|
})
|
|
|
|
it('Verify warning on remove owner for one safe in the group', () => {
|
|
const safe = main.changeSafeChainName(staticSafes.MATIC_STATIC_SAFE_28, 'sep')
|
|
cy.visit(constants.setupUrl + safe)
|
|
|
|
owner.waitForConnectionStatus()
|
|
navigation.verifyTxBtnStatus(constants.enabledStates.enabled)
|
|
suspendOutreachModal()
|
|
owner.openRemoveOwnerWindow(1)
|
|
cy.wait(1000)
|
|
create_wallet.clickOnNextBtn()
|
|
|
|
owner.verifyInconsistentSignersWarning(constants.networks.sepolia)
|
|
})
|
|
|
|
it('Verify warning on change policy for one safe in the group', () => {
|
|
const safe = main.changeSafeChainName(staticSafes.MATIC_STATIC_SAFE_28, 'sep')
|
|
cy.visit(constants.setupUrl + safe)
|
|
owner.waitForConnectionStatus()
|
|
navigation.verifyTxBtnStatus(constants.enabledStates.enabled)
|
|
suspendOutreachModal()
|
|
owner.clickOnChangeThresholdBtn()
|
|
create_wallet.updateThreshold(2)
|
|
owner.clickOnThresholdNextBtn()
|
|
|
|
owner.verifyInconsistentSignersWarning(constants.networks.sepolia)
|
|
})
|
|
|
|
it('Verify warning on swap owner for one safe in the group', () => {
|
|
const safe = main.changeSafeChainName(staticSafes.MATIC_STATIC_SAFE_28, 'sep')
|
|
cy.visit(constants.setupUrl + safe)
|
|
owner.waitForConnectionStatus()
|
|
navigation.verifyTxBtnStatus(constants.enabledStates.enabled)
|
|
suspendOutreachModal()
|
|
owner.openReplaceOwnerWindow(1)
|
|
owner.typeOwnerAddress(constants.SEPOLIA_OWNER_2)
|
|
cy.wait(2000)
|
|
owner.clickOnNextBtn()
|
|
|
|
owner.verifyInconsistentSignersWarning(constants.networks.sepolia)
|
|
})
|
|
})
|