From 7c8b9e30ee6d578bc45720d9631d81036bc4b676 Mon Sep 17 00:00:00 2001 From: mike10ca Date: Thu, 9 Nov 2023 09:01:17 +0100 Subject: [PATCH 1/6] Update test IDs --- cypress/e2e/safe-apps/apps_list.cy.js | 14 ++++---- .../e2e/safe-apps/browser_permissions.cy.js | 4 +-- cypress/e2e/safe-apps/info_modal.cy.js | 6 ++-- .../e2e/safe-apps/permissions_settings.cy.js | 14 ++++---- cypress/e2e/safe-apps/preview_drawer.cy.js | 2 +- cypress/e2e/safe-apps/safe_permissions.cy.js | 4 +-- cypress/e2e/safe-apps/tx_modal.cy.js | 2 +- cypress/e2e/smoke/add_owner.cy.js | 20 +++++------ cypress/e2e/smoke/address_book.cy.js | 12 +++---- cypress/e2e/smoke/assets.cy.js | 36 +++++++++---------- cypress/e2e/smoke/balances.cy.js | 16 ++++----- cypress/e2e/smoke/balances_pagination.cy.js | 4 +-- cypress/e2e/smoke/batch_tx.cy.js | 12 +++---- cypress/e2e/smoke/beamer.cy.js | 2 +- cypress/e2e/smoke/create_safe_simple.cy.js | 26 +++++++------- cypress/e2e/smoke/create_tx.cy.js | 6 ++-- cypress/e2e/smoke/dashboard.cy.js | 8 ++--- cypress/e2e/smoke/import_export_data.cy.js | 10 +++--- cypress/e2e/smoke/landing.cy.js | 2 +- cypress/e2e/smoke/load_safe.cy.js | 10 +++--- cypress/e2e/smoke/nfts.cy.js | 10 +++--- cypress/e2e/smoke/remove_owner.cy.js | 12 +++---- cypress/e2e/smoke/replace_owner.cy.js | 16 ++++----- cypress/e2e/smoke/tx_history.cy.js | 4 +-- 24 files changed, 126 insertions(+), 126 deletions(-) diff --git a/cypress/e2e/safe-apps/apps_list.cy.js b/cypress/e2e/safe-apps/apps_list.cy.js index 4081b813cf..c6b15a2265 100644 --- a/cypress/e2e/safe-apps/apps_list.cy.js +++ b/cypress/e2e/safe-apps/apps_list.cy.js @@ -12,7 +12,7 @@ describe('Safe Apps tests', () => { main.acceptCookies(1) }) - it('Verify app list can be filtered by app name [C56130]', () => { + it('Verify app list can be filtered by app name', () => { // Wait for /safe-apps response cy.intercept('GET', constants.appsEndpoint).then(() => { safeapps.typeAppName(constants.appNames.walletConnect) @@ -20,29 +20,29 @@ describe('Safe Apps tests', () => { }) }) - it('Verify app list can be filtered by app description [C56131]', () => { + it('Verify app list can be filtered by app description', () => { safeapps.typeAppName(constants.appNames.customContract) safeapps.verifyLinkName(safeapps.linkNames.logo) }) - it('Verify error message is displayed when no app found [C56132]', () => { + it('Verify error message is displayed when no app found', () => { safeapps.typeAppName(constants.appNames.noResults) safeapps.verifyNoAppsTextPresent() }) - it('Verify apps can be pinned [C56133]', () => { + it('Verify apps can be pinned ', () => { safeapps.clearSearchAppInput() safeapps.pinApp(safeapps.transactionBuilderStr) safeapps.verifyPinnedAppCount(1) }) - it('Verify apps can be unpinned [C56134]', () => { + it('Verify apps can be unpinned ', () => { safeapps.pinApp(safeapps.transactionBuilderStr) safeapps.pinApp(safeapps.transactionBuilderStr, false) safeapps.verifyPinnedAppCount(0) }) - it('Verify there is an error when the app manifest is invalid [C56135]', () => { + it('Verify there is an error when the app manifest is invalid ', () => { cy.intercept('GET', constants.invalidAppUrl, { name: constants.testAppData.name, }) @@ -52,7 +52,7 @@ describe('Safe Apps tests', () => { safeapps.verifyAppNotSupportedMsg() }) - it('Verify an app can be added to the list within the custom apps section [C56136]', () => { + it('Verify an app can be added to the list within the custom apps section ', () => { cy.intercept('GET', constants.validAppUrlJson, { name: constants.testAppData.name, description: constants.testAppData.descr, diff --git a/cypress/e2e/safe-apps/browser_permissions.cy.js b/cypress/e2e/safe-apps/browser_permissions.cy.js index da34de03ab..14f7bede90 100644 --- a/cypress/e2e/safe-apps/browser_permissions.cy.js +++ b/cypress/e2e/safe-apps/browser_permissions.cy.js @@ -18,13 +18,13 @@ describe('Browser permissions tests', () => { main.acceptCookies(1) }) - it('Verify a permissions slide to the user is displayed [C56137]', () => { + it('Verify a permissions slide to the user is displayed ', () => { safeapps.clickOnContinueBtn() safeapps.verifyCameraCheckBoxExists() safeapps.verifyMicrofoneCheckBoxExists() }) - it('Verify the selection can be changed, accepted and stored [C56138]', () => { + it('Verify the selection can be changed, accepted and stored ', () => { safeapps.verifyMicrofoneCheckBoxExists().click() safeapps.clickOnContinueBtn() safeapps.verifyWarningDefaultAppMsgIsDisplayed() diff --git a/cypress/e2e/safe-apps/info_modal.cy.js b/cypress/e2e/safe-apps/info_modal.cy.js index 650fe1b041..25f8eb4f42 100644 --- a/cypress/e2e/safe-apps/info_modal.cy.js +++ b/cypress/e2e/safe-apps/info_modal.cy.js @@ -9,20 +9,20 @@ describe('Safe Apps info modal tests', () => { main.acceptCookies() }) - it('Verify the disclaimer is displayed when a Safe App is opened [C56139]', () => { + it('Verify the disclaimer is displayed when a Safe App is opened ', () => { safeapps.clickOnApp(safeapps.transactionBuilderStr) safeapps.clickOnOpenSafeAppBtn() }) // Skip tests due to changed logic // TODO: Discuss furthers - it.skip('Verify the permissions slide is shown if the app require permissions [C56140]', () => { + it.skip('Verify the permissions slide is shown if the app require permissions ', () => { safeapps.clickOnContinueBtn() cy.wait(500) // wait for the animation to finish safeapps.verifyCameraCheckBoxExists() }) - it.skip('Verify the permissions and consents decision are stored when accepted [C56141]', () => { + it.skip('Verify the permissions and consents decision are stored when accepted ', () => { safeapps.storeAndVerifyPermissions() }) }) diff --git a/cypress/e2e/safe-apps/permissions_settings.cy.js b/cypress/e2e/safe-apps/permissions_settings.cy.js index ca88523a74..49e40171d0 100644 --- a/cypress/e2e/safe-apps/permissions_settings.cy.js +++ b/cypress/e2e/safe-apps/permissions_settings.cy.js @@ -49,7 +49,7 @@ describe('Safe Apps permissions settings tests', () => { main.acceptCookies() }) - it('Verify for each stored app the permissions configuration is shown [C56142]', () => { + it('Verify for each stored app the permissions configuration is shown ', () => { cy.findAllByRole('heading', { level: 5 }).should('have.length', 4) }) @@ -60,7 +60,7 @@ describe('Safe Apps permissions settings tests', () => { }) }) - it('Verify that app1 has camera, full screen and geo permissions [C56143]', () => { + it('Verify that app1 has camera, full screen and geo permissions ', () => { const app1Data = [ 'app1', safeapps.permissionCheckboxNames.camera, @@ -74,7 +74,7 @@ describe('Safe Apps permissions settings tests', () => { main.verifyCheckboxeState(safeapps.permissionCheckboxes.fullscreen, 0, constants.checkboxStates.checked) }) - it('Verify that app2 has address book and microphone permissions [C56144]', () => { + it('Verify that app2 has address book and microphone permissions ', () => { const app2Data = [ 'app2', safeapps.permissionCheckboxNames.addressbook, @@ -86,21 +86,21 @@ describe('Safe Apps permissions settings tests', () => { main.verifyCheckboxeState(safeapps.permissionCheckboxes.addressbook, 0, constants.checkboxStates.checked) }) - it('Verify that app3 has camera permissions [C56145]', () => { + it('Verify that app3 has camera permissions ', () => { const app3Data = ['app3', safeapps.permissionCheckboxNames.camera] main.checkTextsExistWithinElement($dapps[2], app3Data) main.verifyCheckboxeState(safeapps.permissionCheckboxes.camera, 1, constants.checkboxStates.unchecked) }) - it('Verify that app4 has address book permissions [C56146]', () => { + it('Verify that app4 has address book permissions ', () => { const app4Data = ['app4', safeapps.permissionCheckboxNames.addressbook] main.checkTextsExistWithinElement($dapps[3], app4Data) main.verifyCheckboxeState(safeapps.permissionCheckboxes.addressbook, 1, constants.checkboxStates.checked) }) - it('Verify Allow all or Clear all the checkboxes at once is permitted [C56147]', () => { + it('Verify Allow all or Clear all the checkboxes at once is permitted ', () => { safeapps.uncheckAllPermissions($dapps[1]) main.verifyCheckboxeState(safeapps.permissionCheckboxes.addressbook, 0, constants.checkboxStates.unchecked) main.verifyCheckboxeState(safeapps.permissionCheckboxes.microphone, 0, constants.checkboxStates.unchecked) @@ -110,7 +110,7 @@ describe('Safe Apps permissions settings tests', () => { main.verifyCheckboxeState(safeapps.permissionCheckboxes.microphone, 0, constants.checkboxStates.checked) }) - it('Verify it is permitted to remove apps and reflect it in the localStorage [C56148]', () => { + it('Verify it is permitted to remove apps and reflect it in the localStorage ', () => { cy.wrap($dapps[0]).find('svg').last().click() cy.wrap($dapps[2]) .find('svg') diff --git a/cypress/e2e/safe-apps/preview_drawer.cy.js b/cypress/e2e/safe-apps/preview_drawer.cy.js index 985a1bcad6..e7151b5570 100644 --- a/cypress/e2e/safe-apps/preview_drawer.cy.js +++ b/cypress/e2e/safe-apps/preview_drawer.cy.js @@ -9,7 +9,7 @@ describe('Safe Apps info modal tests', () => { main.acceptCookies() }) - it('Verify the preview drawer is displayed when opening a Safe App from the app list [C56149]', () => { + it('Verify the preview drawer is displayed when opening a Safe App from the app list ', () => { safeapps.clickOnApp(safeapps.transactionBuilderStr) cy.findByRole('presentation').within(() => { diff --git a/cypress/e2e/safe-apps/safe_permissions.cy.js b/cypress/e2e/safe-apps/safe_permissions.cy.js index 771beee30d..85f6c4eb52 100644 --- a/cypress/e2e/safe-apps/safe_permissions.cy.js +++ b/cypress/e2e/safe-apps/safe_permissions.cy.js @@ -15,7 +15,7 @@ describe('Safe permissions system tests', () => { }) }) - it('Verify that requesting permissions with wallet_requestPermissions shows the permissions prompt and return the permissions on accept [C56150]', () => { + it('Verify that requesting permissions with wallet_requestPermissions shows the permissions prompt and return the permissions on accept ', () => { cy.visitSafeApp(constants.testAppUrl + constants.requestPermissionsUrl) main.acceptCookies(1) safeapps.clickOnContinueBtn() @@ -38,7 +38,7 @@ describe('Safe permissions system tests', () => { }) }) - it('Verify that trying to get the current permissions with wallet_getPermissions returns the current permissions [C56151]', () => { + it('Verify that trying to get the current permissions with wallet_getPermissions returns the current permissions ', () => { cy.on('window:before:load', (window) => { window.localStorage.setItem( constants.SAFE_PERMISSIONS_KEY, diff --git a/cypress/e2e/safe-apps/tx_modal.cy.js b/cypress/e2e/safe-apps/tx_modal.cy.js index b2961038a1..6a23a9c1c5 100644 --- a/cypress/e2e/safe-apps/tx_modal.cy.js +++ b/cypress/e2e/safe-apps/tx_modal.cy.js @@ -19,7 +19,7 @@ describe('Transaction modal tests', () => { }) it( - 'Verify that the transaction popup is displayed when sending a transaction from an app [C56152]', + 'Verify that the transaction popup is displayed when sending a transaction from an app ', { defaultCommandTimeout: 12000 }, () => { cy.visitSafeApp(`${constants.testAppUrl}/dummy`) diff --git a/cypress/e2e/smoke/add_owner.cy.js b/cypress/e2e/smoke/add_owner.cy.js index a36635750c..2d1cdf082e 100644 --- a/cypress/e2e/smoke/add_owner.cy.js +++ b/cypress/e2e/smoke/add_owner.cy.js @@ -11,35 +11,35 @@ describe('Add Owners tests', () => { cy.contains(owner.safeAccountNonceStr, { timeout: 10000 }) }) - it('Verify the presence of "Add Owner" button [C56017]', () => { + it('Verify the presence of "Add Owner" button [C56219]', () => { owner.verifyAddOwnerBtnIsEnabled() }) - it('Verify “Add new owner” button tooltip displays correct message for Non-Owner [C56018]', () => { + it('Verify “Add new owner” button tooltip displays correct message for Non-Owner [C56220]', () => { cy.visit(constants.setupUrl + constants.SEPOLIA_TEST_SAFE_2) owner.verifyAddOwnerBtnIsDisabled() }) - it('Verify Tooltip displays correct message for disconnected user [C56019]', () => { + it('Verify Tooltip displays correct message for disconnected user [C56221]', () => { owner.waitForConnectionStatus() owner.clickOnWalletExpandMoreIcon() owner.clickOnDisconnectBtn() owner.verifyAddOwnerBtnIsDisabled() }) - it('Verify the Add New Owner Form can be opened [C56020]', () => { + it('Verify the Add New Owner Form can be opened [C56222]', () => { owner.waitForConnectionStatus() owner.openAddOwnerWindow() }) - it('Verify error message displayed if character limit is exceeded in Name input [C56022]', () => { + it('Verify error message displayed if character limit is exceeded in Name input [C56224]', () => { owner.waitForConnectionStatus() owner.openAddOwnerWindow() owner.typeOwnerName(main.generateRandomString(51)) owner.verifyErrorMsgInvalidAddress(constants.addressBookErrrMsg.exceedChars) }) - it('Verify that the "Name" field is auto-filled with the relevant name from Address Book [C56023]', () => { + it('Verify that the "Name" field is auto-filled with the relevant name from Address Book [C56225]', () => { cy.visit(constants.addressBookUrl + constants.SEPOLIA_TEST_SAFE_1) addressBook.clickOnCreateEntryBtn() addressBook.typeInName(constants.addresBookContacts.user1.name) @@ -54,7 +54,7 @@ describe('Add Owners tests', () => { owner.verifyNewOwnerName(constants.addresBookContacts.user1.name) }) - it('Verify that Name field not mandatory [C56024]', () => { + it('Verify that Name field not mandatory [C56226]', () => { owner.waitForConnectionStatus() owner.openAddOwnerWindow() owner.typeOwnerAddress(constants.SEPOLIA_OWNER_2) @@ -62,7 +62,7 @@ describe('Add Owners tests', () => { owner.verifyConfirmTransactionWindowDisplayed() }) - it('Verify relevant error messages are displayed in Address input [C56025]', () => { + it('Verify relevant error messages are displayed in Address input [C56227]', () => { owner.waitForConnectionStatus() owner.openAddOwnerWindow() owner.typeOwnerAddress(main.generateRandomString(10)) @@ -81,14 +81,14 @@ describe('Add Owners tests', () => { owner.verifyErrorMsgInvalidAddress(constants.addressBookErrrMsg.alreadyAdded) }) - it('Verify default threshold value. Verify correct threshold calculation [C56028]', () => { + it('Verify default threshold value. Verify correct threshold calculation [C56230]', () => { owner.waitForConnectionStatus() owner.openAddOwnerWindow() owner.typeOwnerAddress(constants.DEFAULT_OWNER_ADDRESS) owner.verifyThreshold(1, 2) }) - it('Verify valid Address validation [C56027]', () => { + it('Verify valid Address validation [C56229]', () => { owner.waitForConnectionStatus() owner.openAddOwnerWindow() owner.typeOwnerAddress(constants.SEPOLIA_OWNER_2) diff --git a/cypress/e2e/smoke/address_book.cy.js b/cypress/e2e/smoke/address_book.cy.js index 3a5af18932..000307f444 100644 --- a/cypress/e2e/smoke/address_book.cy.js +++ b/cypress/e2e/smoke/address_book.cy.js @@ -15,12 +15,12 @@ describe('Address book tests', () => { main.acceptCookies(1) }) - it('Verify entry can be added [C56061]', () => { + it('Verify entry can be added ', () => { addressBook.clickOnCreateEntryBtn() addressBook.addEntry(NAME, constants.RECIPIENT_ADDRESS) }) - it('Verify entered entry in Name input can be saved [C56063]', () => { + it('Verify entered entry in Name input can be saved ', () => { addressBook.clickOnCreateEntryBtn() addressBook.addEntry(NAME, constants.RECIPIENT_ADDRESS) addressBook.clickOnEditEntryBtn() @@ -29,7 +29,7 @@ describe('Address book tests', () => { addressBook.verifyNameWasChanged(NAME, EDITED_NAME) }) - it('Verify entry can be deleted [C56062]', () => { + it('Verify entry can be deleted ', () => { addressBook.clickOnCreateEntryBtn() addressBook.addEntry(NAME, constants.RECIPIENT_ADDRESS) // Click the delete button in the first entry @@ -38,14 +38,14 @@ describe('Address book tests', () => { addressBook.verifyEditedNameNotExists(EDITED_NAME) }) - it('Verify csv file can be imported (Goerli) [C56064]', () => { + it('Verify csv file can be imported (Goerli) ', () => { addressBook.clickOnImportFileBtn() addressBook.importFile() addressBook.verifyImportModalIsClosed() addressBook.verifyDataImported(constants.SEPOLIA_CSV_ENTRY.name, constants.SEPOLIA_CSV_ENTRY.address) }) - it.skip('Verify Gnosis Chain imported address can be found [C56066]', () => { + it.skip('Verify Gnosis Chain imported address can be found ', () => { // Go to a Safe on Gnosis Chain cy.get('header') .contains(/^G(ö|oe)rli$/) @@ -63,7 +63,7 @@ describe('Address book tests', () => { cy.contains(constants.GNO_CSV_ENTRY.address).should('exist') }) - it('Verify the address book file can be downloaded [C56065]', () => { + it('Verify the address book file can be downloaded ', () => { addressBook.clickOnImportFileBtn() addressBook.importFile() // Download the export file diff --git a/cypress/e2e/smoke/assets.cy.js b/cypress/e2e/smoke/assets.cy.js index aae5c455ad..5032ab56f4 100644 --- a/cypress/e2e/smoke/assets.cy.js +++ b/cypress/e2e/smoke/assets.cy.js @@ -16,15 +16,15 @@ describe('Assets tests', () => { main.acceptCookies(2) }) - it('Verify that the token tab is selected by default and the table is visible [C56039]', () => { + it('Verify that the token tab is selected by default and the table is visible ', () => { balances.verifyTokensTabIsSelected('true') }) - it('Verify that the native token is visible [C56040]', () => { + it('Verify that the native token is visible ', () => { balances.verifyTokenIsPresent(constants.tokenNames.sepoliaEther) }) - it('Verify that non-native tokens are present and have balance [C56041]', () => { + it('Verify that non-native tokens are present and have balance ', () => { balances.selectTokenList(balances.tokenListOptions.allTokens) balances.verifyBalance(balances.currencyDaiCap, TOKEN_AMOUNT_COLUMN, balances.currencyDaiAlttext) balances.verifyTokenBalanceFormat( @@ -81,7 +81,7 @@ describe('Assets tests', () => { ) }) - it('Verify that every token except the native token has a "go to blockexplorer link" [C56042]', () => { + it('Verify that every token except the native token has a "go to blockexplorer link" ', () => { balances.selectTokenList(balances.tokenListOptions.allTokens) // Specifying true for Sepolia. Will delete the flag once completely migrate to Sepolia balances.verifyAssetNameHasExplorerLink(balances.currencyUSDC, ASSET_NAME_COLUMN, true) @@ -93,7 +93,7 @@ describe('Assets tests', () => { balances.verifyAssetExplorerLinkNotAvailable(constants.tokenNames.sepoliaEther, ASSET_NAME_COLUMN) }) - it('Verify the default Fiat currency and the effects after changing it [C56043]', () => { + it('Verify the default Fiat currency and the effects after changing it ', () => { balances.selectTokenList(balances.tokenListOptions.allTokens) balances.verifyFirstRowDoesNotContainCurrency(balances.currencyEUR, FIAT_AMOUNT_COLUMN) balances.verifyFirstRowContainsCurrency(balances.currencyUSD, FIAT_AMOUNT_COLUMN) @@ -103,12 +103,12 @@ describe('Assets tests', () => { balances.verifyFirstRowContainsCurrency(balances.currencyEUR, FIAT_AMOUNT_COLUMN) }) - it('Verify that a tool tip is shown pointing to "Token list" dropdown [C56044]', () => { + it('Verify that a tool tip is shown pointing to "Token list" dropdown ', () => { //Spam warning message is removed in beforeEach hook cy.reload() }) - it('Verify that Token list dropdown down options show/hide spam tokens [C56045]', () => { + it('Verify that Token list dropdown down options show/hide spam tokens ', () => { let spamTokens = [ balances.currencyAave, balances.currencyTestTokenA, @@ -124,20 +124,20 @@ describe('Assets tests', () => { main.verifyValuesExist(balances.tokenListTable, spamTokens) }) - it('Verify that "Hide token" button is present and opens the "Hide tokens menu" [C56046]', () => { + it('Verify that "Hide token" button is present and opens the "Hide tokens menu" ', () => { balances.selectTokenList(balances.tokenListOptions.allTokens) balances.openHideTokenMenu() balances.verifyEachRowHasCheckbox() }) - it('Verify that checking the checkboxes increases the token selected counter [C56047]', () => { + it('Verify that checking the checkboxes increases the token selected counter ', () => { balances.selectTokenList(balances.tokenListOptions.allTokens) balances.openHideTokenMenu() balances.clickOnTokenCheckbox(balances.currencyLink) balances.checkTokenCounter(1) }) - it('Verify that selecting tokens and saving hides them from the table [C56048]', () => { + it('Verify that selecting tokens and saving hides them from the table ', () => { balances.selectTokenList(balances.tokenListOptions.allTokens) balances.openHideTokenMenu() balances.clickOnTokenCheckbox(balances.currencyLink) @@ -145,7 +145,7 @@ describe('Assets tests', () => { main.verifyValuesDoNotExist(balances.tokenListTable, [balances.currencyLink]) }) - it('Verify that Cancel closes the menu and does not change the table status [C56049]', () => { + it('Verify that Cancel closes the menu and does not change the table status ', () => { balances.selectTokenList(balances.tokenListOptions.allTokens) balances.openHideTokenMenu() balances.clickOnTokenCheckbox(balances.currencyLink) @@ -159,7 +159,7 @@ describe('Assets tests', () => { main.verifyValuesDoNotExist(balances.tokenListTable, [balances.currencyLink, balances.currencyAave]) }) - it('Verify that Deselect All unchecks all tokens from the list [C56050]', () => { + it('Verify that Deselect All unchecks all tokens from the list ', () => { balances.selectTokenList(balances.tokenListOptions.allTokens) balances.openHideTokenMenu() balances.clickOnTokenCheckbox(balances.currencyLink) @@ -168,7 +168,7 @@ describe('Assets tests', () => { balances.verifyEachRowHasCheckbox(constants.checkboxStates.unchecked) }) - it('Verify the Hidden tokens counter works for spam tokens [C56051]', () => { + it('Verify the Hidden tokens counter works for spam tokens ', () => { balances.selectTokenList(balances.tokenListOptions.allTokens) balances.openHideTokenMenu() balances.clickOnTokenCheckbox(balances.currencyLink) @@ -176,19 +176,19 @@ describe('Assets tests', () => { balances.checkHiddenTokenBtnCounter(1) }) - it('Verify the Hidden tokens counter works for native tokens [C56056]', () => { + it('Verify the Hidden tokens counter works for native tokens ', () => { balances.openHideTokenMenu() balances.clickOnTokenCheckbox(constants.tokenNames.sepoliaEther) balances.saveHiddenTokenSelection() balances.checkHiddenTokenBtnCounter(1) }) - it('Verify you can hide tokens from the eye icon in the table rows [C56053]', () => { + it('Verify you can hide tokens from the eye icon in the table rows ', () => { balances.selectTokenList(balances.tokenListOptions.allTokens) balances.hideAsset(balances.currencyLink) }) - it('Verify the sorting of "Assets" and "Balance" in the table [C56052]', () => { + it('Verify the sorting of "Assets" and "Balance" in the table ', () => { balances.selectTokenList(balances.tokenListOptions.allTokens) balances.verifyTableRows(7) balances.clickOnTokenNameSortBtn() @@ -201,12 +201,12 @@ describe('Assets tests', () => { balances.verifyTokenBalanceOrder('descending') }) - it('Verify that clicking the button with an owner opens the Send funds form [C56055]', () => { + it('Verify that clicking the button with an owner opens the Send funds form ', () => { balances.selectTokenList(balances.tokenListOptions.allTokens) balances.clickOnSendBtn(0) }) - it('Verify that the Send button shows when hovering a row [C56054]', () => { + it('Verify that the Send button shows when hovering a row ', () => { owner.clickOnWalletExpandMoreIcon() owner.clickOnDisconnectBtn() balances.selectTokenList(balances.tokenListOptions.allTokens) diff --git a/cypress/e2e/smoke/balances.cy.js b/cypress/e2e/smoke/balances.cy.js index df8e1c3681..61d16d7541 100644 --- a/cypress/e2e/smoke/balances.cy.js +++ b/cypress/e2e/smoke/balances.cy.js @@ -21,25 +21,25 @@ describe('Balance tests', () => { cy.get(balances.balanceSingleRow).should('have.length', ASSETS_LENGTH) }) - it('Verify that token is present: Dai [C56074]', () => { + it('Verify that token is present: Dai ', () => { balances.verityTokenAltImageIsVisible(balances.currencyDaiCap, balances.currencyDaiAlttext) balances.verifyAssetNameHasExplorerLink(balances.currencyDaiCap, ASSET_NAME_COLUMN) balances.verifyBalance(balances.currencyDaiCap, TOKEN_AMOUNT_COLUMN, balances.currencyDaiAlttext) }) - it('Verify that token is present: AAVE [C56075]', () => { + it('Verify that token is present: AAVE ', () => { balances.verityTokenAltImageIsVisible(balances.currencyAave, balances.currencyAaveAlttext) balances.verifyAssetNameHasExplorerLink(balances.currencyAave, ASSET_NAME_COLUMN) balances.verifyBalance(balances.currencyAave, TOKEN_AMOUNT_COLUMN, balances.currencyAaveAlttext) }) - it('Verify that token is present: LINK [C56076]', () => { + it('Verify that token is present: LINK ', () => { balances.verityTokenAltImageIsVisible(balances.currencyLink, balances.currencyLinkAlttext) balances.verifyAssetNameHasExplorerLink(balances.currencyLink, ASSET_NAME_COLUMN) balances.verifyBalance(balances.currencyLink, TOKEN_AMOUNT_COLUMN, balances.currencyLinkAlttext) }) - it('Verify Token and Fiat balances formatted as per specification [C56077]', () => { + it('Verify Token and Fiat balances formatted as per specification ', () => { balances.verifyTokenBalanceFormat( balances.currencyDaiCap, balances.currentcyDaiFormat, @@ -97,23 +97,23 @@ describe('Balance tests', () => { ) }) - it('Verify USD is default currency [C56078]', () => { + it('Verify USD is default currency ', () => { balances.verifyFirstRowDoesNotContainCurrency(balances.currencyEUR, FIAT_AMOUNT_COLUMN) balances.verifyFirstRowContainsCurrency(balances.currencyUSD, FIAT_AMOUNT_COLUMN) }) - it('Verify currency can be changed to EUR [C56079]', () => { + it('Verify currency can be changed to EUR ', () => { balances.clickOnCurrencyDropdown() balances.selectCurrency(balances.currencyEUR) balances.verifyFirstRowDoesNotContainCurrency(balances.currencyUSD, FIAT_AMOUNT_COLUMN) balances.verifyFirstRowContainsCurrency(balances.currencyEUR, FIAT_AMOUNT_COLUMN) }) - it('Verify a token can be hidden [C56080]', () => { + it('Verify a token can be hidden ', () => { balances.hideAsset(balances.currencyDaiCap) }) - it('Verify a token can be unhidden [C56081]', () => { + it('Verify a token can be unhidden ', () => { balances.hideAsset(balances.currencyDaiCap) balances.openHideTokenMenu() balances.clickOnTokenCheckbox(balances.currencyDaiCap) diff --git a/cypress/e2e/smoke/balances_pagination.cy.js b/cypress/e2e/smoke/balances_pagination.cy.js index b1033b9242..d0183a25d7 100644 --- a/cypress/e2e/smoke/balances_pagination.cy.js +++ b/cypress/e2e/smoke/balances_pagination.cy.js @@ -4,7 +4,7 @@ import * as main from '../../e2e/pages/main.page' const ASSETS_LENGTH = 8 -describe('Balance tests', () => { +describe('Balance pagination tests', () => { before(() => { cy.clearLocalStorage() // Open the Safe used for testing @@ -14,7 +14,7 @@ describe('Balance tests', () => { balances.selectTokenList(balances.tokenListOptions.allTokens) }) - it('Verify a user can change rows per page and navigate to next and previous page [C56073]', () => { + it('Verify a user can change rows per page and navigate to next and previous page ', () => { balances.verifyInitialTableState() balances.changeTo10RowsPerPage() balances.verifyTableHas10Rows() diff --git a/cypress/e2e/smoke/batch_tx.cy.js b/cypress/e2e/smoke/batch_tx.cy.js index 507b2b3ecd..2e25a40d8b 100644 --- a/cypress/e2e/smoke/batch_tx.cy.js +++ b/cypress/e2e/smoke/batch_tx.cy.js @@ -13,17 +13,17 @@ describe('Batch transaction tests', () => { main.acceptCookies(2) }) - it('Verify empty batch list can be opened [C56082]', () => { + it('Verify empty batch list can be opened ', () => { batch.openBatchtransactionsModal() batch.openNewTransactionModal() }) - it('Verify the Add batch button is present in a transaction form [C56084]', () => { + it('Verify the Add batch button is present in a transaction form ', () => { //The "true" is to validate that the add to batch button is not visible if "Yes, execute" is selected batch.addNewTransactionToBatch(constants.EOA, currentNonce, funds_first_tx) }) - it('Verify a transaction can be added to the batch [C56085]', () => { + it('Verify a transaction can be added to the batch ', () => { batch.addNewTransactionToBatch(constants.EOA, currentNonce, funds_first_tx) cy.contains(batch.transactionAddedToBatchStr).should('be.visible') //The batch button in the header shows the transaction count @@ -32,7 +32,7 @@ describe('Batch transaction tests', () => { batch.verifyAmountTransactionsInBatch(1) }) - it('Verify a second transaction can be added to the batch [C56086]', () => { + it('Verify a second transaction can be added to the batch ', () => { batch.addNewTransactionToBatch(constants.EOA, currentNonce, funds_first_tx) cy.wait(1000) batch.addNewTransactionToBatch(constants.EOA, currentNonce, funds_first_tx) @@ -41,7 +41,7 @@ describe('Batch transaction tests', () => { batch.verifyAmountTransactionsInBatch(2) }) - it('Verify the batch can be confirmed and related transactions exist in the form [C56088]', () => { + it('Verify the batch can be confirmed and related transactions exist in the form ', () => { batch.addNewTransactionToBatch(constants.EOA, currentNonce, funds_first_tx) cy.wait(1000) batch.addNewTransactionToBatch(constants.EOA, currentNonce, funds_first_tx) @@ -56,7 +56,7 @@ describe('Batch transaction tests', () => { cy.get('@TransactionList').find('li').eq(1).find('span').eq(0).contains(funds_first_tx) }) - it('Verify a transaction can be removed from the batch [C56089]', () => { + it('Verify a transaction can be removed from the batch ', () => { batch.addNewTransactionToBatch(constants.EOA, currentNonce, funds_first_tx) cy.wait(1000) batch.addNewTransactionToBatch(constants.EOA, currentNonce, funds_first_tx) diff --git a/cypress/e2e/smoke/beamer.cy.js b/cypress/e2e/smoke/beamer.cy.js index ab0ee152b9..5f4da54419 100644 --- a/cypress/e2e/smoke/beamer.cy.js +++ b/cypress/e2e/smoke/beamer.cy.js @@ -9,7 +9,7 @@ describe('Beamer tests', () => { main.acceptCookies() }) - it.skip('Verify "Updates" cookie acceptance is required before displaying Beamer [C56090]', () => { + it.skip('Verify "Updates" cookie acceptance is required before displaying Beamer ', () => { addressbook.clickOnWhatsNewBtn() addressbook.acceptBeamerCookies() addressbook.verifyBeamerIsChecked() diff --git a/cypress/e2e/smoke/create_safe_simple.cy.js b/cypress/e2e/smoke/create_safe_simple.cy.js index 9dc90f4723..5deeaee9eb 100644 --- a/cypress/e2e/smoke/create_safe_simple.cy.js +++ b/cypress/e2e/smoke/create_safe_simple.cy.js @@ -11,7 +11,7 @@ describe('Safe creation tests', () => { main.acceptCookies() }) - it('Verify a Wallet can be connected [C56101]', () => { + it('Verify a Wallet can be connected ', () => { owner.waitForConnectionStatus() cy.visit(constants.welcomeUrl) owner.clickOnWalletExpandMoreIcon() @@ -21,7 +21,7 @@ describe('Safe creation tests', () => { createwallet.connectWallet() }) - it('Verify Next button is disabled until switching to network is done [C56102]', () => { + it('Verify Next button is disabled until switching to network is done ', () => { owner.waitForConnectionStatus() createwallet.selectNetwork(constants.networks.ethereum) createwallet.checkNetworkChangeWarningMsg() @@ -30,44 +30,44 @@ describe('Safe creation tests', () => { createwallet.verifyNextBtnIsEnabled() }) - it('Verify that a new Wallet has default name related to the selected network [C56099]', () => { + it('Verify that a new Wallet has default name related to the selected network ', () => { owner.waitForConnectionStatus() createwallet.verifyDefaultWalletName(createwallet.defaltSepoliaPlaceholder) }) - it('Verify error message is displayed if wallet name input exceeds 50 characters [C56098]', () => { + it('Verify error message is displayed if wallet name input exceeds 50 characters ', () => { owner.waitForConnectionStatus() createwallet.typeWalletName(main.generateRandomString(51)) owner.verifyErrorMsgInvalidAddress(constants.addressBookErrrMsg.exceedChars) }) - it('Verify there is no error message is displayed if wallet name input contains less than 50 characters [C56100]', () => { + it('Verify there is no error message is displayed if wallet name input contains less than 50 characters ', () => { owner.waitForConnectionStatus() createwallet.typeWalletName(main.generateRandomString(50)) owner.verifyValidWalletName(constants.addressBookErrrMsg.exceedChars) }) - it('Verify current connected account is shown as default owner [C56091]', () => { + it('Verify current connected account is shown as default owner ', () => { owner.waitForConnectionStatus() owner.clickOnNextBtn() owner.verifyExistingOwnerAddress(0, constants.DEFAULT_OWNER_ADDRESS) }) - it('Verify error message is displayed if owner name input exceeds 50 characters [C56092]', () => { + it('Verify error message is displayed if owner name input exceeds 50 characters ', () => { owner.waitForConnectionStatus() owner.clickOnNextBtn() owner.typeExistingOwnerName(0, main.generateRandomString(51)) owner.verifyErrorMsgInvalidAddress(constants.addressBookErrrMsg.exceedChars) }) - it('Verify there is no error message is displayed if owner name input contains less than 50 characters [C56093]', () => { + it('Verify there is no error message is displayed if owner name input contains less than 50 characters ', () => { owner.waitForConnectionStatus() owner.clickOnNextBtn() owner.typeExistingOwnerName(0, main.generateRandomString(50)) owner.verifyValidWalletName(constants.addressBookErrrMsg.exceedChars) }) - it('Verify Add and Remove Owner Row works as expected [C56094]', () => { + it('Verify Add and Remove Owner Row works as expected ', () => { owner.waitForConnectionStatus() owner.clickOnNextBtn() createwallet.clickOnAddNewOwnerBtn() @@ -80,7 +80,7 @@ describe('Safe creation tests', () => { owner.verifyNumberOfOwners(2) }) - it('Verify Threshold Setup [C56096]', () => { + it('Verify Threshold Setup ', () => { owner.waitForConnectionStatus() owner.clickOnNextBtn() createwallet.clickOnAddNewOwnerBtn() @@ -93,7 +93,7 @@ describe('Safe creation tests', () => { owner.verifyThresholdLimit(1, 2) }) - it('Verify data persistence [C56103]', () => { + it('Verify data persistence ', () => { const ownerName = 'David' owner.waitForConnectionStatus() createwallet.typeWalletName(createwallet.walletName) @@ -123,13 +123,13 @@ describe('Safe creation tests', () => { createwallet.verifyEstimatedFeeInSummaryStep() }) - it('Verify tip is displayed on right side for threshold 1/1 [C56097]', () => { + it('Verify tip is displayed on right side for threshold 1/1 ', () => { owner.waitForConnectionStatus() owner.clickOnNextBtn() createwallet.verifyPolicy1_1() }) - it('Verify address input validation rules [C56095]', () => { + it('Verify address input validation rules ', () => { owner.waitForConnectionStatus() owner.clickOnNextBtn() createwallet.clickOnAddNewOwnerBtn() diff --git a/cypress/e2e/smoke/create_tx.cy.js b/cypress/e2e/smoke/create_tx.cy.js index b1a5e1ac1a..f686ae63d7 100644 --- a/cypress/e2e/smoke/create_tx.cy.js +++ b/cypress/e2e/smoke/create_tx.cy.js @@ -12,7 +12,7 @@ describe('Create transactions tests', () => { main.acceptCookies(2) }) - it('Verify a new send token transaction can be created [C56104]', () => { + it('Verify a new send token transaction can be created ', () => { createtx.clickOnNewtransactionBtn() createtx.clickOnSendTokensBtn() createtx.typeRecipientAddress(constants.EOA) @@ -23,7 +23,7 @@ describe('Create transactions tests', () => { createtx.clickOnNextBtn() }) - it('Verify a transaction can be reviewed, edited and submitted [C56105]', () => { + it('Verify a transaction can be reviewed, edited and submitted ', () => { createtx.verifySubmitBtnIsEnabled() cy.wait(1000) createtx.verifyNativeTokenTransfer() @@ -36,7 +36,7 @@ describe('Create transactions tests', () => { createtx.clickOnSignTransactionBtn() }) - it('Verify that clicking on notification shows the transaction in queue [C56106]', () => { + it('Verify that clicking on notification shows the transaction in queue ', () => { createtx.waitForProposeRequest() createtx.clickViewTransaction() createtx.verifySingleTxPage() diff --git a/cypress/e2e/smoke/dashboard.cy.js b/cypress/e2e/smoke/dashboard.cy.js index ad0ca6d0ee..ef1465618c 100644 --- a/cypress/e2e/smoke/dashboard.cy.js +++ b/cypress/e2e/smoke/dashboard.cy.js @@ -11,19 +11,19 @@ describe('Dashboard tests', () => { dashboard.verifyConnectTransactStrIsVisible() }) - it('Verify the overview widget is displayed [C56107]', () => { + it('Verify the overview widget is displayed ', () => { dashboard.verifyOverviewWidgetData() }) - it('Verify the transaction queue widget is displayed [C56108]', () => { + it('Verify the transaction queue widget is displayed ', () => { dashboard.verifyTxQueueWidget() }) - it('Verify the featured Safe Apps are displayed [C56109]', () => { + it('Verify the featured Safe Apps are displayed ', () => { dashboard.verifyFeaturedAppsSection() }) - it('Verify the Safe Apps Section is displayed [C56110]', () => { + it('Verify the Safe Apps Section is displayed ', () => { dashboard.verifySafeAppsSection() }) }) diff --git a/cypress/e2e/smoke/import_export_data.cy.js b/cypress/e2e/smoke/import_export_data.cy.js index de7e8f3881..e387df862b 100644 --- a/cypress/e2e/smoke/import_export_data.cy.js +++ b/cypress/e2e/smoke/import_export_data.cy.js @@ -11,7 +11,7 @@ describe('Import Export Data tests', () => { file.verifyImportBtnIsVisible() }) - it('Verify Safe can be accessed after test file upload [C56111]', () => { + it('Verify Safe can be accessed after test file upload ', () => { const filePath = '../fixtures/data_import.json' const safe = constants.SEPOLIA_CSV_ENTRY.name @@ -23,20 +23,20 @@ describe('Import Export Data tests', () => { file.clickOnClosePushNotificationsBanner() }) - it('Verify address book imported data [C56112]', () => { + it('Verify address book imported data ', () => { main.acceptCookies(1) file.clickOnAddressBookBtn() file.verifyImportedAddressBookData() }) - it('Verify pinned apps [C56113]', () => { + it('Verify pinned apps ', () => { const appNames = ['Transaction Builder'] file.clickOnAppsBtn() file.verifyAppsAreVisible(appNames) }) - it('Verify imported data in settings [C56114]', () => { + it('Verify imported data in settings ', () => { const unchecked = [file.prependChainPrefixStr, file.copyAddressStr] const checked = [file.darkModeStr] file.clickOnSettingsBtn() @@ -45,7 +45,7 @@ describe('Import Export Data tests', () => { file.verifyCheckboxes(checked, true) }) - it('Verifies data for export in Data tab [C56115]', () => { + it('Verifies data for export in Data tab ', () => { file.clickOnShowMoreTabsBtn() file.verifDataTabBtnIsVisible() file.clickOnDataTab() diff --git a/cypress/e2e/smoke/landing.cy.js b/cypress/e2e/smoke/landing.cy.js index 5482541966..13bd6d4116 100644 --- a/cypress/e2e/smoke/landing.cy.js +++ b/cypress/e2e/smoke/landing.cy.js @@ -1,6 +1,6 @@ import * as constants from '../../support/constants' describe('Landing page tests', () => { - it('Verify a user will be redirected to welcome page [C56116]', () => { + it('Verify a user will be redirected to welcome page ', () => { cy.clearLocalStorage() cy.visit('/') cy.url().should('include', constants.welcomeUrl) diff --git a/cypress/e2e/smoke/load_safe.cy.js b/cypress/e2e/smoke/load_safe.cy.js index 8aaa66bede..0ae0a0f596 100644 --- a/cypress/e2e/smoke/load_safe.cy.js +++ b/cypress/e2e/smoke/load_safe.cy.js @@ -26,7 +26,7 @@ describe('Load Safe tests', () => { cy.wait(2000) }) - it('Verify a network can be selected in the Safe [C56117]', () => { + it('Verify a network can be selected in the Safe ', () => { safe.clickNetworkSelector(constants.networks.sepolia) safe.selectPolygon() cy.wait(2000) @@ -34,7 +34,7 @@ describe('Load Safe tests', () => { safe.selectSepolia() }) - it('Verify only valid Safe name can be accepted [C56118]', () => { + it('Verify only valid Safe name can be accepted ', () => { // alias the address input label cy.get('input[name="address"]').parent().prev('label').as('addressLabel') @@ -62,14 +62,14 @@ describe('Load Safe tests', () => { safe.clickOnNextBtn() }) - it('Verify custom name in the first owner an be set [C56120]', () => { + it('Verify custom name in the first owner an be set ', () => { safe.inputNameAndAddress(testSafeName, constants.SEPOLIA_TEST_SAFE_1) safe.clickOnNextBtn() createwallet.typeOwnerName(testOwnerName, 0) safe.clickOnNextBtn() }) - it('Verify Safe and owner names are displayed in the Review step [C56121]', () => { + it('Verify Safe and owner names are displayed in the Review step ', () => { safe.inputNameAndAddress(testSafeName, constants.SEPOLIA_TEST_SAFE_1) safe.clickOnNextBtn() createwallet.typeOwnerName(testOwnerName, 0) @@ -78,7 +78,7 @@ describe('Load Safe tests', () => { safe.clickOnAddBtn() }) - it('Verify the custom Safe name is successfully loaded [C56122]', () => { + it('Verify the custom Safe name is successfully loaded ', () => { safe.inputNameAndAddress(testSafeName, constants.SEPOLIA_TEST_SAFE_2) safe.clickOnNextBtn() createwallet.typeOwnerName(testOwnerName, 0) diff --git a/cypress/e2e/smoke/nfts.cy.js b/cypress/e2e/smoke/nfts.cy.js index e86f1d671a..95615d6ffc 100644 --- a/cypress/e2e/smoke/nfts.cy.js +++ b/cypress/e2e/smoke/nfts.cy.js @@ -14,27 +14,27 @@ describe('NFTs tests', () => { nfts.clickOnNftsTab() }) - it('Verify that NFTs exist in the table [C56123]', () => { + it('Verify that NFTs exist in the table ', () => { nfts.verifyNFTNumber(20) }) - it('Verify NFT row contains data [C56124]', () => { + it('Verify NFT row contains data ', () => { nfts.verifyDataInTable(nftsName, nftsAddress, nftsTokenID) }) - it('Verify NFT preview window can be opened [C56125]', () => { + it('Verify NFT preview window can be opened ', () => { nfts.openNFT(1) nfts.verifyNameInNFTModal(nftsTokenID) nfts.verifySelectedNetwrokSepolia() nfts.closeNFTModal() }) - it('Verify NFT open does not open if no NFT exits [C56126]', () => { + it('Verify NFT open does not open if no NFT exits ', () => { nfts.clickOn6thNFT() nfts.verifyNFTModalDoesNotExist() }) - it('Verify multipls NFTs can be selected and reviewed [C56127]', () => { + it('Verify multipls NFTs can be selected and reviewed ', () => { nfts.verifyInitialNFTData() nfts.selectNFTs(3) nfts.deselectNFTs([2], 3) diff --git a/cypress/e2e/smoke/remove_owner.cy.js b/cypress/e2e/smoke/remove_owner.cy.js index b22848c26e..9a1d1ae21b 100644 --- a/cypress/e2e/smoke/remove_owner.cy.js +++ b/cypress/e2e/smoke/remove_owner.cy.js @@ -10,18 +10,18 @@ describe('Remove Owners tests', () => { cy.contains(owner.safeAccountNonceStr, { timeout: 10000 }) }) - it('Verify that "Remove" icon is visible [C56030]', () => { + it('Verify that "Remove" icon is visible', () => { cy.visit(constants.setupUrl + constants.SEPOLIA_TEST_SAFE_3) owner.verifyRemoveBtnIsEnabled().should('have.length', 2) }) - it('Verify Tooltip displays correct message for Non-Owner [C56037]', () => { + it('Verify Tooltip displays correct message for Non-Owner ', () => { cy.visit(constants.setupUrl + constants.SEPOLIA_TEST_SAFE_4) owner.waitForConnectionStatus() owner.verifyRemoveBtnIsDisabled() }) - it('Verify Tooltip displays correct message for disconnected user [C56031]', () => { + it('Verify Tooltip displays correct message for disconnected user ', () => { cy.visit(constants.setupUrl + constants.SEPOLIA_TEST_SAFE_3) owner.waitForConnectionStatus() owner.clickOnWalletExpandMoreIcon() @@ -29,13 +29,13 @@ describe('Remove Owners tests', () => { owner.verifyRemoveBtnIsDisabled() }) - it('Verify owner removal form can be opened [C56032]', () => { + it('Verify owner removal form can be opened ', () => { cy.visit(constants.setupUrl + constants.SEPOLIA_TEST_SAFE_3) owner.waitForConnectionStatus() owner.openRemoveOwnerWindow(1) }) - it('Verify threshold input displays the upper limit as the current safe number of owners minus one [C56033]', () => { + it('Verify threshold input displays the upper limit as the current safe number of owners minus one ', () => { cy.visit(constants.setupUrl + constants.SEPOLIA_TEST_SAFE_3) owner.waitForConnectionStatus() owner.openRemoveOwnerWindow(1) @@ -43,7 +43,7 @@ describe('Remove Owners tests', () => { owner.getThresholdOptions().should('have.length', 1) }) - it('Verify owner deletion confirmation is displayed [C56034]', () => { + it('Verify owner deletion confirmation is displayed ', () => { cy.visit(constants.setupUrl + constants.SEPOLIA_TEST_SAFE_3) owner.waitForConnectionStatus() owner.openRemoveOwnerWindow(1) diff --git a/cypress/e2e/smoke/replace_owner.cy.js b/cypress/e2e/smoke/replace_owner.cy.js index b43ec2a8ce..90d200c311 100644 --- a/cypress/e2e/smoke/replace_owner.cy.js +++ b/cypress/e2e/smoke/replace_owner.cy.js @@ -11,36 +11,36 @@ describe('Replace Owners tests', () => { cy.contains(owner.safeAccountNonceStr, { timeout: 10000 }) }) - it('Verify that "Replace" icon is visible [C55998]', () => { + it('Verify that "Replace" icon is visible ', () => { owner.verifyReplaceBtnIsEnabled() }) - it('Verify Tooltip displays correct message for Non-Owner [C56016]', () => { + it('Verify Tooltip displays correct message for Non-Owner ', () => { cy.visit(constants.setupUrl + constants.SEPOLIA_TEST_SAFE_2) owner.waitForConnectionStatus() owner.verifyReplaceBtnIsDisabled() }) - it('Verify Tooltip displays correct message for disconnected user [C56006]', () => { + it('Verify Tooltip displays correct message for disconnected user ', () => { owner.waitForConnectionStatus() owner.clickOnWalletExpandMoreIcon() owner.clickOnDisconnectBtn() owner.verifyReplaceBtnIsDisabled() }) - it('Verify that the owner replacement form is opened [C56007]', () => { + it('Verify that the owner replacement form is opened ', () => { owner.waitForConnectionStatus() owner.openReplaceOwnerWindow() }) - it('Verify max characters in name field [C56008]', () => { + it('Verify max characters in name field ', () => { owner.waitForConnectionStatus() owner.openReplaceOwnerWindow() owner.typeOwnerName(main.generateRandomString(51)) owner.verifyErrorMsgInvalidAddress(constants.addressBookErrrMsg.exceedChars) }) - it('Verify that Address input auto-fills with related value [C56009]', () => { + it('Verify that Address input auto-fills with related value ', () => { cy.visit(constants.addressBookUrl + constants.SEPOLIA_TEST_SAFE_1) addressBook.clickOnCreateEntryBtn() addressBook.typeInName(constants.addresBookContacts.user1.name) @@ -55,7 +55,7 @@ describe('Replace Owners tests', () => { owner.verifyNewOwnerName(constants.addresBookContacts.user1.name) }) - it('Verify that Name field not mandatory. Verify confirmation for owner replacement is displayed [C56011]', () => { + it('Verify that Name field not mandatory. Verify confirmation for owner replacement is displayed ', () => { owner.waitForConnectionStatus() owner.openReplaceOwnerWindow() owner.typeOwnerAddress(constants.SEPOLIA_OWNER_2) @@ -63,7 +63,7 @@ describe('Replace Owners tests', () => { owner.verifyConfirmTransactionWindowDisplayed() }) - it('Verify relevant error messages are displayed in Address input [C56012]', () => { + it('Verify relevant error messages are displayed in Address input ', () => { owner.waitForConnectionStatus() owner.openReplaceOwnerWindow() owner.typeOwnerAddress(main.generateRandomString(10)) diff --git a/cypress/e2e/smoke/tx_history.cy.js b/cypress/e2e/smoke/tx_history.cy.js index 42fdbeeb44..9b41d89c26 100644 --- a/cypress/e2e/smoke/tx_history.cy.js +++ b/cypress/e2e/smoke/tx_history.cy.js @@ -20,7 +20,7 @@ describe('Transaction history tests', () => { main.acceptCookies(1) }) - it('Verify October 29th transactions are displayed [C56128]', () => { + it('Verify October 29th transactions are displayed ', () => { const DATE = 'Oct 29, 2023' const NEXT_DATE_LABEL = 'Oct 20, 2023' const amount = '0.00001 ETH' @@ -45,7 +45,7 @@ describe('Transaction history tests', () => { }) }) - it('Verify transaction can be expanded/collapsed [C56129]', () => { + it('Verify transaction can be expanded/collapsed ', () => { createTx.clickOnTransactionExpandableItem('Oct 20, 2023', () => { createTx.verifyTransactionStrExists(str1) createTx.verifyTransactionStrExists(str2) From 1ea1877a65886570fd224d626cb139417002bd87 Mon Sep 17 00:00:00 2001 From: mike10ca Date: Thu, 9 Nov 2023 13:23:07 +0100 Subject: [PATCH 2/6] Update test IDs --- cypress/e2e/safe-apps/apps_list.cy.js | 2 +- .../e2e/safe-apps/drain_account.spec.cy.js | 2 +- cypress/e2e/safe-apps/info_modal.cy.js | 2 +- .../e2e/safe-apps/permissions_settings.cy.js | 4 +- cypress/e2e/safe-apps/preview_drawer.cy.js | 2 +- cypress/e2e/safe-apps/tx-builder.spec.cy.js | 38 +++++++++---------- cypress/e2e/safe-apps/tx_modal.cy.js | 2 +- cypress/e2e/smoke/pending_actions.cy.js | 2 +- 8 files changed, 27 insertions(+), 27 deletions(-) diff --git a/cypress/e2e/safe-apps/apps_list.cy.js b/cypress/e2e/safe-apps/apps_list.cy.js index c6b15a2265..48f353daca 100644 --- a/cypress/e2e/safe-apps/apps_list.cy.js +++ b/cypress/e2e/safe-apps/apps_list.cy.js @@ -5,7 +5,7 @@ import * as safeapps from '../pages/safeapps.pages' const myCustomAppTitle = 'Cypress Test App' const myCustomAppDescrAdded = 'Cypress Test App Description' -describe('Safe Apps tests', () => { +describe('Safe Apps list tests', () => { beforeEach(() => { cy.clearLocalStorage() cy.visit(constants.SEPOLIA_TEST_SAFE_4 + constants.appsUrl, { failOnStatusCode: false }) diff --git a/cypress/e2e/safe-apps/drain_account.spec.cy.js b/cypress/e2e/safe-apps/drain_account.spec.cy.js index 87d878f307..c9370dbefd 100644 --- a/cypress/e2e/safe-apps/drain_account.spec.cy.js +++ b/cypress/e2e/safe-apps/drain_account.spec.cy.js @@ -3,7 +3,7 @@ import * as constants from '../../support/constants' import * as main from '../pages/main.page' import * as safeapps from '../pages/safeapps.pages' -describe('Drain Account Safe App tests', { defaultCommandTimeout: 12000 }, () => { +describe('Drain Account tests', { defaultCommandTimeout: 12000 }, () => { const appUrl = constants.drainAccount_url const iframeSelector = `iframe[id="iframe-${appUrl}"]` const visitUrl = `/apps/open?safe=${constants.GOERLI_SAFE_APPS_SAFE}&appUrl=${encodeURIComponent(appUrl)}` diff --git a/cypress/e2e/safe-apps/info_modal.cy.js b/cypress/e2e/safe-apps/info_modal.cy.js index 25f8eb4f42..bf52ab1c95 100644 --- a/cypress/e2e/safe-apps/info_modal.cy.js +++ b/cypress/e2e/safe-apps/info_modal.cy.js @@ -2,7 +2,7 @@ import * as constants from '../../support/constants' import * as main from '../pages/main.page' import * as safeapps from '../pages/safeapps.pages' -describe('Safe Apps info modal tests', () => { +describe('Info modal tests', () => { beforeEach(() => { cy.clearLocalStorage() cy.visit(constants.SEPOLIA_TEST_SAFE_5 + constants.appsUrl, { failOnStatusCode: false }) diff --git a/cypress/e2e/safe-apps/permissions_settings.cy.js b/cypress/e2e/safe-apps/permissions_settings.cy.js index 49e40171d0..525305aff4 100644 --- a/cypress/e2e/safe-apps/permissions_settings.cy.js +++ b/cypress/e2e/safe-apps/permissions_settings.cy.js @@ -6,7 +6,7 @@ let $dapps = [] const app1 = 'https://app1.com' const app3 = 'https://app3.com' -describe('Safe Apps permissions settings tests', () => { +describe('Permissions settings tests', () => { before(() => { cy.clearLocalStorage() cy.on('window:before:load', (window) => { @@ -53,7 +53,7 @@ describe('Safe Apps permissions settings tests', () => { cy.findAllByRole('heading', { level: 5 }).should('have.length', 4) }) - describe('Permissions for each app', () => { + describe('Permissions for each Safe app', () => { before(() => { cy.get(safeapps.gridItem).then((items) => { $dapps = items diff --git a/cypress/e2e/safe-apps/preview_drawer.cy.js b/cypress/e2e/safe-apps/preview_drawer.cy.js index e7151b5570..30bf61a963 100644 --- a/cypress/e2e/safe-apps/preview_drawer.cy.js +++ b/cypress/e2e/safe-apps/preview_drawer.cy.js @@ -2,7 +2,7 @@ import * as constants from '../../support/constants' import * as main from '../pages/main.page' import * as safeapps from '../pages/safeapps.pages' -describe('Safe Apps info modal tests', () => { +describe('Preview drawer tests', () => { beforeEach(() => { cy.clearLocalStorage() cy.visit(`/${constants.SEPOLIA_TEST_SAFE_5}/apps`, { failOnStatusCode: false }) diff --git a/cypress/e2e/safe-apps/tx-builder.spec.cy.js b/cypress/e2e/safe-apps/tx-builder.spec.cy.js index fd850c112f..6c7dbeb295 100644 --- a/cypress/e2e/safe-apps/tx-builder.spec.cy.js +++ b/cypress/e2e/safe-apps/tx-builder.spec.cy.js @@ -3,7 +3,7 @@ import * as constants from '../../support/constants' import * as main from '../pages/main.page' import * as safeapps from '../pages/safeapps.pages' -describe('Tx-builder Safe App tests', { defaultCommandTimeout: 20000 }, () => { +describe('Transaction Builder tests', { defaultCommandTimeout: 20000 }, () => { const appUrl = constants.TX_Builder_url const iframeSelector = `iframe[id="iframe-${appUrl}"]` const visitUrl = `/apps/open?safe=${constants.GOERLI_SAFE_APPS_SAFE}&appUrl=${encodeURIComponent(appUrl)}` @@ -14,7 +14,7 @@ describe('Tx-builder Safe App tests', { defaultCommandTimeout: 20000 }, () => { safeapps.clickOnContinueBtn() }) - it('Verify a simple batch can be created [C56609]', () => { + it('Verify a simple batch can be created', () => { cy.enter(iframeSelector).then((getBody) => { getBody().findByLabelText(safeapps.enterAddressStr).type(constants.SAFE_APP_ADDRESS) getBody().find(safeapps.contractMethodIndex).parent().click() @@ -32,7 +32,7 @@ describe('Tx-builder Safe App tests', { defaultCommandTimeout: 20000 }, () => { cy.findAllByText(constants.SAFE_APP_ADDRESS_2_SHORT).should('have.length', 1) }) - it('Verify a complex batch can be created [C56610]', () => { + it('Verify a complex batch can be created', () => { cy.enter(iframeSelector).then((getBody) => { getBody().findByLabelText(safeapps.enterAddressStr).type(constants.SAFE_APP_ADDRESS) getBody().find(safeapps.contractMethodIndex).parent().click() @@ -54,7 +54,7 @@ describe('Tx-builder Safe App tests', { defaultCommandTimeout: 20000 }, () => { cy.findAllByText('True').should('have.length', 3) }) - it('Verify a batch can be created using ENS name [C56611]', () => { + it('Verify a batch can be created using ENS name', () => { cy.enter(iframeSelector).then((getBody) => { getBody().findByLabelText(safeapps.enterAddressStr).type(constants.ENS_TEST_GOERLI) getBody().findByRole('button', { name: safeapps.useImplementationABI }).click() @@ -72,7 +72,7 @@ describe('Tx-builder Safe App tests', { defaultCommandTimeout: 20000 }, () => { cy.findByText(safeapps.thresholdStr2).should('exist') }) - it('Verify a batch can be created from an ABI [C56612]', () => { + it('Verify a batch can be created from an ABI', () => { cy.enter(iframeSelector).then((getBody) => { getBody().findByLabelText(safeapps.enterABIStr).type(safeapps.abi) getBody().findByLabelText(safeapps.toAddressStr).type(constants.SAFE_APP_ADDRESS_2) @@ -85,7 +85,7 @@ describe('Tx-builder Safe App tests', { defaultCommandTimeout: 20000 }, () => { cy.findByText(constants.SAFE_APP_ADDRESS_2).should('be.visible') }) - it('Verify a batch with custom data can be created [C56613]', () => { + it('Verify a batch with custom data can be created', () => { cy.enter(iframeSelector).then((getBody) => { getBody().find('.MuiSwitch-root').click() getBody().findByLabelText(safeapps.enterAddressStr).type(constants.SAFE_APP_ADDRESS_3) @@ -99,7 +99,7 @@ describe('Tx-builder Safe App tests', { defaultCommandTimeout: 20000 }, () => { cy.findByText(constants.SAFE_APP_ADDRESS_3).should('be.visible') }) - it('Verify a batch can be cancelled [C56614]', () => { + it('Verify a batch can be cancelled', () => { cy.enter(iframeSelector).then((getBody) => { getBody().findByLabelText(safeapps.enterAddressStr).type(constants.SAFE_APP_ADDRESS) getBody().find(safeapps.contractMethodIndex).parent().click() @@ -114,7 +114,7 @@ describe('Tx-builder Safe App tests', { defaultCommandTimeout: 20000 }, () => { }) }) - it('Verify cancel operation can be reverted [C56615]', () => { + it('Verify cancel operation can be reverted', () => { cy.enter(iframeSelector).then((getBody) => { getBody().findByLabelText(safeapps.enterAddressStr).type(constants.SAFE_APP_ADDRESS) getBody().find(safeapps.contractMethodIndex).parent().click() @@ -129,7 +129,7 @@ describe('Tx-builder Safe App tests', { defaultCommandTimeout: 20000 }, () => { }) }) - it('Verify it is allowed to go back without removing data and add more transactions to the batch [C56616]', () => { + it('Verify it is allowed to go back without removing data and add more transactions to the batch', () => { cy.enter(iframeSelector).then((getBody) => { getBody().findByLabelText(safeapps.enterAddressStr).type(constants.SAFE_APP_ADDRESS) getBody().find(safeapps.contractMethodIndex).parent().click() @@ -150,14 +150,14 @@ describe('Tx-builder Safe App tests', { defaultCommandTimeout: 20000 }, () => { cy.get('p').contains('2').should('be.visible') }) - it('Verify a batch cannot be created with invalid address [C56617]', () => { + it('Verify a batch cannot be created with invalid address', () => { cy.enter(iframeSelector).then((getBody) => { getBody().findByLabelText(safeapps.enterAddressStr).type(constants.SAFE_APP_ADDRESS_3) getBody().findAllByText(safeapps.addressNotValidStr).should('have.css', 'color', 'rgb(244, 67, 54)') }) }) - it('Verify a batch cannot be created without asset amount [C56618]', () => { + it('Verify a batch cannot be created without asset amount', () => { cy.enter(iframeSelector).then((getBody) => { getBody().findByLabelText(safeapps.enterAddressStr).type(constants.SAFE_APP_ADDRESS_3) getBody().findByText(safeapps.addTransactionStr).click() @@ -165,7 +165,7 @@ describe('Tx-builder Safe App tests', { defaultCommandTimeout: 20000 }, () => { }) }) - it('Verify a batch cannot be created without method data [C56619]', () => { + it('Verify a batch cannot be created without method data', () => { cy.enter(iframeSelector).then((getBody) => { getBody().findByLabelText(safeapps.enterAddressStr).type(constants.SAFE_APP_ADDRESS_2) getBody().findByText(safeapps.addTransactionStr).click() @@ -173,7 +173,7 @@ describe('Tx-builder Safe App tests', { defaultCommandTimeout: 20000 }, () => { }) }) - it('Verify a batch can be uploaded, saved, downloaded and removed [C56620]', () => { + it('Verify a batch can be uploaded, saved, downloaded and removed', () => { cy.enter(iframeSelector).then((getBody) => { getBody().findAllByText('choose a file').attachFile('test-working-batch.json', { subjectType: 'drag-n-drop' }) getBody().findAllByText('uploaded').wait(300) @@ -190,7 +190,7 @@ describe('Tx-builder Safe App tests', { defaultCommandTimeout: 20000 }, () => { cy.readFile('cypress/downloads/E2E test.json').should('exist') }) - it('Verify there is notification if uploaded batch is from a different chain [C56621]', () => { + it('Verify there is notification if uploaded batch is from a different chain', () => { cy.enter(iframeSelector).then((getBody) => { getBody().findAllByText('choose a file').attachFile('test-mainnet-batch.json', { subjectType: 'drag-n-drop' }) getBody().findAllByText(safeapps.warningStr).should('be.visible') @@ -198,7 +198,7 @@ describe('Tx-builder Safe App tests', { defaultCommandTimeout: 20000 }, () => { }) }) - it('Verify there is error message when a modified batch is uploaded [C56622]', () => { + it('Verify there is error message when a modified batch is uploaded', () => { cy.enter(iframeSelector).then((getBody) => { getBody().findAllByText('choose a file').attachFile('test-modified-batch.json', { subjectType: 'drag-n-drop' }) getBody().findAllByText(safeapps.changedPropertiesStr) @@ -206,7 +206,7 @@ describe('Tx-builder Safe App tests', { defaultCommandTimeout: 20000 }, () => { }) }) - it('Verify an invalid batch cannot be uploaded [C56623]', () => { + it('Verify an invalid batch cannot be uploaded', () => { cy.enter(iframeSelector).then((getBody) => { getBody() .findAllByText('choose a file') @@ -216,7 +216,7 @@ describe('Tx-builder Safe App tests', { defaultCommandTimeout: 20000 }, () => { }) }) - it('Verify an empty batch cannot be uploaded [C56624]', () => { + it('Verify an empty batch cannot be uploaded', () => { cy.enter(iframeSelector).then((getBody) => { getBody() .findAllByText('choose a file') @@ -226,7 +226,7 @@ describe('Tx-builder Safe App tests', { defaultCommandTimeout: 20000 }, () => { }) }) - it('Verify a valid batch as successful can be simulated [C56625]', () => { + it('Verify a valid batch as successful can be simulated', () => { cy.enter(iframeSelector).then((getBody) => { getBody().findByLabelText(safeapps.enterAddressStr).type(constants.SAFE_APP_ADDRESS_3) getBody().findByLabelText(safeapps.gorValue).type('0') @@ -238,7 +238,7 @@ describe('Tx-builder Safe App tests', { defaultCommandTimeout: 20000 }, () => { }) }) - it('Verify an invalid batch as failed can be simulated [C56626]', () => { + it('Verify an invalid batch as failed can be simulated', () => { cy.enter(iframeSelector).then((getBody) => { getBody().findByLabelText(safeapps.enterAddressStr).type(constants.TEST_SAFE_2) getBody().findByText(safeapps.keepProxiABIStr).click() diff --git a/cypress/e2e/safe-apps/tx_modal.cy.js b/cypress/e2e/safe-apps/tx_modal.cy.js index 6a23a9c1c5..de4d0564f5 100644 --- a/cypress/e2e/safe-apps/tx_modal.cy.js +++ b/cypress/e2e/safe-apps/tx_modal.cy.js @@ -19,7 +19,7 @@ describe('Transaction modal tests', () => { }) it( - 'Verify that the transaction popup is displayed when sending a transaction from an app ', + 'Verify that the transaction popup is displayed when sending a transaction from an app [C56747]', { defaultCommandTimeout: 12000 }, () => { cy.visitSafeApp(`${constants.testAppUrl}/dummy`) diff --git a/cypress/e2e/smoke/pending_actions.cy.js b/cypress/e2e/smoke/pending_actions.cy.js index 6f8a9edc81..c7f2399954 100644 --- a/cypress/e2e/smoke/pending_actions.cy.js +++ b/cypress/e2e/smoke/pending_actions.cy.js @@ -1,7 +1,7 @@ import * as constants from '../../support/constants' import * as safe from '../pages/load_safe.pages' -describe('Pending actions', () => { +describe('Pending actions tests', () => { before(() => { cy.visit(constants.welcomeUrl) // main.acceptCookies() From cbe3d32e2da320753e2f8d758a7c555e1b8235d4 Mon Sep 17 00:00:00 2001 From: mike10ca Date: Thu, 9 Nov 2023 16:33:49 +0100 Subject: [PATCH 3/6] Update test descriptions to match mapping --- cypress/e2e/safe-apps/apps_list.cy.js | 8 ++--- .../e2e/safe-apps/browser_permissions.cy.js | 4 +-- .../e2e/safe-apps/drain_account.spec.cy.js | 18 +++++----- cypress/e2e/safe-apps/info_modal.cy.js | 6 ++-- .../e2e/safe-apps/permissions_settings.cy.js | 14 ++++---- cypress/e2e/safe-apps/preview_drawer.cy.js | 2 +- cypress/e2e/safe-apps/safe_permissions.cy.js | 4 +-- cypress/e2e/safe-apps/tx_modal.cy.js | 2 +- cypress/e2e/smoke/add_owner.cy.js | 20 +++++------ cypress/e2e/smoke/address_book.cy.js | 12 +++---- cypress/e2e/smoke/assets.cy.js | 36 +++++++++---------- cypress/e2e/smoke/balances.cy.js | 16 ++++----- cypress/e2e/smoke/balances_pagination.cy.js | 2 +- cypress/e2e/smoke/batch_tx.cy.js | 12 +++---- cypress/e2e/smoke/beamer.cy.js | 2 +- cypress/e2e/smoke/create_safe_simple.cy.js | 26 +++++++------- cypress/e2e/smoke/create_tx.cy.js | 6 ++-- cypress/e2e/smoke/dashboard.cy.js | 8 ++--- cypress/e2e/smoke/import_export_data.cy.js | 10 +++--- cypress/e2e/smoke/landing.cy.js | 2 +- cypress/e2e/smoke/load_safe.cy.js | 10 +++--- cypress/e2e/smoke/nfts.cy.js | 10 +++--- cypress/e2e/smoke/remove_owner.cy.js | 10 +++--- cypress/e2e/smoke/replace_owner.cy.js | 16 ++++----- cypress/e2e/smoke/tx_history.cy.js | 4 +-- 25 files changed, 130 insertions(+), 130 deletions(-) diff --git a/cypress/e2e/safe-apps/apps_list.cy.js b/cypress/e2e/safe-apps/apps_list.cy.js index 48f353daca..a1c9dbf63f 100644 --- a/cypress/e2e/safe-apps/apps_list.cy.js +++ b/cypress/e2e/safe-apps/apps_list.cy.js @@ -30,19 +30,19 @@ describe('Safe Apps list tests', () => { safeapps.verifyNoAppsTextPresent() }) - it('Verify apps can be pinned ', () => { + it('Verify apps can be pinned', () => { safeapps.clearSearchAppInput() safeapps.pinApp(safeapps.transactionBuilderStr) safeapps.verifyPinnedAppCount(1) }) - it('Verify apps can be unpinned ', () => { + it('Verify apps can be unpinned', () => { safeapps.pinApp(safeapps.transactionBuilderStr) safeapps.pinApp(safeapps.transactionBuilderStr, false) safeapps.verifyPinnedAppCount(0) }) - it('Verify there is an error when the app manifest is invalid ', () => { + it('Verify there is an error when the app manifest is invalid', () => { cy.intercept('GET', constants.invalidAppUrl, { name: constants.testAppData.name, }) @@ -52,7 +52,7 @@ describe('Safe Apps list tests', () => { safeapps.verifyAppNotSupportedMsg() }) - it('Verify an app can be added to the list within the custom apps section ', () => { + it('Verify an app can be added to the list within the custom apps section', () => { cy.intercept('GET', constants.validAppUrlJson, { name: constants.testAppData.name, description: constants.testAppData.descr, diff --git a/cypress/e2e/safe-apps/browser_permissions.cy.js b/cypress/e2e/safe-apps/browser_permissions.cy.js index 14f7bede90..a8414c0a06 100644 --- a/cypress/e2e/safe-apps/browser_permissions.cy.js +++ b/cypress/e2e/safe-apps/browser_permissions.cy.js @@ -18,13 +18,13 @@ describe('Browser permissions tests', () => { main.acceptCookies(1) }) - it('Verify a permissions slide to the user is displayed ', () => { + it('Verify a permissions slide to the user is displayed', () => { safeapps.clickOnContinueBtn() safeapps.verifyCameraCheckBoxExists() safeapps.verifyMicrofoneCheckBoxExists() }) - it('Verify the selection can be changed, accepted and stored ', () => { + it('Verify the selection can be changed, accepted and stored', () => { safeapps.verifyMicrofoneCheckBoxExists().click() safeapps.clickOnContinueBtn() safeapps.verifyWarningDefaultAppMsgIsDisplayed() diff --git a/cypress/e2e/safe-apps/drain_account.spec.cy.js b/cypress/e2e/safe-apps/drain_account.spec.cy.js index c9370dbefd..b6931cb353 100644 --- a/cypress/e2e/safe-apps/drain_account.spec.cy.js +++ b/cypress/e2e/safe-apps/drain_account.spec.cy.js @@ -19,16 +19,16 @@ describe('Drain Account tests', { defaultCommandTimeout: 12000 }, () => { safeapps.clickOnContinueBtn() }) - it('Verify drain can be created [C56627]', () => { + it('Verify drain can be created', () => { cy.enter(iframeSelector).then((getBody) => { getBody().findByLabelText(safeapps.recipientStr).type(constants.SAFE_APP_ADDRESS_2) - getBody().findAllByText(safeapps.transferEverythingStr).click() + getBody().findAllByText('Test invalid string').click() }) cy.findByRole('button', { name: safeapps.testTransfer1 }) cy.findByRole('button', { name: safeapps.testNativeTransfer2 }) }) - it('Verify partial drain can be created [C56628]', () => { + it('Verify partial drain can be created', () => { cy.enter(iframeSelector).then((getBody) => { getBody().findByLabelText(safeapps.selectAllRowsChbxStr).click() getBody().findAllByLabelText(safeapps.selectRowChbxStr).eq(1).click() @@ -40,7 +40,7 @@ describe('Drain Account tests', { defaultCommandTimeout: 12000 }, () => { cy.findByRole('button', { name: safeapps.testNativeTransfer1 }) }) - it('Verify a drain can be created when a ENS is specified [C56629]', () => { + it('Verify a drain can be created when a ENS is specified', () => { cy.enter(iframeSelector).then((getBody) => { getBody().findByLabelText(safeapps.recipientStr).type('goerli-test-safe.eth').wait(2000) getBody().findAllByText(safeapps.transferEverythingStr).click() @@ -50,7 +50,7 @@ describe('Drain Account tests', { defaultCommandTimeout: 12000 }, () => { }) // TODO: Adjust safe - owner - it.skip('Verify when cancelling a drain, previous data is preserved [C56630]', () => { + it.skip('Verify when cancelling a drain, previous data is preserved', () => { cy.enter(iframeSelector).then((getBody) => { getBody().findByLabelText(safeapps.recipientStr).type(constants.SAFE_APP_ADDRESS_2) getBody().findAllByText(safeapps.transferEverythingStr).click() @@ -61,14 +61,14 @@ describe('Drain Account tests', { defaultCommandTimeout: 12000 }, () => { }) }) - it('Verify a drain cannot be created with no recipient selected [C56631]', () => { + it('Verify a drain cannot be created with no recipient selected', () => { cy.enter(iframeSelector).then((getBody) => { getBody().findAllByText(safeapps.transferEverythingStr).click() getBody().findByText(safeapps.validRecipientAddressStr) }) }) - it('Verify a drain cannot be created with invalid recipient selected [C56632]', () => { + it('Verify a drain cannot be created with invalid recipient selected', () => { cy.enter(iframeSelector).then((getBody) => { getBody().findByLabelText(safeapps.recipientStr).type(constants.SAFE_APP_ADDRESS_2.substring(1)) getBody().findAllByText(safeapps.transferEverythingStr).click() @@ -76,7 +76,7 @@ describe('Drain Account tests', { defaultCommandTimeout: 12000 }, () => { }) }) - it('Verify a drain cannot be created when no assets are selected [C56633]', () => { + it('Verify a drain cannot be created when no assets are selected', () => { cy.enter(iframeSelector).then((getBody) => { getBody().findByLabelText(safeapps.selectAllRowsChbxStr).click() getBody().findByLabelText(safeapps.recipientStr).type(constants.SAFE_APP_ADDRESS_2) @@ -84,7 +84,7 @@ describe('Drain Account tests', { defaultCommandTimeout: 12000 }, () => { }) }) - it('should not allow to perform a drain when no assets and recipient are selected', () => { + it('Verify a drain cannot be created when no assets and recipient are selected', () => { cy.enter(iframeSelector).then((getBody) => { getBody().findByLabelText(safeapps.selectAllRowsChbxStr).click() getBody().findAllByText(safeapps.noTokensSelectedStr).should('be.visible') diff --git a/cypress/e2e/safe-apps/info_modal.cy.js b/cypress/e2e/safe-apps/info_modal.cy.js index bf52ab1c95..7aeab75e2e 100644 --- a/cypress/e2e/safe-apps/info_modal.cy.js +++ b/cypress/e2e/safe-apps/info_modal.cy.js @@ -9,20 +9,20 @@ describe('Info modal tests', () => { main.acceptCookies() }) - it('Verify the disclaimer is displayed when a Safe App is opened ', () => { + it('Verify the disclaimer is displayed when a Safe App is opened', () => { safeapps.clickOnApp(safeapps.transactionBuilderStr) safeapps.clickOnOpenSafeAppBtn() }) // Skip tests due to changed logic // TODO: Discuss furthers - it.skip('Verify the permissions slide is shown if the app require permissions ', () => { + it.skip('Verify the permissions slide is shown if the app require permissions', () => { safeapps.clickOnContinueBtn() cy.wait(500) // wait for the animation to finish safeapps.verifyCameraCheckBoxExists() }) - it.skip('Verify the permissions and consents decision are stored when accepted ', () => { + it.skip('Verify the permissions and consents decision are stored when accepted', () => { safeapps.storeAndVerifyPermissions() }) }) diff --git a/cypress/e2e/safe-apps/permissions_settings.cy.js b/cypress/e2e/safe-apps/permissions_settings.cy.js index 525305aff4..41484f9638 100644 --- a/cypress/e2e/safe-apps/permissions_settings.cy.js +++ b/cypress/e2e/safe-apps/permissions_settings.cy.js @@ -49,7 +49,7 @@ describe('Permissions settings tests', () => { main.acceptCookies() }) - it('Verify for each stored app the permissions configuration is shown ', () => { + it('Verify for each stored app the permissions configuration is shown', () => { cy.findAllByRole('heading', { level: 5 }).should('have.length', 4) }) @@ -60,7 +60,7 @@ describe('Permissions settings tests', () => { }) }) - it('Verify that app1 has camera, full screen and geo permissions ', () => { + it('Verify that app1 has camera, full screen and geo permissions', () => { const app1Data = [ 'app1', safeapps.permissionCheckboxNames.camera, @@ -74,7 +74,7 @@ describe('Permissions settings tests', () => { main.verifyCheckboxeState(safeapps.permissionCheckboxes.fullscreen, 0, constants.checkboxStates.checked) }) - it('Verify that app2 has address book and microphone permissions ', () => { + it('Verify that app2 has address book and microphone permissions', () => { const app2Data = [ 'app2', safeapps.permissionCheckboxNames.addressbook, @@ -86,21 +86,21 @@ describe('Permissions settings tests', () => { main.verifyCheckboxeState(safeapps.permissionCheckboxes.addressbook, 0, constants.checkboxStates.checked) }) - it('Verify that app3 has camera permissions ', () => { + it('Verify that app3 has camera permissions', () => { const app3Data = ['app3', safeapps.permissionCheckboxNames.camera] main.checkTextsExistWithinElement($dapps[2], app3Data) main.verifyCheckboxeState(safeapps.permissionCheckboxes.camera, 1, constants.checkboxStates.unchecked) }) - it('Verify that app4 has address book permissions ', () => { + it('Verify that app4 has address book permissions', () => { const app4Data = ['app4', safeapps.permissionCheckboxNames.addressbook] main.checkTextsExistWithinElement($dapps[3], app4Data) main.verifyCheckboxeState(safeapps.permissionCheckboxes.addressbook, 1, constants.checkboxStates.checked) }) - it('Verify Allow all or Clear all the checkboxes at once is permitted ', () => { + it('Verify Allow all or Clear all the checkboxes at once is permitted', () => { safeapps.uncheckAllPermissions($dapps[1]) main.verifyCheckboxeState(safeapps.permissionCheckboxes.addressbook, 0, constants.checkboxStates.unchecked) main.verifyCheckboxeState(safeapps.permissionCheckboxes.microphone, 0, constants.checkboxStates.unchecked) @@ -110,7 +110,7 @@ describe('Permissions settings tests', () => { main.verifyCheckboxeState(safeapps.permissionCheckboxes.microphone, 0, constants.checkboxStates.checked) }) - it('Verify it is permitted to remove apps and reflect it in the localStorage ', () => { + it('Verify it is permitted to remove apps and reflect it in the localStorage', () => { cy.wrap($dapps[0]).find('svg').last().click() cy.wrap($dapps[2]) .find('svg') diff --git a/cypress/e2e/safe-apps/preview_drawer.cy.js b/cypress/e2e/safe-apps/preview_drawer.cy.js index 30bf61a963..d9d3b35cf4 100644 --- a/cypress/e2e/safe-apps/preview_drawer.cy.js +++ b/cypress/e2e/safe-apps/preview_drawer.cy.js @@ -9,7 +9,7 @@ describe('Preview drawer tests', () => { main.acceptCookies() }) - it('Verify the preview drawer is displayed when opening a Safe App from the app list ', () => { + it('Verify the preview drawer is displayed when opening a Safe App from the app list', () => { safeapps.clickOnApp(safeapps.transactionBuilderStr) cy.findByRole('presentation').within(() => { diff --git a/cypress/e2e/safe-apps/safe_permissions.cy.js b/cypress/e2e/safe-apps/safe_permissions.cy.js index 85f6c4eb52..373e9da3bb 100644 --- a/cypress/e2e/safe-apps/safe_permissions.cy.js +++ b/cypress/e2e/safe-apps/safe_permissions.cy.js @@ -15,7 +15,7 @@ describe('Safe permissions system tests', () => { }) }) - it('Verify that requesting permissions with wallet_requestPermissions shows the permissions prompt and return the permissions on accept ', () => { + it('Verify that requesting permissions with wallet_requestPermissions shows the permissions prompt and return the permissions on accept', () => { cy.visitSafeApp(constants.testAppUrl + constants.requestPermissionsUrl) main.acceptCookies(1) safeapps.clickOnContinueBtn() @@ -38,7 +38,7 @@ describe('Safe permissions system tests', () => { }) }) - it('Verify that trying to get the current permissions with wallet_getPermissions returns the current permissions ', () => { + it('Verify that trying to get the current permissions with wallet_getPermissions returns the current permissions', () => { cy.on('window:before:load', (window) => { window.localStorage.setItem( constants.SAFE_PERMISSIONS_KEY, diff --git a/cypress/e2e/safe-apps/tx_modal.cy.js b/cypress/e2e/safe-apps/tx_modal.cy.js index de4d0564f5..38e46c2d03 100644 --- a/cypress/e2e/safe-apps/tx_modal.cy.js +++ b/cypress/e2e/safe-apps/tx_modal.cy.js @@ -19,7 +19,7 @@ describe('Transaction modal tests', () => { }) it( - 'Verify that the transaction popup is displayed when sending a transaction from an app [C56747]', + 'Verify that the transaction popup is displayed when sending a transaction from an app', { defaultCommandTimeout: 12000 }, () => { cy.visitSafeApp(`${constants.testAppUrl}/dummy`) diff --git a/cypress/e2e/smoke/add_owner.cy.js b/cypress/e2e/smoke/add_owner.cy.js index 2d1cdf082e..ff1653df84 100644 --- a/cypress/e2e/smoke/add_owner.cy.js +++ b/cypress/e2e/smoke/add_owner.cy.js @@ -11,35 +11,35 @@ describe('Add Owners tests', () => { cy.contains(owner.safeAccountNonceStr, { timeout: 10000 }) }) - it('Verify the presence of "Add Owner" button [C56219]', () => { + it('Verify the presence of "Add Owner" button', () => { owner.verifyAddOwnerBtnIsEnabled() }) - it('Verify “Add new owner” button tooltip displays correct message for Non-Owner [C56220]', () => { + it('Verify “Add new owner” button tooltip displays correct message for Non-Owner', () => { cy.visit(constants.setupUrl + constants.SEPOLIA_TEST_SAFE_2) owner.verifyAddOwnerBtnIsDisabled() }) - it('Verify Tooltip displays correct message for disconnected user [C56221]', () => { + it('Verify Tooltip displays correct message for disconnected user', () => { owner.waitForConnectionStatus() owner.clickOnWalletExpandMoreIcon() owner.clickOnDisconnectBtn() owner.verifyAddOwnerBtnIsDisabled() }) - it('Verify the Add New Owner Form can be opened [C56222]', () => { + it('Verify the Add New Owner Form can be opened', () => { owner.waitForConnectionStatus() owner.openAddOwnerWindow() }) - it('Verify error message displayed if character limit is exceeded in Name input [C56224]', () => { + it('Verify error message displayed if character limit is exceeded in Name input', () => { owner.waitForConnectionStatus() owner.openAddOwnerWindow() owner.typeOwnerName(main.generateRandomString(51)) owner.verifyErrorMsgInvalidAddress(constants.addressBookErrrMsg.exceedChars) }) - it('Verify that the "Name" field is auto-filled with the relevant name from Address Book [C56225]', () => { + it('Verify that the "Name" field is auto-filled with the relevant name from Address Book', () => { cy.visit(constants.addressBookUrl + constants.SEPOLIA_TEST_SAFE_1) addressBook.clickOnCreateEntryBtn() addressBook.typeInName(constants.addresBookContacts.user1.name) @@ -54,7 +54,7 @@ describe('Add Owners tests', () => { owner.verifyNewOwnerName(constants.addresBookContacts.user1.name) }) - it('Verify that Name field not mandatory [C56226]', () => { + it('Verify that Name field not mandatory', () => { owner.waitForConnectionStatus() owner.openAddOwnerWindow() owner.typeOwnerAddress(constants.SEPOLIA_OWNER_2) @@ -62,7 +62,7 @@ describe('Add Owners tests', () => { owner.verifyConfirmTransactionWindowDisplayed() }) - it('Verify relevant error messages are displayed in Address input [C56227]', () => { + it('Verify relevant error messages are displayed in Address input', () => { owner.waitForConnectionStatus() owner.openAddOwnerWindow() owner.typeOwnerAddress(main.generateRandomString(10)) @@ -81,14 +81,14 @@ describe('Add Owners tests', () => { owner.verifyErrorMsgInvalidAddress(constants.addressBookErrrMsg.alreadyAdded) }) - it('Verify default threshold value. Verify correct threshold calculation [C56230]', () => { + it('Verify default threshold value. Verify correct threshold calculation', () => { owner.waitForConnectionStatus() owner.openAddOwnerWindow() owner.typeOwnerAddress(constants.DEFAULT_OWNER_ADDRESS) owner.verifyThreshold(1, 2) }) - it('Verify valid Address validation [C56229]', () => { + it('Verify valid Address validation', () => { owner.waitForConnectionStatus() owner.openAddOwnerWindow() owner.typeOwnerAddress(constants.SEPOLIA_OWNER_2) diff --git a/cypress/e2e/smoke/address_book.cy.js b/cypress/e2e/smoke/address_book.cy.js index 000307f444..e909fa9b2a 100644 --- a/cypress/e2e/smoke/address_book.cy.js +++ b/cypress/e2e/smoke/address_book.cy.js @@ -15,12 +15,12 @@ describe('Address book tests', () => { main.acceptCookies(1) }) - it('Verify entry can be added ', () => { + it('Verify entry can be added', () => { addressBook.clickOnCreateEntryBtn() addressBook.addEntry(NAME, constants.RECIPIENT_ADDRESS) }) - it('Verify entered entry in Name input can be saved ', () => { + it('Verify entered entry in Name input can be saved', () => { addressBook.clickOnCreateEntryBtn() addressBook.addEntry(NAME, constants.RECIPIENT_ADDRESS) addressBook.clickOnEditEntryBtn() @@ -29,7 +29,7 @@ describe('Address book tests', () => { addressBook.verifyNameWasChanged(NAME, EDITED_NAME) }) - it('Verify entry can be deleted ', () => { + it('Verify entry can be deleted', () => { addressBook.clickOnCreateEntryBtn() addressBook.addEntry(NAME, constants.RECIPIENT_ADDRESS) // Click the delete button in the first entry @@ -38,14 +38,14 @@ describe('Address book tests', () => { addressBook.verifyEditedNameNotExists(EDITED_NAME) }) - it('Verify csv file can be imported (Goerli) ', () => { + it('Verify csv file can be imported (Goerli)', () => { addressBook.clickOnImportFileBtn() addressBook.importFile() addressBook.verifyImportModalIsClosed() addressBook.verifyDataImported(constants.SEPOLIA_CSV_ENTRY.name, constants.SEPOLIA_CSV_ENTRY.address) }) - it.skip('Verify Gnosis Chain imported address can be found ', () => { + it.skip('Verify Gnosis Chain imported address can be found', () => { // Go to a Safe on Gnosis Chain cy.get('header') .contains(/^G(ö|oe)rli$/) @@ -63,7 +63,7 @@ describe('Address book tests', () => { cy.contains(constants.GNO_CSV_ENTRY.address).should('exist') }) - it('Verify the address book file can be downloaded ', () => { + it('Verify the address book file can be downloaded', () => { addressBook.clickOnImportFileBtn() addressBook.importFile() // Download the export file diff --git a/cypress/e2e/smoke/assets.cy.js b/cypress/e2e/smoke/assets.cy.js index 5032ab56f4..07c23fb5d1 100644 --- a/cypress/e2e/smoke/assets.cy.js +++ b/cypress/e2e/smoke/assets.cy.js @@ -16,15 +16,15 @@ describe('Assets tests', () => { main.acceptCookies(2) }) - it('Verify that the token tab is selected by default and the table is visible ', () => { + it('Verify that the token tab is selected by default and the table is visible', () => { balances.verifyTokensTabIsSelected('true') }) - it('Verify that the native token is visible ', () => { + it('Verify that the native token is visible', () => { balances.verifyTokenIsPresent(constants.tokenNames.sepoliaEther) }) - it('Verify that non-native tokens are present and have balance ', () => { + it('Verify that non-native tokens are present and have balance', () => { balances.selectTokenList(balances.tokenListOptions.allTokens) balances.verifyBalance(balances.currencyDaiCap, TOKEN_AMOUNT_COLUMN, balances.currencyDaiAlttext) balances.verifyTokenBalanceFormat( @@ -81,7 +81,7 @@ describe('Assets tests', () => { ) }) - it('Verify that every token except the native token has a "go to blockexplorer link" ', () => { + it('Verify that every token except the native token has a "go to blockexplorer link"', () => { balances.selectTokenList(balances.tokenListOptions.allTokens) // Specifying true for Sepolia. Will delete the flag once completely migrate to Sepolia balances.verifyAssetNameHasExplorerLink(balances.currencyUSDC, ASSET_NAME_COLUMN, true) @@ -93,7 +93,7 @@ describe('Assets tests', () => { balances.verifyAssetExplorerLinkNotAvailable(constants.tokenNames.sepoliaEther, ASSET_NAME_COLUMN) }) - it('Verify the default Fiat currency and the effects after changing it ', () => { + it('Verify the default Fiat currency and the effects after changing it', () => { balances.selectTokenList(balances.tokenListOptions.allTokens) balances.verifyFirstRowDoesNotContainCurrency(balances.currencyEUR, FIAT_AMOUNT_COLUMN) balances.verifyFirstRowContainsCurrency(balances.currencyUSD, FIAT_AMOUNT_COLUMN) @@ -103,12 +103,12 @@ describe('Assets tests', () => { balances.verifyFirstRowContainsCurrency(balances.currencyEUR, FIAT_AMOUNT_COLUMN) }) - it('Verify that a tool tip is shown pointing to "Token list" dropdown ', () => { + it('Verify that a tool tip is shown pointing to "Token list" dropdown', () => { //Spam warning message is removed in beforeEach hook cy.reload() }) - it('Verify that Token list dropdown down options show/hide spam tokens ', () => { + it('Verify that Token list dropdown down options show/hide spam tokens', () => { let spamTokens = [ balances.currencyAave, balances.currencyTestTokenA, @@ -124,20 +124,20 @@ describe('Assets tests', () => { main.verifyValuesExist(balances.tokenListTable, spamTokens) }) - it('Verify that "Hide token" button is present and opens the "Hide tokens menu" ', () => { + it('Verify that "Hide token" button is present and opens the "Hide tokens menu"', () => { balances.selectTokenList(balances.tokenListOptions.allTokens) balances.openHideTokenMenu() balances.verifyEachRowHasCheckbox() }) - it('Verify that checking the checkboxes increases the token selected counter ', () => { + it('Verify that checking the checkboxes increases the token selected counter', () => { balances.selectTokenList(balances.tokenListOptions.allTokens) balances.openHideTokenMenu() balances.clickOnTokenCheckbox(balances.currencyLink) balances.checkTokenCounter(1) }) - it('Verify that selecting tokens and saving hides them from the table ', () => { + it('Verify that selecting tokens and saving hides them from the table', () => { balances.selectTokenList(balances.tokenListOptions.allTokens) balances.openHideTokenMenu() balances.clickOnTokenCheckbox(balances.currencyLink) @@ -145,7 +145,7 @@ describe('Assets tests', () => { main.verifyValuesDoNotExist(balances.tokenListTable, [balances.currencyLink]) }) - it('Verify that Cancel closes the menu and does not change the table status ', () => { + it('Verify that Cancel closes the menu and does not change the table status', () => { balances.selectTokenList(balances.tokenListOptions.allTokens) balances.openHideTokenMenu() balances.clickOnTokenCheckbox(balances.currencyLink) @@ -159,7 +159,7 @@ describe('Assets tests', () => { main.verifyValuesDoNotExist(balances.tokenListTable, [balances.currencyLink, balances.currencyAave]) }) - it('Verify that Deselect All unchecks all tokens from the list ', () => { + it('Verify that Deselect All unchecks all tokens from the list', () => { balances.selectTokenList(balances.tokenListOptions.allTokens) balances.openHideTokenMenu() balances.clickOnTokenCheckbox(balances.currencyLink) @@ -168,7 +168,7 @@ describe('Assets tests', () => { balances.verifyEachRowHasCheckbox(constants.checkboxStates.unchecked) }) - it('Verify the Hidden tokens counter works for spam tokens ', () => { + it('Verify the Hidden tokens counter works for spam tokens', () => { balances.selectTokenList(balances.tokenListOptions.allTokens) balances.openHideTokenMenu() balances.clickOnTokenCheckbox(balances.currencyLink) @@ -176,19 +176,19 @@ describe('Assets tests', () => { balances.checkHiddenTokenBtnCounter(1) }) - it('Verify the Hidden tokens counter works for native tokens ', () => { + it('Verify the Hidden tokens counter works for native tokens', () => { balances.openHideTokenMenu() balances.clickOnTokenCheckbox(constants.tokenNames.sepoliaEther) balances.saveHiddenTokenSelection() balances.checkHiddenTokenBtnCounter(1) }) - it('Verify you can hide tokens from the eye icon in the table rows ', () => { + it('Verify you can hide tokens from the eye icon in the table rows', () => { balances.selectTokenList(balances.tokenListOptions.allTokens) balances.hideAsset(balances.currencyLink) }) - it('Verify the sorting of "Assets" and "Balance" in the table ', () => { + it('Verify the sorting of "Assets" and "Balance" in the table', () => { balances.selectTokenList(balances.tokenListOptions.allTokens) balances.verifyTableRows(7) balances.clickOnTokenNameSortBtn() @@ -201,12 +201,12 @@ describe('Assets tests', () => { balances.verifyTokenBalanceOrder('descending') }) - it('Verify that clicking the button with an owner opens the Send funds form ', () => { + it('Verify that clicking the button with an owner opens the Send funds form', () => { balances.selectTokenList(balances.tokenListOptions.allTokens) balances.clickOnSendBtn(0) }) - it('Verify that the Send button shows when hovering a row ', () => { + it('Verify that the Send button shows when hovering a row', () => { owner.clickOnWalletExpandMoreIcon() owner.clickOnDisconnectBtn() balances.selectTokenList(balances.tokenListOptions.allTokens) diff --git a/cypress/e2e/smoke/balances.cy.js b/cypress/e2e/smoke/balances.cy.js index 61d16d7541..260938478f 100644 --- a/cypress/e2e/smoke/balances.cy.js +++ b/cypress/e2e/smoke/balances.cy.js @@ -21,25 +21,25 @@ describe('Balance tests', () => { cy.get(balances.balanceSingleRow).should('have.length', ASSETS_LENGTH) }) - it('Verify that token is present: Dai ', () => { + it('Verify that token is present: Dai', () => { balances.verityTokenAltImageIsVisible(balances.currencyDaiCap, balances.currencyDaiAlttext) balances.verifyAssetNameHasExplorerLink(balances.currencyDaiCap, ASSET_NAME_COLUMN) balances.verifyBalance(balances.currencyDaiCap, TOKEN_AMOUNT_COLUMN, balances.currencyDaiAlttext) }) - it('Verify that token is present: AAVE ', () => { + it('Verify that token is present: AAVE', () => { balances.verityTokenAltImageIsVisible(balances.currencyAave, balances.currencyAaveAlttext) balances.verifyAssetNameHasExplorerLink(balances.currencyAave, ASSET_NAME_COLUMN) balances.verifyBalance(balances.currencyAave, TOKEN_AMOUNT_COLUMN, balances.currencyAaveAlttext) }) - it('Verify that token is present: LINK ', () => { + it('Verify that token is present: LINK', () => { balances.verityTokenAltImageIsVisible(balances.currencyLink, balances.currencyLinkAlttext) balances.verifyAssetNameHasExplorerLink(balances.currencyLink, ASSET_NAME_COLUMN) balances.verifyBalance(balances.currencyLink, TOKEN_AMOUNT_COLUMN, balances.currencyLinkAlttext) }) - it('Verify Token and Fiat balances formatted as per specification ', () => { + it('Verify Token and Fiat balances formatted as per specification', () => { balances.verifyTokenBalanceFormat( balances.currencyDaiCap, balances.currentcyDaiFormat, @@ -97,23 +97,23 @@ describe('Balance tests', () => { ) }) - it('Verify USD is default currency ', () => { + it('Verify USD is default currency', () => { balances.verifyFirstRowDoesNotContainCurrency(balances.currencyEUR, FIAT_AMOUNT_COLUMN) balances.verifyFirstRowContainsCurrency(balances.currencyUSD, FIAT_AMOUNT_COLUMN) }) - it('Verify currency can be changed to EUR ', () => { + it('Verify currency can be changed to EUR', () => { balances.clickOnCurrencyDropdown() balances.selectCurrency(balances.currencyEUR) balances.verifyFirstRowDoesNotContainCurrency(balances.currencyUSD, FIAT_AMOUNT_COLUMN) balances.verifyFirstRowContainsCurrency(balances.currencyEUR, FIAT_AMOUNT_COLUMN) }) - it('Verify a token can be hidden ', () => { + it('Verify a token can be hidden', () => { balances.hideAsset(balances.currencyDaiCap) }) - it('Verify a token can be unhidden ', () => { + it('Verify a token can be unhidden', () => { balances.hideAsset(balances.currencyDaiCap) balances.openHideTokenMenu() balances.clickOnTokenCheckbox(balances.currencyDaiCap) diff --git a/cypress/e2e/smoke/balances_pagination.cy.js b/cypress/e2e/smoke/balances_pagination.cy.js index d0183a25d7..9ee15b9a58 100644 --- a/cypress/e2e/smoke/balances_pagination.cy.js +++ b/cypress/e2e/smoke/balances_pagination.cy.js @@ -14,7 +14,7 @@ describe('Balance pagination tests', () => { balances.selectTokenList(balances.tokenListOptions.allTokens) }) - it('Verify a user can change rows per page and navigate to next and previous page ', () => { + it('Verify a user can change rows per page and navigate to next and previous page', () => { balances.verifyInitialTableState() balances.changeTo10RowsPerPage() balances.verifyTableHas10Rows() diff --git a/cypress/e2e/smoke/batch_tx.cy.js b/cypress/e2e/smoke/batch_tx.cy.js index 2e25a40d8b..3a3ccaf6b0 100644 --- a/cypress/e2e/smoke/batch_tx.cy.js +++ b/cypress/e2e/smoke/batch_tx.cy.js @@ -13,17 +13,17 @@ describe('Batch transaction tests', () => { main.acceptCookies(2) }) - it('Verify empty batch list can be opened ', () => { + it('Verify empty batch list can be opened', () => { batch.openBatchtransactionsModal() batch.openNewTransactionModal() }) - it('Verify the Add batch button is present in a transaction form ', () => { + it('Verify the Add batch button is present in a transaction form', () => { //The "true" is to validate that the add to batch button is not visible if "Yes, execute" is selected batch.addNewTransactionToBatch(constants.EOA, currentNonce, funds_first_tx) }) - it('Verify a transaction can be added to the batch ', () => { + it('Verify a transaction can be added to the batch', () => { batch.addNewTransactionToBatch(constants.EOA, currentNonce, funds_first_tx) cy.contains(batch.transactionAddedToBatchStr).should('be.visible') //The batch button in the header shows the transaction count @@ -32,7 +32,7 @@ describe('Batch transaction tests', () => { batch.verifyAmountTransactionsInBatch(1) }) - it('Verify a second transaction can be added to the batch ', () => { + it('Verify a second transaction can be added to the batch', () => { batch.addNewTransactionToBatch(constants.EOA, currentNonce, funds_first_tx) cy.wait(1000) batch.addNewTransactionToBatch(constants.EOA, currentNonce, funds_first_tx) @@ -41,7 +41,7 @@ describe('Batch transaction tests', () => { batch.verifyAmountTransactionsInBatch(2) }) - it('Verify the batch can be confirmed and related transactions exist in the form ', () => { + it('Verify the batch can be confirmed and related transactions exist in the form', () => { batch.addNewTransactionToBatch(constants.EOA, currentNonce, funds_first_tx) cy.wait(1000) batch.addNewTransactionToBatch(constants.EOA, currentNonce, funds_first_tx) @@ -56,7 +56,7 @@ describe('Batch transaction tests', () => { cy.get('@TransactionList').find('li').eq(1).find('span').eq(0).contains(funds_first_tx) }) - it('Verify a transaction can be removed from the batch ', () => { + it('Verify a transaction can be removed from the batch', () => { batch.addNewTransactionToBatch(constants.EOA, currentNonce, funds_first_tx) cy.wait(1000) batch.addNewTransactionToBatch(constants.EOA, currentNonce, funds_first_tx) diff --git a/cypress/e2e/smoke/beamer.cy.js b/cypress/e2e/smoke/beamer.cy.js index 5f4da54419..4280e7f15a 100644 --- a/cypress/e2e/smoke/beamer.cy.js +++ b/cypress/e2e/smoke/beamer.cy.js @@ -9,7 +9,7 @@ describe('Beamer tests', () => { main.acceptCookies() }) - it.skip('Verify "Updates" cookie acceptance is required before displaying Beamer ', () => { + it.skip('Verify "Updates" cookie acceptance is required before displaying Beamer', () => { addressbook.clickOnWhatsNewBtn() addressbook.acceptBeamerCookies() addressbook.verifyBeamerIsChecked() diff --git a/cypress/e2e/smoke/create_safe_simple.cy.js b/cypress/e2e/smoke/create_safe_simple.cy.js index 5deeaee9eb..036cf19b9a 100644 --- a/cypress/e2e/smoke/create_safe_simple.cy.js +++ b/cypress/e2e/smoke/create_safe_simple.cy.js @@ -11,7 +11,7 @@ describe('Safe creation tests', () => { main.acceptCookies() }) - it('Verify a Wallet can be connected ', () => { + it('Verify a Wallet can be connected', () => { owner.waitForConnectionStatus() cy.visit(constants.welcomeUrl) owner.clickOnWalletExpandMoreIcon() @@ -21,7 +21,7 @@ describe('Safe creation tests', () => { createwallet.connectWallet() }) - it('Verify Next button is disabled until switching to network is done ', () => { + it('Verify Next button is disabled until switching to network is done', () => { owner.waitForConnectionStatus() createwallet.selectNetwork(constants.networks.ethereum) createwallet.checkNetworkChangeWarningMsg() @@ -30,44 +30,44 @@ describe('Safe creation tests', () => { createwallet.verifyNextBtnIsEnabled() }) - it('Verify that a new Wallet has default name related to the selected network ', () => { + it('Verify that a new Wallet has default name related to the selected network', () => { owner.waitForConnectionStatus() createwallet.verifyDefaultWalletName(createwallet.defaltSepoliaPlaceholder) }) - it('Verify error message is displayed if wallet name input exceeds 50 characters ', () => { + it('Verify error message is displayed if wallet name input exceeds 50 characters', () => { owner.waitForConnectionStatus() createwallet.typeWalletName(main.generateRandomString(51)) owner.verifyErrorMsgInvalidAddress(constants.addressBookErrrMsg.exceedChars) }) - it('Verify there is no error message is displayed if wallet name input contains less than 50 characters ', () => { + it('Verify there is no error message is displayed if wallet name input contains less than 50 characters', () => { owner.waitForConnectionStatus() createwallet.typeWalletName(main.generateRandomString(50)) owner.verifyValidWalletName(constants.addressBookErrrMsg.exceedChars) }) - it('Verify current connected account is shown as default owner ', () => { + it('Verify current connected account is shown as default owner', () => { owner.waitForConnectionStatus() owner.clickOnNextBtn() owner.verifyExistingOwnerAddress(0, constants.DEFAULT_OWNER_ADDRESS) }) - it('Verify error message is displayed if owner name input exceeds 50 characters ', () => { + it('Verify error message is displayed if owner name input exceeds 50 characters', () => { owner.waitForConnectionStatus() owner.clickOnNextBtn() owner.typeExistingOwnerName(0, main.generateRandomString(51)) owner.verifyErrorMsgInvalidAddress(constants.addressBookErrrMsg.exceedChars) }) - it('Verify there is no error message is displayed if owner name input contains less than 50 characters ', () => { + it('Verify there is no error message is displayed if owner name input contains less than 50 characters', () => { owner.waitForConnectionStatus() owner.clickOnNextBtn() owner.typeExistingOwnerName(0, main.generateRandomString(50)) owner.verifyValidWalletName(constants.addressBookErrrMsg.exceedChars) }) - it('Verify Add and Remove Owner Row works as expected ', () => { + it('Verify Add and Remove Owner Row works as expected', () => { owner.waitForConnectionStatus() owner.clickOnNextBtn() createwallet.clickOnAddNewOwnerBtn() @@ -80,7 +80,7 @@ describe('Safe creation tests', () => { owner.verifyNumberOfOwners(2) }) - it('Verify Threshold Setup ', () => { + it('Verify Threshold Setup', () => { owner.waitForConnectionStatus() owner.clickOnNextBtn() createwallet.clickOnAddNewOwnerBtn() @@ -93,7 +93,7 @@ describe('Safe creation tests', () => { owner.verifyThresholdLimit(1, 2) }) - it('Verify data persistence ', () => { + it('Verify data persistence', () => { const ownerName = 'David' owner.waitForConnectionStatus() createwallet.typeWalletName(createwallet.walletName) @@ -123,13 +123,13 @@ describe('Safe creation tests', () => { createwallet.verifyEstimatedFeeInSummaryStep() }) - it('Verify tip is displayed on right side for threshold 1/1 ', () => { + it('Verify tip is displayed on right side for threshold 1/1', () => { owner.waitForConnectionStatus() owner.clickOnNextBtn() createwallet.verifyPolicy1_1() }) - it('Verify address input validation rules ', () => { + it('Verify address input validation rules', () => { owner.waitForConnectionStatus() owner.clickOnNextBtn() createwallet.clickOnAddNewOwnerBtn() diff --git a/cypress/e2e/smoke/create_tx.cy.js b/cypress/e2e/smoke/create_tx.cy.js index f686ae63d7..f8c6d74dac 100644 --- a/cypress/e2e/smoke/create_tx.cy.js +++ b/cypress/e2e/smoke/create_tx.cy.js @@ -12,7 +12,7 @@ describe('Create transactions tests', () => { main.acceptCookies(2) }) - it('Verify a new send token transaction can be created ', () => { + it('Verify a new send token transaction can be created', () => { createtx.clickOnNewtransactionBtn() createtx.clickOnSendTokensBtn() createtx.typeRecipientAddress(constants.EOA) @@ -23,7 +23,7 @@ describe('Create transactions tests', () => { createtx.clickOnNextBtn() }) - it('Verify a transaction can be reviewed, edited and submitted ', () => { + it('Verify a transaction can be reviewed, edited and submitted', () => { createtx.verifySubmitBtnIsEnabled() cy.wait(1000) createtx.verifyNativeTokenTransfer() @@ -36,7 +36,7 @@ describe('Create transactions tests', () => { createtx.clickOnSignTransactionBtn() }) - it('Verify that clicking on notification shows the transaction in queue ', () => { + it('Verify that clicking on notification shows the transaction in queue', () => { createtx.waitForProposeRequest() createtx.clickViewTransaction() createtx.verifySingleTxPage() diff --git a/cypress/e2e/smoke/dashboard.cy.js b/cypress/e2e/smoke/dashboard.cy.js index ef1465618c..b0e7bd554f 100644 --- a/cypress/e2e/smoke/dashboard.cy.js +++ b/cypress/e2e/smoke/dashboard.cy.js @@ -11,19 +11,19 @@ describe('Dashboard tests', () => { dashboard.verifyConnectTransactStrIsVisible() }) - it('Verify the overview widget is displayed ', () => { + it('Verify the overview widget is displayed', () => { dashboard.verifyOverviewWidgetData() }) - it('Verify the transaction queue widget is displayed ', () => { + it('Verify the transaction queue widget is displayed', () => { dashboard.verifyTxQueueWidget() }) - it('Verify the featured Safe Apps are displayed ', () => { + it('Verify the featured Safe Apps are displayed', () => { dashboard.verifyFeaturedAppsSection() }) - it('Verify the Safe Apps Section is displayed ', () => { + it('Verify the Safe Apps Section is displayed', () => { dashboard.verifySafeAppsSection() }) }) diff --git a/cypress/e2e/smoke/import_export_data.cy.js b/cypress/e2e/smoke/import_export_data.cy.js index e387df862b..553707e14e 100644 --- a/cypress/e2e/smoke/import_export_data.cy.js +++ b/cypress/e2e/smoke/import_export_data.cy.js @@ -11,7 +11,7 @@ describe('Import Export Data tests', () => { file.verifyImportBtnIsVisible() }) - it('Verify Safe can be accessed after test file upload ', () => { + it('Verify Safe can be accessed after test file upload', () => { const filePath = '../fixtures/data_import.json' const safe = constants.SEPOLIA_CSV_ENTRY.name @@ -23,20 +23,20 @@ describe('Import Export Data tests', () => { file.clickOnClosePushNotificationsBanner() }) - it('Verify address book imported data ', () => { + it('Verify address book imported data', () => { main.acceptCookies(1) file.clickOnAddressBookBtn() file.verifyImportedAddressBookData() }) - it('Verify pinned apps ', () => { + it('Verify pinned apps', () => { const appNames = ['Transaction Builder'] file.clickOnAppsBtn() file.verifyAppsAreVisible(appNames) }) - it('Verify imported data in settings ', () => { + it('Verify imported data in settings', () => { const unchecked = [file.prependChainPrefixStr, file.copyAddressStr] const checked = [file.darkModeStr] file.clickOnSettingsBtn() @@ -45,7 +45,7 @@ describe('Import Export Data tests', () => { file.verifyCheckboxes(checked, true) }) - it('Verifies data for export in Data tab ', () => { + it('Verifies data for export in Data tab', () => { file.clickOnShowMoreTabsBtn() file.verifDataTabBtnIsVisible() file.clickOnDataTab() diff --git a/cypress/e2e/smoke/landing.cy.js b/cypress/e2e/smoke/landing.cy.js index 13bd6d4116..206732446f 100644 --- a/cypress/e2e/smoke/landing.cy.js +++ b/cypress/e2e/smoke/landing.cy.js @@ -1,6 +1,6 @@ import * as constants from '../../support/constants' describe('Landing page tests', () => { - it('Verify a user will be redirected to welcome page ', () => { + it('Verify a user will be redirected to welcome page', () => { cy.clearLocalStorage() cy.visit('/') cy.url().should('include', constants.welcomeUrl) diff --git a/cypress/e2e/smoke/load_safe.cy.js b/cypress/e2e/smoke/load_safe.cy.js index 0ae0a0f596..edce4b7f8b 100644 --- a/cypress/e2e/smoke/load_safe.cy.js +++ b/cypress/e2e/smoke/load_safe.cy.js @@ -26,7 +26,7 @@ describe('Load Safe tests', () => { cy.wait(2000) }) - it('Verify a network can be selected in the Safe ', () => { + it('Verify a network can be selected in the Safe', () => { safe.clickNetworkSelector(constants.networks.sepolia) safe.selectPolygon() cy.wait(2000) @@ -34,7 +34,7 @@ describe('Load Safe tests', () => { safe.selectSepolia() }) - it('Verify only valid Safe name can be accepted ', () => { + it('Verify only valid Safe name can be accepted', () => { // alias the address input label cy.get('input[name="address"]').parent().prev('label').as('addressLabel') @@ -62,14 +62,14 @@ describe('Load Safe tests', () => { safe.clickOnNextBtn() }) - it('Verify custom name in the first owner an be set ', () => { + it('Verify custom name in the first owner an be set', () => { safe.inputNameAndAddress(testSafeName, constants.SEPOLIA_TEST_SAFE_1) safe.clickOnNextBtn() createwallet.typeOwnerName(testOwnerName, 0) safe.clickOnNextBtn() }) - it('Verify Safe and owner names are displayed in the Review step ', () => { + it('Verify Safe and owner names are displayed in the Review step', () => { safe.inputNameAndAddress(testSafeName, constants.SEPOLIA_TEST_SAFE_1) safe.clickOnNextBtn() createwallet.typeOwnerName(testOwnerName, 0) @@ -78,7 +78,7 @@ describe('Load Safe tests', () => { safe.clickOnAddBtn() }) - it('Verify the custom Safe name is successfully loaded ', () => { + it('Verify the custom Safe name is successfully loaded', () => { safe.inputNameAndAddress(testSafeName, constants.SEPOLIA_TEST_SAFE_2) safe.clickOnNextBtn() createwallet.typeOwnerName(testOwnerName, 0) diff --git a/cypress/e2e/smoke/nfts.cy.js b/cypress/e2e/smoke/nfts.cy.js index 95615d6ffc..8aaa3c702f 100644 --- a/cypress/e2e/smoke/nfts.cy.js +++ b/cypress/e2e/smoke/nfts.cy.js @@ -14,27 +14,27 @@ describe('NFTs tests', () => { nfts.clickOnNftsTab() }) - it('Verify that NFTs exist in the table ', () => { + it('Verify that NFTs exist in the table', () => { nfts.verifyNFTNumber(20) }) - it('Verify NFT row contains data ', () => { + it('Verify NFT row contains data', () => { nfts.verifyDataInTable(nftsName, nftsAddress, nftsTokenID) }) - it('Verify NFT preview window can be opened ', () => { + it('Verify NFT preview window can be opened', () => { nfts.openNFT(1) nfts.verifyNameInNFTModal(nftsTokenID) nfts.verifySelectedNetwrokSepolia() nfts.closeNFTModal() }) - it('Verify NFT open does not open if no NFT exits ', () => { + it('Verify NFT open does not open if no NFT exits', () => { nfts.clickOn6thNFT() nfts.verifyNFTModalDoesNotExist() }) - it('Verify multipls NFTs can be selected and reviewed ', () => { + it('Verify multipls NFTs can be selected and reviewed', () => { nfts.verifyInitialNFTData() nfts.selectNFTs(3) nfts.deselectNFTs([2], 3) diff --git a/cypress/e2e/smoke/remove_owner.cy.js b/cypress/e2e/smoke/remove_owner.cy.js index 9a1d1ae21b..4643b91e1a 100644 --- a/cypress/e2e/smoke/remove_owner.cy.js +++ b/cypress/e2e/smoke/remove_owner.cy.js @@ -15,13 +15,13 @@ describe('Remove Owners tests', () => { owner.verifyRemoveBtnIsEnabled().should('have.length', 2) }) - it('Verify Tooltip displays correct message for Non-Owner ', () => { + it('Verify Tooltip displays correct message for Non-Owner', () => { cy.visit(constants.setupUrl + constants.SEPOLIA_TEST_SAFE_4) owner.waitForConnectionStatus() owner.verifyRemoveBtnIsDisabled() }) - it('Verify Tooltip displays correct message for disconnected user ', () => { + it('Verify Tooltip displays correct message for disconnected user', () => { cy.visit(constants.setupUrl + constants.SEPOLIA_TEST_SAFE_3) owner.waitForConnectionStatus() owner.clickOnWalletExpandMoreIcon() @@ -29,13 +29,13 @@ describe('Remove Owners tests', () => { owner.verifyRemoveBtnIsDisabled() }) - it('Verify owner removal form can be opened ', () => { + it('Verify owner removal form can be opened', () => { cy.visit(constants.setupUrl + constants.SEPOLIA_TEST_SAFE_3) owner.waitForConnectionStatus() owner.openRemoveOwnerWindow(1) }) - it('Verify threshold input displays the upper limit as the current safe number of owners minus one ', () => { + it('Verify threshold input displays the upper limit as the current safe number of owners minus one', () => { cy.visit(constants.setupUrl + constants.SEPOLIA_TEST_SAFE_3) owner.waitForConnectionStatus() owner.openRemoveOwnerWindow(1) @@ -43,7 +43,7 @@ describe('Remove Owners tests', () => { owner.getThresholdOptions().should('have.length', 1) }) - it('Verify owner deletion confirmation is displayed ', () => { + it('Verify owner deletion confirmation is displayed', () => { cy.visit(constants.setupUrl + constants.SEPOLIA_TEST_SAFE_3) owner.waitForConnectionStatus() owner.openRemoveOwnerWindow(1) diff --git a/cypress/e2e/smoke/replace_owner.cy.js b/cypress/e2e/smoke/replace_owner.cy.js index 90d200c311..ed48056206 100644 --- a/cypress/e2e/smoke/replace_owner.cy.js +++ b/cypress/e2e/smoke/replace_owner.cy.js @@ -11,36 +11,36 @@ describe('Replace Owners tests', () => { cy.contains(owner.safeAccountNonceStr, { timeout: 10000 }) }) - it('Verify that "Replace" icon is visible ', () => { + it('Verify that "Replace" icon is visible', () => { owner.verifyReplaceBtnIsEnabled() }) - it('Verify Tooltip displays correct message for Non-Owner ', () => { + it('Verify Tooltip displays correct message for Non-Owner', () => { cy.visit(constants.setupUrl + constants.SEPOLIA_TEST_SAFE_2) owner.waitForConnectionStatus() owner.verifyReplaceBtnIsDisabled() }) - it('Verify Tooltip displays correct message for disconnected user ', () => { + it('Verify Tooltip displays correct message for disconnected user', () => { owner.waitForConnectionStatus() owner.clickOnWalletExpandMoreIcon() owner.clickOnDisconnectBtn() owner.verifyReplaceBtnIsDisabled() }) - it('Verify that the owner replacement form is opened ', () => { + it('Verify that the owner replacement form is opened', () => { owner.waitForConnectionStatus() owner.openReplaceOwnerWindow() }) - it('Verify max characters in name field ', () => { + it('Verify max characters in name field', () => { owner.waitForConnectionStatus() owner.openReplaceOwnerWindow() owner.typeOwnerName(main.generateRandomString(51)) owner.verifyErrorMsgInvalidAddress(constants.addressBookErrrMsg.exceedChars) }) - it('Verify that Address input auto-fills with related value ', () => { + it('Verify that Address input auto-fills with related value', () => { cy.visit(constants.addressBookUrl + constants.SEPOLIA_TEST_SAFE_1) addressBook.clickOnCreateEntryBtn() addressBook.typeInName(constants.addresBookContacts.user1.name) @@ -55,7 +55,7 @@ describe('Replace Owners tests', () => { owner.verifyNewOwnerName(constants.addresBookContacts.user1.name) }) - it('Verify that Name field not mandatory. Verify confirmation for owner replacement is displayed ', () => { + it('Verify that Name field not mandatory. Verify confirmation for owner replacement is displayed', () => { owner.waitForConnectionStatus() owner.openReplaceOwnerWindow() owner.typeOwnerAddress(constants.SEPOLIA_OWNER_2) @@ -63,7 +63,7 @@ describe('Replace Owners tests', () => { owner.verifyConfirmTransactionWindowDisplayed() }) - it('Verify relevant error messages are displayed in Address input ', () => { + it('Verify relevant error messages are displayed in Address input', () => { owner.waitForConnectionStatus() owner.openReplaceOwnerWindow() owner.typeOwnerAddress(main.generateRandomString(10)) diff --git a/cypress/e2e/smoke/tx_history.cy.js b/cypress/e2e/smoke/tx_history.cy.js index 9b41d89c26..81b2dea87b 100644 --- a/cypress/e2e/smoke/tx_history.cy.js +++ b/cypress/e2e/smoke/tx_history.cy.js @@ -20,7 +20,7 @@ describe('Transaction history tests', () => { main.acceptCookies(1) }) - it('Verify October 29th transactions are displayed ', () => { + it('Verify October 29th transactions are displayed', () => { const DATE = 'Oct 29, 2023' const NEXT_DATE_LABEL = 'Oct 20, 2023' const amount = '0.00001 ETH' @@ -45,7 +45,7 @@ describe('Transaction history tests', () => { }) }) - it('Verify transaction can be expanded/collapsed ', () => { + it('Verify transaction can be expanded/collapsed', () => { createTx.clickOnTransactionExpandableItem('Oct 20, 2023', () => { createTx.verifyTransactionStrExists(str1) createTx.verifyTransactionStrExists(str2) From 6355e7b899f7aa3f88d83cbe1d8eb1841b6a2545 Mon Sep 17 00:00:00 2001 From: mike10ca Date: Fri, 10 Nov 2023 09:42:09 +0100 Subject: [PATCH 4/6] Update tests --- cypress/e2e/smoke/create_safe_simple.cy.js | 6 ++---- cypress/e2e/smoke/import_export_data.cy.js | 2 +- 2 files changed, 3 insertions(+), 5 deletions(-) diff --git a/cypress/e2e/smoke/create_safe_simple.cy.js b/cypress/e2e/smoke/create_safe_simple.cy.js index 036cf19b9a..956be6ae2e 100644 --- a/cypress/e2e/smoke/create_safe_simple.cy.js +++ b/cypress/e2e/smoke/create_safe_simple.cy.js @@ -12,12 +12,10 @@ describe('Safe creation tests', () => { }) it('Verify a Wallet can be connected', () => { - owner.waitForConnectionStatus() - cy.visit(constants.welcomeUrl) + createwallet.clickOnCreateNewSafeBtn() owner.clickOnWalletExpandMoreIcon() owner.clickOnDisconnectBtn() - createwallet.clickOnCreateNewSafeBtn() - owner.clickOnConnectBtn() + createwallet.clickOnConnectWalletBtn() createwallet.connectWallet() }) diff --git a/cypress/e2e/smoke/import_export_data.cy.js b/cypress/e2e/smoke/import_export_data.cy.js index 553707e14e..76101918d3 100644 --- a/cypress/e2e/smoke/import_export_data.cy.js +++ b/cypress/e2e/smoke/import_export_data.cy.js @@ -24,7 +24,7 @@ describe('Import Export Data tests', () => { }) it('Verify address book imported data', () => { - main.acceptCookies(1) + main.acceptCookies() file.clickOnAddressBookBtn() file.verifyImportedAddressBookData() }) From d0400d8e78016933c09b5b7fd2bdb1228be34ba1 Mon Sep 17 00:00:00 2001 From: mike10ca Date: Fri, 10 Nov 2023 10:09:11 +0100 Subject: [PATCH 5/6] Fix test --- cypress/e2e/safe-apps/drain_account.spec.cy.js | 2 +- cypress/e2e/smoke/create_safe_simple.cy.js | 2 -- 2 files changed, 1 insertion(+), 3 deletions(-) diff --git a/cypress/e2e/safe-apps/drain_account.spec.cy.js b/cypress/e2e/safe-apps/drain_account.spec.cy.js index 2b9942d8bf..773559d748 100644 --- a/cypress/e2e/safe-apps/drain_account.spec.cy.js +++ b/cypress/e2e/safe-apps/drain_account.spec.cy.js @@ -22,7 +22,7 @@ describe('Drain Account tests', { defaultCommandTimeout: 12000 }, () => { it('Verify drain can be created', () => { cy.enter(iframeSelector).then((getBody) => { getBody().findByLabelText(safeapps.recipientStr).type(constants.SAFE_APP_ADDRESS_2) - getBody().findAllByText('Test invalid string').click() + getBody().findAllByText(safeapps.transferEverythingStr).click() }) cy.findByRole('button', { name: safeapps.testTransfer1 }) cy.findByRole('button', { name: safeapps.testNativeTransfer2 }) diff --git a/cypress/e2e/smoke/create_safe_simple.cy.js b/cypress/e2e/smoke/create_safe_simple.cy.js index 4d198bb36f..1b924cd743 100644 --- a/cypress/e2e/smoke/create_safe_simple.cy.js +++ b/cypress/e2e/smoke/create_safe_simple.cy.js @@ -9,8 +9,6 @@ describe('Safe creation tests', () => { cy.clearLocalStorage() main.acceptCookies() }) - - it('Verify a Wallet can be connected', () => { createwallet.clickOnCreateNewSafeBtn() owner.clickOnWalletExpandMoreIcon() From d105db7b4cef62d8aad66552e35f2b865c495b15 Mon Sep 17 00:00:00 2001 From: mike10ca Date: Fri, 10 Nov 2023 10:49:54 +0100 Subject: [PATCH 6/6] Update test expectation --- cypress/e2e/smoke/nfts.cy.js | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/cypress/e2e/smoke/nfts.cy.js b/cypress/e2e/smoke/nfts.cy.js index 7f114ba46f..1ecc5c1c20 100644 --- a/cypress/e2e/smoke/nfts.cy.js +++ b/cypress/e2e/smoke/nfts.cy.js @@ -23,7 +23,7 @@ describe('NFTs tests', () => { }) it('Verify NFT preview window can be opened', () => { - nfts.openNFT(1) + nfts.openNFT(0) nfts.verifyNameInNFTModal(nftsTokenID) nfts.verifySelectedNetwrokSepolia() nfts.closeNFTModal()