test: add non-ASCII character embedding test

addaleax · addaleax · commit 4cd7f5f147f5 · 2020-09-23T22:34:05.000+02:00
Add a test that verifies that non-ASCII characters may be used in the source code provided to `LoadEnvironment()`. PR-URL: #33972 Reviewed-By: James M Snell <jasnell@gmail.com>
diff --git a/test/embedding/embedtest.cc b/test/embedding/embedtest.cc
@@ -94,6 +94,7 @@ int RunNodeInstance(MultiIsolatePlatform* platform,
         "const publicRequire ="
         "  require('module').createRequire(process.cwd() + '/');"
         "globalThis.require = publicRequire;"
+        "globalThis.embedVars = { n&ouml;n_asc&#305;&#305;: '&#127987;&#65039;&zwj;&#127752;' };"
         "require('vm').runInThisContext(process.argv[1]);");
 
     if (loadenv_ret.IsEmpty())  // There has been a JS exception.
diff --git a/test/embedding/test-embedding.js b/test/embedding/test-embedding.js
@@ -6,12 +6,23 @@ const child_process = require('child_process');
 const path = require('path');
 
 common.allowGlobals(global.require);
+common.allowGlobals(global.embedVars);
 let binary = `out/${common.buildType}/embedtest`;
 if (common.isWindows) {
   binary += '.exe';
 }
 binary = path.resolve(__dirname, '..', '..', binary);
 
+assert.strictEqual(
+  child_process.spawnSync(binary, ['console.log(42)'])
+    .stdout.toString().trim(),
+  '42');
+
+assert.strictEqual(
+  child_process.spawnSync(binary, ['console.log(embedVars.n&ouml;n_asc&#305;&#305;)'])
+    .stdout.toString().trim(),
+  '&#127987;&#65039;&zwj;&#127752;');
+
 assert.strictEqual(
   child_process.spawnSync(binary, ['console.log(42)'])
     .stdout.toString().trim(),
