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

[GSoC] Addition of unit test for WipeWorkspace extension behavior in GitSCM #852

Merged
merged 3 commits into from
Mar 9, 2020

Conversation

rishabhBudhouliya
Copy link
Contributor

The unit test was added in relevance to one the project goals of Git-Plugin Performance Improvements which is to improve the code coverage.

Screenshot 2020-03-06 at 1 25 22 PM

I ran the coverage report on IntelliJ and most of the report is not relevant to the test, I have just taken out the relevant part.

Copy link
Contributor

@MarkEWaite MarkEWaite left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for the test! Two items that need to be addressed before merge. Merge won't happen until next week when I return from my business trip.

@MarkEWaite MarkEWaite added the test label Mar 6, 2020
@rishabhBudhouliya
Copy link
Contributor Author

The last commit after your approval is just a shift of code from before() method inorder to run equalsContract() test without JenkinsRule.

Thanks for the review!

@MarkEWaite MarkEWaite merged commit f46eebf into jenkinsci:master Mar 9, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants