Comments (11)
@vitalyk-multinarity 35 minutes? That seems intense.
Which configuration do you use?
Given that annotateOnly
has to iterate over all annotations, and uses the GitHub
interface to annotate each and all of them: https://github.com/mikepenz/action-junit-report/blob/main/src/annotator.ts#L31-L45 (with annotateNotice
) it wouldn't surprise me to take longer.
As far as I am aware there is no batch API for these available.
Ways to speed up in that case:
- annotateNotice = false
- use the checks instead of annotate only
I use this action with thousands of test-cases in different projects and it usually completes within seconds.
For these cases we do have annotateOnly = false
and use the normal checks + summary.
from action-junit-report.
@mikepenz - thank you!
I re-read the Readme, but I'm still not sure what is effect of "annotate_notice = true/false".
My configuration is
uses: mikepenz/action-junit-report@v3
if: always()
with:
report_paths: '/tmp/ctest-*-report.xml'
fail_on_failure: true
detailed_summary: true
include_passed: true
which converts to:
Run mikepenz/action-junit-report@v3
with:
report_paths: /tmp/ctest-*-report.xml
fail_on_failure: true
detailed_summary: true
include_passed: true
token: ***
exclude_sources: /build/,/__pycache__/
annotate_only: false
update_check: false
check_name: JUnit Test Report
require_tests: false
check_retries: false
transformers: []
job_summary: true
annotate_notice: true
from action-junit-report.
@vitalyk-multinarity ok given the output you are not using annotate_only: true
so that's not the source of the problem.
Given the other issue you had created I understand that you will want to use include_passed: true
, but would it possible to compare speed with it disabled?
from action-junit-report.
Without "include_passed: true" it takes just 2 seconds :-)
from action-junit-report.
@vitalyk-multinarity if I remember correctly. you use include_passed
mainly to get them also included in the summary. but you actually don't mind the annotations at the specific file locations. correct?
If so, I'll look into it and check optimisations we can apply to skip any file matching and similar.
from action-junit-report.
if I remember correctly. you use include_passed mainly to get them also included in the summary.
Right (to be precise, to see a list of passed tests with their status)!
If so, I'll look into it and check optimisations we can apply to skip any file matching and similar
Many thanks!
from action-junit-report.
Could you please test how it performs using f4441d735cac5cebc8f7c6dadf8eef97865b4d32
?
See the related PR here: #658
- ultimately disabling the resolving of the
path
on the filesystem unless annotation for passed tests is requested.
(still need to adjust tests for this PR due to the adjusted handling for the annotations)
from action-junit-report.
Many thanks - with this commit it takes just one second, as far as I added 'annotate_notice: false':
uses: mikepenz/action-junit-report@f4441d735cac5cebc8f7c6dadf8eef97865b4d32
if: always()
with:
report_paths: '/tmp/ctest-*-report.xml'
fail_on_failure: true
detailed_summary: true
include_passed: true
annotate_notice: false
from action-junit-report.
Thank you for testing this.
from action-junit-report.
This will be in the next release, which will probably be released later this week.
You can use the git commit SHA pinned version until then
from action-junit-report.
Mike, many thanks!
Vitaly
from action-junit-report.
Related Issues (20)
- Doesn't include errors HOT 3
- Annotation truncates failure output/callstack HOT 6
- Skipping all Annotations HOT 14
- Support ESLint or Jest junit reports HOT 6
- Different line number representation (xunit) HOT 3
- Weird behavior with multiple `report_paths` and failures not being processed HOT 4
- Test report stopped working without change HOT 8
- require_tests flag is not working HOT 5
- Editor points out that github_token is required HOT 1
- Skipped tests are show as passed when included_passed set to true HOT 3
- Test failures not correctly recognized HOT 8
- action using deprecate 'set-output' command HOT 2
- Documentation for Maven Surefire HOT 2
- Out of memory error on large project HOT 14
- Failed tests are not shown in Annotations HOT 9
- Check Run Not Created when update_check: true HOT 2
- act error: Error: The runs.using key in action.yml must be one of: [composite docker node12 node16], got node20 HOT 4
- Invalid character entity when using with dastardly HOT 6
- Error: Converting circular structure to JSON HOT 9
- Option to explicitly fail check run HOT 8
Recommend Projects
-
React
A declarative, efficient, and flexible JavaScript library for building user interfaces.
-
Vue.js
🖖 Vue.js is a progressive, incrementally-adoptable JavaScript framework for building UI on the web.
-
Typescript
TypeScript is a superset of JavaScript that compiles to clean JavaScript output.
-
TensorFlow
An Open Source Machine Learning Framework for Everyone
-
Django
The Web framework for perfectionists with deadlines.
-
Laravel
A PHP framework for web artisans
-
D3
Bring data to life with SVG, Canvas and HTML. 📊📈🎉
-
Recommend Topics
-
javascript
JavaScript (JS) is a lightweight interpreted programming language with first-class functions.
-
web
Some thing interesting about web. New door for the world.
-
server
A server is a program made to process requests and deliver data to clients.
-
Machine learning
Machine learning is a way of modeling and interpreting data that allows a piece of software to respond intelligently.
-
Visualization
Some thing interesting about visualization, use data art
-
Game
Some thing interesting about game, make everyone happy.
Recommend Org
-
Facebook
We are working to build community through open source technology. NB: members must have two-factor auth.
-
Microsoft
Open source projects and samples from Microsoft.
-
Google
Google ❤️ Open Source for everyone.
-
Alibaba
Alibaba Open Source for everyone
-
D3
Data-Driven Documents codes.
-
Tencent
China tencent open source team.
from action-junit-report.