Commit b6fb791
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 <moshe@atlow.co.il>
Reviewed-By: Matteo Collina <matteo.collina@gmail.com>
Reviewed-By: Benjamin Gruenbaum <benjamingr@gmail.com>1 parent 1f120a3 commit b6fb791
2 files changed
Lines changed: 3 additions & 3 deletions
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
@@ -155,7 +155,7 @@ class TestCoverage { | |||
| 155 | 155 | }); | |
| 156 | 156 | ||
| 157 | 157 | for (let j = 0; j < functions.length; ++j) { | |
| 158 | - const { functionName, isBlockCoverage, ranges } = functions[j]; | ||
| 158 | + const { isBlockCoverage, ranges } = functions[j]; | ||
| 159 | 159 | ||
| 160 | 160 | for (let k = 0; k < ranges.length; ++k) { | |
| 161 | 161 | const range = ranges[k]; | |
@@ -172,7 +172,7 @@ class TestCoverage { | |||
| 172 | 172 | } | |
| 173 | 173 | } | |
| 174 | 174 | ||
| 175 | - if (functionName.length > 0 && ranges.length > 0) { | ||
| 175 | + if (j > 0 && ranges.length > 0) { | ||
| 176 | 176 | const range = ranges[0]; | |
| 177 | 177 | ||
| 178 | 178 | if (range.count !== 0 || range.ignoredLines === range.lines.length) { | |
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
@@ -29,7 +29,7 @@ if (false) { | |||
| 29 | 29 | require('./invalid-tap.js'); | |
| 30 | 30 | } | |
| 31 | 31 | ||
| 32 | - test('a test', function testHandler() { | ||
| 32 | + test('a test', () => { | ||
| 33 | 33 | const uncalled = () => {}; | |
| 34 | 34 | ||
| 35 | 35 | function fnWithControlFlow(val) { | |
0 commit comments