Skip to content

Commit

Permalink
test: remove unused reject handlers
Browse files Browse the repository at this point in the history
PR-URL: #24540
Reviewed-By: Rich Trott <[email protected]>
Reviewed-By: Anatoli Papirovski <[email protected]>
  • Loading branch information
Dan Foley authored and rvagg committed Nov 28, 2018
1 parent 46b5df0 commit 8fcf3b3
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 8fcf3b3

Please sign in to comment.