Fix Firefox e2e test command on CI (#7000)

* Skip a few Firefox tests to get them passing again
* ci: Fix Firefox e2e test command

This is my bad, introduced in 7fc84f3cc087deab5d937ee589de56fa40cd7ced

1 files changed, 19 insertions, 1 deletions

diff --git a/test/e2e/metamask-ui.spec.js b/test/e2e/metamask-ui.spec.js
index 8d0942dc6..90f320904 100644
--- a/test/e2e/metamask-ui.spec.js
+++ b/test/e2e/metamask-ui.spec.js
@@ -1247,7 +1247,9 @@ describe('MetaMask', function () {
       const transferTokens = await findElement(driver, By.xpath(`//button[contains(text(), 'Approve Tokens')]`))
       await transferTokens.click()
 
-      await closeAllWindowHandlesExcept(driver, [extension, dapp])
+      if (process.env.SELENIUM_BROWSER !== 'firefox') {
+        await closeAllWindowHandlesExcept(driver, [extension, dapp])
+      }
       await driver.switchTo().window(extension)
       await delay(regularDelayMs)
 
@@ -1341,6 +1343,10 @@ describe('MetaMask', function () {
     })
 
     it('finds the transaction in the transactions list', async function () {
+      if (process.env.SELENIUM_BROWSER === 'firefox') {
+        this.skip()
+      }
+
       await driver.wait(async () => {
         const confirmedTxes = await findElements(driver, By.css('.transaction-list__completed-transactions .transaction-list-item'))
         return confirmedTxes.length === 3
@@ -1354,6 +1360,12 @@ describe('MetaMask', function () {
   })
 
   describe('Tranfers a custom token from dapp when no gas value is specified', () => {
+    before(function () {
+      if (process.env.SELENIUM_BROWSER === 'firefox') {
+        this.skip()
+      }
+    })
+
     it('transfers an already created token, without specifying gas', async () => {
       const windowHandles = await driver.getAllWindowHandles()
       const extension = windowHandles[0]
@@ -1403,6 +1415,12 @@ describe('MetaMask', function () {
   })
 
   describe('Approves a custom token from dapp when no gas value is specified', () => {
+    before(function () {
+      if (process.env.SELENIUM_BROWSER === 'firefox') {
+        this.skip()
+      }
+    })
+
     it('approves an already created token', async () => {
       const windowHandles = await driver.getAllWindowHandles()
       const extension = windowHandles[0]