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

How to enhance the reports with test metrics ? #291

Closed
rebse opened this issue Nov 18, 2021 · 6 comments · Fixed by #302
Closed

How to enhance the reports with test metrics ? #291

rebse opened this issue Nov 18, 2021 · 6 comments · Fixed by #302
Assignees
Labels
done This issue was solved but not merged in master enhancement New feature or request
Milestone

Comments

@rebse
Copy link

rebse commented Nov 18, 2021

i would like to extend the reports with test metrics = unit tests (number, errors, failures, skipped, success) , code coverage
Is that possible ?

@rebse rebse added the question Further information is requested label Nov 18, 2021
@Sancretor
Copy link
Contributor

Hi @rebse

Right now, we only show the total test coverage of the project.
I guess we could easily add the data you mentionned (number, errors, success, skipped, etc.), but it will require the development of a new feature.

I can change this issue to be an enhancement for the future release if you'd like.
But I can't make any promise regarding when we'd be able to implement it.
What do you think ?

@Sancretor Sancretor added the enhancement New feature or request label Nov 18, 2021
@rebse
Copy link
Author

rebse commented Nov 18, 2021

Thanks for adding the enhancement label to this issue.
Until now i only implemented a bunch of custom Java rules, but have no experience with other plugins.
Would be great if you could give me some hints to get me going and i would be happy to participate
with a pull request.

@Sancretor
Copy link
Contributor

Hi @rebse
Sorry for the huge delay again...
I don't know if you already took a look into the codebase but here are some information to start with.

This enhancement will only impact the "report" folder inside the sources, the plugin part should remain unchanged.
The "providers" folder contains all classes which retrieve data from SonarQube server.
The "model" may be self explanatory, with classes handling the retrieved data.
The "exporters" folder is ... mostly a huge mess :) But here are classes used to replace placeholders inside the report templates with the retrieved data.
You may have a look at what already exists in "providers" & "model" to add some dedicated classes fort tests data.
In "exporters", you may have a look at "docx/DocXExporter.java" in order to call the dedicated methods to add data inside the report.

@Sancretor Sancretor added this to the 4.1.0 milestone Jan 5, 2022
@Sancretor Sancretor self-assigned this Jan 19, 2022
@Sancretor
Copy link
Contributor

Hi @rebse
Just to let you know that the feature is now implemented and merged in dev branch.
A couple more features/bugs are still to be merged before we could release the new version.
Hopefully, it will be very soon !

@rebse
Copy link
Author

rebse commented Jan 19, 2022

Hi @Sancretor :-)
great news!
And sorry, completely missed your mail 16 days ago :-(
due to stress from log4shell aftershocks ..

Gilbert

@Sancretor
Copy link
Contributor

No worries !
I usually take weeks or months to answer here so ... :)

@Sancretor Sancretor added done This issue was solved but not merged in master and removed question Further information is requested labels Jan 26, 2022
@Sancretor Sancretor mentioned this issue Jan 26, 2022
17 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
done This issue was solved but not merged in master enhancement New feature or request
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants