Move e2e to tests folder and remove unused code

.gitignore

@@ -93,4 +93,4 @@ storybook-static
 .jest-cache
 
 # E2E test reports
-e2e/.results/
+tests/e2e/.results/

README.md

@@ -95,7 +95,7 @@ variables referenced here get updated since your local `.env` file is ignored.
 - `ONYX_METRICS` (optional) - Set this to `true` to capture even more performance metrics and see them in Flipper
    see [React-Native-Onyx#benchmarks](https://github.com/Expensify/react-native-onyx#benchmarks) for more information
 - `E2E_TESTING` (optional) - This needs to be set to `true` when running the e2e tests for performance regression testing. 
-   This happens usually automatically, read [this](e2e/README.md) for more information
+   This happens usually automatically, read [this](tests/e2e/README.md) for more information
 
 ----
 

package.json

@@ -36,7 +36,7 @@
     "analyze-packages": "ANALYZE_BUNDLE=true webpack --config config/webpack/webpack.common.js --env envFile=.env.production",
     "symbolicate:android": "npx metro-symbolicate android/app/build/generated/sourcemaps/react/release/index.android.bundle.map",
     "symbolicate:ios": "npx metro-symbolicate main.jsbundle.map",
-    "test:e2e": "node ./e2e/testRunner.js"
+    "test:e2e": "node tests/e2e/testRunner.js"
   },
   "dependencies": {
     "@expensify/react-native-web": "0.18.9",

src/libs/E2E/client.js

@@ -1,5 +1,5 @@
-import Routes from '../../../e2e/server/routes';
-import Config from '../../../e2e/config';
+import Routes from '../../../tests/e2e/server/routes';
+import Config from '../../../tests/e2e/config';
 
 const SERVER_ADDRESS = `http://localhost:${Config.SERVER_PORT}`;
 

src/libs/E2E/reactNativeLaunchingTest.js

@@ -12,7 +12,7 @@ Performance.markStart('regularAppStart');
 import '../../../index';
 Performance.markEnd('regularAppStart');
 
-import E2EConfig from '../../../e2e/config';
+import E2EConfig from '../../../tests/e2e/config';
 import E2EClient from './client';
 
 console.debug('==========================');

e2e/README.md → tests/e2e/README.md

@@ -32,7 +32,7 @@ It will run the tests of a test case multiple time to average out the results.
 
 ## Adding tests
 
-To add a test checkout the [designed guide](./ADDING_TESTS.md).
+To add a test checkout the [designed guide](tests/e2e/ADDING_TESTS.md).
 
 ## Structure
 

e2e/testRunner.js → tests/e2e/testRunner.js

@@ -24,11 +24,9 @@ const killApp = require('./utils/killApp');
 const launchApp = require('./utils/launchApp');
 const createServerInstance = require('./server');
 const installApp = require('./utils/installApp');
-const reversePort = require('./utils/androidReversePort');
 const math = require('./measure/math');
 const writeTestStats = require('./measure/writeTestStats');
 const withFailTimeout = require('./utils/withFailTimeout');
-const startRecordingVideo = require('./utils/startRecordingVideo');
 
 const args = process.argv.slice(2);
 
@@ -121,8 +119,6 @@ const runTestsOnBranch = async (branch, baselineOrCompare) => {
             const progressText = `(${testIndex + 1}/${numOfTests}) Running test '${config.name}' (iteration ${i + 1}/${RUNS})`;
             testLog.updateText(progressText);
 
-            const stopVideoRecording = startRecordingVideo();
-
             await restartApp();
 
             // Wait for a test to finish by waiting on its done call to the http server
@@ -134,10 +130,8 @@ const runTestsOnBranch = async (branch, baselineOrCompare) => {
                         resolve();
                     });
                 }), progressText);
-                await stopVideoRecording(false);
             } catch (e) {
                 // When we fail due to a timeout it's interesting to take a screenshot of the emulator to see whats going on
-                await stopVideoRecording(true);
                 testLog.done();
                 throw e; // Rethrow to abort execution
             }