From 54ef0fd0106b4eb509c699637d8be86a7a40008f Mon Sep 17 00:00:00 2001
From: Shelley Vohr <shelley.vohr@gmail.com>
Date: Fri, 20 Sep 2019 21:01:43 -0700
Subject: [PATCH] test: --force-context-aware cli flag

PR-URL: https://github.com/nodejs/node/pull/29631
Reviewed-By: Anna Henningsen <anna@addaleax.net>
Reviewed-By: Ben Noordhuis <info@bnoordhuis.nl>
---
 doc/api/cli.md                              |  3 ++-
 test/addons/force-context-aware/binding.cc  |  6 ++++++
 test/addons/force-context-aware/binding.gyp |  9 +++++++++
 test/addons/force-context-aware/index.js    |  4 ++++
 test/addons/force-context-aware/test.js     | 13 +++++++++++++
 test/addons/zlib-binding/test.js            |  2 ++
 6 files changed, 36 insertions(+), 1 deletion(-)
 create mode 100644 test/addons/force-context-aware/binding.cc
 create mode 100644 test/addons/force-context-aware/binding.gyp
 create mode 100644 test/addons/force-context-aware/index.js
 create mode 100644 test/addons/force-context-aware/test.js

diff --git a/doc/api/cli.md b/doc/api/cli.md
index 4753b35591113f..e6cb4b1952f02b 100644
--- a/doc/api/cli.md
+++ b/doc/api/cli.md
@@ -444,7 +444,7 @@ Silence all process warnings (including deprecations).
 added: REPLACEME
 -->
 
-Disable loading non-context-aware native addons.
+Disable loading native addons that are not [context-aware][].
 
 ### `--openssl-config=file`
 <!-- YAML
@@ -1293,3 +1293,4 @@ greater than `4` (its current default value). For more information, see the
 [experimental ECMAScript Module]: esm.html#esm_resolve_hook
 [libuv threadpool documentation]: http://docs.libuv.org/en/latest/threadpool.html
 [remote code execution]: https://www.owasp.org/index.php/Code_Injection
+[context-aware]: addons.html#addons_context_aware_addons
diff --git a/test/addons/force-context-aware/binding.cc b/test/addons/force-context-aware/binding.cc
new file mode 100644
index 00000000000000..4f302fbd558a8a
--- /dev/null
+++ b/test/addons/force-context-aware/binding.cc
@@ -0,0 +1,6 @@
+#include <node.h>
+#include <v8.h>
+
+void init(v8::Local<v8::Object> exports) {}
+
+NODE_MODULE(NODE_GYP_MODULE_NAME, init)
diff --git a/test/addons/force-context-aware/binding.gyp b/test/addons/force-context-aware/binding.gyp
new file mode 100644
index 00000000000000..55fbe7050f18e4
--- /dev/null
+++ b/test/addons/force-context-aware/binding.gyp
@@ -0,0 +1,9 @@
+{
+  'targets': [
+    {
+      'target_name': 'binding',
+      'sources': [ 'binding.cc' ],
+      'includes': ['../common.gypi'],
+    }
+  ]
+}
diff --git a/test/addons/force-context-aware/index.js b/test/addons/force-context-aware/index.js
new file mode 100644
index 00000000000000..17b9a0ad1ef969
--- /dev/null
+++ b/test/addons/force-context-aware/index.js
@@ -0,0 +1,4 @@
+'use strict';
+const common = require('../../common');
+
+require(`./build/${common.buildType}/binding`);
diff --git a/test/addons/force-context-aware/test.js b/test/addons/force-context-aware/test.js
new file mode 100644
index 00000000000000..d5264463a79997
--- /dev/null
+++ b/test/addons/force-context-aware/test.js
@@ -0,0 +1,13 @@
+'use strict';
+const common = require('../../common');
+const childProcess = require('child_process');
+const assert = require('assert');
+const path = require('path');
+
+const mod = path.join('test', 'addons', 'force-context-aware', 'index.js');
+
+const execString = `"${process.execPath}" --force-context-aware ./${mod}`;
+childProcess.exec(execString, common.mustCall((err) => {
+  const errMsg = 'Loading non context-aware native modules has been disabled';
+  assert.strictEqual(err.message.includes(errMsg), true);
+}));
diff --git a/test/addons/zlib-binding/test.js b/test/addons/zlib-binding/test.js
index b50817aa5d4687..7fad9129cd3931 100644
--- a/test/addons/zlib-binding/test.js
+++ b/test/addons/zlib-binding/test.js
@@ -1,3 +1,5 @@
+// Flags: --force-context-aware
+
 'use strict';
 
 const common = require('../../common');