[squash] do not emit init event

Author: addaleax

src/async-wrap.cc

@@ -296,8 +296,17 @@ static void PromiseHook(PromiseHookType type, Local<Promise> promise,
   PromiseWrap* wrap = Unwrap<PromiseWrap>(promise);
   if (type == PromiseHookType::kInit ||
       wrap == nullptr) {
+    uint32_t stored_init_field;
+    if (type != PromiseHookType::kInit) {
+      // Do not emit an init event.
+      stored_init_field = env->async_hooks()->fields()[AsyncHooks::kInit];
+      env->async_hooks()->fields()[AsyncHooks::kInit] = 0;
+    }
     wrap = new PromiseWrap(env, promise);
     wrap->MakeWeak(wrap);
+    if (type != PromiseHookType::kInit) {
+      env->async_hooks()->fields()[AsyncHooks::kInit] = stored_init_field;
+    }
   } else if (type == PromiseHookType::kResolve) {
     // TODO(matthewloring): need to expose this through the async hooks api.
   }

test/parallel/test-async-wrap-promise-after-enabled.js

@@ -13,6 +13,8 @@ const p = new Promise((resolve) => resolve(1));
 p.then(() => seenEvents.push('then'));
 
 const hooks = async_hooks.createHook({
+  init: common.mustNotCall(),
+
   before: common.mustCall((id) => {
     assert.ok(id > 1);
     seenEvents.push('before');
@@ -23,8 +25,10 @@ const hooks = async_hooks.createHook({
     seenEvents.push('after');
     hooks.disable();
   })
-}).enable();
+});
 
 setImmediate(() => {
   assert.deepStrictEqual(seenEvents, ['before', 'then', 'after']);
 });
+
+hooks.enable(); // After `setImmediate` in order to not catch its init event.