.github/workflows/check-dist.yml

@@ -16,9 +16,6 @@ on:
       - '**.md'
   workflow_dispatch:
 
-permissions:
-  contents: read
-
 jobs:
   check-dist:
     runs-on: ubuntu-latest

.github/workflows/ci.yml

@@ -8,9 +8,6 @@ on:
       - main
   workflow_dispatch:
 
-permissions:
-  contents: read
-
 jobs:
   build-test:
     name: Build & Test

.github/workflows/manual-run.yml

@@ -3,9 +3,6 @@ name: Manual run
 on:
   workflow_dispatch:
 
-permissions:
-  contents: read
-
 jobs:
   check-dist:
     runs-on: ubuntu-latest

.github/workflows/test-report.yml

@@ -6,10 +6,6 @@ on:
     types:
       - completed
 
-permissions:
-  contents: read
-  actions: read
-
 jobs:
   report:
     name: Workflow test

.nvmrc

@@ -1 +1 @@
-v24
+v20

CHANGELOG.md

@@ -1,17 +1,5 @@
 # Changelog
 
-## 3.1.0
-* Feature: Add `list-files` input to control test report file listing https://github.com/dorny/test-reporter/pull/773
-* Feature: Add `summary_file` output with the path to the generated summary in Markdown format https://github.com/dorny/test-reporter/pull/772
-
-## 3.0.0
-* Feature: Use NodeJS 24 LTS as default runtime https://github.com/dorny/test-reporter/pull/738
-
-## 2.7.0
-* Feature: Add `slug-prefix` output for link anchors https://github.com/dorny/test-reporter/pull/731
-* Feature: Report `jest-junit` testsuite errors as failures https://github.com/dorny/test-reporter/pull/155
-* Security: Update dependencies to fix reported security vulnerabilities
-
 ## 2.6.0
 * Fix: For `workflow_run` events, resolve the commit of the check run from related pull request head commits first (matching `workflow_run.head_branch`, then first PR), and fall back to `workflow_run.head_sha` for non-PR runs https://github.com/dorny/test-reporter/pull/673
 * Change: The `test-reporter` action will listed all artifacts associated with the build run https://github.com/dorny/test-reporter/pull/693

README.md

@@ -47,12 +47,12 @@ jobs:
     name: Build & Test
     runs-on: ubuntu-latest
     steps:
-      - uses: actions/checkout@v6     # checkout the repo
+      - uses: actions/checkout@v4     # checkout the repo
       - run: npm ci                   # install packages
       - run: npm test                 # run tests (configured to use jest-junit reporter)
 
       - name: Test Report
-        uses: dorny/test-reporter@v3
+        uses: dorny/test-reporter@v2
         if: ${{ !cancelled() }}       # run this step even if previous step failed
         with:
           name: JEST Tests            # Name of the check run which will be created
@@ -78,10 +78,10 @@ jobs:
   build-test:
     runs-on: ubuntu-latest
     steps:
-      - uses: actions/checkout@v6         # checkout the repo
+      - uses: actions/checkout@v4         # checkout the repo
       - run: npm ci                       # install packages
       - run: npm test                     # run tests (configured to use jest-junit reporter)
-      - uses: actions/upload-artifact@v7  # upload test results
+      - uses: actions/upload-artifact@v4  # upload test results
         if: ${{ !cancelled() }}           # run this step even if previous step failed
         with:
           name: test-results
@@ -103,7 +103,7 @@ jobs:
   report:
     runs-on: ubuntu-latest
     steps:
-    - uses: dorny/test-reporter@v3
+    - uses: dorny/test-reporter@v2
       with:
         artifact: test-results            # artifact name
         name: JEST Tests                  # Name of the check run which will be created
@@ -114,7 +114,7 @@ jobs:
 ## Usage
 
 ```yaml
-- uses: dorny/test-reporter@v3
+- uses: dorny/test-reporter@v2
   with:
 
     # Name or regex of artifact containing test results
@@ -184,12 +184,6 @@ jobs:
     #   none
     list-tests: 'all'
 
-    # Limits which test result files are listed:
-    #   all
-    #   failed
-    #   none
-    list-files: 'all'
-
     # Limits number of created annotations with error message and stack trace captured during test execution.
     # Must be less or equal to 50.
     max-annotations: '10'
@@ -200,13 +194,6 @@ jobs:
     # Set this action as failed if no test results were found
     fail-on-empty: 'true'
 
-    # Controls whether test report details are collapsed or expanded.
-    # Supported options:
-    #   auto: Collapse only if all tests pass (default behavior)
-    #   always: Always collapse the report details
-    #   never: Always expand the report details
-    collapsed: 'auto'
-
     # Relative path under $GITHUB_WORKSPACE where the repository was checked out.
     working-directory: ''
 
@@ -225,8 +212,6 @@ jobs:
 | time       | Test execution time [ms] |
 | url        | Check run URL            |
 | url_html   | Check run URL HTML       |
-| summary_file | Path to a file containing the generated test report summary in Markdown format |
-| slug_prefix| Random anchor links slug prefix generated for the summary headers |
 
 ## Supported formats
 

__tests__/__outputs__/jest-test-errors-results.md

@@ -1,22 +0,0 @@
-![Tests failed](https://img.shields.io/badge/tests-2%20failed-critical)
-|Report|Passed|Failed|Skipped|Time|
-|:---|---:|---:|---:|---:|
-|[fixtures/test-errors/jest/jest-test-results.xml](#user-content-r0)||2 ❌||646ms|
-## ❌ <a id="user-content-r0" href="#user-content-r0">fixtures/test-errors/jest/jest-test-results.xml</a>
-**2** tests were completed in **646ms** with **0** passed, **2** failed and **0** skipped.
-|Test suite|Passed|Failed|Skipped|Time|
-|:---|---:|---:|---:|---:|
-|[libs/bar.spec.ts](#user-content-r0s0)||1 ❌||0ms|
-|[libs/foo.spec.ts](#user-content-r0s1)||1 ❌||0ms|
-### ❌ <a id="user-content-r0s0" href="#user-content-r0s0">libs/bar.spec.ts</a>
-```
-Test suite failed to run
-  ❌ libs/bar.spec.ts
-	● Test suite failed to run
-```
-### ❌ <a id="user-content-r0s1" href="#user-content-r0s1">libs/foo.spec.ts</a>
-```
-Test suite failed to run
-  ❌ libs/foo.spec.ts
-	● Test suite failed to run
-```

__tests__/__snapshots__/jest-junit.test.ts.snap

@@ -1,68 +1,5 @@
 // Jest Snapshot v1, https://jestjs.io/docs/snapshot-testing
 
-exports[`jest-junit tests jest testsuite errors example test results matches snapshot 1`] = `
-TestRunResult {
-  "path": "fixtures/test-errors/jest/jest-test-results.xml",
-  "suites": [
-    TestSuiteResult {
-      "groups": [
-        TestGroupResult {
-          "name": "Test suite failed to run",
-          "tests": [
-            TestCaseResult {
-              "error": {
-                "details": "  ● Test suite failed to run
-
-    tsconfig.json:13:3 - error TS6258: 'typeRoots' should be set inside the 'compilerOptions' object of the config json file
-
-    13   "typeRoots": ["./src/lib/types", "./node_modules/@types"],
-         ~~~~~~~~~~~
-",
-                "line": undefined,
-                "path": undefined,
-              },
-              "name": "libs/foo.spec.ts",
-              "result": "failed",
-              "time": 0,
-            },
-          ],
-        },
-      ],
-      "name": "libs/foo.spec.ts",
-      "totalTime": 0,
-    },
-    TestSuiteResult {
-      "groups": [
-        TestGroupResult {
-          "name": "Test suite failed to run",
-          "tests": [
-            TestCaseResult {
-              "error": {
-                "details": "  ● Test suite failed to run
-
-    tsconfig.json:13:3 - error TS6258: 'typeRoots' should be set inside the 'compilerOptions' object of the config json file
-
-    13   "typeRoots": ["./src/lib/types", "./node_modules/@types"],
-         ~~~~~~~~~~~
-",
-                "line": undefined,
-                "path": undefined,
-              },
-              "name": "libs/bar.spec.ts",
-              "result": "failed",
-              "time": 0,
-            },
-          ],
-        },
-      ],
-      "name": "libs/bar.spec.ts",
-      "totalTime": 0,
-    },
-  ],
-  "totalTime": 646,
-}
-`;
-
 exports[`jest-junit tests parsing ESLint report without timing information works - PR #134 1`] = `
 TestRunResult {
   "path": "fixtures/jest-junit-eslint.xml",

__tests__/fixtures/test-errors/jest/files.txt

@@ -1,3 +0,0 @@
-libs/bar.spec.ts
-libs/foo.spec.ts
-tsconfig.json

__tests__/fixtures/test-errors/jest/jest-test-results.xml

@@ -1,25 +0,0 @@
-<?xml version="1.0" encoding="UTF-8"?>
-<testsuites name="jest tests" tests="0" failures="0" errors="2" time="0.646">
-  <testsuite name="libs/foo.spec.ts" errors="1" failures="0" skipped="0" timestamp="1970-01-01T00:00:00" time="0" tests="0">
-    <testcase classname="Test suite failed to run" name="libs/foo.spec.ts" time="0">
-      <error>  ● Test suite failed to run
-
-    tsconfig.json:13:3 - error TS6258: &apos;typeRoots&apos; should be set inside the &apos;compilerOptions&apos; object of the config json file
-
-    13   &quot;typeRoots&quot;: [&quot;./src/lib/types&quot;, &quot;./node_modules/@types&quot;],
-         ~~~~~~~~~~~
-</error>
-    </testcase>
-  </testsuite>
-  <testsuite name="libs/bar.spec.ts" errors="1" failures="0" skipped="0" timestamp="1970-01-01T00:00:00" time="0" tests="0">
-    <testcase classname="Test suite failed to run" name="libs/bar.spec.ts" time="0">
-      <error>  ● Test suite failed to run
-
-    tsconfig.json:13:3 - error TS6258: &apos;typeRoots&apos; should be set inside the &apos;compilerOptions&apos; object of the config json file
-
-    13   &quot;typeRoots&quot;: [&quot;./src/lib/types&quot;, &quot;./node_modules/@types&quot;],
-         ~~~~~~~~~~~
-</error>
-    </testcase>
-  </testsuite>
-</testsuites>

__tests__/jest-junit.test.ts

@@ -351,27 +351,4 @@ describe('jest-junit tests', () => {
     // Report should have the title as the first line
     expect(report).toMatch(/^# My Custom Title\n## 1 passed, 4 failed and 1 skipped\n/)
   })
-
-  it('jest testsuite errors example test results matches snapshot', async () => {
-    const fixturePath = path.join(__dirname, 'fixtures', 'test-errors', 'jest', 'jest-test-results.xml')
-    const trackedFilesPath = path.join(__dirname, 'fixtures', 'test-errors', 'jest', 'files.txt')
-    const outputPath = path.join(__dirname, '__outputs__', 'jest-test-errors-results.md')
-    const filePath = normalizeFilePath(path.relative(__dirname, fixturePath))
-    const fileContent = fs.readFileSync(fixturePath, {encoding: 'utf8'})
-
-    const trackedFiles = fs.readFileSync(trackedFilesPath, {encoding: 'utf8'}).split(/\n\r?/g)
-    const opts: ParseOptions = {
-      parseErrors: true,
-      trackedFiles
-      //workDir: '/home/dorny/dorny/jest/'
-    }
-
-    const parser = new JestJunitParser(opts)
-    const result = await parser.parse(filePath, fileContent)
-    expect(result).toMatchSnapshot()
-
-    const report = getReport([result])
-    fs.mkdirSync(path.dirname(outputPath), {recursive: true})
-    fs.writeFileSync(outputPath, report)
-  })
 })

__tests__/report/get-report.test.ts

@@ -1,5 +1,4 @@
-import {DEFAULT_OPTIONS, getBadge, getReport, ReportOptions} from '../../src/report/get-report.js'
-import {TestCaseResult, TestGroupResult, TestRunResult, TestSuiteResult} from '../../src/test-results.js'
+import {getBadge, DEFAULT_OPTIONS, ReportOptions} from '../../src/report/get-report.js'
 
 describe('getBadge', () => {
   describe('URI encoding with special characters', () => {
@@ -132,147 +131,3 @@ describe('getBadge', () => {
     })
   })
 })
-
-describe('getReport', () => {
-  // Helper function to create test results
-  function createTestResult(path: string, passed: number, failed: number, skipped: number): TestRunResult {
-    const tests: TestCaseResult[] = []
-    for (let i = 0; i < passed; i++) {
-      tests.push(new TestCaseResult(`passed-test-${i}`, 'success', 100))
-    }
-    for (let i = 0; i < failed; i++) {
-      tests.push(
-        new TestCaseResult(`failed-test-${i}`, 'failed', 100, {
-          details: 'Test failed',
-          message: 'Assertion error'
-        })
-      )
-    }
-    for (let i = 0; i < skipped; i++) {
-      tests.push(new TestCaseResult(`skipped-test-${i}`, 'skipped', 0))
-    }
-
-    const group = new TestGroupResult('test-group', tests)
-    const suite = new TestSuiteResult('test-suite', [group])
-    return new TestRunResult(path, [suite])
-  }
-
-  describe('list-files parameter', () => {
-    const results = [
-      createTestResult('passing-file.spec.ts', 5, 0, 0),
-      createTestResult('failing-file.spec.ts', 3, 2, 1),
-      createTestResult('passing-with-skipped-file.spec.ts', 8, 0, 2)
-    ]
-
-    it('shows all files when list-files is "all"', () => {
-      const report = getReport(results, {
-        ...DEFAULT_OPTIONS,
-        listFiles: 'all',
-        listSuites: 'none',
-        listTests: 'none'
-      })
-
-      expect(report).toContain('|Report|Passed|Failed|Skipped|Time|')
-      expect(report).toContain('passing-file.spec.ts')
-      expect(report).toContain('failing-file.spec.ts')
-      expect(report).toContain('passing-with-skipped-file.spec.ts')
-    })
-
-    it('shows only failed files when list-files is "failed"', () => {
-      const report = getReport(results, {
-        ...DEFAULT_OPTIONS,
-        listFiles: 'failed',
-        listSuites: 'none',
-        listTests: 'none'
-      })
-
-      expect(report).toContain('|Report|Passed|Failed|Skipped|Time|')
-      expect(report).not.toContain('passing-file.spec.ts')
-      expect(report).toContain('failing-file.spec.ts')
-      expect(report).not.toContain('passing-with-skipped-file.spec.ts')
-    })
-
-    it('shows no file details when list-files is "none"', () => {
-      const report = getReport(results, {
-        ...DEFAULT_OPTIONS,
-        listFiles: 'none',
-        listSuites: 'none',
-        listTests: 'none'
-      })
-
-      expect(report).toContain('![')
-      expect(report).not.toContain('|Report|Passed|Failed|Skipped|Time|')
-      expect(report).not.toContain('passing-file.spec.ts')
-      expect(report).not.toContain('failing-file.spec.ts')
-      expect(report).not.toContain('passing-with-skipped-file.spec.ts')
-    })
-
-    it('does not show an empty summary table when list-files is "none" and only-summary is enabled', () => {
-      const report = getReport(results, {
-        ...DEFAULT_OPTIONS,
-        listFiles: 'none',
-        listSuites: 'all',
-        onlySummary: true,
-        listTests: 'none'
-      })
-
-      expect(report).toContain('![')
-      expect(report).not.toContain('|Report|Passed|Failed|Skipped|Time|')
-      expect(report).not.toContain('passing-file.spec.ts')
-      expect(report).not.toContain('failing-file.spec.ts')
-      expect(report).not.toContain('passing-with-skipped-file.spec.ts')
-    })
-
-    it('works correctly with list-suites and list-tests when list-files is "failed"', () => {
-      const report = getReport(results, {
-        ...DEFAULT_OPTIONS,
-        listFiles: 'failed',
-        listSuites: 'all',
-        listTests: 'all'
-      })
-
-      expect(report).toContain('|Report|Passed|Failed|Skipped|Time|')
-      expect(report).not.toContain('passing-file.spec.ts')
-      expect(report).toContain('failing-file.spec.ts')
-      expect(report).not.toContain('passing-with-skipped-file.spec.ts')
-      // Should show suite details for the failed file
-      expect(report).toContain('test-suite')
-    })
-
-    it('filters correctly when all files pass and list-files is "failed"', () => {
-      const allPassingResults = [
-        createTestResult('passing-file-1.spec.ts', 5, 0, 0),
-        createTestResult('passing-file-2.spec.ts', 8, 0, 2)
-      ]
-
-      const report = getReport(allPassingResults, {
-        ...DEFAULT_OPTIONS,
-        listFiles: 'failed',
-        listSuites: 'all',
-        listTests: 'none'
-      })
-
-      expect(report).not.toContain('passing-file-1.spec.ts')
-      expect(report).not.toContain('passing-file-2.spec.ts')
-      expect(report).toContain('![')
-      expect(report).not.toContain('|Report|Passed|Failed|Skipped|Time|')
-    })
-
-    it('filters correctly when all files fail and list-files is "failed"', () => {
-      const allFailingResults = [
-        createTestResult('failing-file-1.spec.ts', 0, 5, 0),
-        createTestResult('failing-file-2.spec.ts', 1, 2, 1)
-      ]
-
-      const report = getReport(allFailingResults, {
-        ...DEFAULT_OPTIONS,
-        listFiles: 'failed',
-        listSuites: 'all',
-        listTests: 'none'
-      })
-
-      expect(report).toContain('failing-file-1.spec.ts')
-      expect(report).toContain('failing-file-2.spec.ts')
-    })
-  })
-})

__tests__/utils/slugger.test.ts

@@ -1,28 +0,0 @@
-import {DEFAULT_OPTIONS} from '../../src/report/get-report.js'
-import {slug} from '../../src/utils/slugger.js'
-
-describe('slugger', () => {
-  it('adds prefix from report options to generated slug', () => {
-    const result = slug('r0s1', {
-      ...DEFAULT_OPTIONS,
-      slugPrefix: 'prefix-'
-    })
-
-    expect(result).toEqual({
-      id: 'user-content-prefix-r0s1',
-      link: '#user-content-prefix-r0s1'
-    })
-  })
-
-  it('sanitizes custom prefix using existing slug normalization', () => {
-    const result = slug('r0', {
-      ...DEFAULT_OPTIONS,
-      slugPrefix: ' my /custom_prefix?.'
-    })
-
-    expect(result).toEqual({
-      id: 'user-content-my-customprefix-r0',
-      link: '#user-content-my-customprefix-r0'
-    })
-  })
-})

action.yml

@@ -54,14 +54,6 @@ inputs:
         - none
     required: false
     default: 'all'
-  list-files:
-    description: |
-      Limits which test result files are listed. Supported options:
-        - all
-        - failed
-        - none
-    required: false
-    default: 'all'
   max-annotations:
     description: |
       Limits number of created annotations with error message and stack trace captured during test execution.
@@ -130,12 +122,8 @@ outputs:
     description: Check run URL
   url_html:
     description: Check run URL HTML
-  summary_file:
-    description: Path to a file containing the generated test report summary in Markdown format
-  slug_prefix:
-    description: Random prefix added to generated report anchor slugs for this action run
 runs:
-  using: 'node24'
+  using: 'node20'
   main: 'dist/index.js'
 branding:
   color: blue

dist/licenses.txt

@@ -889,7 +889,7 @@ reusify
 MIT
 The MIT License (MIT)
 
-Copyright (c) 2015-2024 Matteo Collina
+Copyright (c) 2015 Matteo Collina
 
 Permission is hereby granted, free of charge, to any person obtaining a copy
 of this software and associated documentation files (the "Software"), to deal

package.json

@@ -1,6 +1,6 @@
 {
   "name": "test-reporter",
-  "version": "3.0.0",
+  "version": "2.5.0",
   "private": true,
   "description": "Presents test results from popular testing frameworks as Github check run",
   "type": "module",
@@ -37,37 +37,37 @@
   "author": "Michal Dorner <dorner.michal@gmail.com>",
   "license": "MIT",
   "dependencies": {
-    "@actions/core": "^3.0.1",
+    "@actions/core": "^3.0.0",
     "@actions/exec": "^3.0.0",
-    "@actions/github": "^9.1.1",
-    "adm-zip": "^0.5.17",
+    "@actions/github": "^9.0.0",
+    "adm-zip": "^0.5.16",
     "fast-glob": "^3.3.3",
-    "picomatch": "^4.0.4",
+    "picomatch": "^4.0.3",
     "xml2js": "^0.6.2"
   },
   "devDependencies": {
     "@octokit/webhooks-types": "^7.6.1",
-    "@types/adm-zip": "^0.5.8",
+    "@types/adm-zip": "^0.5.7",
     "@types/jest": "^30.0.0",
-    "@types/node": "^24.12.2",
-    "@types/picomatch": "^4.0.3",
+    "@types/node": "^20.19.23",
+    "@types/picomatch": "^4.0.2",
     "@types/xml2js": "^0.4.14",
-    "@typescript-eslint/eslint-plugin": "^8.59.0",
-    "@typescript-eslint/parser": "^8.59.0",
+    "@typescript-eslint/eslint-plugin": "^8.56.1",
+    "@typescript-eslint/parser": "^8.56.1",
     "@vercel/ncc": "^0.38.4",
     "eol-converter-cli": "^1.1.0",
-    "eslint": "^9.39.4",
+    "eslint": "^9.39.3",
     "eslint-import-resolver-typescript": "^4.4.4",
     "eslint-plugin-github": "^6.0.0",
     "eslint-plugin-import": "^2.32.0",
-    "eslint-plugin-jest": "^29.15.2",
+    "eslint-plugin-jest": "^29.15.0",
     "eslint-plugin-prettier": "^5.5.5",
-    "jest": "^30.3.0",
-    "jest-junit": "^17.0.0",
+    "jest": "^30.2.0",
+    "jest-junit": "^16.0.0",
     "js-yaml": "^4.1.1",
-    "prettier": "^3.8.3",
-    "ts-jest": "^29.4.9",
-    "typescript": "^6.0.3"
+    "prettier": "^3.8.1",
+    "ts-jest": "^29.4.6",
+    "typescript": "^5.9.3"
   },
   "overrides": {
     "sax": "^1.4.3"
@@ -83,7 +83,7 @@
     "titleTemplate": "{title}"
   },
   "engines": {
-    "node": ">=24"
+    "node": ">=20"
   },
   "markdownlint-cli2": {
     "ignores": [

reports/jest/package-lock.json

@@ -4548,11 +4548,10 @@
       "dev": true
     },
     "node_modules/picomatch": {
-      "version": "2.3.2",
-      "resolved": "https://registry.npmjs.org/picomatch/-/picomatch-2.3.2.tgz",
-      "integrity": "sha512-V7+vQEJ06Z+c5tSye8S+nHUfI51xoXIXjHQ99cQtKUkQqqO1kO/KCJUfZXuB47h/YBlDhah2H3hdUGXn8ie0oA==",
+      "version": "2.2.2",
+      "resolved": "https://registry.npmjs.org/picomatch/-/picomatch-2.2.2.tgz",
+      "integrity": "sha512-q0M/9eZHzmr0AulXyPwNfZjtwZ/RBZlbN3K3CErVrk50T2ASYI7Bye0EvekFY3IP1Nt2DHu0re+V2ZHIpMkuWg==",
       "dev": true,
-      "license": "MIT",
       "engines": {
         "node": ">=8.6"
       },

src/main.ts

@@ -1,10 +1,6 @@
 import * as core from '@actions/core'
 import * as github from '@actions/github'
 import {GitHub} from '@actions/github/lib/utils'
-import {randomBytes} from 'node:crypto'
-import {writeFileSync} from 'node:fs'
-import {tmpdir} from 'node:os'
-import {join} from 'node:path'
 
 import {ArtifactProvider} from './input-providers/artifact-provider.js'
 import {LocalFileProvider} from './input-providers/local-file-provider.js'
@@ -47,14 +43,12 @@ class TestReporter {
   readonly reporter = core.getInput('reporter', {required: true})
   readonly listSuites = core.getInput('list-suites', {required: true}) as 'all' | 'failed' | 'none'
   readonly listTests = core.getInput('list-tests', {required: true}) as 'all' | 'failed' | 'none'
-  readonly listFiles = core.getInput('list-files', {required: true}) as 'all' | 'failed' | 'none'
   readonly maxAnnotations = parseInt(core.getInput('max-annotations', {required: true}))
   readonly failOnError = core.getInput('fail-on-error', {required: true}) === 'true'
   readonly failOnEmpty = core.getInput('fail-on-empty', {required: true}) === 'true'
   readonly workDirInput = core.getInput('working-directory', {required: false})
   readonly onlySummary = core.getInput('only-summary', {required: false}) === 'true'
   readonly useActionsSummary = core.getInput('use-actions-summary', {required: false}) === 'true'
-  readonly slugPrefix = `tr-${randomBytes(4).toString('base64url')}-`
   readonly badgeTitle = core.getInput('badge-title', {required: false})
   readonly reportTitle = core.getInput('report-title', {required: false})
   readonly collapsed = core.getInput('collapsed', {required: false}) as 'auto' | 'always' | 'never'
@@ -75,11 +69,6 @@ class TestReporter {
       return
     }
 
-    if (this.listFiles !== 'all' && this.listFiles !== 'failed' && this.listFiles !== 'none') {
-      core.setFailed(`Input parameter 'list-files' has invalid value`)
-      return
-    }
-
     if (this.collapsed !== 'auto' && this.collapsed !== 'always' && this.collapsed !== 'never') {
       core.setFailed(`Input parameter 'collapsed' has invalid value`)
       return
@@ -155,7 +144,6 @@ class TestReporter {
     core.setOutput('failed', failed)
     core.setOutput('skipped', skipped)
     core.setOutput('time', time)
-    core.setOutput('slug_prefix', this.slugPrefix)
 
     if (this.failOnError && isFailed) {
       core.setFailed(`Failed test were found and 'fail-on-error' option is set to ${this.failOnError}`)
@@ -186,17 +174,7 @@ class TestReporter {
       }
     }
 
-    const {
-      listSuites,
-      listTests,
-      slugPrefix,
-      listFiles,
-      onlySummary,
-      useActionsSummary,
-      badgeTitle,
-      reportTitle,
-      collapsed
-    } = this
+    const {listSuites, listTests, onlySummary, useActionsSummary, badgeTitle, reportTitle, collapsed} = this
 
     const passed = results.reduce((sum, tr) => sum + tr.passed, 0)
     const failed = results.reduce((sum, tr) => sum + tr.failed, 0)
@@ -210,8 +188,6 @@ class TestReporter {
         {
           listSuites,
           listTests,
-          slugPrefix,
-          listFiles,
           baseUrl,
           onlySummary,
           useActionsSummary,
@@ -224,7 +200,6 @@ class TestReporter {
 
       core.info('Summary content:')
       core.info(summary)
-      this.writeSummaryFile(summary)
       await core.summary.addRaw(summary).write()
     } else {
       core.info(`Creating check run ${name}`)
@@ -244,8 +219,6 @@ class TestReporter {
       const summary = getReport(results, {
         listSuites,
         listTests,
-        slugPrefix,
-        listFiles,
         baseUrl,
         onlySummary,
         useActionsSummary,
@@ -256,7 +229,6 @@ class TestReporter {
 
       core.info('Creating annotations')
       const annotations = getAnnotations(results, this.maxAnnotations)
-      this.writeSummaryFile(summary)
 
       const isFailed = this.failOnError && results.some(tr => tr.result === 'failed')
       const conclusion = isFailed ? 'failure' : 'success'
@@ -283,14 +255,6 @@ class TestReporter {
     return results
   }
 
-  writeSummaryFile(summary: string): void {
-    const dir = process.env.RUNNER_TEMP || tmpdir()
-    const file = join(dir, `test-reporter-summary-${randomBytes(8).toString('hex')}.md`)
-    writeFileSync(file, summary)
-    core.info(`Summary written to ${file}`)
-    core.setOutput('summary_file', file)
-  }
-
   getParser(reporter: string, options: ParseOptions): TestParser {
     switch (reporter) {
       case 'dart-json':

src/parsers/jest-junit/jest-junit-parser.ts

@@ -75,18 +75,17 @@ export class JestJunitParser implements TestParser {
   }
 
   private getTestCaseResult(test: TestCase): TestExecutionResult {
-    if (test.failure || test.error) return 'failed'
+    if (test.failure) return 'failed'
     if (test.skipped) return 'skipped'
     return 'success'
   }
 
   private getTestCaseError(tc: TestCase): TestCaseError | undefined {
-    if (!this.options.parseErrors || !(tc.failure || tc.error)) {
+    if (!this.options.parseErrors || !tc.failure) {
       return undefined
     }
 
-    const message = tc.failure ? tc.failure[0] : tc.error ? tc.error[0] : 'unknown failure'
-    const details = typeof message === 'string' ? message : message['_']
+    const details = typeof tc.failure[0] === 'string' ? tc.failure[0] : tc.failure[0]['_']
     let path
     let line
 

src/parsers/jest-junit/jest-junit-types.ts

@@ -31,5 +31,4 @@ export interface TestCase {
   }
   failure?: string[]
   skipped?: string[]
-  error?: string[]
 }

src/report/get-report.ts

@@ -11,8 +11,6 @@ const MAX_ACTIONS_SUMMARY_LENGTH = 1048576
 export interface ReportOptions {
   listSuites: 'all' | 'failed' | 'none'
   listTests: 'all' | 'failed' | 'none'
-  slugPrefix: string
-  listFiles: 'all' | 'failed' | 'none'
   baseUrl: string
   onlySummary: boolean
   useActionsSummary: boolean
@@ -24,8 +22,6 @@ export interface ReportOptions {
 export const DEFAULT_OPTIONS: ReportOptions = {
   listSuites: 'all',
   listTests: 'all',
-  slugPrefix: '',
-  listFiles: 'all',
   baseUrl: '',
   onlySummary: false,
   useActionsSummary: true,
@@ -175,29 +171,21 @@ function getTestRunsReport(testRuns: TestRunResult[], options: ReportOptions): s
     sections.push(` `)
   }
 
-  // Filter test runs based on list-files option
-  const filteredTestRuns =
-    options.listFiles === 'failed'
-      ? testRuns.filter(tr => tr.result === 'failed')
-      : options.listFiles === 'none'
-        ? []
-        : testRuns
+  if (testRuns.length > 0 || options.onlySummary) {
+    const tableData = testRuns
+      .map((tr, originalIndex) => ({tr, originalIndex}))
+      .filter(({tr}) => tr.passed > 0 || tr.failed > 0 || tr.skipped > 0)
+      .map(({tr, originalIndex}) => {
+        const time = formatTime(tr.time)
+        const name = tr.path
+        const addr = options.baseUrl + makeRunSlug(originalIndex, options).link
+        const nameLink = link(name, addr)
+        const passed = tr.passed > 0 ? `${tr.passed} ${Icon.success}` : ''
+        const failed = tr.failed > 0 ? `${tr.failed} ${Icon.fail}` : ''
+        const skipped = tr.skipped > 0 ? `${tr.skipped} ${Icon.skip}` : ''
+        return [nameLink, passed, failed, skipped, time]
+      })
 
-  const tableData = filteredTestRuns
-    .map((tr, originalIndex) => ({tr, originalIndex}))
-    .filter(({tr}) => tr.passed > 0 || tr.failed > 0 || tr.skipped > 0)
-    .map(({tr, originalIndex}) => {
-      const time = formatTime(tr.time)
-      const name = tr.path
-      const addr = options.baseUrl + makeRunSlug(originalIndex, options).link
-      const nameLink = link(name, addr)
-      const passed = tr.passed > 0 ? `${tr.passed} ${Icon.success}` : ''
-      const failed = tr.failed > 0 ? `${tr.failed} ${Icon.fail}` : ''
-      const skipped = tr.skipped > 0 ? `${tr.skipped} ${Icon.skip}` : ''
-      return [nameLink, passed, failed, skipped, time]
-    })
-
-  if (tableData.length > 0) {
     const resultsTable = table(
       ['Report', 'Passed', 'Failed', 'Skipped', 'Time'],
       [Align.Left, Align.Right, Align.Right, Align.Right, Align.Right],
@@ -207,7 +195,7 @@ function getTestRunsReport(testRuns: TestRunResult[], options: ReportOptions): s
   }
 
   if (options.onlySummary === false) {
-    const suitesReports = filteredTestRuns.map((tr, i) => getSuitesReport(tr, i, options)).flat()
+    const suitesReports = testRuns.map((tr, i) => getSuitesReport(tr, i, options)).flat()
     sections.push(...suitesReports)
   }
 

src/utils/slugger.ts

@@ -4,7 +4,7 @@
 import {ReportOptions} from '../report/get-report.js'
 
 export function slug(name: string, options: ReportOptions): {id: string; link: string} {
-  const slugId = `${options.slugPrefix}${name}`
+  const slugId = name
     .trim()
     .replace(/_/g, '')
     .replace(/[./\\]/g, '-')