.eslintignore

@@ -0,0 +1 @@
+/bin

.eslintrc

@@ -0,0 +1,15 @@
+{
+    "root": true,
+    "env": {
+        "es2022": true,
+        "node": true,
+        "jest/globals": true
+    },
+    "extends": [
+        "eslint:recommended",
+        "plugin:prettier/recommended"
+    ],
+    "plugins": [
+        "jest"
+    ]
+}

.github/workflows/release.yml

@@ -1,6 +1,5 @@
-# This workflow will do a clean install of node dependencies, build the source code and run tests across different versions of node
-# For more information see: https://help.github.com/actions/language-and-framework-guides/using-nodejs-with-github-actions
-name: saucectl pipeline
+# This workflow is triggered by https://github.com/saucelabs/saucectl on release.
+name: Release
 
 on: [repository_dispatch]
 
@@ -13,32 +12,43 @@ jobs:
     steps:
       - name: Event Information
         run: echo "Event '${{ github.event.action }}' received from '${{ github.event.client_payload.version }}'"
-      - uses: actions/checkout@v3
-      - uses: actions/setup-node@v3
+
+      - name: Checkout
+        uses: actions/checkout@v4
+
+      - name: Setup Node
+        uses: actions/setup-node@v3
         with:
-          node-version: 16
-      - run: npm install
+          node-version-file: '.nvmrc'
+
+      - name: Install Dependencies
+        run: npm install
+
       - name: NPM Setup
         run: |
           npm set registry "https://registry.npmjs.org/"
           npm set //registry.npmjs.org/:_authToken $NPM_TOKEN
           npm whoami
+
       - name: Git Setup
         run: |
           git config --global user.name "oss-sauce-bot"
           git config --global user.email "66365630+oss-sauce-bot@users.noreply.github.com"
       - if: github.event.client_payload.version != ''
         run: npx np ${{ github.event.client_payload.version }}
+
   post-deploy-install:
     needs: ["deploy"]
     if: github.event.client_payload.version != ''
     runs-on: ubuntu-latest
     steps:
       - name: Event Information
         run: echo "Event '${{ github.event.action }}' received from '${{ github.event.client_payload.version }}'"
+
       - name: Setup Node
         uses: actions/setup-node@v3
         with:
-          node-version: 16
+          node-version-file: '.nvmrc'
+
       - name: Install saucectl
         run: npm install saucectl@${{ github.event.client_payload.version }}

.github/workflows/test.yml

@@ -1,6 +1,4 @@
-# This workflow will do a clean install of node dependencies, build the source code and run tests across different versions of node
-# For more information see: https://help.github.com/actions/language-and-framework-guides/using-nodejs-with-github-actions
-name: saucectl pipeline
+name: Test
 
 on:
   pull_request:
@@ -9,24 +7,24 @@ on:
       - main
 
 jobs:
-  deploy:
+  test:
     runs-on: ubuntu-latest
     steps:
-      - uses: actions/checkout@v3
-      - uses: actions/setup-node@v3
-        with:
-          node-version: 16
-      - run: npm install
-      - run: npm test
+      - name: Checkout
+        uses: actions/checkout@v4
 
-  deploy-with-github_token:
-    runs-on: ubuntu-latest
-    steps:
-      - uses: actions/checkout@v3
-      - uses: actions/setup-node@v3
+      - name: Setup Node
+        uses: actions/setup-node@v3
         with:
-          node-version: 16
-      - run: npm install
+          node-version-file: '.nvmrc'
+
+      - name: Install Dependencies
+        run: npm install
         env:
           GITHUB_TOKEN: ${{ github.token }}
-      - run: npm test
+
+      - name: Lint
+        run: npm run lint
+
+      - name: Test
+        run: npm test

.nvmrc

@@ -1 +1 @@
-v14.1.0
+v20

.prettierignore

@@ -0,0 +1 @@
+/lib

.prettierrc

@@ -0,0 +1,3 @@
+{
+  "singleQuote": true
+}

README.md

@@ -1,7 +1,7 @@
 Node.js saucectl Wrapper ![build](https://github.com/saucelabs/node-saucectl/workflows/saucectl%20pipeline/badge.svg)
 ========================
 
-Node.js wrapper for saucectl: Sauce Labs Testrunner Toolkit.
+Node.js wrapper for [saucectl](https://github.com/saucelabs/saucectl).
 
 ## Install
 

__tests__/main.test.js

@@ -1,4 +1,4 @@
-jest.mock('child_process')
+jest.mock('child_process');
 const main = require('../');
 const childProcess = require('child_process');
 const { EventEmitter } = require('events');
@@ -13,87 +13,101 @@ const originalOs = process.platform;
 const originalArch = process.arch;
 
 describe('main', function () {
-	let exitSpy, mockSpawnEventEmitter;
-	beforeAll(function () {
-		childProcess.spawn.mockImplementation(() => {
-			mockSpawnEventEmitter = new EventEmitter();
-			return mockSpawnEventEmitter;
-		});
-		exitSpy = jest.spyOn(process, 'exit');
-		exitSpy.mockImplementation(() => { });
-	});
+  let exitSpy, mockSpawnEventEmitter;
+  beforeAll(function () {
+    childProcess.spawn.mockImplementation(() => {
+      mockSpawnEventEmitter = new EventEmitter();
+      return mockSpawnEventEmitter;
+    });
+    exitSpy = jest.spyOn(process, 'exit');
+    exitSpy.mockImplementation(() => {});
+  });
 
-	beforeEach(() => {
-		BinWrapper.mockClear();
+  beforeEach(() => {
+    BinWrapper.mockClear();
 
-		Object.defineProperty(process, 'platform', { value: 'darwin' });
-		Object.defineProperty(process, 'arch', { value: 'arm64' });
+    Object.defineProperty(process, 'platform', { value: 'darwin' });
+    Object.defineProperty(process, 'arch', { value: 'arm64' });
 
-		mockSrc = jest.fn();
+    mockSrc = jest.fn();
 
-		BinWrapper.mockReturnValue({
-			use: jest.fn(),
-			dest: jest.fn(),
-			src: mockSrc,
-			run: jest.fn(),
-		});
-	});
-	afterEach(() => {
-		jest.clearAllMocks();
+    BinWrapper.mockReturnValue({
+      use: jest.fn(),
+      dest: jest.fn(),
+      src: mockSrc,
+      run: jest.fn(),
+    });
+  });
+  afterEach(() => {
+    jest.clearAllMocks();
 
-		Object.defineProperty(process, 'platform', { value: originalOs });
-		Object.defineProperty(process, 'arch', { value: originalArch });
-	});
+    Object.defineProperty(process, 'platform', { value: originalOs });
+    Object.defineProperty(process, 'arch', { value: originalArch });
+  });
 
-	test('should run the executable', async function () {
-		const bin = {
-			run: jest.fn().mockReturnValue(Promise.resolve(0)),
-			path: jest.fn().mockReturnValue('/bin/saucectl')
-		}
-		await main(bin, ['bar', '--foo'])
-		expect(bin.run).toBeCalledTimes(1)
-		expect(childProcess.spawn).toBeCalledWith('/bin/saucectl', ['bar', '--foo'], expect.any(Object))
-		mockSpawnEventEmitter.emit('exit', 0);
-		expect(exitSpy.mock.calls).toEqual([[0]]);
-	});
-	test('should display carry the exitCode', async function () {
-		const bin = {
-			run: jest.fn().mockReturnValue(Promise.resolve(1)),
-			path: jest.fn().mockReturnValue('/bin/saucectl')
-		}
-		await main(bin, ['bar', '--foo'])
-		expect(bin.run).toBeCalledTimes(1)
-		expect(childProcess.spawn).toBeCalledWith('/bin/saucectl', ['bar', '--foo'], expect.any(Object))
-		expect(exitSpy.mock.calls).toEqual([[1]]);
-	});
-	describe('.binWrapper', function () {
-		test('should create a binary wrapper', async function () {
-			await main.binWrapper();
+  test('should run the executable', async function () {
+    const bin = {
+      run: jest.fn().mockReturnValue(Promise.resolve(0)),
+      path: jest.fn().mockReturnValue('/bin/saucectl'),
+    };
+    await main(bin, ['bar', '--foo']);
+    expect(bin.run).toHaveBeenCalledTimes(1);
+    expect(childProcess.spawn).toHaveBeenCalledWith(
+      '/bin/saucectl',
+      ['bar', '--foo'],
+      expect.any(Object),
+    );
+    mockSpawnEventEmitter.emit('exit', 0);
+    expect(exitSpy.mock.calls).toEqual([[0]]);
+  });
+  test('should display carry the exitCode', async function () {
+    const bin = {
+      run: jest.fn().mockReturnValue(Promise.resolve(1)),
+      path: jest.fn().mockReturnValue('/bin/saucectl'),
+    };
+    await main(bin, ['bar', '--foo']);
+    expect(bin.run).toHaveBeenCalledTimes(1);
+    expect(childProcess.spawn).toHaveBeenCalledWith(
+      '/bin/saucectl',
+      ['bar', '--foo'],
+      expect.any(Object),
+    );
+    expect(exitSpy.mock.calls).toEqual([[1]]);
+  });
+  describe('.binWrapper', function () {
+    test('should create a binary wrapper', async function () {
+      await main.binWrapper();
 
-			expect(mockSrc.mock.calls).toEqual([
-				[
-					'https://github.com/saucelabs/saucectl/releases/download/v{{version}}/saucectl_{{version}}_mac_arm64.tar.gz'.replaceAll('{{version}}', packageJson.version),
-					'darwin',
-					'arm64',
-				]
-			]);
-		});
-		test('should respect the SAUCECTL_INSTALL_BINARY wrapper', async function () {
-			await main.binWrapper('http://some-fake-url');
+      expect(mockSrc.mock.calls).toEqual([
+        [
+          'https://github.com/saucelabs/saucectl/releases/download/v{{version}}/saucectl_{{version}}_mac_arm64.tar.gz'.replaceAll(
+            '{{version}}',
+            packageJson.version,
+          ),
+          'darwin',
+          'arm64',
+        ],
+      ]);
+    });
+    test('should respect the SAUCECTL_INSTALL_BINARY wrapper', async function () {
+      await main.binWrapper('http://some-fake-url');
 
-			expect(mockSrc).toMatchSnapshot();
-		});
+      expect(mockSrc).toMatchSnapshot();
+    });
 
-		test('should respect the SAUCECTL_INSTALL_BINARY_MIRROR wrapper', async function () {
-			await main.binWrapper(null, 'https://some-fake-mirror-url');
+    test('should respect the SAUCECTL_INSTALL_BINARY_MIRROR wrapper', async function () {
+      await main.binWrapper(null, 'https://some-fake-mirror-url');
 
-			expect(mockSrc.mock.calls).toEqual([
-				[
-					'https://some-fake-mirror-url/v{{version}}/saucectl_{{version}}_mac_arm64.tar.gz'.replaceAll('{{version}}', packageJson.version),
-					'darwin',
-					'arm64',
-				]
-			]);
-		});
-	});
+      expect(mockSrc.mock.calls).toEqual([
+        [
+          'https://some-fake-mirror-url/v{{version}}/saucectl_{{version}}_mac_arm64.tar.gz'.replaceAll(
+            '{{version}}',
+            packageJson.version,
+          ),
+          'darwin',
+          'arm64',
+        ],
+      ]);
+    });
+  });
 });