E2e improve mocking (#13841)

* improve mocking

* improve mocking

* Unnecessary await

development/mock-e2e.js

@@ -1,7 +1,7 @@
-function setupMocking(server, testSpecificMock) {
-  server.forAnyRequest().thenPassThrough();
+async function setupMocking(server, testSpecificMock) {
+  await server.forAnyRequest().thenPassThrough();
 
-  server
+  await server
     .forOptions('https://gas-api.metaswap.codefi.network/networks/1/gasPrices')
     .thenCallback(() => {
       return {
@@ -14,7 +14,7 @@ function setupMocking(server, testSpecificMock) {
       };
     });
 
-  server
+  await server
     .forGet('https://gas-api.metaswap.codefi.network/networks/1/gasPrices')
     .thenCallback(() => {
       return {

test/e2e/helpers.js

@@ -36,11 +36,12 @@ async function withFixtures(options, testSuite) {
   } = options;
   const fixtureServer = new FixtureServer();
   const ganacheServer = new Ganache();
+  const https = await mockttp.generateCACertificate();
+  const mockServer = mockttp.getLocal({ https });
   let secondaryGanacheServer;
   let dappServer;
   let segmentServer;
   let segmentStub;
-  let mockServer;
 
   let webDriver;
   let failed = false;
@@ -91,9 +92,7 @@ async function withFixtures(options, testSuite) {
       });
       await segmentServer.start(9090);
     }
-    const https = await mockttp.generateCACertificate();
-    mockServer = mockttp.getLocal({ https });
-    setupMocking(mockServer, testSpecificMock);
+    await setupMocking(mockServer, testSpecificMock);
     await mockServer.start(8000);
     if (
       process.env.SELENIUM_BROWSER === 'chrome' &&
@@ -157,12 +156,10 @@ async function withFixtures(options, testSuite) {
       if (segmentServer) {
         await segmentServer.stop();
       }
-      if (mockServer) {
       await mockServer.stop();
     }
   }
 }
-}
 
 /**
  * @param {*} driver - selinium driver

test/e2e/tests/phishing-detection.spec.js

@@ -2,8 +2,8 @@ const { strict: assert } = require('assert');
 const { convertToHexValue, withFixtures } = require('../helpers');
 
 describe('Phishing Detection', function () {
-  function mockPhishingDetection(mockServer) {
-    mockServer
+  async function mockPhishingDetection(mockServer) {
+    await mockServer
       .forGet(
         'https://cdn.jsdelivr.net/gh/MetaMask/eth-phishing-detect@master/src/config.json',
       )