From 35d9d669b37c04130df6d3d9027ad9bad0a60095 Mon Sep 17 00:00:00 2001
From: JckXia <ys3xia@uwaterloo.ca>
Date: Sun, 25 Dec 2022 17:44:36 -0500
Subject: [PATCH] test: add test covg for handle and escapehandle scopes

PR-URL: https://github.com/nodejs/node-addon-api/pull/1263
Reviewed-By: Michael Dawson <midawson@redhat.com
---
 test/handlescope.cc | 26 ++++++++++++++++++++++++++
 test/handlescope.js |  2 ++
 2 files changed, 28 insertions(+)

diff --git a/test/handlescope.cc b/test/handlescope.cc
index 1c7bda393..c68c1c3fa 100644
--- a/test/handlescope.cc
+++ b/test/handlescope.cc
@@ -13,6 +13,16 @@ Value createScope(const CallbackInfo& info) {
   return String::New(info.Env(), "scope");
 }
 
+Value createScopeFromExisting(const CallbackInfo& info) {
+  {
+    napi_handle_scope scope;
+    napi_open_handle_scope(info.Env(), &scope);
+    HandleScope scope_existing(info.Env(), scope);
+    String::New(scope_existing.Env(), "inner-existing-scope");
+  }
+  return String::New(info.Env(), "existing_scope");
+}
+
 Value escapeFromScope(const CallbackInfo& info) {
   Value result;
   {
@@ -22,6 +32,18 @@ Value escapeFromScope(const CallbackInfo& info) {
   return result;
 }
 
+Value escapeFromExistingScope(const CallbackInfo& info) {
+  Value result;
+  {
+    napi_escapable_handle_scope scope;
+    napi_open_escapable_handle_scope(info.Env(), &scope);
+    EscapableHandleScope scope_existing(info.Env(), scope);
+    result = scope_existing.Escape(
+        String::New(scope_existing.Env(), "inner-existing-scope"));
+  }
+  return result;
+}
+
 #define LOOP_MAX 1000000
 Value stressEscapeFromScope(const CallbackInfo& info) {
   Value result;
@@ -52,7 +74,11 @@ Object InitHandleScope(Env env) {
   Object exports = Object::New(env);
 
   exports["createScope"] = Function::New(env, createScope);
+  exports["createScopeFromExisting"] =
+      Function::New(env, createScopeFromExisting);
   exports["escapeFromScope"] = Function::New(env, escapeFromScope);
+  exports["escapeFromExistingScope"] =
+      Function::New(env, escapeFromExistingScope);
   exports["stressEscapeFromScope"] = Function::New(env, stressEscapeFromScope);
   exports["doubleEscapeFromScope"] = Function::New(env, doubleEscapeFromScope);
 
diff --git a/test/handlescope.js b/test/handlescope.js
index e216600ec..ad45f9503 100644
--- a/test/handlescope.js
+++ b/test/handlescope.js
@@ -6,7 +6,9 @@ module.exports = require('./common').runTest(test);
 
 function test (binding) {
   assert.strictEqual(binding.handlescope.createScope(), 'scope');
+  assert.strictEqual(binding.handlescope.createScopeFromExisting(), 'existing_scope');
   assert.strictEqual(binding.handlescope.escapeFromScope(), 'inner-scope');
+  assert.strictEqual(binding.handlescope.escapeFromExistingScope(), 'inner-existing-scope');
   assert.strictEqual(binding.handlescope.stressEscapeFromScope(), 'inner-scope999999');
   assert.throws(() => binding.handlescope.doubleEscapeFromScope(),
     Error,