update summary writing method

Author: mw-kapilg

src/buildSummary.ts

@@ -3,7 +3,7 @@ import * as core from "@actions/core";
 import { join } from 'path';
 import { readFileSync, unlinkSync, existsSync } from 'fs';
 
-export function writeSummary(taskSummaryTableRows: string[][], actionName: string) {
+export function addSummary(taskSummaryTableRows: string[][], actionName: string) {
     try {
         core.summary
             .addHeading("MATLAB Build Results (" + actionName + ") ")
@@ -40,7 +40,7 @@ export function interpretSkipReason(skipReason: string){
     }
 }
 
-export function processAndDisplayBuildSummary(
+export function processAndAddBuildSummary(
     runnerTemp: string,
     runId: string,
     actionName: string
@@ -64,6 +64,6 @@ export function processAndDisplayBuildSummary(
                 console.error(`An error occurred while trying to delete the build summary file ${filePath}:`, e);
             }
         }
-        writeSummary(taskSummaryTable, actionName);
+        addSummary(taskSummaryTable, actionName);
     }
 }

src/testResultsSummary.ts

@@ -61,15 +61,15 @@ export interface TestResultsData {
     Stats: TestStatistics;
 }
 
-export function processAndDisplayTestSummary(
+export function processAndAddTestSummary(
     runnerTemp: string,
     runId: string,
     actionName: string,
     workspace: string,
 ) {
     const testResultsData = getTestResults(runnerTemp, runId, workspace);
     if(testResultsData) {
-        writeSummary(testResultsData, actionName);
+        addSummary(testResultsData, actionName);
     }
 }
 
@@ -132,7 +132,7 @@ export function getTestResults(
     return testResultsData;
 }
 
-export function writeSummary(
+export function addSummary(
     testResultsData: TestResultsData,
     actionName: string,
 ) {

src/testResultsSummary.unit.test.ts

@@ -10,7 +10,6 @@ jest.mock("@actions/core", () => ({
     summary: {
         addHeading: jest.fn().mockReturnThis(),
         addRaw: jest.fn().mockReturnThis(),
-        write: jest.fn().mockReturnThis(),
     },
 }));
 
@@ -141,7 +140,7 @@ describe("Artifact Processing Tests", () => {
     it("should write test results data to the GitHub job summary", () => {
         if (testResultsData) {
             const actionName = process.env.GITHUB_ACTION || "";
-            testResultsSummary.writeSummary(testResultsData, actionName);
+            testResultsSummary.addSummary(testResultsData, actionName);
 
             expect(core.summary.addHeading).toHaveBeenCalledTimes(2);
             expect(core.summary.addHeading).toHaveBeenNthCalledWith(
@@ -165,7 +164,6 @@ describe("Artifact Processing Tests", () => {
             expect(core.summary.addHeading).toHaveBeenNthCalledWith(2, "All tests", 3);
 
             expect(core.summary.addRaw).toHaveBeenCalledTimes(2);
-            expect(core.summary.write).toHaveBeenCalledTimes(1);
         }
     });
 });
@@ -322,7 +320,7 @@ describe("HTML Structure Tests", () => {
 });
 
 describe("Error Handling Tests", () => {
-    it("should handle errors gracefully in writeSummary", () => {
+    it("should handle errors gracefully in addSummary", () => {
         const consoleSpy = jest.spyOn(console, "error").mockImplementation(() => {});
 
         // Mock addHeading to throw an error
@@ -346,7 +344,7 @@ describe("Error Handling Tests", () => {
 
         // This should not throw, but should log the error
         expect(() => {
-            testResultsSummary.writeSummary(mockTestResultsData, "mockAction");
+            testResultsSummary.addSummary(mockTestResultsData, "mockAction");
         }).not.toThrow();
 
         // Verify error was logged