Skip to content

Commit

Permalink
test: increase coverage for diagnostics_channel
Browse files Browse the repository at this point in the history
1. test subscribe with invalid args
https://coverage.nodejs.org/coverage-21f2e8859dfbf09f/lib/diagnostics_channel.js.html#L27

2. test create channel with invalid args
https://coverage.nodejs.org/coverage-21f2e8859dfbf09f/lib/diagnostics_channel.js.html#L98

PR-URL: #36602
Reviewed-By: Gireesh Punathil <[email protected]>
Reviewed-By: Rich Trott <[email protected]>
Reviewed-By: Luigi Pinca <[email protected]>
Reviewed-By: Michael Dawson <[email protected]>
Reviewed-By: Pooja D P <[email protected]>
Reviewed-By: James M Snell <[email protected]>
Reviewed-By: Yash Ladha <[email protected]>
  • Loading branch information
Lxxyx authored and targos committed May 1, 2021
1 parent dcba374 commit ff212f4
Show file tree
Hide file tree
Showing 2 changed files with 10 additions and 0 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -37,3 +37,7 @@ channel.publish(input);
// Should not publish after subscriber is unsubscribed
channel.unsubscribe(subscriber);
assert.ok(!channel.hasSubscribers);

assert.throws(() => {
channel.subscribe(null);
}, { code: 'ERR_INVALID_ARG_TYPE' });
6 changes: 6 additions & 0 deletions test/parallel/test-diagnostics-channel-symbol-named.js
Original file line number Diff line number Diff line change
Expand Up @@ -20,3 +20,9 @@ channel.subscribe(common.mustCall((message, name) => {
}));

channel.publish(input);

{
assert.throws(() => {
dc.channel(null);
}, /ERR_INVALID_ARG_TYPE/);
}

0 comments on commit ff212f4

Please sign in to comment.