Skip to content

Commit 9cc197e

Browse files
committed
- fix group header (no merged cells possible)
- expand README - update test for PR run
1 parent 0d45d52 commit 9cc197e

File tree

6 files changed

+12
-5
lines changed

6 files changed

+12
-5
lines changed

.github/workflows/build.yml

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -95,6 +95,7 @@ jobs:
9595
report_paths: 'test_results/nested/multi-level.xml'
9696
include_passed: true
9797
detailed_summary: true
98+
skip_annotations: true
9899
check_title_template: '{{SUITE_NAME}} | {{TEST_NAME}}'
99100
annotate_only: ${{ github.event_name == 'workflow_dispatch' }}
100101

README.md

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -96,12 +96,14 @@ jobs:
9696
| `job_summary` | Optional. Enables the publishing of the job summary for the results. Defaults to `true`. May be required to disable [Enterprise Server](https://github.com/mikepenz/action-junit-report/issues/637) |
9797
| `detailed_summary` | Optional. Include table with all test results in the summary. Defaults to `false`. |
9898
| `flaky_summary` | Optional. Include table with all falky results in the summary. Defaults to `false`. |
99+
| `group_suite` | Optional. If enabled, will group the testcases by test suite in the `detailed_summary`. Defaults to `false`. |
99100
| `comment` | Optional. Enables a comment being added to the PR with the summary tables (Respects the summary configuration flags). Defaults to `false`. |
100101
| `updateComment` | Optional. If a prior action run comment exists, it is updated. If disabled, new comments are creted for each run. Defaults to `true`. |
101102
| `annotate_notice` | Optional. Annotate passed test results along with warning/failed ones. Defaults to `false`. (Changed in v3.5.0) |
102103
| `follow_symlink` | Optional. Enables to follow symlinks when searching test files via the globber. Defaults to `false`. |
103104
| `job_name` | Optional. Specify the name of a check to update |
104105
| `annotations_limit` | Optional. Specify the limit for annotations. This will also interrupt parsing all test-suites if the limit is reached. Defaults to: `No Limit`. |
106+
| `skip_annotations` | Optional. Setting this flag will result in no annotations being added to the run. Defaults to `false`. |
105107

106108
### Common Configurations
107109

__tests__/table.test.ts

Lines changed: 6 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -137,7 +137,9 @@ describe('buildSummaryTables', () => {
137137
}
138138
],
139139
[
140-
"ABC-0199: XMPP Ping"
140+
"ABC-0199: XMPP Ping",
141+
"",
142+
""
141143
],
142144
[
143145
"",
@@ -150,7 +152,9 @@ describe('buildSummaryTables', () => {
150152
"✅ pass"
151153
],
152154
[
153-
"ABC-0045: Multi-User Chat"
155+
"ABC-0045: Multi-User Chat",
156+
"",
157+
""
154158
],
155159
[
156160
"",

dist/index.js

Lines changed: 1 addition & 1 deletion
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

dist/index.js.map

Lines changed: 1 addition & 1 deletion
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

src/table.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -100,7 +100,7 @@ function appendDetailsTable(
100100
annotation => includePassed || annotation.annotation_level !== 'notice'
101101
)
102102
if (annotations.length > 0) {
103-
detailsTable.push([`${testResult.name}`])
103+
detailsTable.push([`${testResult.name}`, ``, ``])
104104
for (const annotation of annotations) {
105105
detailsTable.push([
106106
``,

0 commit comments

Comments
 (0)