test_runner: support coverage of unnamed functions

This commit updates the code coverage logic to skip the first function in a file (which does not correspond to an actual function) instead of skipping all functions without a name. PR-URL: #47652 Reviewed-By: Moshe Atlow <[email protected]> Reviewed-By: Matteo Collina <[email protected]> Reviewed-By: Benjamin Gruenbaum <[email protected]>
nodejs · May 2, 2023 · b6fb791 · b6fb791
1 parent 1f120a3
commit b6fb791
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 3 deletions.
diff --git a/lib/internal/test_runner/coverage.js b/lib/internal/test_runner/coverage.js
@@ -155,7 +155,7 @@ class TestCoverage {
       });
 
       for (let j = 0; j < functions.length; ++j) {
-        const { functionName, isBlockCoverage, ranges } = functions[j];
+        const { isBlockCoverage, ranges } = functions[j];
 
         for (let k = 0; k < ranges.length; ++k) {
           const range = ranges[k];
@@ -172,7 +172,7 @@ class TestCoverage {
           }
         }
 
-        if (functionName.length > 0 && ranges.length > 0) {
+        if (j > 0 && ranges.length > 0) {
           const range = ranges[0];
 
           if (range.count !== 0 || range.ignoredLines === range.lines.length) {

diff --git a/test/fixtures/test-runner/coverage.js b/test/fixtures/test-runner/coverage.js
@@ -29,7 +29,7 @@ if (false) {
   require('./invalid-tap.js');
 }
 
-test('a test', function testHandler() {
+test('a test', () => {
   const uncalled = () => {};
 
   function fnWithControlFlow(val) {