fix test

Author: mw-kapilg

sample/TheTruth.m

@@ -4,21 +4,5 @@ function testTheTestOfAllTime(testCase)
             onetyone = 11;
             testCase.verifyEqual(onetyone, 11);
         end
-
-        function testAssumptionFailure(testCase)
-            testCase.assumeEqual(1+1, 11);
-        end
-
-        function testVerificationFailure(testCase)
-            % Two verifications to show that the test continues after a verification failure
-            testCase.verifyEqual(1+1, 11);
-            testCase.verifyEqual(1+1, 11);
-        end
-
-        function testAssertionFailure(testCase)
-            % Two assertions to show that the test ends after an assertion failure
-            testCase.assertEqual(1+1, 11);
-            testCase.assertEqual(1+1, 11);
-        end
     end
 end

sample/tAssumptionFailure.m

@@ -0,0 +1,7 @@
+classdef tAssumptionFailure < matlab.unittest.TestCase
+    methods (Test)
+        function testAssumptionFailure(testCase)
+            testCase.assumeEqual(1+1, 11);
+        end
+    end
+end

sample/ValidationTest.m sample/tFatalAssertionFailure.msample/ValidationTest.m renamed to sample/tFatalAssertionFailure.m

@@ -1,6 +1,4 @@
-% Named ValidationTest to run at the end of the suite
-% because it contains a fatal assertion failure.
-classdef ValidationTest < matlab.unittest.TestCase
+classdef tFatalAssertionFailure < matlab.unittest.TestCase
     methods (Test)
         function testFatalAssertionFailure(testCase)
             testCase.fatalAssertEqual(1+1, 11);

sample/tVerificationAndAssertionFailure.m

@@ -0,0 +1,15 @@
+classdef tVerificationAndAssertionFailure < matlab.unittest.TestCase
+    methods (Test)
+        function testVerificationFailure(testCase)
+            % Two verifications to show that the test continues after a verification failure
+            testCase.verifyEqual(1+1, 11);
+            testCase.verifyEqual(1+1, 11);
+        end
+
+        function testAssertionFailure(testCase)
+            % Two assertions to show that the test ends after an assertion failure
+            testCase.assertEqual(1+1, 11);
+            testCase.assertEqual(1+1, 11);
+        end
+    end
+end

src/testResultsSummary.ts

@@ -48,7 +48,7 @@ interface TestResultsData {
 export function writeSummary(testResults: MatlabTestFile[][], stats: TestStatistics) {
     try {
         const helpLink = `<a href="https://github.com/matlab-actions/run-tests/blob/main/README.md"` +
-            `target="_blank" title="View documentation">ℹ️</a>`;
+            ` target="_blank" title="View documentation">ℹ️</a>`;
         const header = getTestHeader(testResults, stats);
         const detailedResults = getDetailedResults(testResults);
 

src/testResultsSummary.unit.test.ts

@@ -138,8 +138,24 @@ describe('Artifact Processing Tests', () => {
         testResultsSummary.writeSummary(testResults, stats);
 
         expect(core.summary.addHeading).toHaveBeenCalledTimes(2);
-        expect(core.summary.addHeading).toHaveBeenNthCalledWith(1, 'MATLAB Test Results (' + process.env.GITHUB_ACTION + ')');
+        expect(core.summary.addHeading).toHaveBeenNthCalledWith(
+            1, 
+            expect.stringContaining('MATLAB Test Results')
+        );
+        expect(core.summary.addHeading).toHaveBeenNthCalledWith(
+            1, 
+            expect.stringContaining('<a href="https://github.com/matlab-actions/run-tests/blob/main/README.md"')
+        );
+        expect(core.summary.addHeading).toHaveBeenNthCalledWith(
+            1, 
+            expect.stringContaining('target="_blank"')
+        );
+        expect(core.summary.addHeading).toHaveBeenNthCalledWith(
+            1, 
+            expect.stringContaining('ℹ️</a>')
+        );
         expect(core.summary.addHeading).toHaveBeenNthCalledWith(2, 'All tests', 3);
+
         expect(core.summary.addRaw).toHaveBeenCalledTimes(2);
         expect(core.summary.write).toHaveBeenCalledTimes(1);
     });