Permalink
Browse files

test: remove unused reject handlers

PR-URL: #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 21, 2018
1 parent a67b22a commit 070995d5866beb31ee2b64ece90b883674a95b40
Showing with 2 additions and 2 deletions.
  1. +2 −2 test/common/inspector-helper.js
@@ -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)))
@@ -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) =>

0 comments on commit 070995d

Please sign in to comment.