From a256482318907007db138dd1c71c98f512570d2f Mon Sep 17 00:00:00 2001
From: Rich Trott <rtrott@gmail.com>
Date: Wed, 22 Nov 2017 12:47:50 -0800
Subject: [PATCH] doc,test: remove unnecessary await with return instances
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

Remove unnecessary `await` in combination with `return` in preparation
for enabling lint rule.

PR-URL: https://github.com/nodejs/node/pull/17265
Reviewed-By: Timothy Gu <timothygu99@gmail.com>
Reviewed-By: Vse Mozhet Byt <vsemozhetbyt@gmail.com>
Reviewed-By: Michaël Zasso <targos@protonmail.com>
Reviewed-By: Anatoli Papirovski <apapirovski@mac.com>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Reviewed-By: Gibson Fahnestock <gibfahn@gmail.com>
---
 doc/api/util.md                        |  2 +-
 test/common/inspector-helper.js        |  2 +-
 test/parallel/test-util-callbackify.js | 10 +++++-----
 3 files changed, 7 insertions(+), 7 deletions(-)

diff --git a/doc/api/util.md b/doc/api/util.md
index ba570749888c88..10474b6f836f5c 100644
--- a/doc/api/util.md
+++ b/doc/api/util.md
@@ -31,7 +31,7 @@ For example:
 const util = require('util');
 
 async function fn() {
-  return await Promise.resolve('hello world');
+  return 'hello world';
 }
 const callbackFunction = util.callbackify(fn);
 
diff --git a/test/common/inspector-helper.js b/test/common/inspector-helper.js
index 81aa046bf3d94d..454eef4c5e26da 100644
--- a/test/common/inspector-helper.js
+++ b/test/common/inspector-helper.js
@@ -390,7 +390,7 @@ class NodeInstance {
     console.log('[test]', 'Connecting to a child Node process');
     const response = await this.httpGet(null, '/json/list');
     const url = response[0]['webSocketDebuggerUrl'];
-    return await this.wsHandshake(url);
+    return this.wsHandshake(url);
   }
 
   expectShutdown() {
diff --git a/test/parallel/test-util-callbackify.js b/test/parallel/test-util-callbackify.js
index 0999fc4fd39f07..84b439e43f0702 100644
--- a/test/parallel/test-util-callbackify.js
+++ b/test/parallel/test-util-callbackify.js
@@ -28,7 +28,7 @@ const values = [
   for (const value of values) {
     // Test and `async function`
     async function asyncFn() {
-      return await Promise.resolve(value);
+      return value;
     }
 
     const cbAsyncFn = callbackify(asyncFn);
@@ -70,7 +70,7 @@ const values = [
   for (const value of values) {
     // Test an `async function`
     async function asyncFn() {
-      return await Promise.reject(value);
+      return Promise.reject(value);
     }
 
     const cbAsyncFn = callbackify(asyncFn);
@@ -142,7 +142,7 @@ const values = [
   for (const value of values) {
     async function asyncFn(arg) {
       assert.strictEqual(arg, value);
-      return await Promise.resolve(arg);
+      return arg;
     }
 
     const cbAsyncFn = callbackify(asyncFn);
@@ -183,7 +183,7 @@ const values = [
     const iAmThat = {
       async fn(arg) {
         assert.strictEqual(this, iAmThat);
-        return await Promise.resolve(arg);
+        return arg;
       },
     };
     iAmThat.cbFn = callbackify(iAmThat.fn);
@@ -241,7 +241,7 @@ const values = [
 
 {
   async function asyncFn() {
-    return await Promise.resolve(42);
+    return 42;
   }
 
   const cb = callbackify(asyncFn);