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

Fix mistake in earlier coverage test commit #145

Merged
merged 2 commits into from
May 11, 2024

Conversation

MarkEWaite
Copy link
Contributor

Fix mistake in earlier coverage test commit

Wrong method name called inside the test. Fixed by using correct method name.

Also increases coverage a little more by testing a few more methods.

Testing done

Confirmed automated tests pass

Submitter checklist

Preview Give feedback

Wrong method name called inside the test.  Fixed by using correct method name.
@MarkEWaite MarkEWaite requested a review from a team as a code owner May 11, 2024 14:38
@MarkEWaite MarkEWaite added the test Improvements to test and test infrastructure label May 11, 2024
@MarkEWaite MarkEWaite merged commit 0bc6477 into jenkinsci:master May 11, 2024
19 checks passed
@MarkEWaite MarkEWaite deleted the fix-coverage-mistake branch May 11, 2024 14:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
test Improvements to test and test infrastructure
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant