update artifact name

Author: mw-kapilg

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") + "_" + getenv("GITHUB_ACTION") + ".json");
+            testArtifactFile = fullfile(getenv("RUNNER_TEMP"), "matlabTestResults" + getenv("GITHUB_RUN_ID") + ".json");
             if isfile(testArtifactFile)
                 testResults = {jsondecode(fileread(testArtifactFile))};
             else

src/testResultsSummary.ts

@@ -199,7 +199,6 @@ export function getStatusEmoji(status: MatlabTestStatus): string {
 export function getTestResults(
     runnerTemp: string,
     runId: string,
-    actionName: string,
     workspace: string,
 ): TestResultsData {
     const testResults: MatlabTestFile[][] = [];
@@ -215,20 +214,7 @@ export function getTestResults(
         TestResults: testResults,
         Stats: stats,
     };
-    const resultsPath = path.join(runnerTemp, `matlabTestResults_${runId}_${actionName}.json`);
-
-    try {
-        const fs = require('fs');
-        core.info(`Contents of runnerTemp directory (${runnerTemp}):`);
-        const files = fs.readdirSync(runnerTemp);
-        files.forEach((file: string) => {
-            const filePath = path.join(runnerTemp, file);
-            const stats = fs.statSync(filePath);
-            core.info(`- ${file} (${stats.isDirectory() ? 'directory' : 'file'}, ${stats.size} bytes)`);
-        });
-    } catch (listError) {
-        core.info(`Error listing files in runnerTemp directory: ${listError}`);
-    }
+    const resultsPath = path.join(runnerTemp, `matlabTestResults${runId}.json`);
 
     if (existsSync(resultsPath)) {
         try {
@@ -263,8 +249,6 @@ export function getTestResults(
                 );
             }
         }
-    } else {
-        console.warn(`Test results summary file ${resultsPath} does not exist.`);
     }
 
     return testResultsData;

src/testResultsSummary.unit.test.ts

@@ -34,7 +34,7 @@ describe("Artifact Processing Tests", () => {
 
         copyTestDataFile(osInfo.osName, runnerTemp, runId, actionName);
 
-        testResultsData = testResultsSummary.getTestResults(runnerTemp, runId, actionName, workspace);
+        testResultsData = testResultsSummary.getTestResults(runnerTemp, runId, workspace);
         testResults = testResultsData.TestResults;
         stats = testResultsData.Stats;
     });
@@ -58,7 +58,7 @@ describe("Artifact Processing Tests", () => {
             osName,
             "matlabTestResults.json",
         );
-        const destinationFilePath = path.join(runnerTemp, "matlabTestResults_" + runId + "_" + actionName + ".json");
+        const destinationFilePath = path.join(runnerTemp, "matlabTestResults" + runId + ".json");
 
         try {
             fs.copyFileSync(sourceFilePath, destinationFilePath);
@@ -363,7 +363,7 @@ describe("Error Handling Tests", () => {
         process.env.GITHUB_ACTION = "run-tests";
 
         // Create a file with invalid JSON
-        const invalidJsonPath = path.join(process.env.RUNNER_TEMP, "matlabTestResults_123_run-tests.json");
+        const invalidJsonPath = path.join(process.env.RUNNER_TEMP, "matlabTestResults123.json");
         fs.writeFileSync(invalidJsonPath, "{ invalid json content");
 
         try {
@@ -413,7 +413,7 @@ describe("Error Handling Tests", () => {
         process.env.GITHUB_ACTION = "run-tests";
 
         // Create a valid JSON file
-        const validJsonPath = path.join(process.env.RUNNER_TEMP, "matlabTestResults_123_run-tests.json");
+        const validJsonPath = path.join(process.env.RUNNER_TEMP, "matlabTestResults123.json");
         fs.writeFileSync(validJsonPath, "[]"); // Empty array - valid JSON
 
         try {