Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Appearance settings

Commit 9970d56

Browse filesBrowse files
dodevcodebytere
authored andcommitted
test: test add and remove for lib/domain
Testing some of the more specific cases of using domain.add and domain.remove. For example, calling domain.add twice with same event emmiter and actually removing an event emitter from the domain. PR-URL: #24163 Reviewed-By: James M Snell <jasnell@gmail.com> Reviewed-By: Colin Ihrig <cjihrig@gmail.com> Reviewed-By: Ruben Bridgewater <ruben@bridgewater.de>
1 parent 733cb1e commit 9970d56
Copy full SHA for 9970d56

File tree

Expand file treeCollapse file tree

1 file changed

+26
-0
lines changed
Open diff view settings
Filter options
Expand file treeCollapse file tree

1 file changed

+26
-0
lines changed
Open diff view settings
Collapse file
+26Lines changed: 26 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,26 @@
1+
'use strict';
2+
3+
require('../common');
4+
const assert = require('assert');
5+
const domain = require('domain');
6+
const EventEmitter = require('events');
7+
8+
const d = new domain.Domain();
9+
const e = new EventEmitter();
10+
const e2 = new EventEmitter();
11+
12+
d.add(e);
13+
assert.strictEqual(e.domain, d);
14+
15+
// Adding the same event to a domain should not change the member count
16+
let previousMemberCount = d.members.length;
17+
d.add(e);
18+
assert.strictEqual(previousMemberCount, d.members.length);
19+
20+
d.add(e2);
21+
assert.strictEqual(e2.domain, d);
22+
23+
previousMemberCount = d.members.length;
24+
d.remove(e2);
25+
assert.notStrictEqual(e2.domain, d);
26+
assert.strictEqual(previousMemberCount - 1, d.members.length);

0 commit comments

Comments
0 (0)
Morty Proxy This is a proxified and sanitized view of the page, visit original site.