nodejs
diff --git a/Collapse file
‎test/common/heap.js‎
Copy file name to clipboardExpand all lines: test/common/heap.js
+19-6Lines changed: 19 additions & 6 deletions b/Collapse file
‎test/common/heap.js‎
Copy file name to clipboardExpand all lines: test/common/heap.js
+19-6Lines changed: 19 additions & 6 deletions
diff --git a/Collapse file
‎test/internet/test-trace-events-dns.js‎
Copy file name to clipboardExpand all lines: test/internet/test-trace-events-dns.js
+6-1Lines changed: 6 additions & 1 deletion b/Collapse file
‎test/internet/test-trace-events-dns.js‎
Copy file name to clipboardExpand all lines: test/internet/test-trace-events-dns.js
+6-1Lines changed: 6 additions & 1 deletion
diff --git a/Collapse file
‎test/parallel/test-trace-events-fs-sync.js‎
Copy file name to clipboardExpand all lines: test/parallel/test-trace-events-fs-sync.js
+6-1Lines changed: 6 additions & 1 deletion b/Collapse file
‎test/parallel/test-trace-events-fs-sync.js‎
Copy file name to clipboardExpand all lines: test/parallel/test-trace-events-fs-sync.js
+6-1Lines changed: 6 additions & 1 deletion
diff --git a/Collapse file
‎test/parallel/test-worker-message-port-transfer-self.js‎
Copy file name to clipboardExpand all lines: test/parallel/test-worker-message-port-transfer-self.js
+8-4Lines changed: 8 additions & 4 deletions b/Collapse file
‎test/parallel/test-worker-message-port-transfer-self.js‎
Copy file name to clipboardExpand all lines: test/parallel/test-worker-message-port-transfer-self.js
+8-4Lines changed: 8 additions & 4 deletions
@@ -33,10 +33,17 @@ class State {
             (node) => [expectedChild.name, 'Node / ' + expectedChild.name]
               .includes(node.name);
 
-          assert(snapshot.some((node) => {
+          const hasChild = snapshot.some((node) => {
             return node.outgoingEdges.map((edge) => edge.toNode).some(check);
-          }), `expected to find child ${util.inspect(expectedChild)} ` +
-              `in ${util.inspect(snapshot)}`);
+          });
+          // Don't use assert with a custom message here. Otherwise the
+          // inspection in the message is done eagerly and wastes a lot of CPU
+          // time.
+          if (!hasChild) {
+            throw new Error(
+              'expected to find child ' +
+              `${util.inspect(expectedChild)} in ${util.inspect(snapshot)}`);
+          }
         }
       }
     }
@@ -57,9 +64,15 @@ class State {
                  node.value.constructor.name === expectedChild.name);
             };
 
-          assert(graph.some((node) => node.edges.some(check)),
-                 `expected to find child ${util.inspect(expectedChild)} ` +
-                 `in ${util.inspect(snapshot)}`);
+          // Don't use assert with a custom message here. Otherwise the
+          // inspection in the message is done eagerly and wastes a lot of CPU
+          // time.
+          const hasChild = graph.some((node) => node.edges.some(check));
+          if (!hasChild) {
+            throw new Error(
+              'expected to find child ' +
+              `${util.inspect(expectedChild)} in ${util.inspect(snapshot)}`);
+          }
         }
       }
     }
 
@@ -49,7 +49,12 @@ for (const tr in tests) {
                             { encoding: 'utf8' });
 
   // Make sure the operation is successful.
-  assert.strictEqual(proc.status, 0, `${tr}:\n${util.inspect(proc)}`);
+  // Don't use assert with a custom message here. Otherwise the
+  // inspection in the message is done eagerly and wastes a lot of CPU
+  // time.
+  if (proc.status !== 0) {
+    throw new Error(`${tr}:\n${util.inspect(proc)}`);
+  }
 
   const file = path.join(tmpdir.path, traceFile);
 
 
@@ -136,7 +136,12 @@ for (const tr in tests) {
   }
 
   // Make sure the operation is successful.
-  assert.strictEqual(proc.status, 0, `${tr}:\n${util.inspect(proc)}`);
+  // Don't use assert with a custom message here. Otherwise the
+  // inspection in the message is done eagerly and wastes a lot of CPU
+  // time.
+  if (proc.status !== 0) {
+    throw new Error(`${tr}:\n${util.inspect(proc)}`);
+  }
 
   // Confirm that trace log file is created.
   assert(fs.existsSync(traceFile));
 
@@ -27,14 +27,18 @@ assert.throws(common.mustCall(() => {
 port2.onmessage = common.mustCall((message) => {
   assert.strictEqual(message, 2);
 
-  assert(util.inspect(port1).includes('active: true'), util.inspect(port1));
-  assert(util.inspect(port2).includes('active: true'), util.inspect(port2));
+  const inspectedPort1 = util.inspect(port1);
+  const inspectedPort2 = util.inspect(port2);
+  assert(inspectedPort1.includes('active: true'), inspectedPort1);
+  assert(inspectedPort2.includes('active: true'), inspectedPort2);
 
   port1.close();
 
   tick(10, () => {
-    assert(util.inspect(port1).includes('active: false'), util.inspect(port1));
-    assert(util.inspect(port2).includes('active: false'), util.inspect(port2));
+    const inspectedPort1 = util.inspect(port1);
+    const inspectedPort2 = util.inspect(port2);
+    assert(inspectedPort1.includes('active: false'), inspectedPort1);
+    assert(inspectedPort2.includes('active: false'), inspectedPort2);
   });
 });
 port1.postMessage(2);