fix: Removed the async from tests

Author: paradowstack

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

@@ -1,52 +1,48 @@
 /* eslint-disable @typescript-eslint/no-require-imports */
 /* eslint-disable no-undef */
 const assert = require("assert");
+const test_async = require("bindings")("addon.node");
 
-module.exports = async () => {
-  const test_async = require("bindings")("addon.node");
-  return new Promise((resolve, reject) => {
-    const test = () =>
-      new Promise((resolve, reject) => {
-        test_async.Test(5, {}, (err, val) => {
-          if (err) {
-            reject(err);
-            return;
-          }
-          try {
-            assert.strictEqual(err, null);
-            assert.strictEqual(val, 10);
-          } catch (e) {
-            reject(e);
-          }
-          resolve();
-        });
-      });
+const test = () =>
+  new Promise((resolve, reject) => {
+    test_async.Test(5, {}, (err, val) => {
+      if (err) {
+        reject(err);
+        return;
+      }
+      try {
+        assert.strictEqual(err, null);
+        assert.strictEqual(val, 10);
+      } catch (e) {
+        reject(e);
+      }
+      resolve();
+    });
+  });
 
-    const testCancel = () =>
-      new Promise((resolve) => {
-        test_async.TestCancel(() => resolve());
-      });
+const testCancel = () =>
+  new Promise((resolve) => {
+    test_async.TestCancel(() => resolve());
+  });
 
-    const doRepeatedWork = (count = 0) =>
-      new Promise((resolve, reject) => {
-        const iterations = 100;
-        const workDone = (status) => {
-          try {
-            assert.strictEqual(status, 0);
-          } catch (e) {
-            reject(e);
-          }
-          if (++count < iterations) {
-            test_async.DoRepeatedWork(workDone);
-          } else {
-            resolve();
-          }
-        };
+const doRepeatedWork = (count = 0) =>
+  new Promise((resolve, reject) => {
+    const iterations = 100;
+    const workDone = (status) => {
+      try {
+        assert.strictEqual(status, 0);
+      } catch (e) {
+        reject(e);
+      }
+      if (++count < iterations) {
         test_async.DoRepeatedWork(workDone);
-      });
-
-    Promise.all([test(), testCancel(), doRepeatedWork()])
-      .then(() => resolve())
-      .catch(reject);
+      } else {
+        resolve();
+      }
+    };
+    test_async.DoRepeatedWork(workDone);
   });
+
+module.exports = () => {
+  return Promise.all([test(), testCancel(), doRepeatedWork()]);
 };

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

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