Merge pull request #1059 from jescalada/upgrade-node-to-20.19

chore: upgrade node in CI to 20.19

Author: JamieSlome

.github/workflows/ci.yml

@@ -19,7 +19,7 @@ jobs:
 
     strategy:
       matrix:
-        node-version: [18.x]
+        node-version: [20.x]
         mongodb-version: [4.4]
 
     steps:

.github/workflows/experimental-inventory-ci.yml

@@ -19,7 +19,7 @@ jobs:
 
     strategy:
       matrix:
-        node-version: [18.x]
+        node-version: [20.x]
         mongodb-version: [4.4]
 
     steps:

.github/workflows/lint.yml

@@ -2,8 +2,8 @@ name: Code Cleanliness
 
 on: [pull_request]
 
-env: # environment variables (available in any part of the action)
-  NODE_VERSION: 18
+env:
+  NODE_VERSION: 20
 
 permissions:
   contents: read

.github/workflows/release.yml

@@ -52,7 +52,7 @@
 #       - uses: actions/checkout@8459bc0 # v4
 #       - uses: actions/setup-node@c2ac33f # v4, Setup .npmrc file to publish to npm
 #         with:
-#           node-version: '18.x'
+#           node-version: '20.x'
 #           registry-url: 'https://registry.npmjs.org'
 #       - run: npm ci
 #       - run: npm publish --access=public

test/plugin/plugin.test.js

@@ -23,15 +23,15 @@ describe('loading plugins from packages', function () {
     spawnSync('npm', ['install'], { cwd: testPackagePath, timeout: 5000 });
   });
 
-  it.skip('should load plugins that are the default export (module.exports = pluginObj)', async function () {
+  it('should load plugins that are the default export (module.exports = pluginObj)', async function () {
     const loader = new PluginLoader([join(testPackagePath, 'default-export.js')]);
     await loader.load();
     expect(loader.pushPlugins.length).to.equal(1);
     expect(loader.pushPlugins.every((p) => isCompatiblePlugin(p))).to.be.true;
     expect(loader.pushPlugins[0]).to.be.an.instanceOf(PushActionPlugin);
   }).timeout(10000);
 
-  it.skip('should load multiple plugins from a module that match the plugin class (module.exports = { pluginFoo, pluginBar })', async function () {
+  it('should load multiple plugins from a module that match the plugin class (module.exports = { pluginFoo, pluginBar })', async function () {
     const loader = new PluginLoader([join(testPackagePath, 'multiple-export.js')]);
     await loader.load();
     expect(loader.pushPlugins.length).to.equal(1);
@@ -45,7 +45,7 @@ describe('loading plugins from packages', function () {
     expect(loader.pullPlugins[0]).to.be.instanceOf(PullActionPlugin);
   }).timeout(10000);
 
-  it.skip('should load plugins that are subclassed from plugin classes', async function () {
+  it('should load plugins that are subclassed from plugin classes', async function () {
     const loader = new PluginLoader([join(testPackagePath, 'subclass.js')]);
     await loader.load();
     expect(loader.pushPlugins.length).to.equal(1);