test-reporter release v2.2.0

Bumps [js-yaml](https://github.com/nodeca/js-yaml) from 4.1.0 to 4.1.1.
- [Changelog](https://github.com/nodeca/js-yaml/blob/master/CHANGELOG.md)
- [Commits](https://github.com/nodeca/js-yaml/compare/4.1.0...4.1.1)

---
updated-dependencies:
- dependency-name: js-yaml
  dependency-version: 4.1.1
  dependency-type: direct:development
  update-type: version-update:semver-patch
...

Signed-off-by: dependabot[bot] <support@github.com>

.github/workflows/check-dist.yml

@@ -21,10 +21,10 @@ jobs:
     runs-on: ubuntu-latest
 
     steps:
-      - uses: actions/checkout@v4
+      - uses: actions/checkout@v5
 
       - name: Set Node.js
-        uses: actions/setup-node@v4
+        uses: actions/setup-node@v6
         with:
           node-version-file: '.nvmrc'
 
@@ -46,7 +46,7 @@ jobs:
         id: diff
 
       # If index.js was different than expected, upload the expected version as an artifact
-      - uses: actions/upload-artifact@v4
+      - uses: actions/upload-artifact@v5
         if: ${{ failure() && steps.diff.conclusion == 'failure' }}
         with:
           name: dist

.github/workflows/ci.yml

@@ -13,8 +13,8 @@ jobs:
     name: Build & Test
     runs-on: ubuntu-latest
     steps:
-      - uses: actions/checkout@v4
+      - uses: actions/checkout@v5
-      - uses: actions/setup-node@v4
+      - uses: actions/setup-node@v6
         with:
           node-version-file: '.nvmrc'
       - run: npm ci
@@ -25,7 +25,7 @@ jobs:
 
       - name: Upload test results
         if: ${{ !cancelled() }}
-        uses: actions/upload-artifact@v4
+        uses: actions/upload-artifact@v5
         with:
           name: test-results
           path: __tests__/__results__/*.xml

.github/workflows/manual-run.yml

@@ -8,7 +8,7 @@ jobs:
     runs-on: ubuntu-latest
 
     steps:
-      - uses: actions/checkout@v4
+      - uses: actions/checkout@v5
       - run: npm ci
       - run: npm run build
       - run: npm test

.github/workflows/test-report.yml

@@ -11,7 +11,7 @@ jobs:
     name: Workflow test
     runs-on: ubuntu-latest
     steps:
-    - uses: actions/checkout@v4
+    - uses: actions/checkout@v5
     - uses: ./
       with:
         artifact: test-results

CHANGELOG.md

@@ -1,5 +1,11 @@
 # Changelog
 
+## 2.2.0
+* Feature: Add collapsed option to control report summary visibility https://github.com/dorny/test-reporter/pull/664
+* Fix badge encoding for values including underscore and hyphens https://github.com/dorny/test-reporter/pull/672
+* Fix missing `report-title` attribute in action definition https://github.com/dorny/test-reporter/pull/637
+* Refactor variable names to fix shadowing issues https://github.com/dorny/test-reporter/pull/630
+
 ## 2.1.1
 * Fix error when a TestMethod element does not have a className attribute in a trx file https://github.com/dorny/test-reporter/pull/623
 * Add stack trace from trx to summary https://github.com/dorny/test-reporter/pull/615

__tests__/__snapshots__/dart-json.test.ts.snap

@@ -1,4 +1,4 @@
-// Jest Snapshot v1, https://goo.gl/fbAQLP
+// Jest Snapshot v1, https://jestjs.io/docs/snapshot-testing
 
 exports[`dart-json tests matches report snapshot 1`] = `
 TestRunResult {

__tests__/__snapshots__/dotnet-nunit.test.ts.snap

@@ -1,4 +1,4 @@
-// Jest Snapshot v1, https://goo.gl/fbAQLP
+// Jest Snapshot v1, https://jestjs.io/docs/snapshot-testing
 
 exports[`dotnet-nunit tests report from ./reports/dotnet test results matches snapshot 1`] = `
 TestRunResult {

__tests__/__snapshots__/dotnet-trx.test.ts.snap

@@ -1,4 +1,4 @@
-// Jest Snapshot v1, https://goo.gl/fbAQLP
+// Jest Snapshot v1, https://jestjs.io/docs/snapshot-testing
 
 exports[`dotnet-trx tests matches dotnet-trx report snapshot 1`] = `
 TestRunResult {

__tests__/__snapshots__/golang-json.test.ts.snap

@@ -1,4 +1,4 @@
-// Jest Snapshot v1, https://goo.gl/fbAQLP
+// Jest Snapshot v1, https://jestjs.io/docs/snapshot-testing
 
 exports[`golang-json tests report from ./reports/dotnet test results matches snapshot 1`] = `
 TestRunResult {

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

@@ -1,4 +1,4 @@
-// Jest Snapshot v1, https://goo.gl/fbAQLP
+// Jest Snapshot v1, https://jestjs.io/docs/snapshot-testing
 
 exports[`java-junit tests report from apache/pulsar single suite test results matches snapshot 1`] = `
 TestRunResult {

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

@@ -1,4 +1,4 @@
-// Jest Snapshot v1, https://goo.gl/fbAQLP
+// Jest Snapshot v1, https://jestjs.io/docs/snapshot-testing
 
 exports[`jest-junit tests parsing ESLint report without timing information works - PR #134 1`] = `
 TestRunResult {

__tests__/__snapshots__/mocha-json.test.ts.snap

@@ -1,4 +1,4 @@
-// Jest Snapshot v1, https://goo.gl/fbAQLP
+// Jest Snapshot v1, https://jestjs.io/docs/snapshot-testing
 
 exports[`mocha-json tests report from ./reports/mocha-json test results matches snapshot 1`] = `
 TestRunResult {

__tests__/__snapshots__/rspec-json.test.ts.snap

@@ -1,4 +1,4 @@
-// Jest Snapshot v1, https://goo.gl/fbAQLP
+// Jest Snapshot v1, https://jestjs.io/docs/snapshot-testing
 
 exports[`rspec-json tests report from ./reports/rspec-json test results matches snapshot 1`] = `
 TestRunResult {

__tests__/__snapshots__/swift-xunit.test.ts.snap

@@ -1,4 +1,4 @@
-// Jest Snapshot v1, https://goo.gl/fbAQLP
+// Jest Snapshot v1, https://jestjs.io/docs/snapshot-testing
 
 exports[`swift-xunit tests report from swift test results matches snapshot 1`] = `
 TestRunResult {

__tests__/jest-junit.test.ts

@@ -207,4 +207,100 @@ describe('jest-junit tests', () => {
     // Report should have the title as the first line
     expect(report).toMatch(/^# My Custom Title\n/)
   })
+
+  it('report can be collapsed when configured', async () => {
+    const fixturePath = path.join(__dirname, 'fixtures', 'jest-junit.xml')
+    const filePath = normalizeFilePath(path.relative(__dirname, fixturePath))
+    const fileContent = fs.readFileSync(fixturePath, {encoding: 'utf8'})
+
+    const opts: ParseOptions = {
+      parseErrors: true,
+      trackedFiles: []
+    }
+
+    const parser = new JestJunitParser(opts)
+    const result = await parser.parse(filePath, fileContent)
+    const report = getReport([result], {
+      ...DEFAULT_OPTIONS,
+      collapsed: 'always'
+    })
+    // Report should include collapsible details
+    expect(report).toContain('<details><summary>Expand for details</summary>')
+    expect(report).toContain('</details>')
+  })
+
+  it('report is not collapsed when configured to never', async () => {
+    const fixturePath = path.join(__dirname, 'fixtures', 'jest-junit.xml')
+    const filePath = normalizeFilePath(path.relative(__dirname, fixturePath))
+    const fileContent = fs.readFileSync(fixturePath, {encoding: 'utf8'})
+
+    const opts: ParseOptions = {
+      parseErrors: true,
+      trackedFiles: []
+    }
+
+    const parser = new JestJunitParser(opts)
+    const result = await parser.parse(filePath, fileContent)
+    const report = getReport([result], {
+      ...DEFAULT_OPTIONS,
+      collapsed: 'never'
+    })
+    // Report should not include collapsible details
+    expect(report).not.toContain('<details><summary>Expand for details</summary>')
+    expect(report).not.toContain('</details>')
+  })
+
+  it('report auto-collapses when all tests pass', async () => {
+    // Test with a fixture that has all passing tests (no failures)
+    const fixturePath = path.join(__dirname, 'fixtures', 'jest-junit-eslint.xml')
+    const filePath = normalizeFilePath(path.relative(__dirname, fixturePath))
+    const fileContent = fs.readFileSync(fixturePath, {encoding: 'utf8'})
+
+    const opts: ParseOptions = {
+      parseErrors: true,
+      trackedFiles: []
+    }
+
+    const parser = new JestJunitParser(opts)
+    const result = await parser.parse(filePath, fileContent)
+
+    // Verify this fixture has no failures
+    expect(result.failed).toBe(0)
+
+    const report = getReport([result], {
+      ...DEFAULT_OPTIONS,
+      collapsed: 'auto'
+    })
+
+    // Should collapse when all tests pass
+    expect(report).toContain('<details><summary>Expand for details</summary>')
+    expect(report).toContain('</details>')
+  })
+
+  it('report does not auto-collapse when tests fail', async () => {
+    // Test with a fixture that has failing tests
+    const fixturePath = path.join(__dirname, 'fixtures', 'jest-junit.xml')
+    const filePath = normalizeFilePath(path.relative(__dirname, fixturePath))
+    const fileContent = fs.readFileSync(fixturePath, {encoding: 'utf8'})
+
+    const opts: ParseOptions = {
+      parseErrors: true,
+      trackedFiles: []
+    }
+
+    const parser = new JestJunitParser(opts)
+    const result = await parser.parse(filePath, fileContent)
+
+    // Verify this fixture has failures
+    expect(result.failed).toBeGreaterThan(0)
+
+    const report = getReport([result], {
+      ...DEFAULT_OPTIONS,
+      collapsed: 'auto'
+    })
+
+    // Should not collapse when there are failures
+    expect(report).not.toContain('<details><summary>Expand for details</summary>')
+    expect(report).not.toContain('</details>')
+  })
 })

__tests__/report/get-report.test.ts

@@ -0,0 +1,120 @@
+import {getBadge, DEFAULT_OPTIONS, ReportOptions} from '../../src/report/get-report'
+
+describe('getBadge', () => {
+  describe('URI encoding with special characters', () => {
+    it('generates correct URI with simple badge title', () => {
+      const options: ReportOptions = {
+        ...DEFAULT_OPTIONS,
+        badgeTitle: 'tests'
+      }
+      const badge = getBadge(5, 0, 1, options)
+      expect(badge).toBe('![Tests passed successfully](https://img.shields.io/badge/tests-5%20passed%2C%201%20skipped-success)')
+    })
+
+    it('handles badge title with single hyphen', () => {
+      const options: ReportOptions = {
+        ...DEFAULT_OPTIONS,
+        badgeTitle: 'unit-tests'
+      }
+      const badge = getBadge(3, 0, 0, options)
+      // The hyphen in the badge title should be encoded as --
+      expect(badge).toBe('![Tests passed successfully](https://img.shields.io/badge/unit--tests-3%20passed-success)')
+    })
+
+    it('handles badge title with multiple hyphens', () => {
+      const options: ReportOptions = {
+        ...DEFAULT_OPTIONS,
+        badgeTitle: 'integration-api-tests'
+      }
+      const badge = getBadge(10, 0, 0, options)
+      // All hyphens in the title should be encoded as --
+      expect(badge).toBe('![Tests passed successfully](https://img.shields.io/badge/integration--api--tests-10%20passed-success)')
+    })
+
+    it('handles badge title with multiple underscores', () => {
+      const options: ReportOptions = {
+        ...DEFAULT_OPTIONS,
+        badgeTitle: 'my_integration_test'
+      }
+      const badge = getBadge(10, 0, 0, options)
+      // All underscores in the title should be encoded as __
+      expect(badge).toBe('![Tests passed successfully](https://img.shields.io/badge/my__integration__test-10%20passed-success)')
+    })
+
+    it('handles badge title with version format containing hyphen', () => {
+      const options: ReportOptions = {
+        ...DEFAULT_OPTIONS,
+        badgeTitle: 'MariaDb 12.0-ubi database tests'
+      }
+      const badge = getBadge(1, 0, 0, options)
+      // The hyphen in "12.0-ubi" should be encoded as --
+      expect(badge).toBe('![Tests passed successfully](https://img.shields.io/badge/MariaDb%2012.0--ubi%20database%20tests-1%20passed-success)')
+    })
+
+    it('handles badge title with dots and hyphens', () => {
+      const options: ReportOptions = {
+        ...DEFAULT_OPTIONS,
+        badgeTitle: 'v1.2.3-beta-test'
+      }
+      const badge = getBadge(4, 1, 0, options)
+      expect(badge).toBe('![Tests failed](https://img.shields.io/badge/v1.2.3--beta--test-4%20passed%2C%201%20failed-critical)')
+    })
+
+    it('preserves structural hyphens between label and message', () => {
+      const options: ReportOptions = {
+        ...DEFAULT_OPTIONS,
+        badgeTitle: 'test-suite'
+      }
+      const badge = getBadge(2, 3, 1, options)
+      // The URI should have literal hyphens separating title-message-color
+      expect(badge).toBe('![Tests failed](https://img.shields.io/badge/test--suite-2%20passed%2C%203%20failed%2C%201%20skipped-critical)')
+    })
+  })
+
+  describe('generates test outcome as color name for imgshields', () => {
+    it('uses success color when all tests pass', () => {
+      const options: ReportOptions = {...DEFAULT_OPTIONS}
+      const badge = getBadge(5, 0, 0, options)
+      expect(badge).toContain('-success)')
+    })
+
+    it('uses critical color when tests fail', () => {
+      const options: ReportOptions = {...DEFAULT_OPTIONS}
+      const badge = getBadge(5, 2, 0, options)
+      expect(badge).toContain('-critical)')
+    })
+
+    it('uses yellow color when no tests found', () => {
+      const options: ReportOptions = {...DEFAULT_OPTIONS}
+      const badge = getBadge(0, 0, 0, options)
+      expect(badge).toContain('-yellow)')
+    })
+  })
+
+  describe('badge message composition', () => {
+    it('includes only passed count when no failures or skips', () => {
+      const options: ReportOptions = {...DEFAULT_OPTIONS}
+      const badge = getBadge(5, 0, 0, options)
+      expect(badge).toBe('![Tests passed successfully](https://img.shields.io/badge/tests-5%20passed-success)')
+    })
+
+    it('includes passed and failed counts', () => {
+      const options: ReportOptions = {...DEFAULT_OPTIONS}
+      const badge = getBadge(5, 2, 0, options)
+      expect(badge).toBe('![Tests failed](https://img.shields.io/badge/tests-5%20passed%2C%202%20failed-critical)')
+    })
+
+    it('includes passed, failed and skipped counts', () => {
+      const options: ReportOptions = {...DEFAULT_OPTIONS}
+      const badge = getBadge(5, 2, 1, options)
+      expect(badge).toBe('![Tests failed](https://img.shields.io/badge/tests-5%20passed%2C%202%20failed%2C%201%20skipped-critical)')
+    })
+
+    it('uses "none" message when no tests', () => {
+      const options: ReportOptions = {...DEFAULT_OPTIONS}
+      const badge = getBadge(0, 0, 0, options)
+      expect(badge).toBe('![Tests passed successfully](https://img.shields.io/badge/tests-none-yellow)')
+    })
+  })
+})
+

__tests__/utils/parse-utils.test.ts

@@ -32,6 +32,6 @@ describe('parseNetDuration', () => {
   })
 
   it('throws when string has invalid format', () => {
-    expect(() => parseNetDuration('12:34:56 not a duration')).toThrowError(/^Invalid format/)
+    expect(() => parseNetDuration('12:34:56 not a duration')).toThrow(/^Invalid format/)
   })
 })

action.yml

@@ -68,6 +68,10 @@ inputs:
   working-directory:
     description: Relative path under $GITHUB_WORKSPACE where the repository was checked out
     required: false
+  report-title:
+    description: Title for the test report summary
+    required: false
+    default: ''
   only-summary:
     description: |
       Allows you to generate only the summary.
@@ -85,6 +89,14 @@ inputs:
     description: Customize badge title
     required: false
     default: 'tests'
+  collapsed:
+    description: |
+      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
+    required: false
+    default: 'auto'
   token:
     description: GitHub Access Token
     required: false

dist/index.js

@@ -309,6 +309,7 @@ class TestReporter {
     useActionsSummary = core.getInput('use-actions-summary', { required: false }) === 'true';
     badgeTitle = core.getInput('badge-title', { required: false });
     reportTitle = core.getInput('report-title', { required: false });
+    collapsed = core.getInput('collapsed', { required: false });
     token = core.getInput('token', { required: true });
     octokit;
     context = (0, github_utils_1.getCheckRunContext)();
@@ -322,6 +323,10 @@ class TestReporter {
             core.setFailed(`Input parameter 'list-tests' has invalid value`);
             return;
         }
+        if (this.collapsed !== 'auto' && this.collapsed !== 'always' && this.collapsed !== 'never') {
+            core.setFailed(`Input parameter 'collapsed' has invalid value`);
+            return;
+        }
         if (isNaN(this.maxAnnotations) || this.maxAnnotations < 0 || this.maxAnnotations > 50) {
             core.setFailed(`Input parameter 'max-annotations' has invalid value`);
             return;
@@ -401,7 +406,7 @@ class TestReporter {
                 throw error;
             }
         }
-        const { listSuites, listTests, onlySummary, useActionsSummary, badgeTitle, reportTitle } = 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);
         const skipped = results.reduce((sum, tr) => sum + tr.skipped, 0);
@@ -415,7 +420,8 @@ class TestReporter {
                 onlySummary,
                 useActionsSummary,
                 badgeTitle,
-                reportTitle
+                reportTitle,
+                collapsed
             });
             core.info('Summary content:');
             core.info(summary);
@@ -443,7 +449,8 @@ class TestReporter {
                 onlySummary,
                 useActionsSummary,
                 badgeTitle,
-                reportTitle
+                reportTitle,
+                collapsed
             });
             core.info('Creating annotations');
             const annotations = (0, get_annotations_1.getAnnotations)(results, this.maxAnnotations);
@@ -837,12 +844,12 @@ class DotnetNunitParser {
             .map(suite => suite.$.name)
             .join('.');
         const groupName = suitesWithoutTheories[suitesWithoutTheories.length - 1].$.name;
-        let existingSuite = result.find(existingSuite => existingSuite.name === suiteName);
+        let existingSuite = result.find(suite => suite.name === suiteName);
         if (existingSuite === undefined) {
             existingSuite = new test_results_1.TestSuiteResult(suiteName, []);
             result.push(existingSuite);
         }
-        let existingGroup = existingSuite.groups.find(existingGroup => existingGroup.name === groupName);
+        let existingGroup = existingSuite.groups.find(group => group.name === groupName);
         if (existingGroup === undefined) {
             existingGroup = new test_results_1.TestGroupResult(groupName, []);
             existingSuite.groups.push(existingGroup);
@@ -1909,6 +1916,7 @@ var __importStar = (this && this.__importStar) || (function () {
 Object.defineProperty(exports, "__esModule", ({ value: true }));
 exports.DEFAULT_OPTIONS = void 0;
 exports.getReport = getReport;
+exports.getBadge = getBadge;
 const core = __importStar(__nccwpck_require__(7484));
 const markdown_utils_1 = __nccwpck_require__(5129);
 const node_utils_1 = __nccwpck_require__(5384);
@@ -1923,7 +1931,8 @@ exports.DEFAULT_OPTIONS = {
     onlySummary: false,
     useActionsSummary: true,
     badgeTitle: 'tests',
-    reportTitle: ''
+    reportTitle: '',
+    collapsed: 'auto'
 };
 function getReport(results, options = exports.DEFAULT_OPTIONS) {
     core.info('Generating check run summary');
@@ -2022,13 +2031,17 @@ function getBadge(passed, failed, skipped, options) {
         color = 'yellow';
     }
     const hint = failed > 0 ? 'Tests failed' : 'Tests passed successfully';
-    const uri = encodeURIComponent(`${options.badgeTitle}-${message}-${color}`);
+    const encodedBadgeTitle = encodeImgShieldsURIComponent(options.badgeTitle);
-    return `![${hint}](https://img.shields.io/badge/${uri})`;
+    const encodedMessage = encodeImgShieldsURIComponent(message);
+    const encodedColor = encodeImgShieldsURIComponent(color);
+    return `![${hint}](https://img.shields.io/badge/${encodedBadgeTitle}-${encodedMessage}-${encodedColor})`;
 }
 function getTestRunsReport(testRuns, options) {
     const sections = [];
     const totalFailed = testRuns.reduce((sum, tr) => sum + tr.failed, 0);
-    if (totalFailed === 0) {
+    // Determine if report should be collapsed based on collapsed option
+    const shouldCollapse = options.collapsed === 'always' || (options.collapsed === 'auto' && totalFailed === 0);
+    if (shouldCollapse) {
         sections.push(`<details><summary>Expand for details</summary>`);
         sections.push(` `);
     }
@@ -2053,7 +2066,7 @@ function getTestRunsReport(testRuns, options) {
         const suitesReports = testRuns.map((tr, i) => getSuitesReport(tr, i, options)).flat();
         sections.push(...suitesReports);
     }
-    if (totalFailed === 0) {
+    if (shouldCollapse) {
         sections.push(`</details>`);
     }
     return sections;
@@ -2153,6 +2166,9 @@ function getResultIcon(result) {
             return '';
     }
 }
+function encodeImgShieldsURIComponent(component) {
+    return encodeURIComponent(component).replace(/-/g, '--').replace(/_/g, '__');
+}
 
 
 /***/ }),
@@ -27847,6 +27863,7 @@ module.exports = {
 
   // Replace globs with equivalent patterns to reduce parsing time.
   REPLACEMENTS: {
+    __proto__: null,
     '***': '*',
     '**/**': '**',
     '**/**/**': '**'

package.json

@@ -1,6 +1,6 @@
 {
   "name": "test-reporter",
-  "version": "2.1.1",
+  "version": "2.2.0",
   "private": true,
   "description": "Presents test results from popular testing frameworks as Github check run",
   "main": "lib/main.js",
@@ -42,33 +42,32 @@
     "adm-zip": "^0.5.16",
     "fast-glob": "^3.3.3",
     "got": "^11.8.6",
-    "picomatch": "^4.0.2",
+    "picomatch": "^4.0.3",
     "xml2js": "^0.6.2"
   },
   "devDependencies": {
     "@octokit/webhooks-types": "^7.6.1",
     "@types/adm-zip": "^0.5.7",
-    "@types/jest": "^29.5.14",
+    "@types/jest": "^30.0.0",
-    "@types/node": "^20.19.2",
+    "@types/node": "^20.19.23",
-    "@types/picomatch": "^2.3.4",
+    "@types/picomatch": "^4.0.2",
     "@types/xml2js": "^0.4.14",
     "@typescript-eslint/eslint-plugin": "^7.18.0",
     "@typescript-eslint/parser": "^7.18.0",
-    "@vercel/ncc": "^0.38.3",
+    "@vercel/ncc": "^0.38.4",
     "eol-converter-cli": "^1.1.0",
     "eslint": "^8.57.1",
     "eslint-import-resolver-typescript": "^3.10.1",
     "eslint-plugin-github": "^4.10.2",
     "eslint-plugin-import": "^2.32.0",
     "eslint-plugin-jest": "^28.14.0",
-    "eslint-plugin-prettier": "^5.5.1",
+    "eslint-plugin-prettier": "^5.5.4",
-    "jest": "^29.7.0",
+    "jest": "^30.2.0",
-    "jest-circus": "^29.7.0",
     "jest-junit": "^16.0.0",
-    "js-yaml": "^4.1.0",
+    "js-yaml": "^4.1.1",
     "prettier": "^3.6.2",
-    "ts-jest": "^29.4.0",
+    "ts-jest": "^29.4.5",
-    "typescript": "^5.8.3"
+    "typescript": "^5.9.3"
   },
   "jest-junit": {
     "suiteName": "jest tests",

src/main.ts

@@ -49,6 +49,7 @@ class TestReporter {
   readonly useActionsSummary = core.getInput('use-actions-summary', {required: false}) === 'true'
   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'
   readonly token = core.getInput('token', {required: true})
   readonly octokit: InstanceType<typeof GitHub>
   readonly context = getCheckRunContext()
@@ -66,6 +67,11 @@ class TestReporter {
       return
     }
 
+    if (this.collapsed !== 'auto' && this.collapsed !== 'always' && this.collapsed !== 'never') {
+      core.setFailed(`Input parameter 'collapsed' has invalid value`)
+      return
+    }
+
     if (isNaN(this.maxAnnotations) || this.maxAnnotations < 0 || this.maxAnnotations > 50) {
       core.setFailed(`Input parameter 'max-annotations' has invalid value`)
       return
@@ -166,7 +172,7 @@ class TestReporter {
       }
     }
 
-    const {listSuites, listTests, onlySummary, useActionsSummary, badgeTitle, reportTitle} = 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)
@@ -182,7 +188,8 @@ class TestReporter {
         onlySummary,
         useActionsSummary,
         badgeTitle,
-        reportTitle
+        reportTitle,
+        collapsed
       })
 
       core.info('Summary content:')
@@ -211,7 +218,8 @@ class TestReporter {
         onlySummary,
         useActionsSummary,
         badgeTitle,
-        reportTitle
+        reportTitle,
+        collapsed
       })
 
       core.info('Creating annotations')

src/parsers/dotnet-nunit/dotnet-nunit-parser.ts

@@ -77,13 +77,13 @@ export class DotnetNunitParser implements TestParser {
       .join('.')
     const groupName = suitesWithoutTheories[suitesWithoutTheories.length - 1].$.name
 
-    let existingSuite = result.find(existingSuite => existingSuite.name === suiteName)
+    let existingSuite = result.find(suite => suite.name === suiteName)
     if (existingSuite === undefined) {
       existingSuite = new TestSuiteResult(suiteName, [])
       result.push(existingSuite)
     }
 
-    let existingGroup = existingSuite.groups.find(existingGroup => existingGroup.name === groupName)
+    let existingGroup = existingSuite.groups.find(group => group.name === groupName)
     if (existingGroup === undefined) {
       existingGroup = new TestGroupResult(groupName, [])
       existingSuite.groups.push(existingGroup)

src/report/get-report.ts

@@ -16,6 +16,7 @@ export interface ReportOptions {
   useActionsSummary: boolean
   badgeTitle: string
   reportTitle: string
+  collapsed: 'auto' | 'always' | 'never'
 }
 
 export const DEFAULT_OPTIONS: ReportOptions = {
@@ -25,7 +26,8 @@ export const DEFAULT_OPTIONS: ReportOptions = {
   onlySummary: false,
   useActionsSummary: true,
   badgeTitle: 'tests',
-  reportTitle: ''
+  reportTitle: '',
+  collapsed: 'auto'
 }
 
 export function getReport(results: TestRunResult[], options: ReportOptions = DEFAULT_OPTIONS): string {
@@ -125,7 +127,7 @@ function getReportBadge(results: TestRunResult[], options: ReportOptions): strin
   return getBadge(passed, failed, skipped, options)
 }
 
-function getBadge(passed: number, failed: number, skipped: number, options: ReportOptions): string {
+export function getBadge(passed: number, failed: number, skipped: number, options: ReportOptions): string {
   const text = []
   if (passed > 0) {
     text.push(`${passed} passed`)
@@ -145,14 +147,20 @@ function getBadge(passed: number, failed: number, skipped: number, options: Repo
     color = 'yellow'
   }
   const hint = failed > 0 ? 'Tests failed' : 'Tests passed successfully'
-  const uri = encodeURIComponent(`${options.badgeTitle}-${message}-${color}`)
+  const encodedBadgeTitle = encodeImgShieldsURIComponent(options.badgeTitle)
-  return `![${hint}](https://img.shields.io/badge/${uri})`
+  const encodedMessage = encodeImgShieldsURIComponent(message)
+  const encodedColor = encodeImgShieldsURIComponent(color)
+  return `![${hint}](https://img.shields.io/badge/${encodedBadgeTitle}-${encodedMessage}-${encodedColor})`
 }
 
 function getTestRunsReport(testRuns: TestRunResult[], options: ReportOptions): string[] {
   const sections: string[] = []
   const totalFailed = testRuns.reduce((sum, tr) => sum + tr.failed, 0)
-  if (totalFailed === 0) {
+
+  // Determine if report should be collapsed based on collapsed option
+  const shouldCollapse = options.collapsed === 'always' || (options.collapsed === 'auto' && totalFailed === 0)
+
+  if (shouldCollapse) {
     sections.push(`<details><summary>Expand for details</summary>`)
     sections.push(` `)
   }
@@ -185,7 +193,7 @@ function getTestRunsReport(testRuns: TestRunResult[], options: ReportOptions): s
     sections.push(...suitesReports)
   }
 
-  if (totalFailed === 0) {
+  if (shouldCollapse) {
     sections.push(`</details>`)
   }
   return sections
@@ -305,3 +313,7 @@ function getResultIcon(result: TestExecutionResult): string {
       return ''
   }
 }
+
+function encodeImgShieldsURIComponent(component: string): string {
+  return encodeURIComponent(component).replace(/-/g, '--').replace(/_/g, '__')
+}