Test: E2e fix create tx test (#3205)

safe-global · Feb 6, 2024 · b2e774b · b2e774b
1 parent f220e81
commit b2e774b
Show file tree

Hide file tree

Showing 4 changed files with 22 additions and 32 deletions.
diff --git a/cypress/e2e/regression/remove_owner.cy.js b/cypress/e2e/regression/remove_owner.cy.js
@@ -1,6 +1,8 @@
 import * as constants from '../../support/constants'
 import * as main from '../../e2e/pages/main.page'
 import * as owner from '../pages/owners.pages'
+import * as createwallet from '../pages/create_wallet.pages'
+import * as createTx from '../pages/create_tx.pages.js'
 
 describe('Remove Owners tests', () => {
   beforeEach(() => {
@@ -38,4 +40,20 @@ describe('Remove Owners tests', () => {
     owner.verifyThresholdLimit(1, 1)
     owner.getThresholdOptions().should('have.length', 1)
   })
+
+  it('Verify owner deletion transaction has been created', () => {
+    owner.waitForConnectionStatus()
+    owner.openRemoveOwnerWindow(1)
+    cy.wait(3000)
+    createwallet.clickOnNextBtn()
+    //This method creates the @removedAddress alias
+    owner.getAddressToBeRemoved()
+    owner.verifyOwnerDeletionWindowDisplayed()
+    createTx.changeNonce(10)
+    createTx.clickOnSignTransactionBtn()
+    createTx.waitForProposeRequest()
+    createTx.clickViewTransaction()
+    createTx.clickOnTransactionItemByName('removeOwner')
+    createTx.verifyTxDestinationAddress('@removedAddress')
+  })
 })
diff --git a/cypress/e2e/smoke/create_tx.cy.js b/cypress/e2e/smoke/create_tx.cy.js
@@ -3,7 +3,7 @@ import * as main from '../../e2e/pages/main.page'
 import * as createtx from '../../e2e/pages/create_tx.pages'
 
 const sendValue = 0.00002
-const currentNonce = 1
+const currentNonce = 2
 
 function happyPathToStepTwo() {
   createtx.typeRecipientAddress(constants.EOA)
@@ -15,7 +15,7 @@ function happyPathToStepTwo() {
 describe('[SMOKE] Create transactions tests', () => {
   beforeEach(() => {
     cy.clearLocalStorage()
-    cy.visit(constants.BALANCE_URL + constants.SEPOLIA_TEST_SAFE_7)
+    cy.visit(constants.BALANCE_URL + constants.SEPOLIA_TEST_SAFE_16_CREATE_TX)
     main.acceptCookies()
     createtx.clickOnNewtransactionBtn()
     createtx.clickOnSendTokensBtn()
@@ -41,7 +41,7 @@ describe('[SMOKE] Create transactions tests', () => {
     createtx.verifyMaxAmount(constants.tokenNames.sepoliaEther, constants.tokenAbbreviation.sep)
   })
 
-  it.skip('[SMOKE] Verify nonce tooltip warning messages', () => {
+  it('[SMOKE] Verify nonce tooltip warning messages', () => {
     createtx.changeNonce(0)
     createtx.verifyTooltipMessage(constants.nonceTooltipMsg.lowerThanCurrent + currentNonce.toString())
     createtx.changeNonce(currentNonce + 53)

diff --git a/cypress/e2e/smoke/remove_owner.cy.js b/cypress/e2e/smoke/remove_owner.cy.js
diff --git a/cypress/support/constants.js b/cypress/support/constants.js
@@ -28,6 +28,7 @@ export const SEPOLIA_TEST_SAFE_13_SHORT = '0x9870...fec0'
 export const SEPOLIA_TEST_SAFE_14 = 'sep:0xC23e061252BFc7967203D054136d8fA7c7df2fc4'
 // SAFE 15 is a safe with native tokens but the automation user is not its owner
 export const SEPOLIA_TEST_SAFE_15_TOKEN = 'sep:0xfC0A7ac73Fde7547ac0792Cca1D8A50CE0AFC4Df'
+export const SEPOLIA_TEST_SAFE_16_CREATE_TX = 'sep:0xc2F3645bfd395516d1a18CA6ad9298299d328C01'
 export const SEPOLIA_CONTRACT_SHORT = '0x11AB...34aF'
 export const SEPOLIA_RECIPIENT_ADDR_SHORT = '0x4DD4...7bde'
 export const GNO_TEST_SAFE = 'gno:0xB8d760a90a5ed54D3c2b3EFC231277e99188642A'