From ba999e1b9e96370e074f4227fd59db52d7554a54 Mon Sep 17 00:00:00 2001
From: Braden Whitten <bwhitten518@gmail.com>
Date: Fri, 6 Oct 2017 16:14:31 -0700
Subject: [PATCH] test: more AsyncWrap constructor validation tests
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

Fixes a line of uncovered code in the AsyncWrap class's constructor.
Specifically this covers validtion of the 'promiseResolve' argument.

PR-URL: https://github.com/nodejs/node/pull/16025
Reviewed-By: Ruben Bridgewater <ruben@bridgewater.de>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: Benjamin Gruenbaum <benjamingr@gmail.com>
Reviewed-By: Tobias Nießen <tniessen@tnie.de>
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Refael Ackermann <refack@gmail.com>
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Reviewed-By: Yuta Hiroto <hello@about-hiroppy.com>
---
 test/parallel/test-async-wrap-constructor.js | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/test/parallel/test-async-wrap-constructor.js b/test/parallel/test-async-wrap-constructor.js
index 86fce0e3f39..5fdf9e4fdf8 100644
--- a/test/parallel/test-async-wrap-constructor.js
+++ b/test/parallel/test-async-wrap-constructor.js
@@ -7,7 +7,8 @@ const assert = require('assert');
 const async_hooks = require('async_hooks');
 
 for (const badArg of [0, 1, false, true, null, 'hello']) {
-  for (const field of ['init', 'before', 'after', 'destroy']) {
+  const hookNames = ['init', 'before', 'after', 'destroy', 'promiseResolve'];
+  for (const field of hookNames) {
     assert.throws(() => {
       async_hooks.createHook({ [field]: badArg });
     }, common.expectsError({