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

Improve Test Coverage #213

Open
SudharakaP opened this issue Aug 8, 2020 · 7 comments
Open

Improve Test Coverage #213

SudharakaP opened this issue Aug 8, 2020 · 7 comments
Labels
Type: Enhancement 🚀 Something that modifies or adds to existing functionality

Comments

@SudharakaP
Copy link
Member

We should improve the test coverage of the project; https://sonarcloud.io/dashboard?id=jhipster_jhipster-online

@SudharakaP SudharakaP added Type: Enhancement 🚀 Something that modifies or adds to existing functionality Good First Issue Good as a startring point of contributions to this repo labels Aug 8, 2020
@Strat1987
Copy link
Contributor

@SudharakaP would this one be eligible to get a hacktoberfest label?

@SudharakaP
Copy link
Member Author

@Strat1987 : Yes, you are most welcome to do it. Please feel free to suggest a PR. I will apply the Hacktoberfest label. 😄

@SudharakaP SudharakaP added the Hacktoberfest 🖥️ Issue is eligible for Hacktoberfest 2020 label Oct 7, 2020
@Strat1987
Copy link
Contributor

I'm currently up to 64% and trying to triage on what seems sensible to test, is there any specific target set?

@SudharakaP
Copy link
Member Author

@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.

  • JHipsterService.java, GeneratorService.java - This is where we have the methods to generate the projects, the jdl configs, the ci-cd configs etc.

  • GitService.java - I am not sure how easy this is to test as it involves api calls to the git api.

  • JdlService.java and JdlMetadataService.java

  • GithubService.java and GitlabService.java

  • StatisticsService.java

@pascalgrimaud @jdubois @PierreBesson : Let us know if you have any ideas on this one. 😄

@pascalgrimaud
Copy link
Member

Agree with your suggestions, @SudharakaP

@SudharakaP
Copy link
Member Author

@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 good-first-issue label from this as I feel this might take some time to achieve as some test cases will be a bit complicated to write. However all contributions are welcome and please feel free to suggest multiple PRs. 😄

@SudharakaP SudharakaP removed the Good First Issue Good as a startring point of contributions to this repo label Oct 10, 2020
@Strat1987
Copy link
Contributor

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.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Type: Enhancement 🚀 Something that modifies or adds to existing functionality
Projects
None yet
Development

No branches or pull requests

3 participants