Add report_url output with test report URLs

Co-authored-by: mikepenz <[email protected]>

Author: Copilot

__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('')
+  })
+})

__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)

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'

dist/index.js

@@ -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`)
     }

dist/index.js.map

@@ -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}])