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]>
231 lines
10 KiB
JavaScript
231 lines
10 KiB
JavaScript
import * as constants from '../../support/constants.js'
|
|
import * as copilot from '../pages/copilot.js'
|
|
import * as createtx from '../pages/create_tx.pages.js'
|
|
import * as main from '../pages/main.page.js'
|
|
import { getSafes, CATEGORIES } from '../../support/safes/safesHandler.js'
|
|
import * as wallet from '../../support/utils/wallet.js'
|
|
import * as ls from '../../support/localstorage_data.js'
|
|
|
|
const walletCredentials = JSON.parse(Cypress.env('CYPRESS_WALLET_CREDENTIALS'))
|
|
const signer = walletCredentials.OWNER_4_PRIVATE_KEY
|
|
let staticSafes = []
|
|
|
|
describe('Safe Copilot tests', { defaultCommandTimeout: 30000 }, () => {
|
|
before(async () => {
|
|
staticSafes = await getSafes(CATEGORIES.static)
|
|
})
|
|
|
|
// ========================================
|
|
// 1. Widget General
|
|
// ========================================
|
|
|
|
it('[Widget General] Verify that Safe Shield empty state is shown on New Transaction start before scanning', () => {
|
|
wallet.connectSignerViaStorage(signer, constants.BALANCE_URL + staticSafes.MATIC_STATIC_SAFE_30)
|
|
createtx.clickOnNewtransactionBtn()
|
|
createtx.clickOnSendTokensBtn()
|
|
copilot.verifySafeShieldDisplayed()
|
|
copilot.verifyEmptyState()
|
|
copilot.verifySecuredByFooter()
|
|
})
|
|
|
|
it('[Widget General] Verify that Risk detected requires Risk Confirmation checkbox to continue', () => {
|
|
copilot.navigateToTransactionAndSetupCopilot(copilot.testTransactions.threatAnalysisMaliciousApproval, signer)
|
|
|
|
copilot.verifyRiskDetected()
|
|
|
|
// Verify risk confirmation checkbox is unchecked and continue button is disabled
|
|
copilot.verifyRiskConfirmationCheckboxUnchecked()
|
|
copilot.verifyContinueButtonDisabled()
|
|
|
|
// Check the risk confirmation checkbox and continue
|
|
copilot.checkRiskConfirmationCheckbox()
|
|
copilot.verifyContinueButtonEnabled()
|
|
createtx.clickOnContinueSignTransactionBtn()
|
|
cy.contains(createtx.txDetailsStr).should('be.visible')
|
|
})
|
|
|
|
// ========================================
|
|
// 2. Recipient Analyse
|
|
// ========================================
|
|
//Only 4A - Incompatible Safe version is not covered
|
|
|
|
it('[Recipient Analyse] Verify that Known recipient is shown when address is in address book - 1A', () => {
|
|
copilot.setupRecipientAnalysis(
|
|
copilot.testTransactions.recipientAnalysisKnownUnknown,
|
|
signer,
|
|
ls.addressBookData.safeSchiledAddressBook,
|
|
)
|
|
main.verifyTextVisibility([copilot.addressInAddressBookStr])
|
|
})
|
|
|
|
it('[Recipient Analyse] Verify that Known recipient is shown when recipient is a Safe you own - 1A', () => {
|
|
copilot.setupRecipientAnalysis(copilot.testTransactions.recipientAnalysisSafeYouOwn, signer)
|
|
main.verifyTextVisibility([copilot.addressIsSafeYouOwnStr])
|
|
})
|
|
|
|
it('[Recipient Analyse] Verify that Unknown recipient is shown when address is not in address book - 1B', () => {
|
|
copilot.setupRecipientAnalysis(copilot.testTransactions.recipientAnalysisKnownUnknown, signer)
|
|
main.verifyTextVisibility([copilot.unknownRecipientStr, copilot.addressNotInAddressBookStr])
|
|
})
|
|
|
|
it('[Recipient Analyse] Verify that New recipient is shown for first time interaction - 3A', () => {
|
|
copilot.setupRecipientAnalysis(copilot.testTransactions.recipientAnalysisSafeYouOwn, signer)
|
|
main.verifyTextVisibility([copilot.firstTimeInteractionStr])
|
|
})
|
|
|
|
it('[Recipient Analyse] Verify that Recurring recipient is shown with interaction count - 3B', () => {
|
|
copilot.setupRecipientAnalysis(
|
|
copilot.testTransactions.recipientAnalysisKnownUnknown,
|
|
signer,
|
|
ls.addressBookData.safeSchiledAddressBook,
|
|
)
|
|
main.verifyTextVisibility([copilot.recurringRecipientStr, copilot.interactedTwoTimesStr])
|
|
})
|
|
|
|
it('[Recipient Analyse] Verify that Low activity recipient warning is shown for address with few transactions - 2', () => {
|
|
copilot.setupRecipientAnalysis(copilot.testTransactions.recipientAnalysisLowActivity, signer)
|
|
main.verifyTextVisibility([copilot.lowActivityRecipientStr, copilot.fewTransactionsStr])
|
|
})
|
|
|
|
it('[Recipient Analyse] Verify that Missing ownership warning is shown - 4B', () => {
|
|
copilot.setupRecipientAnalysis(
|
|
copilot.testTransactions.recipientAnalysisMissingOwnership,
|
|
signer,
|
|
null,
|
|
staticSafes.MATIC_STATIC_SAFE_28,
|
|
)
|
|
main.verifyTextVisibility([copilot.missingOwnershipStr, copilot.missingOwnershipMessageStr])
|
|
})
|
|
|
|
it('[Recipient Analyse] Verify that Unsupported network warning is shown - 4C', () => {
|
|
copilot.setupRecipientAnalysis(copilot.testTransactions.recipientAnalysisUnsupportedNetwork, signer)
|
|
main.verifyTextVisibility([copilot.unsupportedNetworkStr, copilot.unsupportedNetworkMessageStr])
|
|
})
|
|
|
|
it('[Recipient Analyse] Verify that Different setup warning is shown - 4D', () => {
|
|
copilot.setupRecipientAnalysis(copilot.testTransactions.recipientAnalysisDifferentSetup, signer)
|
|
main.verifyTextVisibility([copilot.differentSetupMessageStr])
|
|
})
|
|
|
|
// ========================================
|
|
// 3. Threat Analyse
|
|
// ========================================
|
|
|
|
it('[Threat Analyse] Verify that Safe Shield shows warning details for reverted txs-9B', () => {
|
|
copilot.navigateToTransactionAndSetupCopilot(copilot.testTransactions.threatAnalysisFailed, signer)
|
|
|
|
copilot.verifyIssuesFoundWarningHeader()
|
|
copilot.verifyThreatAnalysisGroupCard()
|
|
copilot.verifyThreatAnalysisWarningState()
|
|
copilot.expandThreatAnalysisCard()
|
|
copilot.verifyThreatAnalysisFailedDetails()
|
|
})
|
|
|
|
it('[Threat Analyse] Verify that Safe Shield shows no threat detected-9C', () => {
|
|
copilot.navigateToTransactionAndSetupCopilot(copilot.testTransactions.threatAnalysisNoThreat, signer)
|
|
|
|
copilot.verifyThreatAnalysisGroupCard()
|
|
copilot.verifyThreatAnalysisNoThreatState()
|
|
copilot.expandThreatAnalysisCard()
|
|
copilot.verifyThreatAnalysisFoundNoIssues()
|
|
main.verifyTextNotVisible(['Malicious threat detected', 'Threat analysis failed'])
|
|
})
|
|
|
|
it('[Threat Analyse] Verify Malicious Approval detection - 9A', () => {
|
|
copilot.navigateToTransactionAndSetupCopilot(copilot.testTransactions.threatAnalysisMaliciousApproval, signer)
|
|
|
|
copilot.verifyRiskDetected()
|
|
copilot.verifyThreatAnalysisGroupCard()
|
|
copilot.verifyThreatAnalysisMaliciousState()
|
|
copilot.expandThreatAnalysisCard()
|
|
main.verifyTextVisibility([copilot.drainerApprovalMessageStr, copilot.drainerActivityStr])
|
|
})
|
|
|
|
it('[Threat Analyse] Verify Malicious Transfer detection - 9A', () => {
|
|
copilot.navigateToTransactionAndSetupCopilot(copilot.testTransactions.threatAnalysisMaliciousTransfer, signer)
|
|
|
|
copilot.verifyRiskDetected()
|
|
copilot.verifyThreatAnalysisGroupCard()
|
|
copilot.verifyThreatAnalysisMaliciousState()
|
|
copilot.expandThreatAnalysisCard()
|
|
main.verifyTextVisibility([copilot.drainerTransferMessageStr, copilot.drainerActivityStr])
|
|
})
|
|
|
|
it('[Threat Analyse] Verify Malicious Native Transfer detection - 9A', () => {
|
|
copilot.navigateToTransactionAndSetupCopilot(copilot.testTransactions.threatAnalysisMaliciousNativeTransfer, signer)
|
|
|
|
copilot.verifyRiskDetected()
|
|
copilot.verifyThreatAnalysisGroupCard()
|
|
copilot.verifyThreatAnalysisMaliciousState()
|
|
copilot.expandThreatAnalysisCard()
|
|
main.verifyTextVisibility([copilot.drainerNativeTransferMessageStr, copilot.drainerActivityStr])
|
|
})
|
|
|
|
it('[Threat Analyse] Verify Malicious wallet_sendCalls detection - 9A', () => {
|
|
copilot.navigateToTransactionAndSetupCopilot(copilot.testTransactions.threatAnalysisMaliciousAddress, signer)
|
|
|
|
copilot.verifyRiskDetected()
|
|
copilot.verifyThreatAnalysisGroupCard()
|
|
copilot.verifyThreatAnalysisMaliciousState()
|
|
copilot.expandThreatAnalysisCard()
|
|
main.verifyTextVisibility([copilot.maliciousAddressMessageStr, copilot.maliciousActivityStr])
|
|
})
|
|
|
|
it('[Threat Analyse] Verify Malicious wallet_sendCalls(Eth) detection - 9A', () => {
|
|
copilot.navigateToTransactionAndSetupCopilot(copilot.testTransactions.threatAnalysisMaliciousAddressEth, signer)
|
|
|
|
copilot.verifyRiskDetected()
|
|
copilot.verifyThreatAnalysisGroupCard()
|
|
copilot.verifyThreatAnalysisMaliciousState()
|
|
copilot.expandThreatAnalysisCard()
|
|
main.verifyTextVisibility([copilot.maliciousAddressMessageStr, copilot.maliciousActivityStr])
|
|
})
|
|
//TODO: Add tests for offchain messages when implemented
|
|
// ========================================
|
|
// 4. Contract Analyse
|
|
// ========================================
|
|
|
|
// TODO: Add Contract Analyse tests
|
|
|
|
// ========================================
|
|
// 5. Tenderly Simulation
|
|
// ========================================
|
|
|
|
it('[Tenderly Simulation] Verify that tenderly section is presented in the safe shield', () => {
|
|
copilot.navigateToTransactionAndSetupCopilot(copilot.testTransactions.tenderlySimulation, signer)
|
|
|
|
copilot.verifyTenderlySimulation()
|
|
cy.get(copilot.tenderlySimulation).should('contain.text', 'Transaction simulation')
|
|
cy.get(copilot.tenderlySimulation).find(copilot.runSimulationBtn).should('be.visible')
|
|
cy.contains('Run').should('be.visible')
|
|
})
|
|
|
|
it('[Tenderly Simulation] Verify success simulation state', () => {
|
|
copilot.navigateToTransactionAndSetupCopilot(copilot.testTransactions.tenderlySimulation, signer)
|
|
|
|
copilot.verifyTenderlySimulation()
|
|
cy.get(copilot.tenderlySimulation, { timeout: 15000 }).find(copilot.runSimulationBtn).click()
|
|
cy.get(copilot.tenderlySimulation).find(copilot.runSimulationBtn).should('contain.text', 'Running...')
|
|
cy.contains('Simulation successful', { timeout: 10000 }).should('be.visible')
|
|
cy.get(copilot.tenderlySimulation).should('contain.text', 'Simulation successful')
|
|
cy.contains('View').should('be.visible')
|
|
cy.get(copilot.tenderlySimulation).find(copilot.runSimulationBtn).should('not.exist')
|
|
main.verifyLinkContainsUrl('View', copilot.tenderlySimulationUrl)
|
|
})
|
|
|
|
it('[Tenderly Simulation] Verify failed simulation state', () => {
|
|
copilot.navigateToTransactionAndSetupCopilot(copilot.testTransactions.threatAnalysisFailed, signer)
|
|
|
|
copilot.verifyTenderlySimulation()
|
|
cy.get(copilot.tenderlySimulation, { timeout: 15000 }).find(copilot.runSimulationBtn).click()
|
|
cy.get(copilot.tenderlySimulation).find(copilot.runSimulationBtn).should('contain.text', 'Running...')
|
|
cy.contains('Simulation failed', { timeout: 10000 }).should('be.visible')
|
|
cy.get(copilot.tenderlySimulation).should('contain.text', 'Simulation failed')
|
|
cy.contains('View').should('be.visible')
|
|
cy.get(copilot.tenderlySimulation).find(copilot.runSimulationBtn).should('not.exist')
|
|
main.verifyLinkContainsUrl('View', copilot.tenderlySimulationUrl)
|
|
})
|
|
|
|
//it('[Tenderly Simulation] Verify original and nested txs simulations in Tenderly card'
|
|
})
|