Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merge reports of multiple checks #195

Closed
mikepenz opened this issue Jun 9, 2021 · 5 comments
Closed

Merge reports of multiple checks #195

mikepenz opened this issue Jun 9, 2021 · 5 comments

Comments

@mikepenz
Copy link
Owner

mikepenz commented Jun 9, 2021

Running many checks with different check_name will result in the sidebar to contain all those entries.

It would be great if we could merge together the report so only one entry will be shown

Suggested by @iBotPeaches in this ticket:
#194

@mbien
Copy link

mbien commented Jul 2, 2022

github recently added job summaries:
https://github.blog/2022-05-09-supercharging-github-actions-with-job-summaries/

this reads like it might solve this problem too. (also requires less permissions as bonus)

@mikepenz
Copy link
Owner Author

mikepenz commented Jul 22, 2022

You may want to try the following PR which adds ability configure multiple imports in one step:

      - name: Test Multi test import
        uses: ./
        if: endsWith(github.ref, 'main') == false
        with:
          check_name: |-
            Example Multi JUnit Test Report
            Example Multi Pytest Report
          report_paths: |-
            **/surefire-reports/TEST-*.xml
            test_results/python/report.xml
          summary: |-
            <table><thead><tr><th> Application (src/applications) </th></tr></thead><tbody><tr><td> multi test </td></tr></tbody></table>
            \n
          check_title_template: |-
            {{SUITE_NAME}} | {{TEST_NAME}}
            \n

#606

@mikepenz
Copy link
Owner Author

This is now included in v3.3.0

@laughedelic
Copy link
Contributor

laughedelic commented Sep 19, 2022

Hi @mikepenz! I couldn't make this new feature work for the original use case:

Running many checks with different check_name will result in the sidebar to contain all those entries.
It would be great if we could merge together the report so only one entry will be shown

From what I see in your example in #195 (comment), you produce several sets of reports in the same job and use one action step to handle them all, via multiple check_names and the corresponding report_paths. But this seems to be a very different use case.

My understanding was that now it's possible to run the action on multiple parallel jobs with the same check_name and their results will get merged together. But when I tried that, I actually haven't got any check results.

Could you clarify how the new functionality covers the original use case and what's the intended usage?

@mikepenz
Copy link
Owner Author

@laughedelic with the API you outlined, it's possible to have multiple reports imported at once.

The idea is that this can be used if there are multiple different languages for example or different test report types. It's still possible to have individual runs though if this is preferred

Given the way GitHub handles multiple parallel jobs, the action can't actually combine them this way directly.
If you want to import those at once, what you would usually need is something like:

  • archive and upload the result files
  • have a step which is run after your tests / matrix build, ...
  • use the download action from GitHub to collect the reports
  • import all of them via the junit action

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants