Fix failing e2e

[michalinacienciala]

The e2e tests (regular and fork-based) were failing on the main branch. This PR incorporates a couple changes that fix:

• The issue with RECOVERY_PHRASE environment variable is not defined. error in e2e-tests job
• The issue with unzip: cannot find or open chrome-fork.zip, chrome-fork.zip.zip or chrome-fork.zip.ZIP. error in e2e-tests-fork job.

The PR does not address the following problems (we'll deal with them separately):

• The issue with Transaction signed, broadcasting... not being visible during User can send base asset test in e2e-tests-fork job. (Possibly an issue with cache or caused b the same issue as mentioned in the point below)
• The issue with Expected pattern: /^Ethereum$/ vs Received string: "" in User can open ERC-20 transfer from asset list and can reject the transfer test in e2e-tests-fork job. (Actual bug with rejecting txs - Extension tx screen re-opening after approving / rejecting tx #2876 - Reject button needs to be clicked twice)

Latest build: extension-builds-3567 (as of Mon, 24 Jul 2023 18:35:33 GMT).

[michalinacienciala]

It's likely that the two remaining problems will be fixed by #3569. I merged this change to the PR branch, let's wait for the tests results and see if everything is green.

[michalinacienciala]

It's likely that the two remaining problems will be fixed by #3569. I merged this change to the PR branch, let's wait for the tests results and see if everything is green.

The tests executed after merging main to the feature branch have failed on one test in Onboarding:

1) [chromium] › onboarding.spec.ts:7:7 › Onboarding › User can onboard a read-only address ───────

    locator.innerText: Target closed
    =========================== logs ===========================
    waiting for getByTestId('wallet_balance')
    ============================================================

      21 |     }).toPass()
      22 |
    > 23 |     expect(popup.getByTestId("wallet_balance").innerText()).not.toContain("$0")
         |                                                ^
      24 |   })
      25 |
      26 |   test("User can onboard with a existing seed-phrase", async ({

        at /home/runner/work/extension/extension/e2e-tests/regular/onboarding.spec.ts:[23](https://github.com/tahowallet/extension/actions/runs/5648091731/job/15299648051#step:9:24):48

I've never seen this test failing on this line before...

After re-running I got a green run. I would suggest merging this PR and observing if the mentioned issue appear again.

[Shadowfiend]

All righty let's do this thing!