-
-
Notifications
You must be signed in to change notification settings - Fork 141
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
Improve Test Coverage #213
Comments
@SudharakaP would this one be eligible to get a hacktoberfest label? |
@Strat1987 : Yes, you are most welcome to do it. Please feel free to suggest a PR. I will apply the Hacktoberfest label. 😄 |
I'm currently up to 64% and trying to triage on what seems sensible to test, is there any specific target set? |
@Strat1987 : Some of the classed I think are quite nice to have a good test coverage are highlighted below; maybe some others who has more experience working with jhipster-online will have some further ideas on this.
@pascalgrimaud @jdubois @PierreBesson : Let us know if you have any ideas on this one. 😄 |
Agree with your suggestions, @SudharakaP |
@Strat1987 : Also feel free to improve any uncovered classes in the list; https://sonarcloud.io/component_measures?id=jhipster_jhipster-online&metric=coverage&view=list I have removed the |
Of the list mentioned above I covered the JdlMetadataService. I have to admit that most of the listed once would take up more time / complexity than I can spare on this. |
This improves the test coverage of JdlService class. Related to #213
We should improve the test coverage of the project; https://sonarcloud.io/dashboard?id=jhipster_jhipster-online
The text was updated successfully, but these errors were encountered: