Commit 4aedde8
test: expand test coverage of events.js
* test else path in emitMany function
* test calling removeAllListeners() in a event emitter instance
with no events at all
* test calling removeListener() passing a event type that does
not exist
* test calling eventNames() in a event emitter instance
with no events at all
Refs: https://coverage.nodejs.org/coverage-ba776b3a56642d4c/root/events.js.html
PR-URL: #10947
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Michael Dawson <michael_dawson@ca.ibm.com>
Reviewed-By: Gibson Fahnestock <gibfahn@gmail.com>1 parent b48b80f commit 4aedde8Copy full SHA for 4aedde8
File tree
Expand file treeCollapse file tree
4 files changed
+25
-5
lines changedOpen diff view settings
Filter options
- test/parallel
Expand file treeCollapse file tree
4 files changed
+25
-5
lines changedOpen diff view settings
Collapse file
test/parallel/test-event-emitter-num-args.js
Copy file name to clipboardExpand all lines: test/parallel/test-event-emitter-num-args.js+12-5Lines changed: 12 additions & 5 deletions
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| ||
4 | 4 | |
5 | 5 | |
6 | 6 | |
7 | | - |
| 7 | + |
8 | 8 | |
9 | 9 | |
10 | 10 | |
11 | | - |
12 | | - |
| 11 | + |
13 | 12 | |
14 | 13 | |
15 | | - |
| 14 | + |
| 15 | + |
| 16 | + |
| 17 | + |
| 18 | + |
| 19 | + |
| 20 | + |
16 | 21 | |
17 | 22 | |
18 | 23 | |
| ||
21 | 26 | |
22 | 27 | |
23 | 28 | |
| 29 | + |
| 30 | + |
24 | 31 | |
25 | | - |
| 32 | + |
26 | 33 | |
Collapse file
test/parallel/test-event-emitter-remove-all-listeners.js
Copy file name to clipboardExpand all lines: test/parallel/test-event-emitter-remove-all-listeners.js+5Lines changed: 5 additions & 0 deletions
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| ||
77 | 77 | |
78 | 78 | |
79 | 79 | |
| 80 | + |
| 81 | + |
| 82 | + |
| 83 | + |
| 84 | + |
Collapse file
test/parallel/test-event-emitter-remove-listeners.js
Copy file name to clipboardExpand all lines: test/parallel/test-event-emitter-remove-listeners.js+6Lines changed: 6 additions & 0 deletions
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| ||
116 | 116 | |
117 | 117 | |
118 | 118 | |
| 119 | + |
| 120 | + |
| 121 | + |
| 122 | + |
| 123 | + |
| 124 | + |
119 | 125 | |
120 | 126 | |
121 | 127 | |
|
Collapse file
test/parallel/test-event-emitter-special-event-names.js
Copy file name to clipboardExpand all lines: test/parallel/test-event-emitter-special-event-names.js+2Lines changed: 2 additions & 0 deletions
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| ||
7 | 7 | |
8 | 8 | |
9 | 9 | |
| 10 | + |
| 11 | + |
10 | 12 | |
11 | 13 | |
12 | 14 | |
|
0 commit comments