{
const rmcPath = getRunMATLABCommandScriptPath(platform, architecture);
await fs.chmod(rmcPath, 0o777);
@@ -62,7 +68,7 @@ export async function runCommand(hs: HelperScript, platform: string, architectur
let execArgs = [rmcArg];
if (args) {
- execArgs = execArgs.concat(args);
+ execArgs = execArgs.concat(args);
}
const exitCode = await fn(rmcPath, execArgs);
@@ -76,10 +82,12 @@ export async function runCommand(hs: HelperScript, platform: string, architectur
*
* @param platform Operating system of the runner (e.g., "win32" or "linux")
* @param architecture Architecture of the runner (e.g., "x64")
-*/
+ */
export function getRunMATLABCommandScriptPath(platform: string, architecture: string): string {
if (architecture != "x64" && !(platform == "darwin" && architecture == "arm64")) {
- throw new Error(`This action is not supported on ${platform} runners using the ${architecture} architecture.`);
+ throw new Error(
+ `This action is not supported on ${platform} runners using the ${architecture} architecture.`,
+ );
}
let ext;
let platformDir;
@@ -101,9 +109,10 @@ export function getRunMATLABCommandScriptPath(platform: string, architecture: st
platformDir = "glnxa64";
break;
default:
- throw new Error(`This action is not supported on ${platform} runners using the ${architecture} architecture.`);
+ throw new Error(
+ `This action is not supported on ${platform} runners using the ${architecture} architecture.`,
+ );
}
const rmcPath = path.join(import.meta.dirname, "bin", platformDir, `run-matlab-command${ext}`);
return rmcPath;
-
}
diff --git a/src/matlab.unit.test.ts b/src/matlab.unit.test.ts
index 3bca266..93d891a 100644
--- a/src/matlab.unit.test.ts
+++ b/src/matlab.unit.test.ts
@@ -60,7 +60,7 @@ describe("script generation", () => {
describe("run command", () => {
const helperScript = { dir: "/home/sweet/home", command: "disp('hello, world');" };
const platform = "win32";
- const architecture = "x64"
+ const architecture = "x64";
it("ideally works", async () => {
const chmod = jest.spyOn(fs, "chmod");
@@ -80,7 +80,11 @@ describe("run command", () => {
chmod.mockResolvedValue(undefined);
execFn.mockResolvedValue(0);
- const actual = matlab.runCommand(helperScript, platform, architecture, execFn, ["-nojvm", "-logfile", "file"]);
+ const actual = matlab.runCommand(helperScript, platform, architecture, execFn, [
+ "-nojvm",
+ "-logfile",
+ "file",
+ ]);
await expect(actual).resolves.toBeUndefined();
expect(execFn.mock.calls[0][1]![1]).toBe("-nojvm");
expect(execFn.mock.calls[0][1]![2]).toBe("-logfile");
@@ -127,8 +131,8 @@ describe("run command", () => {
});
describe("ci helper path", () => {
- const platform = "linux"
- const architecture = "x64"
+ const platform = "linux";
+ const architecture = "x64";
const testExtension = (platform: string, ext: string) => {
it(`considers the appropriate script on ${platform}`, () => {
const actualPath = matlab.getRunMATLABCommandScriptPath(platform, architecture);
@@ -143,7 +147,7 @@ describe("ci helper path", () => {
expect(actualPath).toContain(subdirectory);
});
};
-
+
testExtension("win32", "exe");
testExtension("darwin", "");
testExtension("linux", "");
@@ -154,10 +158,10 @@ describe("ci helper path", () => {
testDirectory("linux", "x64", "glnxa64");
it("errors on unsupported platform", () => {
- expect(() => matlab.getRunMATLABCommandScriptPath('sunos',architecture)).toThrow();
- })
+ expect(() => matlab.getRunMATLABCommandScriptPath("sunos", architecture)).toThrow();
+ });
it("errors on unsupported architecture", () => {
- expect(() => matlab.getRunMATLABCommandScriptPath(platform, 'x86')).toThrow();
- })
+ expect(() => matlab.getRunMATLABCommandScriptPath(platform, "x86")).toThrow();
+ });
});
diff --git a/src/script.ts b/src/script.ts
index 5b3e185..78d10d7 100644
--- a/src/script.ts
+++ b/src/script.ts
@@ -10,10 +10,8 @@ import * as path from "path";
* @returns MATLAB command.
*/
export function prepare(command: string): string {
- const pluginsPath = path.join(import.meta.dirname, "plugins").replaceAll("'","''");
- return `cd(getenv('MW_ORIG_WORKING_FOLDER')); ` +
- `addpath('` + pluginsPath + `'); `
- + command;
+ const pluginsPath = path.join(import.meta.dirname, "plugins").replaceAll("'", "''");
+ return `cd(getenv('MW_ORIG_WORKING_FOLDER')); ` + `addpath('` + pluginsPath + `'); ` + command;
}
/**
diff --git a/src/script.unit.test.ts b/src/script.unit.test.ts
index 79fe7f8..1c8e4a0 100644
--- a/src/script.unit.test.ts
+++ b/src/script.unit.test.ts
@@ -8,8 +8,9 @@ describe("call generation", () => {
it("ideally works", () => {
// I know what your thinking
const testCommand = "disp('hello world')";
- const pluginsPath = path.join(import.meta.dirname, "plugins").replaceAll("'","''");
- const expectedString = `cd(getenv('MW_ORIG_WORKING_FOLDER')); addpath('` + pluginsPath + `'); ${testCommand}`;
+ const pluginsPath = path.join(import.meta.dirname, "plugins").replaceAll("'", "''");
+ const expectedString =
+ `cd(getenv('MW_ORIG_WORKING_FOLDER')); addpath('` + pluginsPath + `'); ${testCommand}`;
expect(script.prepare(testCommand)).toMatch(expectedString);
});
diff --git a/src/testResultsSummary.ts b/src/testResultsSummary.ts
index 714efa8..a835b19 100644
--- a/src/testResultsSummary.ts
+++ b/src/testResultsSummary.ts
@@ -70,7 +70,7 @@ export function processAndAddTestSummary(
) {
const testResultsData = getTestResults(runnerTemp, runId, workspace);
const coverageResultsData = getCoverageResults(runnerTemp, runId);
- if(testResultsData || coverageResultsData) {
+ if (testResultsData || coverageResultsData) {
addSummary(testResultsData, coverageResultsData, actionName);
}
}
@@ -143,8 +143,8 @@ export function addSummary(
// Add test results table if available
if (testResultsData) {
const helpLink =
- `âšī¸`;
+ `âšī¸`;
const header = getTestHeader(testResultsData.Stats);
core.summary
@@ -161,9 +161,7 @@ export function addSummary(
// Add detailed test results
if (testResultsData) {
const detailedResults = getDetailedResults(testResultsData.TestResults);
- core.summary
- .addHeading("All tests", 3)
- .addRaw(detailedResults, true);
+ core.summary.addHeading("All tests", 3).addRaw(detailedResults, true);
}
} catch (e) {
console.error("An error occurred while adding the test results to the summary:", e);
@@ -175,19 +173,39 @@ export function getTestHeader(stats: TestStatistics): string {
`
| Total tests |
- Passed ` + getStatusEmoji(MatlabTestStatus.PASSED) + ` |
- Failed ` + getStatusEmoji(MatlabTestStatus.FAILED) + ` |
- Incomplete ` + getStatusEmoji(MatlabTestStatus.INCOMPLETE) + ` |
- Not Run ` + getStatusEmoji(MatlabTestStatus.NOT_RUN) + ` |
+ Passed ` +
+ getStatusEmoji(MatlabTestStatus.PASSED) +
+ ` |
+ Failed ` +
+ getStatusEmoji(MatlabTestStatus.FAILED) +
+ ` |
+ Incomplete ` +
+ getStatusEmoji(MatlabTestStatus.INCOMPLETE) +
+ ` |
+ Not Run ` +
+ getStatusEmoji(MatlabTestStatus.NOT_RUN) +
+ ` |
Duration(s) â |
- | ` + stats.Total + ` |
- ` + stats.Passed + ` |
- ` + stats.Failed + ` |
- ` + stats.Incomplete + ` |
- ` + stats.NotRun + ` |
- ` + stats.Duration.toFixed(2) + ` |
+ ` +
+ stats.Total +
+ ` |
+ ` +
+ stats.Passed +
+ ` |
+ ` +
+ stats.Failed +
+ ` |
+ ` +
+ stats.Incomplete +
+ ` |
+ ` +
+ stats.NotRun +
+ ` |
+ ` +
+ stats.Duration.toFixed(2) +
+ ` |
`
);
@@ -200,10 +218,10 @@ export function getDetailedResults(testResults: MatlabTestFile[][]): string {
Test File |
Duration(s) |
` +
- testResults
- .flat()
- .map((file) => generateTestFileRow(file))
- .join("") +
+ testResults
+ .flat()
+ .map((file) => generateTestFileRow(file))
+ .join("") +
``
);
}
@@ -216,11 +234,17 @@ function generateTestFileRow(file: MatlabTestFile): string {
return (
`
-
+
- ` +
- statusEmoji + ` ` + file.Name +
- `
+ ` +
+ statusEmoji +
+ ` ` +
+ file.Name +
+ `
@@ -229,15 +253,15 @@ function generateTestFileRow(file: MatlabTestFile): string {
| Diagnostics |
Duration(s) |
` +
- file.TestCases.map((tc) => generateTestCaseRow(tc)).join("") +
- `
+ file.TestCases.map((tc) => generateTestCaseRow(tc)).join("") +
+ `
|
` +
- `` +
- file.Duration.toFixed(2) +
- `` +
- ` |
+ `` +
+ file.Duration.toFixed(2) +
+ `` +
+ `
`
);
}
@@ -246,26 +270,32 @@ function generateTestCaseRow(testCase: MatlabTestCase): string {
const statusEmoji = getStatusEmoji(testCase.Status);
const diagnosticsColumn =
testCase.Diagnostics.length > 0
- ? testCase.Diagnostics
- .map(
- (diagnostic) =>
- `` +
- `` +
- diagnostic.Event +
- `
` +
- `` +
- diagnostic.Report.replace(/\n/g, "
").trim() +
- `
` +
- ` `,
- )
- .join("")
+ ? testCase.Diagnostics.map(
+ (diagnostic) =>
+ `` +
+ `` +
+ diagnostic.Event +
+ `
` +
+ `` +
+ diagnostic.Report.replace(/\n/g, "
").trim() +
+ `
` +
+ ` `,
+ ).join("")
: "";
return (
`` +
- `| ` + statusEmoji + ` ` + testCase.Name + ` | ` +
- `` + diagnosticsColumn + ` | ` +
- `` + testCase.Duration.toFixed(2) + ` | ` +
+ `` +
+ statusEmoji +
+ ` ` +
+ testCase.Name +
+ ` | ` +
+ `` +
+ diagnosticsColumn +
+ ` | ` +
+ `` +
+ testCase.Duration.toFixed(2) +
+ ` | ` +
`
`
);
}
@@ -337,7 +367,9 @@ function determineTestStatus(testResult: MatlabTestResultJson): MatlabTestStatus
}
}
-function processDiagnostics(diagnostics: MatlabTestDiagnostics | MatlabTestDiagnostics[] | undefined): MatlabTestDiagnostics[] {
+function processDiagnostics(
+ diagnostics: MatlabTestDiagnostics | MatlabTestDiagnostics[] | undefined,
+): MatlabTestDiagnostics[] {
if (!diagnostics) return [];
return Array.isArray(diagnostics) ? diagnostics : [diagnostics];
diff --git a/src/testResultsSummary.unit.test.ts b/src/testResultsSummary.unit.test.ts
index 700f57d..73fddf1 100644
--- a/src/testResultsSummary.unit.test.ts
+++ b/src/testResultsSummary.unit.test.ts
@@ -1,6 +1,6 @@
// Copyright 2025-2026 The MathWorks, Inc.
-import {jest, describe, it, expect, beforeAll} from "@jest/globals";
+import { jest, describe, it, expect, beforeAll } from "@jest/globals";
import * as path from "path";
import * as os from "os";
import * as nodeFs from "fs";
@@ -29,7 +29,7 @@ jest.unstable_mockModule("fs", () => ({
const core = await import("@actions/core");
const fs = await import("fs");
const testResultsSummary = await import("./testResultsSummary.js");
-const {MatlabTestStatus} = testResultsSummary;
+const { MatlabTestStatus } = testResultsSummary;
describe("Artifact Processing Tests", () => {
// Shared test data
@@ -59,11 +59,17 @@ describe("Artifact Processing Tests", () => {
return { osName: "windows", workspaceParent: "C:\\" };
if (platform.includes("linux") || platform.includes("unix") || platform.includes("aix"))
return { osName: "linux", workspaceParent: "/home/user/" };
- if (platform.includes("darwin")) return { osName: "mac", workspaceParent: "/Users/username/" };
+ if (platform.includes("darwin"))
+ return { osName: "mac", workspaceParent: "/Users/username/" };
throw new Error(`Unsupported OS: ${platform}`);
}
- function copyTestDataFile(osName: string, runnerTemp: string, runId: string, actionName: string) {
+ function copyTestDataFile(
+ osName: string,
+ runnerTemp: string,
+ runId: string,
+ actionName: string,
+ ) {
const sourceFilePath = path.join(
import.meta.dirname,
"test-data",
@@ -120,18 +126,10 @@ describe("Artifact Processing Tests", () => {
expect(testResults[0][0].TestCases[8].Name).toBe("testInvalidDateFormat");
expect(testResults[1][0].TestCases[0].Name).toBe("testNonLeapYear");
- expect(testResults[0][0].TestCases[0].Status).toBe(
- MatlabTestStatus.PASSED,
- );
- expect(testResults[0][0].TestCases[4].Status).toBe(
- MatlabTestStatus.FAILED,
- );
- expect(testResults[0][0].TestCases[8].Status).toBe(
- MatlabTestStatus.NOT_RUN,
- );
- expect(testResults[1][0].TestCases[0].Status).toBe(
- MatlabTestStatus.INCOMPLETE,
- );
+ expect(testResults[0][0].TestCases[0].Status).toBe(MatlabTestStatus.PASSED);
+ expect(testResults[0][0].TestCases[4].Status).toBe(MatlabTestStatus.FAILED);
+ expect(testResults[0][0].TestCases[8].Status).toBe(MatlabTestStatus.NOT_RUN);
+ expect(testResults[1][0].TestCases[0].Status).toBe(MatlabTestStatus.INCOMPLETE);
expect(testResults[0][0].TestCases[0].Duration).toBeCloseTo(0.1);
expect(testResults[0][0].TestCases[1].Duration).toBeCloseTo(0.11);
@@ -382,7 +380,11 @@ describe("Error Handling Tests", () => {
fs.writeFileSync(invalidJsonPath, "{ invalid json content");
try {
- const result = testResultsSummary.getTestResults(process.env.RUNNER_TEMP, process.env.GITHUB_RUN_ID, "");
+ const result = testResultsSummary.getTestResults(
+ process.env.RUNNER_TEMP,
+ process.env.GITHUB_RUN_ID,
+ "",
+ );
expect(result).toBeNull();
// Verify error was logged
@@ -419,9 +421,13 @@ describe("Error Handling Tests", () => {
fs.writeFileSync(validJsonPath, "[]"); // Empty array - valid JSON
try {
- const result = testResultsSummary.getTestResults(process.env.RUNNER_TEMP, process.env.GITHUB_RUN_ID, "");
+ const result = testResultsSummary.getTestResults(
+ process.env.RUNNER_TEMP,
+ process.env.GITHUB_RUN_ID,
+ "",
+ );
- if(result){
+ if (result) {
// Should still return results even if deletion fails
expect(result).toBeDefined();
expect(result.TestResults).toEqual([]);
@@ -450,4 +456,4 @@ describe("Error Handling Tests", () => {
}
}
});
-});
\ No newline at end of file
+});