refactor: Move 'bindings' require statement inside module exports for better scoping

Author: paradowstack

packages/node-addon-examples/tests/async/addon.js

@@ -1,9 +1,9 @@
 /* eslint-disable @typescript-eslint/no-require-imports */
 /* eslint-disable no-undef */
-const test_async = require("bindings")("addon.node");
 const assert = require("assert");
 
 module.exports = async () => {
+  const test_async = require("bindings")("addon.node");
   return new Promise((resolve, reject) => {
     const test = () =>
       new Promise((resolve, reject) => {

packages/node-addon-examples/tests/buffers/addon.js

@@ -1,6 +1,5 @@
 /* eslint-disable @typescript-eslint/no-require-imports */
 /* eslint-disable no-undef */
-const addon = require("bindings")("addon.node");
 const assert = require("assert");
 
 const toLocaleString = (text) => {
@@ -12,6 +11,7 @@ const toLocaleString = (text) => {
 };
 
 module.exports = async () => {
+  const addon = require("bindings")("addon.node");
   assert.strictEqual(toLocaleString(addon.newBuffer()), addon.theText);
   assert.strictEqual(toLocaleString(addon.newExternalBuffer()), addon.theText);
   assert.strictEqual(toLocaleString(addon.copyBuffer()), addon.theText);