Skip to content

Commit

Permalink
test: remove unused reject handlers
Browse files Browse the repository at this point in the history
PR-URL: nodejs#24540
Reviewed-By: Rich Trott <rtrott@gmail.com>
Reviewed-By: Anatoli Papirovski <apapirovski@mac.com>
  • Loading branch information
Dan Foley authored and Trott committed Nov 22, 2018
1 parent a67b22a commit 070995d
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions test/common/inspector-helper.js
Original file line number Diff line number Diff line change
Expand Up @@ -416,7 +416,7 @@ class NodeInstance extends EventEmitter {
async connectInspectorSession() {
console.log('[test]', 'Connecting to a child Node process');
const upgradeRequest = await this.sendUpgradeRequest();
return new Promise((resolve, reject) => {
return new Promise((resolve) => {
upgradeRequest
.on('upgrade',
(message, socket) => resolve(new InspectorSession(socket, this)))
Expand All @@ -427,7 +427,7 @@ class NodeInstance extends EventEmitter {
async expectConnectionDeclined() {
console.log('[test]', 'Checking upgrade is not possible');
const upgradeRequest = await this.sendUpgradeRequest();
return new Promise((resolve, reject) => {
return new Promise((resolve) => {
upgradeRequest
.on('upgrade', common.mustNotCall('Upgrade was received'))
.on('response', (response) =>
Expand Down

0 comments on commit 070995d

Please sign in to comment.