From c3f6e1754625eb9980b7e575400b4291581ef102 Mon Sep 17 00:00:00 2001 From: Brad Decker Date: Tue, 31 Oct 2023 09:04:38 -0500 Subject: [PATCH] dev(chore): log test names and suites (#21570) MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit ## **Description** Before #21441 the test reporter would output the test path it was executing which made figuring out where tests were failing at fairly easy. The updates in #21441 make tests run faster but it doesn't help when trying to ascertain where failures occur. To make it slightly easier this PR adds some reporting to the run-all script and the withFixtures and verboseReportOnFailure functions to explicitly call out where breakdowns happen. This PR also changes from using `this.test.title` to `this.test.fullTitle()` which includes the name spacing of Describe Blocks for easier access. ## **Related issues** ## **Manual testing steps** 1. View CircleCI output on develop for any of the E2E tests. 2. Observe that the test steps are written out verbosely, but no information about what file or test is being ran. 3. Do the same on this branch's CircleCI output and see the the test files and names are logged. 4. Repeat steps locally on develop and this branch to see local terminal console outputs file paths and test names. ## **Screenshots/Recordings** ### **Before** Screenshot 2023-10-26 at 1 48 27 PM ### **After** Screenshot 2023-10-26 at 2 15 21 PM Screenshot 2023-10-26 at 2 15 41 PM ## **Pre-merge author checklist** - [x] I’ve followed [MetaMask Coding Standards](https://github.com/MetaMask/metamask-extension/blob/develop/.github/guidelines/CODING_GUIDELINES.md). - [x] I've clearly explained what problem this PR is solving and how it is solved. - [ ] I've linked related issues - [x] I've included manual testing steps - [x] I've included screenshots/recordings if applicable - [x] I’ve included tests if applicable - [x] I’ve documented my code using [JSDoc](https://jsdoc.app/) format if applicable - [x] I’ve applied the right labels on the PR (see [labeling guidelines](https://github.com/MetaMask/metamask-extension/blob/develop/.github/guidelines/LABELING_GUIDELINES.md)). Not required for external contributors. - [x] I’ve properly set the pull request status: - [ ] In case it's not yet "ready for review", I've set it to "draft". - [x] In case it's "ready for review", I've changed it from "draft" to "non-draft". ## **Pre-merge reviewer checklist** - [ ] I've manually tested the PR (e.g. pull and build branch, run the app, test code being changed). - [ ] I confirm that this PR addresses all acceptance criteria described in the ticket it closes and includes the necessary testing evidence such as recordings and or screenshots. --------- Co-authored-by: Howard Braham --- .../accounts/test-create-snap-account.spec.js | 6 ++-- .../test-remove-accounts-snap.spec.js | 2 +- test/e2e/accounts/test-snap-accounts.spec.js | 14 ++++----- test/e2e/flask/petnames.spec.js | 6 ++-- test/e2e/helpers.js | 8 ++++- test/e2e/json-rpc/eth_accounts.spec.js | 2 +- test/e2e/json-rpc/eth_call.spec.js | 2 +- test/e2e/json-rpc/eth_chainId.spec.js | 2 +- test/e2e/json-rpc/eth_estimateGas.spec.js | 2 +- test/e2e/json-rpc/eth_gasPrice.spec.js | 2 +- test/e2e/json-rpc/eth_newBlockFilter.spec.js | 2 +- test/e2e/json-rpc/eth_sendTransaction.spec.js | 4 +-- test/e2e/json-rpc/switchEthereumChain.spec.js | 8 ++--- test/e2e/provider/eip-6963.spec.js | 2 +- test/e2e/run-all.js | 1 + test/e2e/snaps/test-snap-bip-32.spec.js | 2 +- test/e2e/snaps/test-snap-bip-44.spec.js | 2 +- test/e2e/snaps/test-snap-cronjob.spec.js | 2 +- test/e2e/snaps/test-snap-dialog.spec.js | 2 +- test/e2e/snaps/test-snap-ethprovider.spec.js | 2 +- test/e2e/snaps/test-snap-get-locale.spec.js | 2 +- test/e2e/snaps/test-snap-getentropy.spec.js | 2 +- test/e2e/snaps/test-snap-installed.spec.js | 2 +- test/e2e/snaps/test-snap-lifecycle.spec.js | 2 +- test/e2e/snaps/test-snap-management.spec.js | 2 +- test/e2e/snaps/test-snap-managestate.spec.js | 2 +- .../e2e/snaps/test-snap-networkaccess.spec.js | 2 +- test/e2e/snaps/test-snap-notification.spec.js | 2 +- test/e2e/snaps/test-snap-revoke-perm.spec.js | 2 +- test/e2e/snaps/test-snap-rpc.spec.js | 2 +- test/e2e/snaps/test-snap-txinsights.spec.js | 2 +- test/e2e/snaps/test-snap-update.spec.js | 2 +- test/e2e/snaps/test-snap-wasm.spec.js | 2 +- test/e2e/tests/4byte-directory.spec.js | 4 +-- test/e2e/tests/account-details.spec.js | 12 ++++---- test/e2e/tests/account-token-list.spec.js | 4 +-- test/e2e/tests/add-account.spec.js | 6 ++-- test/e2e/tests/add-custom-network.spec.js | 18 +++++------ test/e2e/tests/add-hide-token.spec.js | 8 ++--- test/e2e/tests/add-multiple-tokens.spec.js | 2 +- test/e2e/tests/address-book.spec.js | 6 ++-- test/e2e/tests/advanced-settings.spec.js | 4 +-- test/e2e/tests/auto-lock.spec.js | 2 +- test/e2e/tests/backup-restore.spec.js | 4 +-- test/e2e/tests/block-explorer.spec.js | 6 ++-- test/e2e/tests/chain-interactions.spec.js | 4 +-- test/e2e/tests/clear-activity.spec.js | 2 +- test/e2e/tests/contract-interactions.spec.js | 2 +- test/e2e/tests/custom-rpc-history.spec.js | 12 ++++---- .../tests/custom-token-add-approve.spec.js | 10 +++---- test/e2e/tests/dapp-interactions.spec.js | 4 +-- test/e2e/tests/dapp-tx-edit.spec.js | 4 +-- test/e2e/tests/edit-gas-fee.spec.js | 6 ++-- test/e2e/tests/encrypt-decrypt.spec.js | 8 ++--- test/e2e/tests/ens.spec.js | 2 +- test/e2e/tests/errors.spec.js | 30 +++++++++---------- test/e2e/tests/eth-sign.spec.js | 6 ++-- test/e2e/tests/eth-subscribe.spec.js | 2 +- test/e2e/tests/failing-contract.spec.js | 4 +-- test/e2e/tests/gas-estimates.spec.js | 12 ++++---- test/e2e/tests/import-flow.spec.js | 12 ++++---- test/e2e/tests/import-tokens.spec.js | 2 +- test/e2e/tests/incremental-security.spec.js | 2 +- test/e2e/tests/ipfs-ens-resolution.spec.js | 2 +- test/e2e/tests/ipfs-toggle.spec.js | 2 +- test/e2e/tests/localization.spec.js | 2 +- test/e2e/tests/lock-account.spec.js | 2 +- test/e2e/tests/lockdown.spec.js | 2 +- test/e2e/tests/metamask-responsive-ui.spec.js | 6 ++-- .../metrics/permissions-approved.spec.js | 2 +- .../tests/metrics/signature-approved.spec.js | 10 +++---- test/e2e/tests/metrics/swaps.spec.js | 2 +- .../metrics/transaction-finalized.spec.js | 2 +- test/e2e/tests/metrics/unlock-wallet.spec.js | 2 +- test/e2e/tests/metrics/wallet-created.spec.js | 4 +-- test/e2e/tests/multiple-transactions.spec.js | 4 +-- test/e2e/tests/navigate-transactions.spec.js | 10 +++---- test/e2e/tests/network-error.spec.js | 2 +- test/e2e/tests/nft/auto-detect-nft.spec.js | 2 +- .../e2e/tests/nft/erc1155-interaction.spec.js | 8 ++--- test/e2e/tests/nft/erc721-interaction.spec.js | 12 ++++---- test/e2e/tests/nft/import-erc1155.spec.js | 4 +-- test/e2e/tests/nft/import-nft.spec.js | 4 +-- test/e2e/tests/nft/remove-erc1155.spec.js | 2 +- test/e2e/tests/nft/remove-nft.spec.js | 2 +- test/e2e/tests/nft/send-nft.spec.js | 2 +- .../tests/nft/view-erc1155-details.spec.js | 2 +- test/e2e/tests/nft/view-nft-details.spec.js | 2 +- test/e2e/tests/onboarding.spec.js | 16 +++++----- test/e2e/tests/permissions.spec.js | 2 +- test/e2e/tests/personal-sign.spec.js | 4 +-- .../phishing-detection.spec.js | 14 ++++----- test/e2e/tests/portfolio-site.spec.js | 2 +- ...ppom-blockaid-alert-erc20-approval.spec.js | 2 +- ...ppom-blockaid-alert-erc20-transfer.spec.js | 2 +- test/e2e/tests/ppom-blockaid-alert.spec.js | 6 ++-- test/e2e/tests/ppom-toggle-settings.spec.js | 4 +-- test/e2e/tests/provider-api.spec.js | 4 +-- test/e2e/tests/security-provider.spec.js | 8 ++--- test/e2e/tests/send-edit.spec.js | 4 +-- test/e2e/tests/send-eth.spec.js | 12 ++++---- test/e2e/tests/send-hex-address.spec.js | 8 ++--- test/e2e/tests/send-to-contract.spec.js | 2 +- test/e2e/tests/settings-general.spec.js | 2 +- test/e2e/tests/settings-search.spec.js | 18 +++++------ .../settings-security-reveal-srp.spec.js | 6 ++-- test/e2e/tests/signature-request.spec.js | 4 +-- test/e2e/tests/signin-with-ethereum.spec.js | 2 +- test/e2e/tests/simple-send.spec.js | 2 +- test/e2e/tests/state-logs.spec.js | 2 +- .../tests/stuck-approved-transaction.spec.js | 2 +- test/e2e/tests/swaps/swap-eth.spec.js | 4 +-- .../tests/swaps/swaps-notifications.spec.js | 8 ++--- test/e2e/tests/switch-custom-network.spec.js | 2 +- test/e2e/tests/terms-of-use.spec.js | 2 +- test/e2e/tests/token-details.spec.js | 2 +- test/e2e/webdriver/driver.js | 12 +++++++- 117 files changed, 284 insertions(+), 267 deletions(-) diff --git a/test/e2e/accounts/test-create-snap-account.spec.js b/test/e2e/accounts/test-create-snap-account.spec.js index eb4ffdf93a3e..649f681350d6 100644 --- a/test/e2e/accounts/test-create-snap-account.spec.js +++ b/test/e2e/accounts/test-create-snap-account.spec.js @@ -15,7 +15,7 @@ describe('Create Snap Account', function () { fixtures: new FixtureBuilder().build(), ganacheOptions: defaultGanacheOptions, failOnConsoleError: false, - title: this.test.title, + title: this.test.fullTitle(), }, async ({ driver }) => { await driver.navigate(); @@ -101,7 +101,7 @@ describe('Create Snap Account', function () { fixtures: new FixtureBuilder().build(), ganacheOptions: defaultGanacheOptions, failOnConsoleError: false, - title: this.test.title, + title: this.test.fullTitle(), }, async ({ driver }) => { await driver.navigate(); @@ -196,7 +196,7 @@ describe('Create Snap Account', function () { fixtures: new FixtureBuilder().build(), ganacheOptions: defaultGanacheOptions, failOnConsoleError: false, - title: this.test.title, + title: this.test.fullTitle(), }, async ({ driver }) => { await driver.navigate(); diff --git a/test/e2e/accounts/test-remove-accounts-snap.spec.js b/test/e2e/accounts/test-remove-accounts-snap.spec.js index a014baafe460..47750c4d094c 100644 --- a/test/e2e/accounts/test-remove-accounts-snap.spec.js +++ b/test/e2e/accounts/test-remove-accounts-snap.spec.js @@ -16,7 +16,7 @@ describe('Remove Account Snap', function () { fixtures: new FixtureBuilder().build(), ganacheOptions: defaultGanacheOptions, failOnConsoleError: false, - title: this.test.title, + title: this.test.fullTitle(), }, async ({ driver }) => { await driver.navigate(); diff --git a/test/e2e/accounts/test-snap-accounts.spec.js b/test/e2e/accounts/test-snap-accounts.spec.js index 7fb9cc347399..32bd2604af81 100644 --- a/test/e2e/accounts/test-snap-accounts.spec.js +++ b/test/e2e/accounts/test-snap-accounts.spec.js @@ -51,7 +51,7 @@ describe('Test Snap Account', function () { it('can create a new Snap account', async function () { await withFixtures( - accountSnapFixtures(this.test.title), + accountSnapFixtures(this.test.fullTitle()), async ({ driver }) => { await installSnapSimpleKeyring(driver, false); @@ -66,7 +66,7 @@ describe('Test Snap Account', function () { fixtures: new FixtureBuilder().build(), ganacheOptions: defaultGanacheOptions, failOnConsoleError: false, - title: this.test.title, + title: this.test.fullTitle(), }, async ({ driver }) => { await installSnapSimpleKeyring(driver, false); @@ -135,7 +135,7 @@ describe('Test Snap Account', function () { it('can import a private key and transfer 1 ETH (sync flow)', async function () { await withFixtures( - accountSnapFixtures(this.test.title), + accountSnapFixtures(this.test.fullTitle()), async ({ driver }) => { await importPrivateKeyAndTransfer1ETH(driver, 'sync'); }, @@ -144,7 +144,7 @@ describe('Test Snap Account', function () { it('can import a private key and transfer 1 ETH (async flow approve)', async function () { await withFixtures( - accountSnapFixtures(this.test.title), + accountSnapFixtures(this.test.fullTitle()), async ({ driver }) => { await importPrivateKeyAndTransfer1ETH(driver, 'approve'); }, @@ -153,7 +153,7 @@ describe('Test Snap Account', function () { it('can import a private key and transfer 1 ETH (async flow reject)', async function () { await withFixtures( - accountSnapFixtures(this.test.title), + accountSnapFixtures(this.test.fullTitle()), async ({ driver }) => { await importPrivateKeyAndTransfer1ETH(driver, 'reject'); }, @@ -188,7 +188,7 @@ describe('Test Snap Account', function () { it(title, async function () { await withFixtures( - accountSnapFixtures(this.test.title), + accountSnapFixtures(this.test.fullTitle()), async ({ driver }) => { const isAsyncFlow = flowType !== 'sync'; @@ -211,7 +211,7 @@ describe('Test Snap Account', function () { fixtures: new FixtureBuilder().build(), ganacheOptions, failOnConsoleError: false, - title: this.test.title, + title: this.test.fullTitle(), }, async ({ driver }) => { const flowType = 'approve'; diff --git a/test/e2e/flask/petnames.spec.js b/test/e2e/flask/petnames.spec.js index 5c8dd62c0d65..2f754913bac0 100644 --- a/test/e2e/flask/petnames.spec.js +++ b/test/e2e/flask/petnames.spec.js @@ -157,7 +157,7 @@ describe('Petnames', function () { .withNoNames() .build(), ganacheOptions, - title: this.test.title, + title: this.test.fullTitle(), }, async ({ driver }) => { await login(driver); @@ -193,7 +193,7 @@ describe('Petnames', function () { .withNoNames() .build(), ganacheOptions, - title: this.test.title, + title: this.test.fullTitle(), }, async ({ driver }) => { await login(driver); @@ -232,7 +232,7 @@ describe('Petnames', function () { .withNoNames() .build(), ganacheOptions, - title: this.test.title, + title: this.test.fullTitle(), }, async ({ driver }) => { await login(driver); diff --git a/test/e2e/helpers.js b/test/e2e/helpers.js index 3ca6188e0ed4..01c3fca5ae07 100644 --- a/test/e2e/helpers.js +++ b/test/e2e/helpers.js @@ -149,6 +149,8 @@ async function withFixtures(options, testSuite) { }); } + console.log(`\nExecuting test suite: ${title}\n`); + await testSuite({ driver: driverProxy ?? driver, contractRegistry, @@ -157,6 +159,10 @@ async function withFixtures(options, testSuite) { mockedEndpoint, }); + // At this point the suite has executed successfully, so we can log out a + // success message. + console.log(`\nSuccess on test suite: '${title}'\n`); + // Evaluate whether any new hosts received network requests during E2E test // suite execution. If so, fail the test unless the // --update-privacy-snapshot was specified. In that case, update the @@ -199,7 +205,7 @@ async function withFixtures(options, testSuite) { failed = true; if (webDriver) { try { - await driver.verboseReportOnFailure(title); + await driver.verboseReportOnFailure(title, error); } catch (verboseReportError) { console.error(verboseReportError); } diff --git a/test/e2e/json-rpc/eth_accounts.spec.js b/test/e2e/json-rpc/eth_accounts.spec.js index ecc15a5d0d24..641e0286134b 100644 --- a/test/e2e/json-rpc/eth_accounts.spec.js +++ b/test/e2e/json-rpc/eth_accounts.spec.js @@ -13,7 +13,7 @@ describe('eth_accounts', function () { .withPermissionControllerConnectedToTestDapp() .build(), ganacheOptions: defaultGanacheOptions, - title: this.test.title, + title: this.test.fullTitle(), }, async ({ driver }) => { await driver.navigate(); diff --git a/test/e2e/json-rpc/eth_call.spec.js b/test/e2e/json-rpc/eth_call.spec.js index 592a298ff891..142726268eb6 100644 --- a/test/e2e/json-rpc/eth_call.spec.js +++ b/test/e2e/json-rpc/eth_call.spec.js @@ -24,7 +24,7 @@ describe('eth_call', function () { .build(), ganacheOptions, smartContract, - title: this.test.title, + title: this.test.fullTitle(), }, async ({ driver, _, contractRegistry }) => { const contract = contractRegistry.getContractAddress(smartContract); diff --git a/test/e2e/json-rpc/eth_chainId.spec.js b/test/e2e/json-rpc/eth_chainId.spec.js index d86ccea60eaa..2921069ffaf8 100644 --- a/test/e2e/json-rpc/eth_chainId.spec.js +++ b/test/e2e/json-rpc/eth_chainId.spec.js @@ -20,7 +20,7 @@ describe('eth_chainId', function () { .withPermissionControllerConnectedToTestDapp() .build(), ganacheOptions, - title: this.test.title, + title: this.test.fullTitle(), }, async ({ driver }) => { await driver.navigate(); diff --git a/test/e2e/json-rpc/eth_estimateGas.spec.js b/test/e2e/json-rpc/eth_estimateGas.spec.js index 6a875094e3de..959056253a82 100644 --- a/test/e2e/json-rpc/eth_estimateGas.spec.js +++ b/test/e2e/json-rpc/eth_estimateGas.spec.js @@ -11,7 +11,7 @@ describe('eth_estimateGas', function () { .withPermissionControllerConnectedToTestDapp() .build(), ganacheOptions: defaultGanacheOptions, - title: this.test.title, + title: this.test.fullTitle(), }, async ({ driver }) => { await driver.navigate(); diff --git a/test/e2e/json-rpc/eth_gasPrice.spec.js b/test/e2e/json-rpc/eth_gasPrice.spec.js index 28053cef442d..6c8d879ab91b 100644 --- a/test/e2e/json-rpc/eth_gasPrice.spec.js +++ b/test/e2e/json-rpc/eth_gasPrice.spec.js @@ -11,7 +11,7 @@ describe('eth_gasPrice', function () { .withPermissionControllerConnectedToTestDapp() .build(), ganacheOptions: defaultGanacheOptions, - title: this.test.title, + title: this.test.fullTitle(), }, async ({ driver }) => { await driver.navigate(); diff --git a/test/e2e/json-rpc/eth_newBlockFilter.spec.js b/test/e2e/json-rpc/eth_newBlockFilter.spec.js index 6784eae8d9a1..b476811c7036 100644 --- a/test/e2e/json-rpc/eth_newBlockFilter.spec.js +++ b/test/e2e/json-rpc/eth_newBlockFilter.spec.js @@ -15,7 +15,7 @@ describe('eth_newBlockFilter', function () { .withPermissionControllerConnectedToTestDapp() .build(), ganacheOptions, - title: this.test.title, + title: this.test.fullTitle(), }, async ({ driver }) => { await driver.navigate(); diff --git a/test/e2e/json-rpc/eth_sendTransaction.spec.js b/test/e2e/json-rpc/eth_sendTransaction.spec.js index 90da37a4b1e1..df67c1d74b40 100644 --- a/test/e2e/json-rpc/eth_sendTransaction.spec.js +++ b/test/e2e/json-rpc/eth_sendTransaction.spec.js @@ -23,7 +23,7 @@ describe('eth_sendTransaction', function () { .withPermissionControllerConnectedToTestDapp() .build(), ganacheOptions, - title: this.test.title, + title: this.test.fullTitle(), }, async ({ driver }) => { await driver.navigate(); @@ -70,7 +70,7 @@ describe('eth_sendTransaction', function () { .withPermissionControllerConnectedToTestDapp() .build(), ganacheOptions, - title: this.test.title, + title: this.test.fullTitle(), }, async ({ driver }) => { await driver.navigate(); diff --git a/test/e2e/json-rpc/switchEthereumChain.spec.js b/test/e2e/json-rpc/switchEthereumChain.spec.js index e88424c01876..64cbb7c570d2 100644 --- a/test/e2e/json-rpc/switchEthereumChain.spec.js +++ b/test/e2e/json-rpc/switchEthereumChain.spec.js @@ -23,7 +23,7 @@ describe('Switch Ethereum Chain for two dapps', function () { ...defaultGanacheOptions, concurrent: { port: 8546, chainId: 1338 }, }, - title: this.test.title, + title: this.test.fullTitle(), }, async ({ driver }) => { await driver.navigate(); @@ -97,7 +97,7 @@ describe('Switch Ethereum Chain for two dapps', function () { ...defaultGanacheOptions, concurrent: { port: 8546, chainId: 1338 }, }, - title: this.test.title, + title: this.test.fullTitle(), }, async ({ driver }) => { await driver.navigate(); @@ -170,7 +170,7 @@ describe('Switch Ethereum Chain for two dapps', function () { ...defaultGanacheOptions, concurrent: { port: 8546, chainId: 1338 }, }, - title: this.test.title, + title: this.test.fullTitle(), }, async ({ driver }) => { await driver.navigate(); @@ -244,7 +244,7 @@ describe('Switch Ethereum Chain for two dapps', function () { ...defaultGanacheOptions, concurrent: { port: 8546, chainId: 1338 }, }, - title: this.test.title, + title: this.test.fullTitle(), }, async ({ driver }) => { await driver.navigate(); diff --git a/test/e2e/provider/eip-6963.spec.js b/test/e2e/provider/eip-6963.spec.js index 751a4e674633..23031af1c67a 100644 --- a/test/e2e/provider/eip-6963.spec.js +++ b/test/e2e/provider/eip-6963.spec.js @@ -27,7 +27,7 @@ describe('EIP-6963 Provider', function () { .withPermissionControllerConnectedToTestDapp() .build(), ganacheOptions, - title: this.test.title, + title: this.test.fullTitle(), }, async ({ driver }) => { await driver.navigate(); diff --git a/test/e2e/run-all.js b/test/e2e/run-all.js index e9e94ee83d54..117deb611139 100644 --- a/test/e2e/run-all.js +++ b/test/e2e/run-all.js @@ -204,6 +204,7 @@ async function main() { for (let testPath of myTestList) { if (testPath !== '') { testPath = testPath.replace('\n', ''); // sometimes there's a newline at the end of the testPath + console.log(`\nExecuting testPath: ${testPath}\n`); await runInShell('node', [...args, testPath]); } } diff --git a/test/e2e/snaps/test-snap-bip-32.spec.js b/test/e2e/snaps/test-snap-bip-32.spec.js index 5b69c576619c..fd4100bf6d99 100644 --- a/test/e2e/snaps/test-snap-bip-32.spec.js +++ b/test/e2e/snaps/test-snap-bip-32.spec.js @@ -17,7 +17,7 @@ describe('Test Snap bip-32', function () { fixtures: new FixtureBuilder().build(), ganacheOptions, failOnConsoleError: false, - title: this.test.title, + title: this.test.fullTitle(), }, async ({ driver }) => { await driver.navigate(); diff --git a/test/e2e/snaps/test-snap-bip-44.spec.js b/test/e2e/snaps/test-snap-bip-44.spec.js index 48b2bbfc26a3..1eb563ac564c 100644 --- a/test/e2e/snaps/test-snap-bip-44.spec.js +++ b/test/e2e/snaps/test-snap-bip-44.spec.js @@ -18,7 +18,7 @@ describe('Test Snap bip-44', function () { fixtures: new FixtureBuilder().build(), ganacheOptions, failOnConsoleError: false, - title: this.test.title, + title: this.test.fullTitle(), }, async ({ driver }) => { await driver.navigate(); diff --git a/test/e2e/snaps/test-snap-cronjob.spec.js b/test/e2e/snaps/test-snap-cronjob.spec.js index 8495e953d409..91466b94c32b 100644 --- a/test/e2e/snaps/test-snap-cronjob.spec.js +++ b/test/e2e/snaps/test-snap-cronjob.spec.js @@ -19,7 +19,7 @@ describe('Test Snap Cronjob', function () { fixtures: new FixtureBuilder().build(), ganacheOptions, failOnConsoleError: false, - title: this.test.title, + title: this.test.fullTitle(), }, async ({ driver }) => { await driver.navigate(); diff --git a/test/e2e/snaps/test-snap-dialog.spec.js b/test/e2e/snaps/test-snap-dialog.spec.js index e675a9910019..507f3c115394 100644 --- a/test/e2e/snaps/test-snap-dialog.spec.js +++ b/test/e2e/snaps/test-snap-dialog.spec.js @@ -19,7 +19,7 @@ describe('Test Snap Dialog', function () { fixtures: new FixtureBuilder().build(), ganacheOptions, failOnConsoleError: false, - title: this.test.title, + title: this.test.fullTitle(), }, async ({ driver }) => { await driver.navigate(); diff --git a/test/e2e/snaps/test-snap-ethprovider.spec.js b/test/e2e/snaps/test-snap-ethprovider.spec.js index 0baa9e805460..003a508d08a6 100644 --- a/test/e2e/snaps/test-snap-ethprovider.spec.js +++ b/test/e2e/snaps/test-snap-ethprovider.spec.js @@ -18,7 +18,7 @@ describe('Test Snap ethereum_provider', function () { fixtures: new FixtureBuilder().build(), ganacheOptions, failOnConsoleError: false, - title: this.test.title, + title: this.test.fullTitle(), }, async ({ driver }) => { await driver.navigate(); diff --git a/test/e2e/snaps/test-snap-get-locale.spec.js b/test/e2e/snaps/test-snap-get-locale.spec.js index bf270935762d..1cc6f9006a17 100644 --- a/test/e2e/snaps/test-snap-get-locale.spec.js +++ b/test/e2e/snaps/test-snap-get-locale.spec.js @@ -19,7 +19,7 @@ describe('Test Snap Get Locale', function () { fixtures: new FixtureBuilder().build(), ganacheOptions, failOnConsoleError: false, - title: this.test.title, + title: this.test.fullTitle(), }, async ({ driver }) => { await driver.navigate(); diff --git a/test/e2e/snaps/test-snap-getentropy.spec.js b/test/e2e/snaps/test-snap-getentropy.spec.js index de6ecc4dcf80..ceece8776280 100644 --- a/test/e2e/snaps/test-snap-getentropy.spec.js +++ b/test/e2e/snaps/test-snap-getentropy.spec.js @@ -18,7 +18,7 @@ describe('Test Snap getEntropy', function () { fixtures: new FixtureBuilder().build(), ganacheOptions, failOnConsoleError: false, - title: this.test.title, + title: this.test.fullTitle(), }, async ({ driver }) => { await driver.navigate(); diff --git a/test/e2e/snaps/test-snap-installed.spec.js b/test/e2e/snaps/test-snap-installed.spec.js index f3ee0ba6e937..4988b4f0e64a 100644 --- a/test/e2e/snaps/test-snap-installed.spec.js +++ b/test/e2e/snaps/test-snap-installed.spec.js @@ -18,7 +18,7 @@ describe('Test Snap Installed', function () { fixtures: new FixtureBuilder().build(), ganacheOptions, failOnConsoleError: false, - title: this.test.title, + title: this.test.fullTitle(), }, async ({ driver }) => { await driver.navigate(); diff --git a/test/e2e/snaps/test-snap-lifecycle.spec.js b/test/e2e/snaps/test-snap-lifecycle.spec.js index 4da6c1e941d8..024e9df29f8f 100644 --- a/test/e2e/snaps/test-snap-lifecycle.spec.js +++ b/test/e2e/snaps/test-snap-lifecycle.spec.js @@ -19,7 +19,7 @@ describe('Test Snap Lifecycle Hooks', function () { fixtures: new FixtureBuilder().build(), ganacheOptions, failOnConsoleError: false, - title: this.test.title, + title: this.test.fullTitle(), }, async ({ driver }) => { await driver.navigate(); diff --git a/test/e2e/snaps/test-snap-management.spec.js b/test/e2e/snaps/test-snap-management.spec.js index f286d1c5af02..0e428f3869de 100644 --- a/test/e2e/snaps/test-snap-management.spec.js +++ b/test/e2e/snaps/test-snap-management.spec.js @@ -19,7 +19,7 @@ describe('Test Snap Management', function () { fixtures: new FixtureBuilder().build(), ganacheOptions, failOnConsoleError: false, - title: this.test.title, + title: this.test.fullTitle(), }, async ({ driver }) => { await driver.navigate(); diff --git a/test/e2e/snaps/test-snap-managestate.spec.js b/test/e2e/snaps/test-snap-managestate.spec.js index dbd4f0891153..90f20b714320 100644 --- a/test/e2e/snaps/test-snap-managestate.spec.js +++ b/test/e2e/snaps/test-snap-managestate.spec.js @@ -20,7 +20,7 @@ describe('Test Snap manageState', function () { fixtures: new FixtureBuilder().build(), ganacheOptions, failOnConsoleError: false, - title: this.test.title, + title: this.test.fullTitle(), }, async ({ driver }) => { await driver.navigate(); diff --git a/test/e2e/snaps/test-snap-networkaccess.spec.js b/test/e2e/snaps/test-snap-networkaccess.spec.js index 0a65386804fc..0301cbf776b7 100644 --- a/test/e2e/snaps/test-snap-networkaccess.spec.js +++ b/test/e2e/snaps/test-snap-networkaccess.spec.js @@ -18,7 +18,7 @@ describe('Test Snap networkAccess', function () { fixtures: new FixtureBuilder().build(), ganacheOptions, failOnConsoleError: false, - title: this.test.title, + title: this.test.fullTitle(), }, async ({ driver }) => { await driver.navigate(); diff --git a/test/e2e/snaps/test-snap-notification.spec.js b/test/e2e/snaps/test-snap-notification.spec.js index fd272ea2ca16..0d062826e554 100644 --- a/test/e2e/snaps/test-snap-notification.spec.js +++ b/test/e2e/snaps/test-snap-notification.spec.js @@ -19,7 +19,7 @@ describe('Test Snap Notification', function () { fixtures: new FixtureBuilder().build(), ganacheOptions, failOnConsoleError: false, - title: this.test.title, + title: this.test.fullTitle(), }, async ({ driver }) => { await driver.navigate(); diff --git a/test/e2e/snaps/test-snap-revoke-perm.spec.js b/test/e2e/snaps/test-snap-revoke-perm.spec.js index b500f583de21..6aa8e115018e 100644 --- a/test/e2e/snaps/test-snap-revoke-perm.spec.js +++ b/test/e2e/snaps/test-snap-revoke-perm.spec.js @@ -18,7 +18,7 @@ describe('Test Snap revoke permission', function () { fixtures: new FixtureBuilder().build(), ganacheOptions, failOnConsoleError: false, - title: this.test.title, + title: this.test.fullTitle(), }, async ({ driver }) => { await driver.navigate(); diff --git a/test/e2e/snaps/test-snap-rpc.spec.js b/test/e2e/snaps/test-snap-rpc.spec.js index 2c955e9e7d6e..35d12927ad0d 100644 --- a/test/e2e/snaps/test-snap-rpc.spec.js +++ b/test/e2e/snaps/test-snap-rpc.spec.js @@ -18,7 +18,7 @@ describe('Test Snap RPC', function () { fixtures: new FixtureBuilder().build(), ganacheOptions, failOnConsoleError: false, - title: this.test.title, + title: this.test.fullTitle(), }, async ({ driver }) => { await driver.navigate(); diff --git a/test/e2e/snaps/test-snap-txinsights.spec.js b/test/e2e/snaps/test-snap-txinsights.spec.js index 637dab89e2e2..a283736a6cbb 100644 --- a/test/e2e/snaps/test-snap-txinsights.spec.js +++ b/test/e2e/snaps/test-snap-txinsights.spec.js @@ -18,7 +18,7 @@ describe('Test Snap TxInsights', function () { fixtures: new FixtureBuilder().build(), ganacheOptions, failOnConsoleError: false, - title: this.test.title, + title: this.test.fullTitle(), }, async ({ driver }) => { await driver.navigate(); diff --git a/test/e2e/snaps/test-snap-update.spec.js b/test/e2e/snaps/test-snap-update.spec.js index ea8dcbc11962..e882cf381423 100644 --- a/test/e2e/snaps/test-snap-update.spec.js +++ b/test/e2e/snaps/test-snap-update.spec.js @@ -18,7 +18,7 @@ describe('Test Snap update', function () { fixtures: new FixtureBuilder().build(), ganacheOptions, failOnConsoleError: false, - title: this.test.title, + title: this.test.fullTitle(), }, async ({ driver }) => { await driver.navigate(); diff --git a/test/e2e/snaps/test-snap-wasm.spec.js b/test/e2e/snaps/test-snap-wasm.spec.js index f0e301ca53e5..f3b6e1a2ec0b 100644 --- a/test/e2e/snaps/test-snap-wasm.spec.js +++ b/test/e2e/snaps/test-snap-wasm.spec.js @@ -18,7 +18,7 @@ describe('Test Snap WASM', function () { fixtures: new FixtureBuilder().build(), ganacheOptions, failOnConsoleError: false, - title: this.test.title, + title: this.test.fullTitle(), }, async ({ driver }) => { await driver.navigate(); diff --git a/test/e2e/tests/4byte-directory.spec.js b/test/e2e/tests/4byte-directory.spec.js index a8a5f07c76fe..300094defa22 100644 --- a/test/e2e/tests/4byte-directory.spec.js +++ b/test/e2e/tests/4byte-directory.spec.js @@ -20,7 +20,7 @@ describe('4byte setting', function () { .withPermissionControllerConnectedToTestDapp() .build(), smartContract, - title: this.test.title, + title: this.test.fullTitle(), }, async ({ driver, contractRegistry }) => { const contractAddress = await contractRegistry.getContractAddress( @@ -61,7 +61,7 @@ describe('4byte setting', function () { .withPermissionControllerConnectedToTestDapp() .build(), smartContract, - title: this.test.title, + title: this.test.fullTitle(), }, async ({ driver, contractRegistry }) => { const contractAddress = await contractRegistry.getContractAddress( diff --git a/test/e2e/tests/account-details.spec.js b/test/e2e/tests/account-details.spec.js index b6601fb51b5c..7769dedb2cc9 100644 --- a/test/e2e/tests/account-details.spec.js +++ b/test/e2e/tests/account-details.spec.js @@ -54,7 +54,7 @@ describe('Show account details', function () { { fixtures: new FixtureBuilder().build(), ganacheOptions, - title: this.test.title, + title: this.test.fullTitle(), }, async ({ driver }) => { await driver.navigate(); @@ -77,7 +77,7 @@ describe('Show account details', function () { { fixtures: new FixtureBuilder().build(), ganacheOptions, - title: this.test.title, + title: this.test.fullTitle(), }, async ({ driver }) => { await driver.navigate(); @@ -97,7 +97,7 @@ describe('Show account details', function () { { fixtures: new FixtureBuilder().build(), ganacheOptions, - title: this.test.title, + title: this.test.fullTitle(), }, async ({ driver }) => { await driver.navigate(); @@ -128,7 +128,7 @@ describe('Show account details', function () { { fixtures: new FixtureBuilder().build(), ganacheOptions, - title: this.test.title, + title: this.test.fullTitle(), }, async ({ driver }) => { await driver.navigate(); @@ -148,7 +148,7 @@ describe('Show account details', function () { { fixtures: new FixtureBuilder().build(), ganacheOptions, - title: this.test.title, + title: this.test.fullTitle(), }, async ({ driver }) => { await driver.navigate(); @@ -178,7 +178,7 @@ describe('Show account details', function () { await withFixtures( { fixtures: new FixtureBuilder().build(), - title: this.test.title, + title: this.test.fullTitle(), failOnConsoleError: false, }, async ({ driver }) => { diff --git a/test/e2e/tests/account-token-list.spec.js b/test/e2e/tests/account-token-list.spec.js index 65d1dd8f31ba..d203a9dabb91 100644 --- a/test/e2e/tests/account-token-list.spec.js +++ b/test/e2e/tests/account-token-list.spec.js @@ -16,7 +16,7 @@ describe('Settings', function () { fixtures: new FixtureBuilder().build(), defaultGanacheOptions, smartContract, - title: this.test.title, + title: this.test.fullTitle(), }, async ({ driver }) => { await driver.navigate(); @@ -49,7 +49,7 @@ describe('Settings', function () { fixtures: new FixtureBuilder().build(), defaultGanacheOptions, smartContract, - title: this.test.title, + title: this.test.fullTitle(), }, async ({ driver }) => { await driver.navigate(); diff --git a/test/e2e/tests/add-account.spec.js b/test/e2e/tests/add-account.spec.js index a72072f783c4..6c9d76174984 100644 --- a/test/e2e/tests/add-account.spec.js +++ b/test/e2e/tests/add-account.spec.js @@ -34,7 +34,7 @@ describe('Add account', function () { { fixtures: new FixtureBuilder().build(), ganacheOptions, - title: this.test.title, + title: this.test.fullTitle(), }, async ({ driver }) => { await driver.navigate(); @@ -66,7 +66,7 @@ describe('Add account', function () { { fixtures: new FixtureBuilder({ onboarding: true }).build(), ganacheOptions, - title: this.test.title, + title: this.test.fullTitle(), failOnConsoleError: false, }, async ({ driver }) => { @@ -188,7 +188,7 @@ describe('Add account', function () { { fixtures: new FixtureBuilder().build(), ganacheOptions, - title: this.test.title, + title: this.test.fullTitle(), }, async ({ driver }) => { await driver.navigate(); diff --git a/test/e2e/tests/add-custom-network.spec.js b/test/e2e/tests/add-custom-network.spec.js index c4c3c3c1a31c..d5eeda09690d 100644 --- a/test/e2e/tests/add-custom-network.spec.js +++ b/test/e2e/tests/add-custom-network.spec.js @@ -86,7 +86,7 @@ describe('Custom network', function () { .withPermissionControllerConnectedToTestDapp() .build(), ganacheOptions, - title: this.test.title, + title: this.test.fullTitle(), }, async ({ driver }) => { await driver.navigate(); @@ -165,7 +165,7 @@ describe('Custom network', function () { .withPermissionControllerConnectedToTestDapp() .withPreferencesController({ useSafeChainsListValidation: true }) .build(), - title: this.test.title, + title: this.test.fullTitle(), }, async ({ driver }) => { await driver.navigate(); @@ -272,7 +272,7 @@ describe('Custom network', function () { .withPermissionControllerConnectedToTestDapp() .withPreferencesController({ useSafeChainsListValidation: false }) .build(), - title: this.test.title, + title: this.test.fullTitle(), testSpecificMock: mockRPCURLAndChainId, }, async ({ driver }) => { @@ -327,7 +327,7 @@ describe('Custom network', function () { .withPermissionControllerConnectedToTestDapp() .build(), ganacheOptions, - title: this.test.title, + title: this.test.fullTitle(), }, async ({ driver }) => { await driver.navigate(); @@ -388,7 +388,7 @@ describe('Custom network', function () { { fixtures: new FixtureBuilder().build(), ganacheOptions, - title: this.test.title, + title: this.test.fullTitle(), }, async ({ driver }) => { await driver.navigate(); @@ -475,7 +475,7 @@ describe('Custom network', function () { { fixtures: new FixtureBuilder().build(), ganacheOptions, - title: this.test.title, + title: this.test.fullTitle(), }, async ({ driver }) => { await driver.navigate(); @@ -532,7 +532,7 @@ describe('Custom network', function () { }) .build(), ganacheOptions, - title: this.test.title, + title: this.test.fullTitle(), }, async ({ driver }) => { await driver.navigate(); @@ -583,7 +583,7 @@ describe('Custom network', function () { { fixtures: new FixtureBuilder().build(), ganacheOptions, - title: this.test.title, + title: this.test.fullTitle(), testSpecificMock: mockRPCURLAndChainId, }, async ({ driver }) => { @@ -625,7 +625,7 @@ describe('Custom network', function () { { fixtures: new FixtureBuilder().build(), ganacheOptions, - title: this.test.title, + title: this.test.fullTitle(), testSpecificMock: mockRPCURLAndChainId, }, async ({ driver }) => { diff --git a/test/e2e/tests/add-hide-token.spec.js b/test/e2e/tests/add-hide-token.spec.js index 533611ff2665..7bdf1cfdf931 100644 --- a/test/e2e/tests/add-hide-token.spec.js +++ b/test/e2e/tests/add-hide-token.spec.js @@ -43,7 +43,7 @@ describe('Hide token', function () { }) .build(), ganacheOptions, - title: this.test.title, + title: this.test.fullTitle(), }, async ({ driver }) => { await driver.navigate(); @@ -100,7 +100,7 @@ describe('Add existing token using search', function () { .withPreferencesController({ useTokenDetection: true }) .build(), ganacheOptions, - title: this.test.title, + title: this.test.fullTitle(), }, async ({ driver }) => { await driver.navigate(); @@ -146,7 +146,7 @@ describe('Add token using wallet_watchAsset', function () { .withPermissionControllerConnectedToTestDapp() .build(), ganacheOptions, - title: this.test.title, + title: this.test.fullTitle(), }, async ({ driver }) => { await driver.navigate(); @@ -199,7 +199,7 @@ describe('Add token using wallet_watchAsset', function () { .withPermissionControllerConnectedToTestDapp() .build(), ganacheOptions, - title: this.test.title, + title: this.test.fullTitle(), }, async ({ driver }) => { await driver.navigate(); diff --git a/test/e2e/tests/add-multiple-tokens.spec.js b/test/e2e/tests/add-multiple-tokens.spec.js index a249e3ce36b1..85f8e38d52eb 100644 --- a/test/e2e/tests/add-multiple-tokens.spec.js +++ b/test/e2e/tests/add-multiple-tokens.spec.js @@ -19,7 +19,7 @@ describe('Multiple ERC20 Watch Asset', function () { .withPermissionControllerConnectedToTestDapp() .build(), ganacheOptions: defaultGanacheOptions, - title: this.test.title, + title: this.test.fullTitle(), }, async ({ driver }) => { await driver.navigate(); diff --git a/test/e2e/tests/address-book.spec.js b/test/e2e/tests/address-book.spec.js index 0627cf3564e9..32679712978c 100644 --- a/test/e2e/tests/address-book.spec.js +++ b/test/e2e/tests/address-book.spec.js @@ -37,7 +37,7 @@ describe('Address Book', function () { }) .build(), ganacheOptions, - title: this.test.title, + title: this.test.fullTitle(), }, async ({ driver, ganacheServer }) => { await driver.navigate(); @@ -98,7 +98,7 @@ describe('Address Book', function () { }) .build(), ganacheOptions, - title: this.test.title, + title: this.test.fullTitle(), }, async ({ driver }) => { await driver.navigate(); @@ -161,7 +161,7 @@ describe('Address Book', function () { }) .build(), ganacheOptions, - title: this.test.title, + title: this.test.fullTitle(), }, async ({ driver }) => { await driver.navigate(); diff --git a/test/e2e/tests/advanced-settings.spec.js b/test/e2e/tests/advanced-settings.spec.js index aff3137f1ab9..03f8f1359390 100644 --- a/test/e2e/tests/advanced-settings.spec.js +++ b/test/e2e/tests/advanced-settings.spec.js @@ -19,7 +19,7 @@ describe('Advanced Settings', function () { dapp: true, fixtures: new FixtureBuilder().build(), ganacheOptions, - title: this.test.title, + title: this.test.fullTitle(), }, async ({ driver }) => { await driver.navigate(); @@ -61,7 +61,7 @@ describe('Advanced Settings', function () { dapp: true, fixtures: new FixtureBuilder().build(), ganacheOptions, - title: this.test.title, + title: this.test.fullTitle(), }, async ({ driver }) => { await driver.navigate(); diff --git a/test/e2e/tests/auto-lock.spec.js b/test/e2e/tests/auto-lock.spec.js index 40a1009f296b..8dfb42eeccf3 100644 --- a/test/e2e/tests/auto-lock.spec.js +++ b/test/e2e/tests/auto-lock.spec.js @@ -18,7 +18,7 @@ describe('Auto-Lock Timer', function () { { fixtures: new FixtureBuilder().build(), ganacheOptions, - title: this.test.title, + title: this.test.fullTitle(), }, async ({ driver }) => { await driver.navigate(); diff --git a/test/e2e/tests/backup-restore.spec.js b/test/e2e/tests/backup-restore.spec.js index 43690cbc1e2b..960a6c2ea5e3 100644 --- a/test/e2e/tests/backup-restore.spec.js +++ b/test/e2e/tests/backup-restore.spec.js @@ -64,7 +64,7 @@ describe('Backup and Restore', function () { { fixtures: new FixtureBuilder().build(), ganacheOptions, - title: this.test.title, + title: this.test.fullTitle(), failOnConsoleError: false, }, async ({ driver }) => { @@ -109,7 +109,7 @@ describe('Backup and Restore', function () { { fixtures: new FixtureBuilder().build(), ganacheOptions, - title: this.test.title, + title: this.test.fullTitle(), }, async ({ driver }) => { await driver.navigate(); diff --git a/test/e2e/tests/block-explorer.spec.js b/test/e2e/tests/block-explorer.spec.js index 4627d4df5322..6148e8774cb2 100644 --- a/test/e2e/tests/block-explorer.spec.js +++ b/test/e2e/tests/block-explorer.spec.js @@ -15,7 +15,7 @@ describe('Block Explorer', function () { }) .build(), ganacheOptions: defaultGanacheOptions, - title: this.test.title, + title: this.test.fullTitle(), }, async ({ driver }) => { await driver.navigate(); @@ -61,7 +61,7 @@ describe('Block Explorer', function () { .build(), ganacheOptions: defaultGanacheOptions, smartContract: SMART_CONTRACTS.HST, - title: this.test.title, + title: this.test.fullTitle(), }, async ({ driver }) => { await driver.navigate(); @@ -110,7 +110,7 @@ describe('Block Explorer', function () { .withTransactionControllerCompletedTransaction() .build(), ganacheOptions: defaultGanacheOptions, - title: this.test.title, + title: this.test.fullTitle(), }, async ({ driver }) => { await driver.navigate(); diff --git a/test/e2e/tests/chain-interactions.spec.js b/test/e2e/tests/chain-interactions.spec.js index 3ed9be54fdb4..776af7e9ea83 100644 --- a/test/e2e/tests/chain-interactions.spec.js +++ b/test/e2e/tests/chain-interactions.spec.js @@ -21,7 +21,7 @@ describe('Chain Interactions', function () { dapp: true, fixtures: new FixtureBuilder().build(), ganacheOptions, - title: this.test.title, + title: this.test.fullTitle(), }, async ({ driver }) => { await driver.navigate(); @@ -76,7 +76,7 @@ describe('Chain Interactions', function () { dapp: true, fixtures: new FixtureBuilder().build(), ganacheOptions, - title: this.test.title, + title: this.test.fullTitle(), }, async ({ driver }) => { await driver.navigate(); diff --git a/test/e2e/tests/clear-activity.spec.js b/test/e2e/tests/clear-activity.spec.js index 77d9fdb8b698..fd1b94fad84c 100644 --- a/test/e2e/tests/clear-activity.spec.js +++ b/test/e2e/tests/clear-activity.spec.js @@ -24,7 +24,7 @@ describe('Clear account activity', function () { .withTransactionControllerCompletedAndIncomingTransaction() .build(), ganacheOptions, - title: this.test.title, + title: this.test.fullTitle(), }, async ({ driver }) => { await driver.navigate(); diff --git a/test/e2e/tests/contract-interactions.spec.js b/test/e2e/tests/contract-interactions.spec.js index 4a98d9d48ad9..0cc6092bc941 100644 --- a/test/e2e/tests/contract-interactions.spec.js +++ b/test/e2e/tests/contract-interactions.spec.js @@ -30,7 +30,7 @@ describe('Deploy contract and call contract methods', function () { .build(), ganacheOptions, smartContract, - title: this.test.title, + title: this.test.fullTitle(), }, async ({ driver, contractRegistry, ganacheServer }) => { const contractAddress = await contractRegistry.getContractAddress( diff --git a/test/e2e/tests/custom-rpc-history.spec.js b/test/e2e/tests/custom-rpc-history.spec.js index 93709a98817b..b45e1855802c 100644 --- a/test/e2e/tests/custom-rpc-history.spec.js +++ b/test/e2e/tests/custom-rpc-history.spec.js @@ -24,7 +24,7 @@ describe('Stores custom RPC history', function () { { fixtures: new FixtureBuilder().build(), ganacheOptions: { ...ganacheOptions, concurrent: { port, chainId } }, - title: this.test.title, + title: this.test.fullTitle(), }, async ({ driver }) => { await driver.navigate(); @@ -80,7 +80,7 @@ describe('Stores custom RPC history', function () { { fixtures: new FixtureBuilder().build(), ganacheOptions, - title: this.test.title, + title: this.test.fullTitle(), }, async ({ driver }) => { await driver.navigate(); @@ -122,7 +122,7 @@ describe('Stores custom RPC history', function () { { fixtures: new FixtureBuilder().build(), ganacheOptions, - title: this.test.title, + title: this.test.fullTitle(), failOnConsoleError: false, }, async ({ driver }) => { @@ -175,7 +175,7 @@ describe('Stores custom RPC history', function () { { fixtures: new FixtureBuilder().build(), ganacheOptions, - title: this.test.title, + title: this.test.fullTitle(), }, async ({ driver }) => { await driver.navigate(); @@ -216,7 +216,7 @@ describe('Stores custom RPC history', function () { }) .build(), ganacheOptions, - title: this.test.title, + title: this.test.fullTitle(), }, async ({ driver }) => { await driver.navigate(); @@ -268,7 +268,7 @@ describe('Stores custom RPC history', function () { }) .build(), ganacheOptions, - title: this.test.title, + title: this.test.fullTitle(), failOnConsoleError: false, }, async ({ driver }) => { diff --git a/test/e2e/tests/custom-token-add-approve.spec.js b/test/e2e/tests/custom-token-add-approve.spec.js index 03aaecac7b1f..1687f40cab1c 100644 --- a/test/e2e/tests/custom-token-add-approve.spec.js +++ b/test/e2e/tests/custom-token-add-approve.spec.js @@ -25,7 +25,7 @@ describe('Create token, approve token and approve token without gas', function ( .build(), ganacheOptions, smartContract, - title: this.test.title, + title: this.test.fullTitle(), }, async ({ driver, contractRegistry }) => { const contractAddress = await contractRegistry.getContractAddress( @@ -90,7 +90,7 @@ describe('Create token, approve token and approve token without gas', function ( .build(), ganacheOptions, smartContract, - title: this.test.title, + title: this.test.fullTitle(), }, async ({ driver, contractRegistry }) => { const contractAddress = await contractRegistry.getContractAddress( @@ -208,7 +208,7 @@ describe('Create token, approve token and approve token without gas', function ( .build(), ganacheOptions, smartContract, - title: this.test.title, + title: this.test.fullTitle(), }, async ({ driver, contractRegistry }) => { const contractAddress = await contractRegistry.getContractAddress( @@ -350,7 +350,7 @@ describe('Create token, approve token and approve token without gas', function ( .build(), ganacheOptions, smartContract, - title: this.test.title, + title: this.test.fullTitle(), }, async ({ driver, contractRegistry }) => { const contractAddress = await contractRegistry.getContractAddress( @@ -434,7 +434,7 @@ describe('Create token, approve token and approve token without gas', function ( .build(), ganacheOptions, smartContract, - title: this.test.title, + title: this.test.fullTitle(), }, async ({ driver, contractRegistry }) => { const contractAddress = await contractRegistry.getContractAddress( diff --git a/test/e2e/tests/dapp-interactions.spec.js b/test/e2e/tests/dapp-interactions.spec.js index 4fb9a99a2faf..678c77df1484 100644 --- a/test/e2e/tests/dapp-interactions.spec.js +++ b/test/e2e/tests/dapp-interactions.spec.js @@ -29,7 +29,7 @@ describe('Dapp interactions', function () { ...ganacheOptions, concurrent: { port: 8546, chainId: 1338 }, }, - title: this.test.title, + title: this.test.fullTitle(), }, async ({ driver }) => { await driver.navigate(); @@ -59,7 +59,7 @@ describe('Dapp interactions', function () { .build(), ganacheOptions, dappOptions: { numberOfDapps: 2 }, - title: this.test.title, + title: this.test.fullTitle(), }, async ({ driver }) => { await driver.navigate(); diff --git a/test/e2e/tests/dapp-tx-edit.spec.js b/test/e2e/tests/dapp-tx-edit.spec.js index c572956986aa..d9b9fa7c2fb6 100644 --- a/test/e2e/tests/dapp-tx-edit.spec.js +++ b/test/e2e/tests/dapp-tx-edit.spec.js @@ -23,7 +23,7 @@ describe('Editing confirmations of dapp initiated contract interactions', functi .build(), ganacheOptions, smartContract, - title: this.test.title, + title: this.test.fullTitle(), }, async ({ driver, contractRegistry }) => { const contractAddress = await contractRegistry.getContractAddress( @@ -70,7 +70,7 @@ describe('Editing confirmations of dapp initiated contract interactions', functi .build(), ganacheOptions, smartContract, - title: this.test.title, + title: this.test.fullTitle(), }, async ({ driver }) => { await driver.navigate(); diff --git a/test/e2e/tests/edit-gas-fee.spec.js b/test/e2e/tests/edit-gas-fee.spec.js index 38a71361359f..81ad6bbe65fb 100644 --- a/test/e2e/tests/edit-gas-fee.spec.js +++ b/test/e2e/tests/edit-gas-fee.spec.js @@ -25,7 +25,7 @@ describe('Editing Confirm Transaction', function () { .withTransactionControllerTypeTwoTransaction() .build(), ganacheOptions, - title: this.test.title, + title: this.test.fullTitle(), }, async ({ driver }) => { await driver.navigate(); @@ -111,7 +111,7 @@ describe('Editing Confirm Transaction', function () { .withTransactionControllerTypeTwoTransaction() .build(), ganacheOptions, - title: this.test.title, + title: this.test.fullTitle(), }, async ({ driver }) => { await driver.navigate(); @@ -196,7 +196,7 @@ describe('Editing Confirm Transaction', function () { .withPermissionControllerConnectedToTestDapp() .build(), ganacheOptions, - title: this.test.title, + title: this.test.fullTitle(), dapp: true, }, async ({ driver }) => { diff --git a/test/e2e/tests/encrypt-decrypt.spec.js b/test/e2e/tests/encrypt-decrypt.spec.js index bdce53f8f0eb..b89fcd1ebc7d 100644 --- a/test/e2e/tests/encrypt-decrypt.spec.js +++ b/test/e2e/tests/encrypt-decrypt.spec.js @@ -81,7 +81,7 @@ describe('Encrypt Decrypt', function () { .withPermissionControllerConnectedToTestDapp() .build(), ganacheOptions, - title: this.test.title, + title: this.test.fullTitle(), }, async ({ driver }) => { await driver.navigate(); @@ -122,7 +122,7 @@ describe('Encrypt Decrypt', function () { .withPermissionControllerConnectedToTestDapp() .build(), ganacheOptions, - title: this.test.title, + title: this.test.fullTitle(), }, async ({ driver }) => { await driver.navigate(); @@ -179,7 +179,7 @@ describe('Encrypt Decrypt', function () { .withPermissionControllerConnectedToTestDapp() .build(), ganacheOptions, - title: this.test.title, + title: this.test.fullTitle(), }, async ({ driver }) => { await driver.navigate(); @@ -221,7 +221,7 @@ describe('Encrypt Decrypt', function () { }) .build(), ganacheOptions, - title: this.test.title, + title: this.test.fullTitle(), }, async ({ driver }) => { await driver.navigate(); diff --git a/test/e2e/tests/ens.spec.js b/test/e2e/tests/ens.spec.js index 58b0e18514d1..583d918ea91a 100644 --- a/test/e2e/tests/ens.spec.js +++ b/test/e2e/tests/ens.spec.js @@ -84,7 +84,7 @@ describe('ENS', function () { { fixtures: new FixtureBuilder().withNetworkControllerOnMainnet().build(), ganacheOptions, - title: this.test.title, + title: this.test.fullTitle(), testSpecificMock: mockInfura, }, async ({ driver }) => { diff --git a/test/e2e/tests/errors.spec.js b/test/e2e/tests/errors.spec.js index 8043cd79ae4b..62a03e2b38b5 100644 --- a/test/e2e/tests/errors.spec.js +++ b/test/e2e/tests/errors.spec.js @@ -222,7 +222,7 @@ describe('Sentry errors', function () { meta: undefined, }, ganacheOptions, - title: this.test.title, + title: this.test.fullTitle(), failOnConsoleError: false, testSpecificMock: mockSentryMigratorError, }, @@ -250,7 +250,7 @@ describe('Sentry errors', function () { }) .build(), ganacheOptions, - title: this.test.title, + title: this.test.fullTitle(), failOnConsoleError: false, testSpecificMock: mockSentryTestError, }, @@ -289,7 +289,7 @@ describe('Sentry errors', function () { meta: undefined, }, ganacheOptions, - title: this.test.title, + title: this.test.fullTitle(), failOnConsoleError: false, testSpecificMock: mockSentryMigratorError, }, @@ -329,7 +329,7 @@ describe('Sentry errors', function () { meta: undefined, }, ganacheOptions, - title: this.test.title, + title: this.test.fullTitle(), failOnConsoleError: false, testSpecificMock: mockSentryMigratorError, }, @@ -385,7 +385,7 @@ describe('Sentry errors', function () { .build(), }, ganacheOptions, - title: this.test.title, + title: this.test.fullTitle(), failOnConsoleError: false, testSpecificMock: mockSentryInvariantMigrationError, }, @@ -430,7 +430,7 @@ describe('Sentry errors', function () { }) .build(), ganacheOptions, - title: this.test.title, + title: this.test.fullTitle(), failOnConsoleError: false, testSpecificMock: mockSentryTestError, }, @@ -473,7 +473,7 @@ describe('Sentry errors', function () { }) .build(), ganacheOptions, - title: this.test.title, + title: this.test.fullTitle(), failOnConsoleError: false, testSpecificMock: mockSentryTestError, }, @@ -535,7 +535,7 @@ describe('Sentry errors', function () { }) .build(), ganacheOptions, - title: this.test.title, + title: this.test.fullTitle(), failOnConsoleError: false, testSpecificMock: mockSentryTestError, }, @@ -568,7 +568,7 @@ describe('Sentry errors', function () { }) .build(), ganacheOptions, - title: this.test.title, + title: this.test.fullTitle(), failOnConsoleError: false, testSpecificMock: mockSentryTestError, }, @@ -601,7 +601,7 @@ describe('Sentry errors', function () { }) .build(), ganacheOptions, - title: this.test.title, + title: this.test.fullTitle(), failOnConsoleError: false, testSpecificMock: mockSentryTestError, }, @@ -645,7 +645,7 @@ describe('Sentry errors', function () { }) .build(), ganacheOptions, - title: this.test.title, + title: this.test.fullTitle(), failOnConsoleError: false, testSpecificMock: mockSentryTestError, }, @@ -700,7 +700,7 @@ describe('Sentry errors', function () { }) .build(), ganacheOptions, - title: this.test.title, + title: this.test.fullTitle(), failOnConsoleError: false, testSpecificMock: mockSentryTestError, }, @@ -741,7 +741,7 @@ describe('Sentry errors', function () { }) .build(), ganacheOptions, - title: this.test.title, + title: this.test.fullTitle(), failOnConsoleError: false, testSpecificMock: mockSentryTestError, }, @@ -790,7 +790,7 @@ describe('Sentry errors', function () { { fixtures: new FixtureBuilder().build(), ganacheOptions, - title: this.test.title, + title: this.test.fullTitle(), }, async ({ driver }) => { await driver.navigate(); @@ -838,7 +838,7 @@ describe('Sentry errors', function () { { fixtures: new FixtureBuilder().build(), ganacheOptions, - title: this.test.title, + title: this.test.fullTitle(), }, async ({ driver }) => { await driver.navigate(); diff --git a/test/e2e/tests/eth-sign.spec.js b/test/e2e/tests/eth-sign.spec.js index 682cb9f5e8cc..27ceefb7ec84 100644 --- a/test/e2e/tests/eth-sign.spec.js +++ b/test/e2e/tests/eth-sign.spec.js @@ -18,7 +18,7 @@ describe('Eth sign', function () { .withPermissionControllerConnectedToTestDapp() .build(), ganacheOptions: defaultGanacheOptions, - title: this.test.title, + title: this.test.fullTitle(), }, async ({ driver }) => { await driver.navigate(); @@ -53,7 +53,7 @@ describe('Eth sign', function () { .withPermissionControllerConnectedToTestDapp() .build(), ganacheOptions: defaultGanacheOptions, - title: this.test.title, + title: this.test.fullTitle(), }, async ({ driver }) => { await driver.navigate(); @@ -108,7 +108,7 @@ describe('Eth sign', function () { .withPermissionControllerConnectedToTestDapp() .build(), ganacheOptions: defaultGanacheOptions, - title: this.test.title, + title: this.test.fullTitle(), }, async ({ driver }) => { await driver.navigate(); diff --git a/test/e2e/tests/eth-subscribe.spec.js b/test/e2e/tests/eth-subscribe.spec.js index 8bfdfd99d0a5..3e80008f5ba5 100644 --- a/test/e2e/tests/eth-subscribe.spec.js +++ b/test/e2e/tests/eth-subscribe.spec.js @@ -26,7 +26,7 @@ describe('eth_subscribe', function () { .build(), ganacheOptions, dappOptions: { numberOfDapps: 2 }, - title: this.test.title, + title: this.test.fullTitle(), }, async ({ driver }) => { await driver.navigate(); diff --git a/test/e2e/tests/failing-contract.spec.js b/test/e2e/tests/failing-contract.spec.js index 6a896b47eb74..e5632cdfb284 100644 --- a/test/e2e/tests/failing-contract.spec.js +++ b/test/e2e/tests/failing-contract.spec.js @@ -24,7 +24,7 @@ describe('Failing contract interaction ', function () { .build(), ganacheOptions, smartContract, - title: this.test.title, + title: this.test.fullTitle(), }, async ({ driver, contractRegistry }) => { const contractAddress = await contractRegistry.getContractAddress( @@ -102,7 +102,7 @@ describe('Failing contract interaction on non-EIP1559 network', function () { .build(), ganacheOptions, smartContract, - title: this.test.title, + title: this.test.fullTitle(), }, async ({ driver, contractRegistry }) => { const contractAddress = await contractRegistry.getContractAddress( diff --git a/test/e2e/tests/gas-estimates.spec.js b/test/e2e/tests/gas-estimates.spec.js index a9cd69d66f0a..5af04c6764fa 100644 --- a/test/e2e/tests/gas-estimates.spec.js +++ b/test/e2e/tests/gas-estimates.spec.js @@ -32,7 +32,7 @@ describe('Gas estimates generated by MetaMask', function () { { fixtures: new FixtureBuilder().build(), ganacheOptions: postLondonGanacheOptions, - title: this.test.title, + title: this.test.fullTitle(), }, async ({ driver, ganacheServer }) => { await driver.navigate(); @@ -77,7 +77,7 @@ describe('Gas estimates generated by MetaMask', function () { }; }); }, - title: this.test.title, + title: this.test.fullTitle(), }, async ({ driver, ganacheServer }) => { await driver.navigate(); @@ -119,7 +119,7 @@ describe('Gas estimates generated by MetaMask', function () { }; }); }, - title: this.test.title, + title: this.test.fullTitle(), }, async ({ driver, ganacheServer }) => { await driver.navigate(); @@ -155,7 +155,7 @@ describe('Gas estimates generated by MetaMask', function () { ...preLondonGanacheOptions, chainId: parseInt(CHAIN_IDS.BSC, 16), }, - title: this.test.title, + title: this.test.fullTitle(), }, async ({ driver, ganacheServer }) => { await driver.navigate(); @@ -203,7 +203,7 @@ describe('Gas estimates generated by MetaMask', function () { }; }); }, - title: this.test.title, + title: this.test.fullTitle(), }, async ({ driver, ganacheServer }) => { await driver.navigate(); @@ -234,7 +234,7 @@ describe('Gas estimates generated by MetaMask', function () { { fixtures: new FixtureBuilder().build(), ganacheOptions: preLondonGanacheOptions, - title: this.test.title, + title: this.test.fullTitle(), }, async ({ driver, ganacheServer }) => { await driver.navigate(); diff --git a/test/e2e/tests/import-flow.spec.js b/test/e2e/tests/import-flow.spec.js index 793c2e396424..e8652b502475 100644 --- a/test/e2e/tests/import-flow.spec.js +++ b/test/e2e/tests/import-flow.spec.js @@ -45,7 +45,7 @@ describe('Import flow @no-mmi', function () { { fixtures: new FixtureBuilder({ onboarding: true }).build(), ganacheOptions, - title: this.test.title, + title: this.test.fullTitle(), failOnConsoleError: false, }, async ({ driver }) => { @@ -166,7 +166,7 @@ describe('Import flow @no-mmi', function () { { fixtures: new FixtureBuilder({ onboarding: true }).build(), ganacheOptions, - title: this.test.title, + title: this.test.fullTitle(), failOnConsoleError: false, }, async ({ driver }) => { @@ -210,7 +210,7 @@ describe('Import flow @no-mmi', function () { .withPreferencesControllerImportedAccountIdentities() .build(), ganacheOptions, - title: this.test.title, + title: this.test.fullTitle(), }, async ({ driver }) => { await driver.navigate(); @@ -296,7 +296,7 @@ describe('Import flow @no-mmi', function () { .withPreferencesControllerImportedAccountIdentities() .build(), ganacheOptions, - title: this.test.title, + title: this.test.fullTitle(), }, async ({ driver }) => { await driver.navigate(); @@ -362,7 +362,7 @@ describe('Import flow @no-mmi', function () { .withPreferencesControllerImportedAccountIdentities() .build(), ganacheOptions, - title: this.test.title, + title: this.test.fullTitle(), }, async ({ driver }) => { await driver.navigate(); @@ -401,7 +401,7 @@ describe('Import flow @no-mmi', function () { { fixtures: new FixtureBuilder().build(), ganacheOptions, - title: this.test.title, + title: this.test.fullTitle(), testSpecificMock: mockTrezor, }, async ({ driver }) => { diff --git a/test/e2e/tests/import-tokens.spec.js b/test/e2e/tests/import-tokens.spec.js index 01adbb2ac3a2..bd8f3ba3e4c5 100644 --- a/test/e2e/tests/import-tokens.spec.js +++ b/test/e2e/tests/import-tokens.spec.js @@ -23,7 +23,7 @@ describe('Import flow', function () { { fixtures: new FixtureBuilder().build(), ganacheOptions, - title: this.test.title, + title: this.test.fullTitle(), }, async ({ driver }) => { await driver.navigate(); diff --git a/test/e2e/tests/incremental-security.spec.js b/test/e2e/tests/incremental-security.spec.js index 961dcc904959..2842bc4db56f 100644 --- a/test/e2e/tests/incremental-security.spec.js +++ b/test/e2e/tests/incremental-security.spec.js @@ -23,7 +23,7 @@ describe('Incremental Security', function () { dapp: true, fixtures: new FixtureBuilder({ onboarding: true }).build(), ganacheOptions, - title: this.test.title, + title: this.test.fullTitle(), failOnConsoleError: false, dappPath: 'send-eth-with-private-key-test', }, diff --git a/test/e2e/tests/ipfs-ens-resolution.spec.js b/test/e2e/tests/ipfs-ens-resolution.spec.js index ba7a1357d283..964c06efda42 100644 --- a/test/e2e/tests/ipfs-ens-resolution.spec.js +++ b/test/e2e/tests/ipfs-ens-resolution.spec.js @@ -37,7 +37,7 @@ describe('Settings', function () { await withFixtures( { fixtures: new FixtureBuilder().build(), - title: this.test.title, + title: this.test.fullTitle(), testSpecificMock: (mockServer) => { server = mockServer; }, diff --git a/test/e2e/tests/ipfs-toggle.spec.js b/test/e2e/tests/ipfs-toggle.spec.js index 74cfa4fb6122..024fb8396127 100644 --- a/test/e2e/tests/ipfs-toggle.spec.js +++ b/test/e2e/tests/ipfs-toggle.spec.js @@ -16,7 +16,7 @@ describe('Settings', function () { fixtures: new FixtureBuilder().withNftControllerERC1155().build(), defaultGanacheOptions, smartContract, - title: this.test.title, + title: this.test.fullTitle(), }, async ({ driver }) => { await driver.navigate(); diff --git a/test/e2e/tests/localization.spec.js b/test/e2e/tests/localization.spec.js index e188d61198a0..cfcc362691ac 100644 --- a/test/e2e/tests/localization.spec.js +++ b/test/e2e/tests/localization.spec.js @@ -26,7 +26,7 @@ describe('Localization', function () { }) .build(), ganacheOptions, - title: this.test.title, + title: this.test.fullTitle(), }, async ({ driver }) => { await driver.navigate(); diff --git a/test/e2e/tests/lock-account.spec.js b/test/e2e/tests/lock-account.spec.js index 957be232e1ba..ae7b6bb0ca4e 100644 --- a/test/e2e/tests/lock-account.spec.js +++ b/test/e2e/tests/lock-account.spec.js @@ -17,7 +17,7 @@ describe('Lock and unlock', function () { { fixtures: new FixtureBuilder().build(), ganacheOptions, - title: this.test.title, + title: this.test.fullTitle(), }, async ({ driver }) => { await driver.navigate(); diff --git a/test/e2e/tests/lockdown.spec.js b/test/e2e/tests/lockdown.spec.js index f49baa331df9..988463bc13ae 100644 --- a/test/e2e/tests/lockdown.spec.js +++ b/test/e2e/tests/lockdown.spec.js @@ -68,7 +68,7 @@ describe('lockdown', function () { // The fixtures used here is arbitrary. Any fixture would do. fixtures: new FixtureBuilder().build(), ganacheOptions, - title: this.test.title, + title: this.test.fullTitle(), }, async ({ driver }) => { await driver.navigate(PAGES.HOME); diff --git a/test/e2e/tests/metamask-responsive-ui.spec.js b/test/e2e/tests/metamask-responsive-ui.spec.js index f8d94707513e..f13a076daa6f 100644 --- a/test/e2e/tests/metamask-responsive-ui.spec.js +++ b/test/e2e/tests/metamask-responsive-ui.spec.js @@ -16,7 +16,7 @@ describe('MetaMask Responsive UI', function () { { fixtures: new FixtureBuilder({ onboarding: true }).build(), driverOptions, - title: this.test.title, + title: this.test.fullTitle(), failOnConsoleError: false, }, async ({ driver }) => { @@ -89,7 +89,7 @@ describe('MetaMask Responsive UI', function () { { fixtures: new FixtureBuilder().build(), driverOptions, - title: this.test.title, + title: this.test.fullTitle(), failOnConsoleError: false, }, async ({ driver, ganacheServer }) => { @@ -133,7 +133,7 @@ describe('MetaMask Responsive UI', function () { fixtures: new FixtureBuilder().build(), driverOptions, ganacheOptions, - title: this.test.title, + title: this.test.fullTitle(), }, async ({ driver }) => { await driver.navigate(); diff --git a/test/e2e/tests/metrics/permissions-approved.spec.js b/test/e2e/tests/metrics/permissions-approved.spec.js index 08db0511fe99..2c558c727e80 100644 --- a/test/e2e/tests/metrics/permissions-approved.spec.js +++ b/test/e2e/tests/metrics/permissions-approved.spec.js @@ -56,7 +56,7 @@ describe('Permissions Approved Event', function () { }) .build(), defaultGanacheOptions, - title: this.test.title, + title: this.test.fullTitle(), testSpecificMock: mockSegment, }, async ({ driver, mockedEndpoint: mockedEndpoints }) => { diff --git a/test/e2e/tests/metrics/signature-approved.spec.js b/test/e2e/tests/metrics/signature-approved.spec.js index 53b512c4cd48..6862b203c76f 100644 --- a/test/e2e/tests/metrics/signature-approved.spec.js +++ b/test/e2e/tests/metrics/signature-approved.spec.js @@ -60,7 +60,7 @@ describe('Signature Approved Event @no-mmi', function () { }) .build(), defaultGanacheOptions, - title: this.test.title, + title: this.test.fullTitle(), testSpecificMock: mockSegment, }, async ({ driver, mockedEndpoint: mockedEndpoints }) => { @@ -109,7 +109,7 @@ describe('Signature Approved Event @no-mmi', function () { }) .build(), defaultGanacheOptions, - title: this.test.title, + title: this.test.fullTitle(), testSpecificMock: mockSegment, }, async ({ driver, mockedEndpoint: mockedEndpoints }) => { @@ -158,7 +158,7 @@ describe('Signature Approved Event @no-mmi', function () { }) .build(), defaultGanacheOptions, - title: this.test.title, + title: this.test.fullTitle(), testSpecificMock: mockSegment, }, async ({ driver, mockedEndpoint: mockedEndpoints }) => { @@ -206,7 +206,7 @@ describe('Signature Approved Event @no-mmi', function () { }) .build(), defaultGanacheOptions, - title: this.test.title, + title: this.test.fullTitle(), testSpecificMock: mockSegment, }, async ({ driver, mockedEndpoint: mockedEndpoints }) => { @@ -259,7 +259,7 @@ describe('Signature Approved Event @no-mmi', function () { }) .build(), defaultGanacheOptions, - title: this.test.title, + title: this.test.fullTitle(), testSpecificMock: mockSegment, }, async ({ driver, mockedEndpoint: mockedEndpoints }) => { diff --git a/test/e2e/tests/metrics/swaps.spec.js b/test/e2e/tests/metrics/swaps.spec.js index 68006315ea10..bfa206070d70 100644 --- a/test/e2e/tests/metrics/swaps.spec.js +++ b/test/e2e/tests/metrics/swaps.spec.js @@ -119,7 +119,7 @@ describe('Swap Eth for another Token @no-mmi', function () { }, ], }), - title: this.test.title, + title: this.test.fullTitle(), testSpecificMock: mockSegmentAndMetaswapRequests, }, async ({ driver, mockedEndpoint: mockedEndpoints }) => { diff --git a/test/e2e/tests/metrics/transaction-finalized.spec.js b/test/e2e/tests/metrics/transaction-finalized.spec.js index d905c29da6a5..1aa6cfb34611 100644 --- a/test/e2e/tests/metrics/transaction-finalized.spec.js +++ b/test/e2e/tests/metrics/transaction-finalized.spec.js @@ -144,7 +144,7 @@ describe('Transaction Finalized Event', function () { }) .build(), ganacheOptions: defaultGanacheOptions, - title: this.test.title, + title: this.test.fullTitle(), testSpecificMock: mockSegment, }, async ({ driver, mockedEndpoint: mockedEndpoints, ganacheServer }) => { diff --git a/test/e2e/tests/metrics/unlock-wallet.spec.js b/test/e2e/tests/metrics/unlock-wallet.spec.js index db3142775927..e92b32a75715 100644 --- a/test/e2e/tests/metrics/unlock-wallet.spec.js +++ b/test/e2e/tests/metrics/unlock-wallet.spec.js @@ -30,7 +30,7 @@ describe('Unlock wallet', function () { }) .build(), ganacheOptions: defaultGanacheOptions, - title: this.test.title, + title: this.test.fullTitle(), testSpecificMock: mockSegment, }, async ({ driver, mockedEndpoint }) => { diff --git a/test/e2e/tests/metrics/wallet-created.spec.js b/test/e2e/tests/metrics/wallet-created.spec.js index c699d1e7c3a3..cff720062818 100644 --- a/test/e2e/tests/metrics/wallet-created.spec.js +++ b/test/e2e/tests/metrics/wallet-created.spec.js @@ -59,7 +59,7 @@ describe('Wallet Created Events @no-mmi', function () { }) .build(), defaultGanacheOptions, - title: this.test.title, + title: this.test.fullTitle(), testSpecificMock: mockSegment, }, async ({ driver, mockedEndpoint: mockedEndpoints }) => { @@ -101,7 +101,7 @@ describe('Wallet Created Events @no-mmi', function () { }) .build(), defaultGanacheOptions, - title: this.test.title, + title: this.test.fullTitle(), testSpecificMock: mockSegment, }, async ({ driver, mockedEndpoint: mockedEndpoints }) => { diff --git a/test/e2e/tests/multiple-transactions.spec.js b/test/e2e/tests/multiple-transactions.spec.js index 4bac6e54fff6..f0325a517a83 100644 --- a/test/e2e/tests/multiple-transactions.spec.js +++ b/test/e2e/tests/multiple-transactions.spec.js @@ -28,7 +28,7 @@ describe('Multiple transactions', function () { .withPermissionControllerConnectedToTestDapp() .build(), ganacheOptions, - title: this.test.title, + title: this.test.fullTitle(), }, async ({ driver }) => { await driver.navigate(); @@ -97,7 +97,7 @@ describe('Multiple transactions', function () { .withPermissionControllerConnectedToTestDapp() .build(), ganacheOptions, - title: this.test.title, + title: this.test.fullTitle(), }, async ({ driver }) => { await driver.navigate(); diff --git a/test/e2e/tests/navigate-transactions.spec.js b/test/e2e/tests/navigate-transactions.spec.js index 507071f4a24d..72f826ce32a7 100644 --- a/test/e2e/tests/navigate-transactions.spec.js +++ b/test/e2e/tests/navigate-transactions.spec.js @@ -25,7 +25,7 @@ describe('Navigate transactions', function () { .withTransactionControllerMultipleTransactions() .build(), ganacheOptions, - title: this.test.title, + title: this.test.fullTitle(), }, async ({ driver }) => { await driver.navigate(); @@ -116,7 +116,7 @@ describe('Navigate transactions', function () { .withTransactionControllerMultipleTransactions() .build(), ganacheOptions, - title: this.test.title, + title: this.test.fullTitle(), }, async ({ driver }) => { await driver.navigate(); @@ -162,7 +162,7 @@ describe('Navigate transactions', function () { .withTransactionControllerMultipleTransactions() .build(), ganacheOptions, - title: this.test.title, + title: this.test.fullTitle(), }, async ({ driver }) => { await driver.navigate(); @@ -192,7 +192,7 @@ describe('Navigate transactions', function () { .withTransactionControllerMultipleTransactions() .build(), ganacheOptions, - title: this.test.title, + title: this.test.fullTitle(), }, async ({ driver }) => { await driver.navigate(); @@ -222,7 +222,7 @@ describe('Navigate transactions', function () { .withTransactionControllerMultipleTransactions() .build(), ganacheOptions, - title: this.test.title, + title: this.test.fullTitle(), }, async ({ driver, ganacheServer }) => { await driver.navigate(); diff --git a/test/e2e/tests/network-error.spec.js b/test/e2e/tests/network-error.spec.js index 05a723a7fee6..4bb0b443ae11 100644 --- a/test/e2e/tests/network-error.spec.js +++ b/test/e2e/tests/network-error.spec.js @@ -65,7 +65,7 @@ describe('Gas API fallback', function () { fixtures: new FixtureBuilder().build(), testSpecificMock: mockGasApiDown, ganacheOptions, - title: this.test.title, + title: this.test.fullTitle(), }, async ({ driver, ganacheServer }) => { await driver.navigate(); diff --git a/test/e2e/tests/nft/auto-detect-nft.spec.js b/test/e2e/tests/nft/auto-detect-nft.spec.js index 4a621e9fca6f..4b26d7c19a10 100644 --- a/test/e2e/tests/nft/auto-detect-nft.spec.js +++ b/test/e2e/tests/nft/auto-detect-nft.spec.js @@ -18,7 +18,7 @@ describe('NFT detection', function () { .build(), driverOptions, ganacheOptions: defaultGanacheOptions, - title: this.test.title, + title: this.test.fullTitle(), testSpecificMock: setupAutoDetectMocking, }, async ({ driver }) => { diff --git a/test/e2e/tests/nft/erc1155-interaction.spec.js b/test/e2e/tests/nft/erc1155-interaction.spec.js index 5b2ab2d36d4a..c3471b53c118 100644 --- a/test/e2e/tests/nft/erc1155-interaction.spec.js +++ b/test/e2e/tests/nft/erc1155-interaction.spec.js @@ -30,7 +30,7 @@ describe('ERC1155 NFTs testdapp interaction', function () { .build(), ganacheOptions, smartContract, - title: this.test.title, + title: this.test.fullTitle(), failOnConsoleError: false, }, async ({ driver, _, contractRegistry }) => { @@ -87,7 +87,7 @@ describe('ERC1155 NFTs testdapp interaction', function () { .build(), ganacheOptions, smartContract, - title: this.test.title, + title: this.test.fullTitle(), failOnConsoleError: false, }, async ({ driver, _, contractRegistry }) => { @@ -145,7 +145,7 @@ describe('ERC1155 NFTs testdapp interaction', function () { .build(), ganacheOptions, smartContract, - title: this.test.title, + title: this.test.fullTitle(), }, async ({ driver, _, contractRegistry }) => { const contract = contractRegistry.getContractAddress(smartContract); @@ -233,7 +233,7 @@ describe('ERC1155 NFTs testdapp interaction', function () { .build(), ganacheOptions, smartContract, - title: this.test.title, + title: this.test.fullTitle(), }, async ({ driver, _, contractRegistry }) => { const contract = contractRegistry.getContractAddress(smartContract); diff --git a/test/e2e/tests/nft/erc721-interaction.spec.js b/test/e2e/tests/nft/erc721-interaction.spec.js index cb4f09050d80..d2a264d527af 100644 --- a/test/e2e/tests/nft/erc721-interaction.spec.js +++ b/test/e2e/tests/nft/erc721-interaction.spec.js @@ -29,7 +29,7 @@ describe('ERC721 NFTs testdapp interaction', function () { .build(), ganacheOptions, smartContract, - title: this.test.title, + title: this.test.fullTitle(), failOnConsoleError: false, }, async ({ driver, _, contractRegistry }) => { @@ -151,7 +151,7 @@ describe('ERC721 NFTs testdapp interaction', function () { .build(), ganacheOptions, smartContract, - title: this.test.title, + title: this.test.fullTitle(), failOnConsoleError: false, }, async ({ driver, _, contractRegistry }) => { @@ -247,7 +247,7 @@ describe('ERC721 NFTs testdapp interaction', function () { .build(), ganacheOptions, smartContract, - title: this.test.title, + title: this.test.fullTitle(), failOnConsoleError: false, }, async ({ driver, _, contractRegistry }) => { @@ -298,7 +298,7 @@ describe('ERC721 NFTs testdapp interaction', function () { .build(), ganacheOptions, smartContract, - title: this.test.title, + title: this.test.fullTitle(), failOnConsoleError: false, }, async ({ driver, _, contractRegistry }) => { @@ -369,7 +369,7 @@ describe('ERC721 NFTs testdapp interaction', function () { .build(), ganacheOptions, smartContract, - title: this.test.title, + title: this.test.fullTitle(), failOnConsoleError: false, }, async ({ driver, _, contractRegistry }) => { @@ -439,7 +439,7 @@ describe('ERC721 NFTs testdapp interaction', function () { .build(), ganacheOptions, smartContract, - title: this.test.title, + title: this.test.fullTitle(), failOnConsoleError: false, }, async ({ driver, _, contractRegistry }) => { diff --git a/test/e2e/tests/nft/import-erc1155.spec.js b/test/e2e/tests/nft/import-erc1155.spec.js index 0f598330a601..111905d2ad39 100644 --- a/test/e2e/tests/nft/import-erc1155.spec.js +++ b/test/e2e/tests/nft/import-erc1155.spec.js @@ -28,7 +28,7 @@ describe('Import ERC1155 NFT', function () { .build(), ganacheOptions, smartContract, - title: this.test.title, + title: this.test.fullTitle(), }, async ({ driver, _, contractRegistry }) => { const contractAddress = @@ -77,7 +77,7 @@ describe('Import ERC1155 NFT', function () { .build(), ganacheOptions, smartContract, - title: this.test.title, + title: this.test.fullTitle(), }, async ({ driver, _, contractRegistry }) => { const contractAddress = diff --git a/test/e2e/tests/nft/import-nft.spec.js b/test/e2e/tests/nft/import-nft.spec.js index 320e43ad78a1..3d8d027fe310 100644 --- a/test/e2e/tests/nft/import-nft.spec.js +++ b/test/e2e/tests/nft/import-nft.spec.js @@ -28,7 +28,7 @@ describe('Import NFT', function () { .build(), ganacheOptions, smartContract, - title: this.test.title, + title: this.test.fullTitle(), }, async ({ driver, _, contractRegistry }) => { const contractAddress = @@ -76,7 +76,7 @@ describe('Import NFT', function () { .build(), ganacheOptions, smartContract, - title: this.test.title, + title: this.test.fullTitle(), }, async ({ driver, _, contractRegistry }) => { const contractAddress = diff --git a/test/e2e/tests/nft/remove-erc1155.spec.js b/test/e2e/tests/nft/remove-erc1155.spec.js index e79afc37400e..776a5d4ae9d5 100644 --- a/test/e2e/tests/nft/remove-erc1155.spec.js +++ b/test/e2e/tests/nft/remove-erc1155.spec.js @@ -26,7 +26,7 @@ describe('Remove ERC1155 NFT', function () { fixtures: new FixtureBuilder().withNftControllerERC1155().build(), ganacheOptions, smartContract, - title: this.test.title, + title: this.test.fullTitle(), }, async ({ driver }) => { await driver.navigate(); diff --git a/test/e2e/tests/nft/remove-nft.spec.js b/test/e2e/tests/nft/remove-nft.spec.js index aa08f43435ba..2039f736b19b 100644 --- a/test/e2e/tests/nft/remove-nft.spec.js +++ b/test/e2e/tests/nft/remove-nft.spec.js @@ -26,7 +26,7 @@ describe('Remove NFT', function () { fixtures: new FixtureBuilder().withNftControllerERC721().build(), ganacheOptions, smartContract, - title: this.test.title, + title: this.test.fullTitle(), }, async ({ driver }) => { await driver.navigate(); diff --git a/test/e2e/tests/nft/send-nft.spec.js b/test/e2e/tests/nft/send-nft.spec.js index 590664d2737e..f627ae72eb84 100644 --- a/test/e2e/tests/nft/send-nft.spec.js +++ b/test/e2e/tests/nft/send-nft.spec.js @@ -26,7 +26,7 @@ describe('Send NFT', function () { fixtures: new FixtureBuilder().withNftControllerERC721().build(), ganacheOptions, smartContract, - title: this.test.title, + title: this.test.fullTitle(), }, async ({ driver }) => { await driver.navigate(); diff --git a/test/e2e/tests/nft/view-erc1155-details.spec.js b/test/e2e/tests/nft/view-erc1155-details.spec.js index 7407f5b9aa81..870326e6e8a9 100644 --- a/test/e2e/tests/nft/view-erc1155-details.spec.js +++ b/test/e2e/tests/nft/view-erc1155-details.spec.js @@ -26,7 +26,7 @@ describe('View ERC1155 NFT details', function () { fixtures: new FixtureBuilder().withNftControllerERC1155().build(), ganacheOptions, smartContract, - title: this.test.title, + title: this.test.fullTitle(), }, async ({ driver }) => { await driver.navigate(); diff --git a/test/e2e/tests/nft/view-nft-details.spec.js b/test/e2e/tests/nft/view-nft-details.spec.js index a2ebedaa531f..fbd731aafc5e 100644 --- a/test/e2e/tests/nft/view-nft-details.spec.js +++ b/test/e2e/tests/nft/view-nft-details.spec.js @@ -26,7 +26,7 @@ describe('View NFT details', function () { fixtures: new FixtureBuilder().withNftControllerERC721().build(), ganacheOptions, smartContract, - title: this.test.title, + title: this.test.fullTitle(), }, async ({ driver }) => { await driver.navigate(); diff --git a/test/e2e/tests/onboarding.spec.js b/test/e2e/tests/onboarding.spec.js index 6bcd8ee9b311..5ab4cb9d30b4 100644 --- a/test/e2e/tests/onboarding.spec.js +++ b/test/e2e/tests/onboarding.spec.js @@ -35,7 +35,7 @@ describe('MetaMask onboarding @no-mmi', function () { { fixtures: new FixtureBuilder({ onboarding: true }).build(), ganacheOptions: defaultGanacheOptions, - title: this.test.title, + title: this.test.fullTitle(), failOnConsoleError: false, }, async ({ driver }) => { @@ -56,7 +56,7 @@ describe('MetaMask onboarding @no-mmi', function () { { fixtures: new FixtureBuilder({ onboarding: true }).build(), ganacheOptions: defaultGanacheOptions, - title: this.test.title, + title: this.test.fullTitle(), failOnConsoleError: false, }, async ({ driver }) => { @@ -81,7 +81,7 @@ describe('MetaMask onboarding @no-mmi', function () { { fixtures: new FixtureBuilder({ onboarding: true }).build(), ganacheOptions: defaultGanacheOptions, - title: this.test.title, + title: this.test.fullTitle(), failOnConsoleError: false, }, async ({ driver }) => { @@ -103,7 +103,7 @@ describe('MetaMask onboarding @no-mmi', function () { { fixtures: new FixtureBuilder({ onboarding: true }).build(), ganacheOptions: defaultGanacheOptions, - title: this.test.title, + title: this.test.fullTitle(), failOnConsoleError: false, }, async ({ driver }) => { @@ -143,7 +143,7 @@ describe('MetaMask onboarding @no-mmi', function () { { fixtures: new FixtureBuilder({ onboarding: true }).build(), ganacheOptions: defaultGanacheOptions, - title: this.test.title, + title: this.test.fullTitle(), failOnConsoleError: false, }, async ({ driver }) => { @@ -182,7 +182,7 @@ describe('MetaMask onboarding @no-mmi', function () { { fixtures: new FixtureBuilder({ onboarding: true }).build(), ganacheOptions: defaultGanacheOptions, - title: this.test.title, + title: this.test.fullTitle(), failOnConsoleError: false, }, async ({ driver }) => { @@ -206,7 +206,7 @@ describe('MetaMask onboarding @no-mmi', function () { { fixtures: new FixtureBuilder({ onboarding: true }).build(), ganacheOptions: defaultGanacheOptions, - title: this.test.title, + title: this.test.fullTitle(), failOnConsoleError: false, }, async ({ driver }) => { @@ -252,7 +252,7 @@ describe('MetaMask onboarding @no-mmi', function () { ...defaultGanacheOptions, concurrent: { port, chainId, ganacheOptions2 }, }, - title: this.test.title, + title: this.test.fullTitle(), }, async ({ driver, secondaryGanacheServer }) => { diff --git a/test/e2e/tests/permissions.spec.js b/test/e2e/tests/permissions.spec.js index be4fa78e7020..1e3e6d6ae813 100644 --- a/test/e2e/tests/permissions.spec.js +++ b/test/e2e/tests/permissions.spec.js @@ -18,7 +18,7 @@ describe('Permissions', function () { dapp: true, fixtures: new FixtureBuilder().build(), ganacheOptions, - title: this.test.title, + title: this.test.fullTitle(), }, async ({ driver, ganacheServer }) => { const addresses = await ganacheServer.getAccounts(); diff --git a/test/e2e/tests/personal-sign.spec.js b/test/e2e/tests/personal-sign.spec.js index 780d84796e6c..d1519f212a03 100644 --- a/test/e2e/tests/personal-sign.spec.js +++ b/test/e2e/tests/personal-sign.spec.js @@ -25,7 +25,7 @@ describe('Personal sign', function () { .withPermissionControllerConnectedToTestDapp() .build(), ganacheOptions, - title: this.test.title, + title: this.test.fullTitle(), }, async ({ driver, ganacheServer }) => { const addresses = await ganacheServer.getAccounts(); @@ -73,7 +73,7 @@ describe('Personal sign', function () { .withPermissionControllerConnectedToTestDapp() .build(), ganacheOptions, - title: this.test.title, + title: this.test.fullTitle(), }, async ({ driver, ganacheServer }) => { const addresses = await ganacheServer.getAccounts(); diff --git a/test/e2e/tests/phishing-controller/phishing-detection.spec.js b/test/e2e/tests/phishing-controller/phishing-detection.spec.js index 823432c35015..4f344343ec0b 100644 --- a/test/e2e/tests/phishing-controller/phishing-detection.spec.js +++ b/test/e2e/tests/phishing-controller/phishing-detection.spec.js @@ -43,7 +43,7 @@ describe('Phishing Detection', function () { { fixtures: new FixtureBuilder().build(), ganacheOptions, - title: this.test.title, + title: this.test.fullTitle(), testSpecificMock: async (mockServer) => { return setupPhishingDetectionMocks(mockServer, { blockProvider: BlockProvider.MetaMask, @@ -76,7 +76,7 @@ describe('Phishing Detection', function () { { fixtures: new FixtureBuilder().build(), ganacheOptions, - title: this.test.title, + title: this.test.fullTitle(), testSpecificMock: async (mockServer) => { return setupPhishingDetectionMocks(mockServer, { blockProvider: BlockProvider.MetaMask, @@ -117,7 +117,7 @@ describe('Phishing Detection', function () { { fixtures: new FixtureBuilder().build(), ganacheOptions, - title: this.test.title, + title: this.test.fullTitle(), testSpecificMock: async (mockServer) => { return setupPhishingDetectionMocks(mockServer, { blockProvider: BlockProvider.MetaMask, @@ -161,7 +161,7 @@ describe('Phishing Detection', function () { { fixtures: new FixtureBuilder().build(), ganacheOptions, - title: this.test.title, + title: this.test.fullTitle(), testSpecificMock: (mockServer) => { setupPhishingDetectionMocks(mockServer, { blockProvider: BlockProvider.MetaMask, @@ -201,7 +201,7 @@ describe('Phishing Detection', function () { { fixtures: new FixtureBuilder().build(), ganacheOptions, - title: this.test.title, + title: this.test.fullTitle(), testSpecificMock: async (mockServer) => { return setupPhishingDetectionMocks(mockServer, { blockProvider: BlockProvider.MetaMask, @@ -239,7 +239,7 @@ describe('Phishing Detection', function () { { fixtures: new FixtureBuilder().build(), ganacheOptions, - title: this.test.title, + title: this.test.fullTitle(), testSpecificMock: async (mockServer) => { return setupPhishingDetectionMocks(mockServer, { blockProvider: BlockProvider.PhishFort, @@ -275,7 +275,7 @@ describe('Phishing Detection', function () { { fixtures: new FixtureBuilder().build(), ganacheOptions, - title: this.test.title, + title: this.test.fullTitle(), testSpecificMock: async (mockServer) => { return setupPhishingDetectionMocks(mockServer, { blockProvider: BlockProvider.MetaMask, diff --git a/test/e2e/tests/portfolio-site.spec.js b/test/e2e/tests/portfolio-site.spec.js index 57ee1ac1b74c..94635a57c924 100644 --- a/test/e2e/tests/portfolio-site.spec.js +++ b/test/e2e/tests/portfolio-site.spec.js @@ -35,7 +35,7 @@ describe('Portfolio site', function () { dapp: true, fixtures: new FixtureBuilder().build(), ganacheOptions, - title: this.test.title, + title: this.test.fullTitle(), testSpecificMock: mockPortfolioSite, }, async ({ driver }) => { diff --git a/test/e2e/tests/ppom-blockaid-alert-erc20-approval.spec.js b/test/e2e/tests/ppom-blockaid-alert-erc20-approval.spec.js index 8de02f62edaf..c1fbd1e38c8e 100644 --- a/test/e2e/tests/ppom-blockaid-alert-erc20-approval.spec.js +++ b/test/e2e/tests/ppom-blockaid-alert-erc20-approval.spec.js @@ -177,7 +177,7 @@ describe('PPOM Blockaid Alert - Malicious ERC20 Approval @no-mmi', function () { .build(), defaultGanacheOptions, testSpecificMock: mockInfura, - title: this.test.title, + title: this.test.fullTitle(), }, async ({ driver }) => { diff --git a/test/e2e/tests/ppom-blockaid-alert-erc20-transfer.spec.js b/test/e2e/tests/ppom-blockaid-alert-erc20-transfer.spec.js index 795f0cc056b0..08e09625dbbe 100644 --- a/test/e2e/tests/ppom-blockaid-alert-erc20-transfer.spec.js +++ b/test/e2e/tests/ppom-blockaid-alert-erc20-transfer.spec.js @@ -174,7 +174,7 @@ describe('PPOM Blockaid Alert - Malicious ERC20 Transfer @no-mmi', function () { .build(), defaultGanacheOptions, testSpecificMock: mockInfura, - title: this.test.title, + title: this.test.fullTitle(), }, async ({ driver }) => { diff --git a/test/e2e/tests/ppom-blockaid-alert.spec.js b/test/e2e/tests/ppom-blockaid-alert.spec.js index 7afb4131ffee..1d7fbefda3a5 100644 --- a/test/e2e/tests/ppom-blockaid-alert.spec.js +++ b/test/e2e/tests/ppom-blockaid-alert.spec.js @@ -158,7 +158,7 @@ describe('Confirmation Security Alert - Blockaid @no-mmi', function () { .build(), defaultGanacheOptions, testSpecificMock: mockInfura, - title: this.test.title, + title: this.test.fullTitle(), }, async ({ driver }) => { @@ -226,7 +226,7 @@ describe('Confirmation Security Alert - Blockaid @no-mmi', function () { .build(), defaultGanacheOptions, testSpecificMock: mockInfuraWithMaliciousResponses, - title: this.test.title, + title: this.test.fullTitle(), }, async ({ driver }) => { @@ -286,7 +286,7 @@ describe('Confirmation Security Alert - Blockaid @no-mmi', function () { }) .build(), defaultGanacheOptions, - title: this.test.title, + title: this.test.fullTitle(), }, async ({ driver }) => { diff --git a/test/e2e/tests/ppom-toggle-settings.spec.js b/test/e2e/tests/ppom-toggle-settings.spec.js index fc2035746cbf..445fc700af7e 100644 --- a/test/e2e/tests/ppom-toggle-settings.spec.js +++ b/test/e2e/tests/ppom-toggle-settings.spec.js @@ -18,7 +18,7 @@ describe('PPOM Settings @no-mmi', function () { .withPermissionControllerConnectedToTestDapp() .build(), ganacheOptions: defaultGanacheOptions, - title: this.test.title, + title: this.test.fullTitle(), }, async ({ driver }) => { await driver.navigate(); @@ -46,7 +46,7 @@ describe('PPOM Settings @no-mmi', function () { .withPermissionControllerConnectedToTestDapp() .build(), ganacheOptions: defaultGanacheOptions, - title: this.test.title, + title: this.test.fullTitle(), }, async ({ driver }) => { await driver.navigate(); diff --git a/test/e2e/tests/provider-api.spec.js b/test/e2e/tests/provider-api.spec.js index eee5daa32d3b..8c5c505499bf 100644 --- a/test/e2e/tests/provider-api.spec.js +++ b/test/e2e/tests/provider-api.spec.js @@ -22,7 +22,7 @@ describe('MetaMask', function () { .withPermissionControllerConnectedToTestDapp() .build(), ganacheOptions, - title: this.test.title, + title: this.test.fullTitle(), }, async ({ driver, ganacheServer }) => { const addresses = await ganacheServer.getAccounts(); @@ -66,7 +66,7 @@ describe('MetaMask', function () { .withPermissionControllerConnectedToTestDapp() .build(), ganacheOptions, - title: this.test.title, + title: this.test.fullTitle(), }, async ({ driver }) => { await driver.navigate(); diff --git a/test/e2e/tests/security-provider.spec.js b/test/e2e/tests/security-provider.spec.js index 9ffb5a3529b1..153de5aaefa5 100644 --- a/test/e2e/tests/security-provider.spec.js +++ b/test/e2e/tests/security-provider.spec.js @@ -83,7 +83,7 @@ describe('Transaction security provider', function () { .withPermissionControllerConnectedToTestDapp() .build(), ganacheOptions, - title: this.test.title, + title: this.test.fullTitle(), testSpecificMock: async (mockServer) => await mockSecurityProviderDetection(mockServer, 'malicious'), dapp: true, @@ -123,7 +123,7 @@ describe('Transaction security provider', function () { .withPermissionControllerConnectedToTestDapp() .build(), ganacheOptions, - title: this.test.title, + title: this.test.fullTitle(), testSpecificMock: async (mockServer) => await mockSecurityProviderDetection(mockServer, 'notSafe'), dapp: true, @@ -163,7 +163,7 @@ describe('Transaction security provider', function () { .withPermissionControllerConnectedToTestDapp() .build(), ganacheOptions, - title: this.test.title, + title: this.test.fullTitle(), testSpecificMock: async (mockServer) => await mockSecurityProviderDetection(mockServer, 'notMalicious'), dapp: true, @@ -203,7 +203,7 @@ describe('Transaction security provider', function () { .withPermissionControllerConnectedToTestDapp() .build(), ganacheOptions, - title: this.test.title, + title: this.test.fullTitle(), testSpecificMock: async (mockServer) => await mockSecurityProviderDetection(mockServer, 'requestNotVerified'), dapp: true, diff --git a/test/e2e/tests/send-edit.spec.js b/test/e2e/tests/send-edit.spec.js index 6893d8096a0d..1691f0fbfb91 100644 --- a/test/e2e/tests/send-edit.spec.js +++ b/test/e2e/tests/send-edit.spec.js @@ -19,7 +19,7 @@ describe('Editing Confirm Transaction', function () { .withTransactionControllerTypeOneTransaction() .build(), ganacheOptions, - title: this.test.title, + title: this.test.fullTitle(), failOnConsoleError: false, }, async ({ driver }) => { @@ -101,7 +101,7 @@ describe('Editing Confirm Transaction', function () { .withTransactionControllerTypeTwoTransaction() .build(), ganacheOptions, - title: this.test.title, + title: this.test.fullTitle(), failOnConsoleError: false, }, async ({ driver }) => { diff --git a/test/e2e/tests/send-eth.spec.js b/test/e2e/tests/send-eth.spec.js index c826761e9bc1..c64efe1fc694 100644 --- a/test/e2e/tests/send-eth.spec.js +++ b/test/e2e/tests/send-eth.spec.js @@ -25,7 +25,7 @@ describe('Send ETH from inside MetaMask using default gas', function () { { fixtures: new FixtureBuilder().build(), ganacheOptions, - title: this.test.title, + title: this.test.fullTitle(), }, async ({ driver, ganacheServer }) => { await driver.navigate(); @@ -117,7 +117,7 @@ describe('Send ETH non-contract address with data that matches ERC20 transfer da }) .build(), ganacheOptions, - title: this.test.title, + title: this.test.fullTitle(), }, async ({ driver, ganacheServer }) => { await driver.navigate(); @@ -171,7 +171,7 @@ describe('Send ETH from inside MetaMask using advanced gas modal', function () { { fixtures: new FixtureBuilder().build(), ganacheOptions, - title: this.test.title, + title: this.test.fullTitle(), }, async ({ driver }) => { await driver.navigate(); @@ -239,7 +239,7 @@ describe('Send ETH from dapp using advanced gas controls', function () { .withPermissionControllerConnectedToTestDapp() .build(), ganacheOptions, - title: this.test.title, + title: this.test.fullTitle(), }, async ({ driver }) => { await driver.navigate(); @@ -318,7 +318,7 @@ describe('Send ETH from dapp using advanced gas controls', function () { ...ganacheOptions, hardfork: 'london', }, - title: this.test.title, + title: this.test.fullTitle(), }, async ({ driver }) => { await driver.navigate(); @@ -414,7 +414,7 @@ describe('Send ETH from inside MetaMask to a Multisig Address', function () { fixtures: new FixtureBuilder().build(), ganacheOptions, smartContract, - title: this.test.title, + title: this.test.fullTitle(), }, async ({ driver, contractRegistry, ganacheServer }) => { const contractAddress = await contractRegistry.getContractAddress( diff --git a/test/e2e/tests/send-hex-address.spec.js b/test/e2e/tests/send-hex-address.spec.js index c01290e7b257..686160af7abb 100644 --- a/test/e2e/tests/send-hex-address.spec.js +++ b/test/e2e/tests/send-hex-address.spec.js @@ -25,7 +25,7 @@ describe('Send ETH to a 40 character hexadecimal address', function () { { fixtures: new FixtureBuilder().build(), ganacheOptions, - title: this.test.title, + title: this.test.fullTitle(), failOnConsoleError: false, }, async ({ driver, ganacheServer }) => { @@ -71,7 +71,7 @@ describe('Send ETH to a 40 character hexadecimal address', function () { { fixtures: new FixtureBuilder().build(), ganacheOptions, - title: this.test.title, + title: this.test.fullTitle(), failOnConsoleError: false, }, async ({ driver, ganacheServer }) => { @@ -131,7 +131,7 @@ describe('Send ERC20 to a 40 character hexadecimal address', function () { fixtures: new FixtureBuilder().withTokensControllerERC20().build(), ganacheOptions, smartContract, - title: this.test.title, + title: this.test.fullTitle(), failOnConsoleError: false, }, async ({ driver, ganacheServer }) => { @@ -194,7 +194,7 @@ describe('Send ERC20 to a 40 character hexadecimal address', function () { fixtures: new FixtureBuilder().withTokensControllerERC20().build(), ganacheOptions, smartContract, - title: this.test.title, + title: this.test.fullTitle(), failOnConsoleError: false, }, async ({ driver, ganacheServer }) => { diff --git a/test/e2e/tests/send-to-contract.spec.js b/test/e2e/tests/send-to-contract.spec.js index 68db254a3e5b..b5b5d4a2598d 100644 --- a/test/e2e/tests/send-to-contract.spec.js +++ b/test/e2e/tests/send-to-contract.spec.js @@ -21,7 +21,7 @@ describe('Send ERC20 token to contract address', function () { fixtures: new FixtureBuilder().withTokensControllerERC20().build(), ganacheOptions, smartContract, - title: this.test.title, + title: this.test.fullTitle(), failOnConsoleError: false, }, async ({ driver, contractRegistry }) => { diff --git a/test/e2e/tests/settings-general.spec.js b/test/e2e/tests/settings-general.spec.js index ad937acd05ec..c9c08a3e4b44 100644 --- a/test/e2e/tests/settings-general.spec.js +++ b/test/e2e/tests/settings-general.spec.js @@ -18,7 +18,7 @@ describe('Settings', function () { { fixtures: new FixtureBuilder().build(), ganacheOptions, - title: this.test.title, + title: this.test.fullTitle(), }, async ({ driver }) => { await driver.navigate(); diff --git a/test/e2e/tests/settings-search.spec.js b/test/e2e/tests/settings-search.spec.js index 39513874fff4..35b9f3d9a6eb 100644 --- a/test/e2e/tests/settings-search.spec.js +++ b/test/e2e/tests/settings-search.spec.js @@ -28,7 +28,7 @@ describe('Settings Search', function () { { fixtures: new FixtureBuilder().build(), ganacheOptions, - title: this.test.title, + title: this.test.fullTitle(), }, async ({ driver }) => { await driver.navigate(); @@ -56,7 +56,7 @@ describe('Settings Search', function () { { fixtures: new FixtureBuilder().build(), ganacheOptions, - title: this.test.title, + title: this.test.fullTitle(), }, async ({ driver }) => { await driver.navigate(); @@ -85,7 +85,7 @@ describe('Settings Search', function () { { fixtures: new FixtureBuilder().build(), ganacheOptions, - title: this.test.title, + title: this.test.fullTitle(), }, async ({ driver }) => { await driver.navigate(); @@ -114,7 +114,7 @@ describe('Settings Search', function () { { fixtures: new FixtureBuilder().build(), ganacheOptions, - title: this.test.title, + title: this.test.fullTitle(), }, async ({ driver }) => { await driver.navigate(); @@ -143,7 +143,7 @@ describe('Settings Search', function () { { fixtures: new FixtureBuilder().build(), ganacheOptions, - title: this.test.title, + title: this.test.fullTitle(), }, async ({ driver }) => { await driver.navigate(); @@ -172,7 +172,7 @@ describe('Settings Search', function () { { fixtures: new FixtureBuilder().build(), ganacheOptions, - title: this.test.title, + title: this.test.fullTitle(), }, async ({ driver }) => { await driver.navigate(); @@ -201,7 +201,7 @@ describe('Settings Search', function () { { fixtures: new FixtureBuilder().build(), ganacheOptions, - title: this.test.title, + title: this.test.fullTitle(), }, async ({ driver }) => { await driver.navigate(); @@ -230,7 +230,7 @@ describe('Settings Search', function () { { fixtures: new FixtureBuilder().build(), ganacheOptions, - title: this.test.title, + title: this.test.fullTitle(), }, async ({ driver }) => { await driver.navigate(); @@ -259,7 +259,7 @@ describe('Settings Search', function () { { fixtures: new FixtureBuilder().build(), ganacheOptions, - title: this.test.title, + title: this.test.fullTitle(), }, async ({ driver }) => { await driver.navigate(); diff --git a/test/e2e/tests/settings-security-reveal-srp.spec.js b/test/e2e/tests/settings-security-reveal-srp.spec.js index ea84cb6bc749..6433b39c9321 100644 --- a/test/e2e/tests/settings-security-reveal-srp.spec.js +++ b/test/e2e/tests/settings-security-reveal-srp.spec.js @@ -19,7 +19,7 @@ describe('Reveal SRP through settings', function () { await withFixtures( { fixtures: new FixtureBuilder().build(), - title: this.test.title, + title: this.test.fullTitle(), failOnConsoleError: false, }, async ({ driver }) => { @@ -40,7 +40,7 @@ describe('Reveal SRP through settings', function () { await withFixtures( { fixtures: new FixtureBuilder().build(), - title: this.test.title, + title: this.test.fullTitle(), }, async ({ driver }) => { await passwordUnlockOpenSRPRevealQuiz(driver); @@ -78,7 +78,7 @@ describe('Reveal SRP through settings', function () { await withFixtures( { fixtures: new FixtureBuilder().build(), - title: this.test.title, + title: this.test.fullTitle(), }, async ({ driver }) => { await passwordUnlockOpenSRPRevealQuiz(driver); diff --git a/test/e2e/tests/signature-request.spec.js b/test/e2e/tests/signature-request.spec.js index bbe628d9a831..80bc5c86718b 100644 --- a/test/e2e/tests/signature-request.spec.js +++ b/test/e2e/tests/signature-request.spec.js @@ -71,7 +71,7 @@ describe('Sign Typed Data Signature Request', function () { .withPermissionControllerConnectedToTestDapp() .build(), ganacheOptions, - title: this.test.title, + title: this.test.fullTitle(), }, async ({ driver, ganacheServer }) => { const addresses = await ganacheServer.getAccounts(); @@ -132,7 +132,7 @@ describe('Sign Typed Data Signature Request', function () { .withPermissionControllerConnectedToTestDapp() .build(), ganacheOptions, - title: this.test.title, + title: this.test.fullTitle(), }, async ({ driver, ganacheServer }) => { const addresses = await ganacheServer.getAccounts(); diff --git a/test/e2e/tests/signin-with-ethereum.spec.js b/test/e2e/tests/signin-with-ethereum.spec.js index 43436ae3012b..39e8a9560bee 100644 --- a/test/e2e/tests/signin-with-ethereum.spec.js +++ b/test/e2e/tests/signin-with-ethereum.spec.js @@ -33,7 +33,7 @@ describe('Sign in with ethereum', function () { ganacheOptions: { ...ganacheOptions, }, - title: this.test.title, + title: this.test.fullTitle(), }, async ({ driver }) => { await driver.navigate(); diff --git a/test/e2e/tests/simple-send.spec.js b/test/e2e/tests/simple-send.spec.js index 3e692ed0e599..ffbf93992e82 100644 --- a/test/e2e/tests/simple-send.spec.js +++ b/test/e2e/tests/simple-send.spec.js @@ -21,7 +21,7 @@ describe('Simple send', function () { { fixtures: new FixtureBuilder().build(), ganacheOptions, - title: this.test.title, + title: this.test.fullTitle(), }, async ({ driver, ganacheServer }) => { await driver.navigate(); diff --git a/test/e2e/tests/state-logs.spec.js b/test/e2e/tests/state-logs.spec.js index 8cccbcf911bf..6c29b298b0d3 100644 --- a/test/e2e/tests/state-logs.spec.js +++ b/test/e2e/tests/state-logs.spec.js @@ -40,7 +40,7 @@ describe('State logs', function () { { fixtures: new FixtureBuilder().build(), ganacheOptions, - title: this.test.title, + title: this.test.fullTitle(), failOnConsoleError: false, }, async ({ driver }) => { diff --git a/test/e2e/tests/stuck-approved-transaction.spec.js b/test/e2e/tests/stuck-approved-transaction.spec.js index 74c031385832..83b917762d68 100644 --- a/test/e2e/tests/stuck-approved-transaction.spec.js +++ b/test/e2e/tests/stuck-approved-transaction.spec.js @@ -20,7 +20,7 @@ describe('Editing Confirm Transaction', function () { .withTransactionControllerApprovedTransaction() .build(), ganacheOptions, - title: this.test.title, + title: this.test.fullTitle(), }, async ({ driver }) => { await driver.navigate(); diff --git a/test/e2e/tests/swaps/swap-eth.spec.js b/test/e2e/tests/swaps/swap-eth.spec.js index 6f8bdad1fdfe..691bbcc9127c 100644 --- a/test/e2e/tests/swaps/swap-eth.spec.js +++ b/test/e2e/tests/swaps/swap-eth.spec.js @@ -17,7 +17,7 @@ describe('Swap Eth for another Token @no-mmi', function () { { ...withFixturesOptions, failOnConsoleError: false, - title: this.test.title, + title: this.test.fullTitle(), }, async ({ driver }) => { await loadExtension(driver); @@ -62,7 +62,7 @@ describe('Swap Eth for another Token @no-mmi', function () { await withFixtures( { ...withFixturesOptions, - title: this.test.title, + title: this.test.fullTitle(), }, async ({ driver }) => { await loadExtension(driver); diff --git a/test/e2e/tests/swaps/swaps-notifications.spec.js b/test/e2e/tests/swaps/swaps-notifications.spec.js index 6abf5312dcc0..91bc643f6111 100644 --- a/test/e2e/tests/swaps/swaps-notifications.spec.js +++ b/test/e2e/tests/swaps/swaps-notifications.spec.js @@ -60,7 +60,7 @@ describe('Swaps - notifications @no-mmi', function () { { ...withFixturesOptions, testSpecificMock: mockTradesApiPriceSlippageError, - title: this.test.title, + title: this.test.fullTitle(), }, async ({ driver }) => { await loadExtension(driver); @@ -99,7 +99,7 @@ describe('Swaps - notifications @no-mmi', function () { await withFixtures( { ...withFixturesOptions, - title: this.test.title, + title: this.test.fullTitle(), }, async ({ driver }) => { await loadExtension(driver); @@ -129,7 +129,7 @@ describe('Swaps - notifications @no-mmi', function () { await withFixtures( { ...withFixturesOptions, - title: this.test.title, + title: this.test.fullTitle(), }, async ({ driver }) => { await loadExtension(driver); @@ -151,7 +151,7 @@ describe('Swaps - notifications @no-mmi', function () { await withFixtures( { ...withFixturesOptions, - title: this.test.title, + title: this.test.fullTitle(), }, async ({ driver }) => { await loadExtension(driver); diff --git a/test/e2e/tests/switch-custom-network.spec.js b/test/e2e/tests/switch-custom-network.spec.js index ca40232d13a4..bb066ebc8359 100644 --- a/test/e2e/tests/switch-custom-network.spec.js +++ b/test/e2e/tests/switch-custom-network.spec.js @@ -22,7 +22,7 @@ describe('Switch ethereum chain', function () { .withPermissionControllerConnectedToTestDapp() .build(), ganacheOptions, - title: this.test.title, + title: this.test.fullTitle(), failOnConsoleError: false, }, async ({ driver }) => { diff --git a/test/e2e/tests/terms-of-use.spec.js b/test/e2e/tests/terms-of-use.spec.js index a8be7b5f1cb1..95ad93665726 100644 --- a/test/e2e/tests/terms-of-use.spec.js +++ b/test/e2e/tests/terms-of-use.spec.js @@ -22,7 +22,7 @@ describe('Terms of use', function () { }) .build(), ganacheOptions, - title: this.test.title, + title: this.test.fullTitle(), }, async ({ driver }) => { await driver.navigate(); diff --git a/test/e2e/tests/token-details.spec.js b/test/e2e/tests/token-details.spec.js index e365b760d260..1863547888a4 100644 --- a/test/e2e/tests/token-details.spec.js +++ b/test/e2e/tests/token-details.spec.js @@ -17,7 +17,7 @@ describe('Token Details', function () { { fixtures: new FixtureBuilder().build(), ganacheOptions, - title: this.test.title, + title: this.test.fullTitle(), }, async ({ driver }) => { await driver.navigate(); diff --git a/test/e2e/webdriver/driver.js b/test/e2e/webdriver/driver.js index 1061fe1b59bd..7d1da18245c2 100644 --- a/test/e2e/webdriver/driver.js +++ b/test/e2e/webdriver/driver.js @@ -505,7 +505,17 @@ class Driver { // Error handling - async verboseReportOnFailure(title) { + async verboseReportOnFailure(title, error) { + if (process.env.CIRCLECI) { + console.error( + `Failure in ${title}, for more information see the artifacts tab in CI\n`, + ); + } else { + console.error( + `Failure in ${title}, for more information see the test-artifacts folder\n`, + ); + } + console.error(`${error}\n`); const artifactDir = `./test-artifacts/${this.browser}/${title}`; const filepathBase = `${artifactDir}/test-failure`; await fs.mkdir(artifactDir, { recursive: true });