Commit 8e2cc5e
report: refactor triggerReport()
This commit fixes the triggerReport() argument validation. The
existing test is also updated, as it was not passing the Error
object to triggerReport().
PR-URL: #26268
Reviewed-By: Richard Lau <riclau@uk.ibm.com>
Reviewed-By: Ruben Bridgewater <ruben@bridgewater.de>
Reviewed-By: James M Snell <jasnell@gmail.com>1 parent b0c310d commit 8e2cc5eCopy full SHA for 8e2cc5e
File tree
Expand file treeCollapse file tree
3 files changed
+17
-20
lines changedOpen diff view settings
Filter options
- lib/internal/process
- src
- test/node-report
Expand file treeCollapse file tree
3 files changed
+17
-20
lines changedOpen diff view settings
Collapse file
lib/internal/process/report.js
Copy file name to clipboardExpand all lines: lib/internal/process/report.js+11-13Lines changed: 11 additions & 13 deletions
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| ||
75 | 75 | |
76 | 76 | |
77 | 77 | |
78 | | - |
79 | | - |
80 | | - |
81 | | - |
82 | | - |
83 | | - |
84 | | - |
85 | | - |
86 | | - |
87 | | - |
88 | | - |
89 | | - |
90 | | - |
| 78 | + |
| 79 | + |
| 80 | + |
| 81 | + |
| 82 | + |
91 | 83 | |
| 84 | + |
| 85 | + |
| 86 | + |
| 87 | + |
| 88 | + |
| 89 | + |
92 | 90 | |
93 | 91 | |
94 | 92 | |
|
Collapse file
+4-5Lines changed: 4 additions & 5 deletions
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| ||
48 | 48 | |
49 | 49 | |
50 | 50 | |
51 | | - |
52 | | - |
53 | | - |
| 51 | + |
| 52 | + |
| 53 | + |
| 54 | + |
54 | 55 | |
55 | | - |
56 | | - |
57 | 56 | |
58 | 57 | |
59 | 58 | |
|
Collapse file
test/node-report/test-api-pass-error.js
Copy file name to clipboardExpand all lines: test/node-report/test-api-pass-error.js+2-2Lines changed: 2 additions & 2 deletions
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| ||
7 | 7 | |
8 | 8 | |
9 | 9 | |
10 | | - |
11 | | - |
| 10 | + |
| 11 | + |
12 | 12 | |
13 | 13 | |
14 | 14 | |
|
0 commit comments