Fix test formatter

Author: leeyi45

.github/actions/src/__tests__/git.test.ts

@@ -8,7 +8,7 @@ vi.mock(import('lodash/memoize.js'), () => ({
 
 const mockedExecOutput = vi.spyOn(exec, 'getExecOutput');
 
-describe('Test checkForChanges', () => {
+describe(git.checkForChanges, () => {
   function mockChanges(value: boolean) {
     mockedExecOutput.mockResolvedValueOnce({
       exitCode: value ? 1 : 0, stdout: '', stderr: ''

.github/actions/src/info/__tests__/sorter.test.ts

@@ -1,7 +1,7 @@
 import { describe, expect, test } from 'vitest';
 import { topoSortPackages } from '../sorter.js';
 
-describe('Test topoSorter', () => {
+describe(topoSortPackages, () => {
   test('Without a cycle', () => {
     const result = topoSortPackages({
       '@sourceacademy/0': {

lib/vitest-reporter/build/test-reporter.js

@@ -23,6 +23,11 @@ function* formatTestSuite(suite) {
   }
   yield '</ul>\n';
 }
+function formatRow(...items) {
+  const tds = items.map((item) => `<td>${item}</td>`);
+  return `<tr>${tds.join('')}</tr>
+`;
+}
 function getTestCount(item) {
   if (item.type === 'test') return 1;
   let output = 0;
@@ -44,15 +49,10 @@ var GithubActionsSummaryReporter = class {
     if (!this.writeStream) return;
     this.writeStream.write('## Test Results\n');
     for (const testModule of modules) {
-      const formatRow2 = function(...items) {
-        const tds = items.map((item) => `<td>${item}</td>`);
-        return `<tr>${tds.join('')}</tr>
-`;
-      };
-      var formatRow = formatRow2;
       const passed = testModule.ok();
+      const fileName = testModule.location ?? 'undefined`';
       const testCount = getTestCount(testModule);
-      this.writeStream.write(`${passed ? '\u2705' : '\u274C'} (fileName) (${testCount} test${testCount === 1 ? '' : 's'}) 
+      this.writeStream.write(`${passed ? '\u2705' : '\u274C'} \`${fileName}\` (${testCount} test${testCount === 1 ? '' : 's'}) 
 `);
       this.writeStream.write('<ul>');
       for (const child of testModule.children) {
@@ -64,12 +64,13 @@ var GithubActionsSummaryReporter = class {
       const diagnostics = testModule.diagnostic();
       const totalDuration = diagnostics.duration.toFixed(0);
       this.writeStream.write('\n\n');
-      this.writeStream.write('#### Summary\n');
-      this.writeStream.write('<table>');
-      this.writeStream.write(formatRow2('Test Files', ''));
-      this.writeStream.write(formatRow2('Tests', testCount.toString()));
-      this.writeStream.write(formatRow2('Start at', ''));
-      this.writeStream.write(formatRow2('Duration', `${totalDuration}ms`));
+      this.writeStream.write(`#### Summary for \`${fileName}\`
+`);
+      this.writeStream.write('<table>\n');
+      this.writeStream.write(formatRow('Test Files', ''));
+      this.writeStream.write(formatRow('Tests', testCount.toString()));
+      this.writeStream.write(formatRow('Start at', ''));
+      this.writeStream.write(formatRow('Duration', `${totalDuration}ms`));
       this.writeStream.write('</table>');
     }
     this.writeStream.close();

lib/vitest-reporter/src/test-reporter.ts

@@ -28,6 +28,11 @@ function* formatTestSuite(suite: TestSuite): Generator<string> {
   yield '</ul>\n';
 }
 
+function formatRow(...items: string[]) {
+  const tds = items.map(item => `<td>${item}</td>`);
+  return `<tr>${tds.join('')}</tr>\n`;
+}
+
 function getTestCount(item: TestModule | TestSuite | TestCase): number {
   if (item.type === 'test') return 1;
 
@@ -57,9 +62,10 @@ export default class GithubActionsSummaryReporter implements Reporter {
     this.writeStream.write('## Test Results\n');
     for (const testModule of modules) {
       const passed = testModule.ok();
+      const fileName: string = testModule.location ?? 'undefined`';
       const testCount = getTestCount(testModule);
 
-      this.writeStream.write(`${passed ? '✅' : '❌'} (fileName) (${testCount} test${testCount === 1 ? '' : 's'}) \n`);
+      this.writeStream.write(`${passed ? '✅' : '❌'} \`${fileName}\` (${testCount} test${testCount === 1 ? '' : 's'}) \n`);
 
       this.writeStream.write('<ul>');
       for (const child of testModule.children) {
@@ -73,14 +79,8 @@ export default class GithubActionsSummaryReporter implements Reporter {
       const totalDuration = diagnostics.duration.toFixed(0);
 
       this.writeStream.write('\n\n');
-      this.writeStream.write('#### Summary\n');
-      this.writeStream.write('<table>');
-
-      function formatRow(...items: string[]) {
-        const tds = items.map(item => `<td>${item}</td>`);
-        return `<tr>${tds.join('')}</tr>\n`;
-      }
-
+      this.writeStream.write(`#### Summary for \`${fileName}\`\n`);
+      this.writeStream.write('<table>\n');
       this.writeStream.write(formatRow('Test Files', ''));
       this.writeStream.write(formatRow('Tests', testCount.toString()));
       this.writeStream.write(formatRow('Start at', ''));