Remove unused assertions and assert methods #794

MarkEWaite · 2021-12-30T14:45:54Z

Remove unused assertions

Earlier splits of tests from one large file into smaller files did not consider that the tests in the smaller files did not need all the infrastructure that had been included when all the tests were in a single, larger file.

Checklist

I have read the CONTRIBUTING doc
I have added tests that verify my changes
Unit tests pass locally with my changes
I have added documentation as necessary
No Javadoc warnings were introduced with my changes
No spotbugs warnings were introduced with my changes

Types of changes

Test improvements

Earlier splits of tests from one large file into smaller files did not consider that the tests in the smaller files did not need all the infrastructure that had been included when all the tests were in a single, larger file.

Remove unused assertions and assert methods

e41127c

Earlier splits of tests from one large file into smaller files did not consider that the tests in the smaller files did not need all the infrastructure that had been included when all the tests were in a single, larger file.

github-actions bot added the test Automated test addition or improvement label Dec 30, 2021

MarkEWaite merged commit 236b0bf into jenkinsci:master Dec 30, 2021

MarkEWaite deleted the remove-unused-assertions branch December 30, 2021 15:23

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Remove unused assertions and assert methods #794

Remove unused assertions and assert methods #794

MarkEWaite commented Dec 30, 2021 •

edited

Remove unused assertions and assert methods #794

Remove unused assertions and assert methods #794

Conversation

MarkEWaite commented Dec 30, 2021 • edited