worker: add worker name to report

Author: theanarkh

src/node_report.cc

@@ -234,9 +234,9 @@ static void WriteNodeReport(Isolate* isolate,
     env->ForEachWorker([&](Worker* w) {
       expected_results += w->RequestInterrupt([&](Environment* env) {
         std::ostringstream os;
-
-        GetNodeReport(
-            env, "Worker thread subreport", trigger, Local<Value>(), os);
+        std::string name =
+            "Worker thread subreport [" + std::string(w->name()) + "]";
+        GetNodeReport(env, name.c_str(), trigger, Local<Value>(), os);
 
         Mutex::ScopedLock lock(workers_mutex);
         worker_infos.emplace_back(os.str());

src/node_worker.h

@@ -62,6 +62,7 @@ class Worker : public AsyncWrap {
   bool is_stopped() const;
   const SnapshotData* snapshot_data() const { return snapshot_data_; }
   bool is_internal() const { return is_internal_; }
+  std::string_view name() const { return name_; }
 
   static void New(const v8::FunctionCallbackInfo<v8::Value>& args);
   static void CloneParentEnvVars(

test/report/test-report-worker.js

@@ -13,7 +13,7 @@ async function basic() {
     parentPort.once('message', () => {
       /* Wait for message to stop the Worker */
     });
-  `, { eval: true });
+  `, { eval: true, name: "hello" });
 
   await once(w, 'online');
 
@@ -22,7 +22,8 @@ async function basic() {
   assert.strictEqual(report.workers.length, 1);
   helper.validateContent(report.workers[0]);
   assert.strictEqual(report.workers[0].header.threadId, w.threadId);
-
+  assert.strictEqual(report.workers[0].header.event,
+                    'Worker thread subreport [hello]');
   w.postMessage({});
 
   await once(w, 'exit');