Merge pull request #1456 from mikepenz/copilot/add-test-report-url-output

mikepenz · web-flow · commit f210c1272c16 · 2025-11-01T08:20:40.000+01:00
Add report_url output for test report URLs
diff --git a/README.md b/README.md
@@ -180,6 +180,7 @@ A full set list of possible output values for this action.
 | `outputs.summary`          | The short summary of the junit report. In html format (as also constructed by GitHub for the summary).              |
 | `outputs.detailed_summary` | The full table with all test results in a summary. In html format (as also constructed by GitHub for the summary).  |
 | `outputs.flaky_summary`    | The full table with all flaky results in a summary. In html format (as also constructed by GitHub for the summary). |
+| `outputs.report_url`       | The URL(s) to the test report(s). If multiple reports are created, they are separated by newlines.                  |
 
 ### PR run permissions
 
diff --git a/__tests__/outputs.test.ts b/__tests__/outputs.test.ts
@@ -0,0 +1,49 @@
+import {describe, it, expect} from 'vitest'
+import {CheckInfo} from '../src/annotator.js'
+
+/**
+ *   Copyright 2024 Mike Penz
+ */
+
+describe('report_url output', () => {
+  it('should format single URL correctly', () => {
+    const checkInfos: CheckInfo[] = [
+      {
+        name: 'Test Report 1',
+        url: 'https://github.com/owner/repo/runs/123'
+      }
+    ]
+
+    const reportUrls = checkInfos.map(info => info.url).join('\n')
+    expect(reportUrls).toBe('https://github.com/owner/repo/runs/123')
+  })
+
+  it('should format multiple URLs with newline separation', () => {
+    const checkInfos: CheckInfo[] = [
+      {
+        name: 'Test Report 1',
+        url: 'https://github.com/owner/repo/runs/123'
+      },
+      {
+        name: 'Test Report 2',
+        url: 'https://github.com/owner/repo/runs/456'
+      },
+      {
+        name: 'Test Report 3',
+        url: 'https://github.com/owner/repo/runs/789'
+      }
+    ]
+
+    const reportUrls = checkInfos.map(info => info.url).join('\n')
+    expect(reportUrls).toBe(
+      'https://github.com/owner/repo/runs/123\nhttps://github.com/owner/repo/runs/456\nhttps://github.com/owner/repo/runs/789'
+    )
+  })
+
+  it('should handle empty checkInfos array', () => {
+    const checkInfos: CheckInfo[] = []
+
+    const reportUrls = checkInfos.map(info => info.url).join('\n')
+    expect(reportUrls).toBe('')
+  })
+})
diff --git a/__tests__/table.test.ts b/__tests__/table.test.ts
@@ -141,9 +141,7 @@ describe('buildSummaryTables', () => {
 
     // Check that the detail table doesn't include skipped tests
     const flatResults = detailTable.flat()
-    const hasSkippedTests = flatResults.some(
-      cell => typeof cell === 'string' && cell.includes('⚠️ skipped')
-    )
+    const hasSkippedTests = flatResults.some(cell => typeof cell === 'string' && cell.includes('⚠️ skipped'))
     expect(hasSkippedTests).toBe(false)
 
     // Test with includeSkipped = true (should include skipped tests in detailed table)
diff --git a/action.yml b/action.yml
@@ -188,6 +188,8 @@ outputs:
     description: 'The full table with all test results in a summary. In html format (as also constructed by GitHub for the summary).'
   flaky_summary:
     description: 'The full table with all flaky results in a summary. In html format (as also constructed by GitHub for the summary).'
+  report_url:
+    description: 'The URL(s) to the test report(s). If multiple reports are created, they are separated by newlines.'
 runs:
   using: 'node24'
   main: 'dist/index.js'
diff --git a/dist/index.js b/dist/index.js
diff --git a/dist/index.js.map b/dist/index.js.map
diff --git a/src/main.ts b/src/main.ts
@@ -219,6 +219,10 @@ export async function run(): Promise<void> {
     core.setOutput('detailed_summary', buildTable(detailTable))
     core.setOutput('flaky_summary', buildTable(flakyTable))
 
+    // Set report URLs as output (newline-separated for multiple reports)
+    const reportUrls = checkInfos.map(info => info.url).join('\n')
+    core.setOutput('report_url', reportUrls)
+
     if (failOnFailure && conclusion === 'failure') {
       core.setFailed(`❌ Tests reported ${mergedResult.failed} failures`)
     }
diff --git a/src/table.ts b/src/table.ts
@@ -92,7 +92,7 @@ export function buildSummaryTables(
     table.push(row)
 
     const annotations = testResult.globalAnnotations.filter(
-      annotation => 
+      annotation =>
         (includePassed || annotation.annotation_level !== 'notice') &&
         (includeSkipped || annotation.status !== 'skipped')
     )
@@ -170,9 +170,8 @@ function appendDetailsTable(
 ): void {
   const colspan = includeTimeInSummary ? '3' : '2'
   const annotations = testResult.annotations.filter(
-    annotation => 
-      (includePassed || annotation.annotation_level !== 'notice') &&
-      (includeSkipped || annotation.status !== 'skipped')
+    annotation =>
+      (includePassed || annotation.annotation_level !== 'notice') && (includeSkipped || annotation.status !== 'skipped')
   )
   if (annotations.length > 0) {
     detailsTable.push([{data: `<em>${testResult.name}</em>`, colspan}])
