feat: enable use of global node context (#270)

Author: Alex Plischke

package-lock.json

@@ -37,7 +37,7 @@
     "dotenv": "16.4.5",
     "lodash": "4.17.21",
     "playwright": "1.45.2",
-    "sauce-testrunner-utils": "3.0.0",
+    "sauce-testrunner-utils": "3.1.0",
     "xml-js": "1.6.11"
   },
   "devDependencies": {

package.json

@@ -5,6 +5,7 @@ import { prepareNpmEnv, preExec } from 'sauce-testrunner-utils';
 
 import type { CucumberRunnerConfig } from './types';
 import * as utils from './utils';
+import { NodeContext } from 'sauce-testrunner-utils/lib/types';
 
 function buildArgs(runCfg: CucumberRunnerConfig, cucumberBin: string) {
   const paths: string[] = [];
@@ -88,7 +89,11 @@ export async function runCucumber(
     'bin',
     'npm-cli.js',
   );
-  const nodeCtx = { nodePath: nodeBin, npmPath: npmBin };
+  const nodeCtx: NodeContext = {
+    nodePath: nodeBin,
+    npmPath: npmBin,
+    useGlobals: !!runCfg.nodeVersion,
+  };
 
   // Install NPM dependencies
   await prepareNpmEnv(runCfg, nodeCtx);

src/cucumber-runner.ts

@@ -18,6 +18,7 @@ import * as convert from 'xml-js';
 import { runCucumber } from './cucumber-runner';
 import type { CucumberRunnerConfig, RunnerConfig } from './types';
 import * as utils from './utils';
+import { NodeContext } from 'sauce-testrunner-utils/lib/types';
 
 function getPlatformName(platformName: string) {
   if (process.platform.toLowerCase() === 'linux') {
@@ -359,7 +360,11 @@ async function runPlaywright(
     'bin',
     'npm-cli.js',
   );
-  const nodeCtx = { nodePath: nodeBin, npmPath: npmBin };
+  const nodeCtx: NodeContext = {
+    nodePath: nodeBin,
+    npmPath: npmBin,
+    useGlobals: !!runCfg.nodeVersion,
+  };
 
   // runCfg.path must be set for prepareNpmEnv to find node_modules. :(
   await prepareNpmEnv(runCfg, nodeCtx);

src/playwright-runner.ts

@@ -18,6 +18,8 @@ export interface RunnerConfig {
     version: string;
   };
 
+  nodeVersion?: string;
+
   // WARN: The following properties are set dynamically by the runner and are not
   // deserialized from the runner config json. They should technically be marked
   // as optional, but the runners treat them as required so type them as such.
@@ -72,6 +74,7 @@ export interface CucumberRunnerConfig {
     configFile?: string;
     version: string;
   };
+  nodeVersion?: string;
   suite: CucumberSuite;
   assetsDir: string;
   sauceReportFile: string;