Skip to content

Commit

Permalink
- fix group header (no merged cells possible)
Browse files Browse the repository at this point in the history
- expand README
- update test for PR run
  • Loading branch information
mikepenz committed Nov 3, 2024
1 parent 0d45d52 commit 9cc197e
Show file tree
Hide file tree
Showing 6 changed files with 12 additions and 5 deletions.
1 change: 1 addition & 0 deletions .github/workflows/build.yml
Original file line number Diff line number Diff line change
Expand Up @@ -95,6 +95,7 @@ jobs:
report_paths: 'test_results/nested/multi-level.xml'
include_passed: true
detailed_summary: true
skip_annotations: true
check_title_template: '{{SUITE_NAME}} | {{TEST_NAME}}'
annotate_only: ${{ github.event_name == 'workflow_dispatch' }}

Expand Down
2 changes: 2 additions & 0 deletions README.md
Original file line number Diff line number Diff line change
Expand Up @@ -96,12 +96,14 @@ jobs:
| `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) |
| `detailed_summary` | Optional. Include table with all test results in the summary. Defaults to `false`. |
| `flaky_summary` | Optional. Include table with all falky results in the summary. Defaults to `false`. |
| `group_suite` | Optional. If enabled, will group the testcases by test suite in the `detailed_summary`. Defaults to `false`. |
| `comment` | Optional. Enables a comment being added to the PR with the summary tables (Respects the summary configuration flags). Defaults to `false`. |
| `updateComment` | Optional. If a prior action run comment exists, it is updated. If disabled, new comments are creted for each run. Defaults to `true`. |
| `annotate_notice` | Optional. Annotate passed test results along with warning/failed ones. Defaults to `false`. (Changed in v3.5.0) |
| `follow_symlink` | Optional. Enables to follow symlinks when searching test files via the globber. Defaults to `false`. |
| `job_name` | Optional. Specify the name of a check to update |
| `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`. |
| `skip_annotations` | Optional. Setting this flag will result in no annotations being added to the run. Defaults to `false`. |

### Common Configurations

Expand Down
8 changes: 6 additions & 2 deletions __tests__/table.test.ts
Original file line number Diff line number Diff line change
Expand Up @@ -137,7 +137,9 @@ describe('buildSummaryTables', () => {
}
],
[
"ABC-0199: XMPP Ping"
"ABC-0199: XMPP Ping",
"",
""
],
[
"",
Expand All @@ -150,7 +152,9 @@ describe('buildSummaryTables', () => {
"✅ pass"
],
[
"ABC-0045: Multi-User Chat"
"ABC-0045: Multi-User Chat",
"",
""
],
[
"",
Expand Down
2 changes: 1 addition & 1 deletion dist/index.js

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

2 changes: 1 addition & 1 deletion dist/index.js.map

Large diffs are not rendered by default.

2 changes: 1 addition & 1 deletion src/table.ts
Original file line number Diff line number Diff line change
Expand Up @@ -100,7 +100,7 @@ function appendDetailsTable(
annotation => includePassed || annotation.annotation_level !== 'notice'
)
if (annotations.length > 0) {
detailsTable.push([`${testResult.name}`])
detailsTable.push([`${testResult.name}`, ``, ``])
for (const annotation of annotations) {
detailsTable.push([
``,
Expand Down

0 comments on commit 9cc197e

Please sign in to comment.