From f1788433da211048eecf836795f51849ed464f6a Mon Sep 17 00:00:00 2001
From: kirillzyusko <kiryl@margelo.io>
Date: Tue, 18 Jun 2024 12:05:03 +0300
Subject: [PATCH] chore: fixes after code review

---
 tests/e2e/testRunner.ts    | 10 +++++-----
 tests/e2e/utils/measure.ts |  5 +++--
 2 files changed, 8 insertions(+), 7 deletions(-)

diff --git a/tests/e2e/testRunner.ts b/tests/e2e/testRunner.ts
index cc5f494b9154..b674ac418f3e 100644
--- a/tests/e2e/testRunner.ts
+++ b/tests/e2e/testRunner.ts
@@ -26,7 +26,7 @@ import installApp from './utils/installApp';
 import killApp from './utils/killApp';
 import launchApp from './utils/launchApp';
 import * as Logger from './utils/logger';
-import {start, stop} from './utils/measure';
+import * as MeasureUtils from './utils/measure';
 import sleep from './utils/sleep';
 import withFailTimeout from './utils/withFailTimeout';
 
@@ -144,13 +144,13 @@ const runTests = async (): Promise<void> => {
         Logger.log('Launching', appPackage);
         await launchApp('android', appPackage, config.ACTIVITY_PATH, launchArgs);
 
-        start(appPackage);
+        MeasureUtils.start(appPackage);
         await withFailTimeout(
             new Promise<void>((resolve) => {
-                const subscription = server.addTestDoneListener(() => {
+                const removeListener = server.addTestDoneListener(() => {
                     Logger.success(iterationText);
 
-                    const metrics = stop();
+                    const metrics = MeasureUtils.stop();
                     const test = server.getTestConfig();
 
                     if (server.isReadyToAcceptTestResults) {
@@ -170,7 +170,7 @@ const runTests = async (): Promise<void> => {
                             duration: metrics.ram,
                         });
                     }
-                    subscription();
+                    removeListener();
                     resolve();
                 });
             }),
diff --git a/tests/e2e/utils/measure.ts b/tests/e2e/utils/measure.ts
index 9b072879743e..c7e7e1837e14 100644
--- a/tests/e2e/utils/measure.ts
+++ b/tests/e2e/utils/measure.ts
@@ -1,11 +1,12 @@
 import {profiler} from '@perf-profiler/profiler';
 import {getAverageCpuUsage, getAverageFPSUsage, getAverageRAMUsage} from '@perf-profiler/reporter';
 import type {Measure} from '@perf-profiler/types';
-import noop from 'lodash/noop';
 
 let measures: Measure[] = [];
 let polling = {
-    stop: noop,
+    stop: (): void => {
+        throw new Error('Cannot stop polling on a stopped profiler');
+    },
 };
 
 const start = (bundleId: string) => {