V3.0 release (#39)

* v3.0: Upgrade to Node 24 and modernize codebase

* update actions versions referenced in README

* major version numbers

Author: danjhd

.eslintrc.json

@@ -2,19 +2,41 @@
 
 All notable changes to this project will be documented in this file.
 
+## 3.0 (2026-03-24)
+
+> **_BREAKING CHANGES_**
+>
+> Action runtime upgraded from `node20` to `node24`. This requires GitHub Actions runners with Node.js 24 support.
+
+### Features
+
+* Upgraded action runtime from `node20` to `node24`
+
+### Maintenance
+
+* Modernized codebase to ES modules (internal change)
+* Upgraded all dependencies to latest versions
+  * `@actions/core` to v3.0.0
+  * `@actions/github` to v9.0.0
+  * `@aws-sdk/client-device-farm` to v3.1015.0
+  * `axios` to v1.13.6
+* Removed `fast-xml-parser` security override (resolved in dependency updates)
+* Switched build toolchain from `@vercel/ncc` to `rollup` (following GitHub's official documentation)
+* Migrated test framework from Jest to Vitest for native ES modules support
+* Updated ESLint to v10 with new flat config format
+* All tests passing with 100% code coverage
+
 ## 2.3 (2024-08-14)
 
 > Fixed dependabot security finding [#7](https://github.com/aws-actions/aws-devicefarm-mobile-device-testing/security/dependabot/7).
 
-
 ## 2.2 (2024-07-30)
 
 > Fixed dependabot security finding [#6](https://github.com/aws-actions/aws-devicefarm-mobile-device-testing/security/dependabot/6).
 
-
 ## 2.1 (2024-06-04)
 
-> Fixed bug identified in issue [#8](#8) where `configuration` section in `run-settings-json` could not be excluded.
+> Fixed bug identified in issue [#8](https://github.com/aws-actions/aws-devicefarm-mobile-device-testing/issues/8) where `configuration` section in `run-settings-json` could not be excluded.
 
 ## 2.0 (2023-10-13)
 

CHANGELOG.md

@@ -60,10 +60,10 @@ For example, testing new versions of code being committed to a branch to ensure
 
 ```yaml
       - name: Checkout repo
-        uses: actions/checkout@v3
+        uses: actions/checkout@v6
 
       - name: Configure AWS credentials
-        uses: aws-actions/configure-aws-credentials@v2 # More information on this action can be found below in the 'AWS Credentials' section
+        uses: aws-actions/configure-aws-credentials@v6 # More information on this action can be found below in the 'AWS Credentials' section
         with:
           role-to-assume: arn:aws:iam::123456789012:role/my-github-actions-role
           aws-region: us-west-2
@@ -74,7 +74,7 @@ For example, testing new versions of code being committed to a branch to ensure
 ```yaml
       - name: Schedule Device Farm Automated Test
         id: run-test
-        uses: aws-actions/aws-devicefarm-mobile-device-testing@v2.0
+        uses: aws-actions/aws-devicefarm-mobile-device-testing@v3
         with:
           run-settings-json: |
             {
@@ -90,7 +90,7 @@ For example, testing new versions of code being committed to a branch to ensure
             }
           artifact-types: ALL
 
-      - uses: actions/upload-artifact@v3
+      - uses: actions/upload-artifact@v7
         if: always() # This ensures the artifacts are uploaded even if the Test Run Fails
         with:
           name: AutomatedTestOutputFiles
@@ -102,7 +102,7 @@ For example, testing new versions of code being committed to a branch to ensure
 ```yaml
       - name: Schedule Device Farm Automated Test
         id: run-test
-        uses: aws-actions/aws-devicefarm-mobile-device-testing@v2.0
+        uses: aws-actions/aws-devicefarm-mobile-device-testing@v3
         with:
           run-settings-json: |
             {
@@ -118,7 +118,7 @@ For example, testing new versions of code being committed to a branch to ensure
             }
           artifact-types: VIDEO,SCREENSHOT
 
-      - uses: actions/upload-artifact@v3
+      - uses: actions/upload-artifact@v7
         if: always() # This ensures the artifacts are uploaded even if the Test Run Fails
         with:
           name: AutomatedTestOutputFiles
@@ -130,7 +130,7 @@ For example, testing new versions of code being committed to a branch to ensure
 ```yaml
       - name: Schedule Device Farm Automated Test
         id: run-test
-        uses: aws-actions/aws-devicefarm-mobile-device-testing@v2.0
+        uses: aws-actions/aws-devicefarm-mobile-device-testing@v3
         with:
           run-settings-json: |
             {
@@ -156,14 +156,14 @@ This action relies on the [default behaviour of the AWS SDK for JavaScript](http
 
 ```yaml
       - name: Configure AWS credentials
-        uses: aws-actions/configure-aws-credentials@v2
+        uses: aws-actions/configure-aws-credentials@v6
         with:
           role-to-assume: arn:aws:iam::123456789012:role/my-github-actions-role
           aws-region: us-west-2
 
       - name: Schedule Device Farm Automated Test
         id: run-test
-        uses: aws-actions/aws-devicefarm-mobile-device-testing@v2.0
+        uses: aws-actions/aws-devicefarm-mobile-device-testing@v3
         with:
           run-settings-json: |
             {

README.md

@@ -52,6 +52,6 @@ outputs:
   console-url:
     description: 'The AWS Console URL for the test run'
 runs:
-  using: 'node20'
+  using: 'node24'
   main: 'dist/index.js'
   post: 'dist/cleanup/index.js'

action.yml

@@ -1,13 +1,13 @@
-const {
+import {
     DeviceFarmClient,
     paginateListUploads,
     DeleteUploadCommand
-} = require("@aws-sdk/client-device-farm");
-const core = require("@actions/core");
-const github = require("@actions/github");
-const fs = require("fs/promises");
-const { existsSync } = require("fs");
-const { UPLOAD } = require("./constants");
+} from "@aws-sdk/client-device-farm";
+import * as core from "@actions/core";
+import * as github from "@actions/github";
+import fs from "fs/promises";
+import { existsSync } from "fs";
+import { UPLOAD } from "./constants.js";
 
 const deviceFarm = new DeviceFarmClient();
 
@@ -39,11 +39,11 @@ async function cleanup() {
     }
 }
 
-module.exports = {
+export {
     cleanup
 };
 
 /* istanbul ignore next */
-if (require.main === module) {
+if (import.meta.url === `file://${process.argv[1]}`) {
     cleanup();
 }

cleanup.js

@@ -1,24 +1,25 @@
-const { cleanup } = require("./cleanup.js");
-const {
+import { vi } from "vitest";
+import { cleanup } from "./cleanup.js";
+import {
     DeviceFarmClient,
     ListUploadsCommand,
     DeleteUploadCommand
-} = require("@aws-sdk/client-device-farm");
-const core = require("@actions/core");
-const github = require("@actions/github");
-const fs = require("fs/promises");
-const { UPLOAD } = require("./constants");
-const { mockClient } = require("aws-sdk-client-mock");
-require("aws-sdk-client-mock-jest");
-const mock = require("mock-fs");
-
-jest.mock("@actions/core");
-jest.mock("@actions/github", () => ({
+} from "@aws-sdk/client-device-farm";
+import * as core from "@actions/core";
+import * as github from "@actions/github";
+import fs from "fs/promises";
+import { UPLOAD } from "./constants.js";
+import { mockClient } from "aws-sdk-client-mock";
+import "aws-sdk-client-mock-vitest";
+import mock from "mock-fs";
+
+vi.mock("@actions/core");
+vi.mock("@actions/github", () => ({
     context: {
         runId: 1
     }
 }));
-jest.mock("fs/promises");
+vi.mock("fs/promises");
 
 function mockGetState(requestResponse) {
     return function (name, options) { // eslint-disable-line no-unused-vars
@@ -36,8 +37,8 @@ const mockDeviceFarm = mockClient(DeviceFarmClient);
 describe("Cleanup", () => {
 
     beforeEach(() => {
-        jest.clearAllMocks();
-        core.getState = jest.fn().mockImplementation(mockGetState(STATES));
+        vi.clearAllMocks();
+        core.getState = vi.fn().mockImplementation(mockGetState(STATES));
         mockDeviceFarm.reset();
     });
 
@@ -86,10 +87,10 @@ describe("Cleanup", () => {
         await cleanup();
 
         expect(github.context.runId).toBe(1);
-        expect(mockDeviceFarm).toHaveReceivedNthSpecificCommandWith(1, ListUploadsCommand, {
+        expect(mockDeviceFarm).toHaveReceivedCommandWith(ListUploadsCommand, {
             arn: "fake-project-arn"
         });
-        expect(mockDeviceFarm).toHaveReceivedNthSpecificCommandWith(1, DeleteUploadCommand, {
+        expect(mockDeviceFarm).toHaveReceivedCommandWith(DeleteUploadCommand, {
             arn: "fake-upload-arn-1"
         });
         expect(core.setFailed).toHaveBeenCalledTimes(0);

cleanup.test.js

@@ -1,19 +1,19 @@
-const INPUTS = {
+export const INPUTS = {
     runSettingsJson: "run-settings-json",
     artifactTypes: "artifact-types",
     uploadPollInterval: "upload-poll-interval",
     runPollInterval: "run-poll-interval",
 };
 
-const OUTPUTS = {
+export const OUTPUTS = {
     arn: "arn",
     status: "status",
     result: "result",
     artifactFolder: "artifact-folder",
     consoleUrl: "console-url",
 };
 
-const UPLOAD = {
+export const UPLOAD = {
     STATUS: {
         INITIALIZED: "INITIALIZED",
         PROCESSING: "PROCESSING",
@@ -26,7 +26,7 @@ const UPLOAD = {
     },
 };
 
-const RUN = {
+export const RUN = {
     STATUS: {
         PENDING: "PENDING",
         PENDING_CONCURRENCY: "PENDING_CONCURRENCY",
@@ -53,10 +53,3 @@ const RUN = {
         "LOG",
     ]
 };
-
-module.exports = {
-    INPUTS,
-    OUTPUTS,
-    UPLOAD,
-    RUN,
-};