test: remove unused reject handlers

Dan Foley · rvagg · commit 8fcf3b3c59de · 2018-11-28T11:25:32.000+11:00
PR-URL: #24540 Reviewed-By: Rich Trott <rtrott@gmail.com> Reviewed-By: Anatoli Papirovski <apapirovski@mac.com>
diff --git a/test/common/inspector-helper.js b/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) =>
