Report source locations to ResultDB

This adds source file information for Interaction and E2E tests. It is not possible atm to add that info for Unit tests, and line number information is expensive to extract. See bug below for details.

Bug: 389873145
Change-Id: Ie640f8cc617685ba4faff9f0b28f4ca411e5eede
Reviewed-on: https://chromium-review.googlesource.com/c/devtools/devtools-frontend/+/6175033
Reviewed-by: Eric Leese <[email protected]>
Reviewed-by: Philip Pfaffe <[email protected]>
Commit-Queue: Liviu Rau <[email protected]>

Author: Liviu Rau

test/conductor/resultsdb.ts

@@ -4,6 +4,11 @@
 
 import * as fs from 'fs';
 import * as http from 'http';
+import * as path from 'path';
+
+import {GEN_DIR} from './paths.js';
+
+export const REPO = 'https://chromium.googlesource.com/devtools/devtools-frontend';
 
 // This type mirrors test_result.proto.
 // https://source.chromium.org/chromium/infra/infra/+/main:recipes-py/recipe_proto/go.chromium.org/luci/resultdb/proto/sink/v1/test_result.proto
@@ -19,6 +24,13 @@ export interface TestResult {
       filePath: string,
     },
   };
+  testMetadata?: {
+    name: string,
+    location: {
+      repo: string,
+      fileName?: string,
+    },
+  };
 }
 
 class SanitizedTestIdTag {
@@ -94,3 +106,7 @@ export function sendTestResult(results: TestResult): void {
   request.write(data);
   request.end();
 }
+
+export function testLocation(file?: string): string|undefined {
+  return file ? `//${path.relative(GEN_DIR, file).replace('.js', '.ts')}` : undefined;
+}

test/shared/mocha-resultsdb-reporter.ts

@@ -136,6 +136,13 @@ class ResultsDbReporter extends Mocha.reporters.Spec {
       testId: ResultsDb.sanitizedTestId(testId),
       duration: `${test.duration || 0}ms`,
       tags: [{key: 'run', value: String(testRetry.currentRetry() + 1)}],
+      testMetadata: {
+        name: test.title,
+        location: {
+          repo: ResultsDb.REPO,
+          fileName: ResultsDb.testLocation(test.file),
+        }
+      }
     };
     const hookName = this.maybeHook(test);
     if (hookName) {