update artifact name

Author: mw-kapilg

.github/workflows/publish.yml

@@ -29,6 +29,7 @@ jobs:
           VERSION=$(npm version "${{ github.event.release.tag_name }}" --no-git-tag-version)
           git add package.json package-lock.json
           git commit -m "[skip ci] Bump $VERSION"
+          git pull --rebase origin main
           git push origin HEAD:main
           
       # Build and test

plugins/+ciplugins/+github/TestResultsSummaryPlugin.m

@@ -13,7 +13,7 @@ function reportFinalizedSuite(plugin, pluginData)
             end
 
             % If test results artifact exists, update the same file
-            testArtifactFile = fullfile(getenv("RUNNER_TEMP"), "matlabTestResults" + getenv("GITHUB_RUN_ID") + ".json");
+            testArtifactFile = fullfile(getenv("RUNNER_TEMP"), "matlabTestResults_" + getenv("GITHUB_RUN_ID") + "_" + getenv("GITHUB_ACTION") + ".json");
             if isfile(testArtifactFile)
                 testResults = {jsondecode(fileread(testArtifactFile))};
             else

src/testResultsSummary.ts

@@ -199,6 +199,7 @@ export function getStatusEmoji(status: MatlabTestStatus): string {
 export function getTestResults(
     runnerTemp: string,
     runId: string,
+    actionName: string,
     workspace: string,
 ): TestResultsData {
     const testResults: MatlabTestFile[][] = [];
@@ -214,7 +215,7 @@ export function getTestResults(
         TestResults: testResults,
         Stats: stats,
     };
-    const resultsPath = path.join(runnerTemp, `matlabTestResults${runId}.json`);
+    const resultsPath = path.join(runnerTemp, `matlabTestResults_${runId}_${actionName}.json`);
 
     if (existsSync(resultsPath)) {
         try {

src/testResultsSummary.unit.test.ts

@@ -27,13 +27,14 @@ describe("Artifact Processing Tests", () => {
 
     beforeAll(() => {
         const runnerTemp = path.join(__dirname, "..");
-        const runId = "";
+        const runId = "123";
+        const actionName = "run-tests";
         const osInfo = getOSInfo();
         const workspace = path.join(osInfo.workspaceParent, "workspace");
 
-        copyTestDataFile(osInfo.osName, runnerTemp);
+        copyTestDataFile(osInfo.osName, runnerTemp, runId, actionName);
 
-        testResultsData = testResultsSummary.getTestResults(runnerTemp, runId, workspace);
+        testResultsData = testResultsSummary.getTestResults(runnerTemp, runId, actionName, workspace);
         testResults = testResultsData.TestResults;
         stats = testResultsData.Stats;
     });
@@ -48,7 +49,7 @@ describe("Artifact Processing Tests", () => {
         throw new Error(`Unsupported OS: ${os}`);
     }
 
-    function copyTestDataFile(osName: string, runnerTemp: string) {
+    function copyTestDataFile(osName: string, runnerTemp: string, runId: string, actionName: string) {
         const sourceFilePath = path.join(
             __dirname,
             "test-data",
@@ -57,7 +58,7 @@ describe("Artifact Processing Tests", () => {
             osName,
             "matlabTestResults.json",
         );
-        const destinationFilePath = path.join(runnerTemp!, "matlabTestResults.json");
+        const destinationFilePath = path.join(runnerTemp, "matlabTestResults_" + runId + "_" + actionName + ".json");
 
         try {
             fs.copyFileSync(sourceFilePath, destinationFilePath);
@@ -358,14 +359,15 @@ describe("Error Handling Tests", () => {
 
         // Set up environment variables
         process.env.RUNNER_TEMP = path.join(__dirname, "..");
-        process.env.GITHUB_RUN_ID = "";
+        process.env.GITHUB_RUN_ID = "123";
+        process.env.GITHUB_ACTION = "run-tests";
 
         // Create a file with invalid JSON
-        const invalidJsonPath = path.join(process.env.RUNNER_TEMP, "matlabTestResults.json");
+        const invalidJsonPath = path.join(process.env.RUNNER_TEMP, "matlabTestResults_123_run-tests.json");
         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, process.env.GITHUB_ACTION, "");
 
             // Should return empty results
             expect(result.TestResults).toEqual([]);
@@ -407,14 +409,15 @@ describe("Error Handling Tests", () => {
 
         // Set up environment variables
         process.env.RUNNER_TEMP = path.join(__dirname, "..");
-        process.env.GITHUB_RUN_ID = "";
+        process.env.GITHUB_RUN_ID = "123";
+        process.env.GITHUB_ACTION = "run-tests";
 
         // Create a valid JSON file
-        const validJsonPath = path.join(process.env.RUNNER_TEMP, "matlabTestResults.json");
+        const validJsonPath = path.join(process.env.RUNNER_TEMP, "matlabTestResults_123_run-tests.json");
         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, process.env.GITHUB_ACTION, "");
 
             // Should still return results even if deletion fails
             expect(result).toBeDefined();