Report real duration in E2E non hosted mode

Mocha track the full time including setup for the test.
We create our own timer and wrap the Test object with a Proxy to
make sure the reports get the correct time.

Bug: none
Change-Id: Ifab37c7854549ac6dc5a79676c42819bdc25fcaa
Reviewed-on: https://chromium-review.googlesource.com/c/devtools/devtools-frontend/+/6694755
Reviewed-by: Alex Rudenko <[email protected]>
Commit-Queue: Nikolay Vitkov <[email protected]>

Author: Lightning00Blade

test/e2e_non_hosted/conductor/mocha-interface-helpers.ts

@@ -103,10 +103,15 @@ export class InstrumentedTestFunction {
       // eslint-disable-next-line no-debugger
       debugger;  // If you're paused here while debugging, stepping into the next line will step into your test.
     }
+    const start = performance.now();
     const testResult =
         await (this.state === undefined ?
                    this.fn.call(context) :
                    (this.fn as unknown as E2E.TestAsyncCallbackWithState).call(undefined, this.state.state));
+
+    if (context.test) {
+      (context.test as Mocha.Test).realDuration = Math.ceil(performance.now() - start);
+    }
     dumpCollectedErrors();
     return testResult;
   }

test/e2e_non_hosted/conductor/mocha-interface.ts

@@ -112,10 +112,23 @@ function customIt(testImplementation: TestFunctions, suite: Mocha.Suite, file: s
   function createTest(title: string, itBodyFn?: Mocha.AsyncFunc) {
     const test = new Mocha.Test(
         title,
-        suite.isPending() || !itBodyFn ? undefined : InstrumentedTestFunction.instrument(itBodyFn, 'test', suite));
+        suite.isPending() || !itBodyFn ? undefined : InstrumentedTestFunction.instrument(itBodyFn, 'test', suite),
+    );
     test.file = file;
-    suite.addTest(test);
-    return test;
+
+    // Creates a proxy that changes the duration to return
+    // our own timing.
+    const proxyTest = new Proxy(test, {
+      get(target, property, receiver) {
+        if (property === 'duration' && target.realDuration) {
+          return Reflect.get(target, 'realDuration', receiver) ?? Reflect.get(target, property, receiver);
+        }
+        return Reflect.get(target, property, receiver);
+      },
+    });
+
+    suite.addTest(proxyTest);
+    return proxyTest;
   }
 
   // Regular mocha it returns the test instance.

test/e2e_non_hosted/types.d.ts

@@ -48,6 +48,10 @@ declare global {
       beforeEach: undefined;
       afterEach: undefined;
     }
+
+    export interface Test {
+      realDuration?: number;
+    }
   }
   namespace E2E {
     export type HarnessSettings = BrowserSettings&DevtoolsSettings;