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

Missing test coverage #56

Open
jonesbusy opened this issue Oct 29, 2023 · 4 comments
Open

Missing test coverage #56

jonesbusy opened this issue Oct 29, 2023 · 4 comments

Comments

@jonesbusy
Copy link
Contributor

What feature do you want to see added?

This plugin miss some essential testing to validate non-regression

Unit tests
HTMLUnit for UI configuration
Jenkins Configuration Test

More information on https://www.jenkins.io/doc/developer/testing/

Upstream changes

No response

@Rachit1313
Copy link

Rachit1313 commented Oct 29, 2023

Could you please provide a bit info like you need tests for which files?

@jonesbusy
Copy link
Contributor Author

This plugin doesn't have any test (You can check that there is no 'test' folder in 'src'). So any test is better than nothing.

Feel free to submit a PR

@debayangg
Copy link
Contributor

I added one unit test in NextBuildsTest. Should I create a PR or add some more tests? Also could you advise on which functions should be tested.

@jonesbusy
Copy link
Contributor Author

I added one unit test in NextBuildsTest. Should I create a PR or add some more tests? Also could you advise on which functions should be tested.

Any test is better than nothing. You can run mvn -Penable-jacoco clean verify and check the report on target/site/jacoco/index.html to check the coverage and check code not covered.

Feel free to open a PR and I will review it

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