After merge of #16314
We have ability to update all/most test methods in SarifLoggerTest.java to use verifyWithInlineConfigParserAndLogger (this method should be created to accept any logger)
convert usage of verifyContent to verifyWithInlineConfigParserAndLogger or similar method
Update is required for:
Please send PR for each test method separately, to ease review.
Example of update https://github.com/checkstyle/checkstyle/pull/16698/files
Goal: use it as much as possible, ideally in all test methods.
After merge of #16314
We have ability to update all/most test methods in SarifLoggerTest.java to use verifyWithInlineConfigParserAndLogger (this method should be created to accept any logger)
convert usage of
verifyContenttoverifyWithInlineConfigParserAndLoggeror similar methodUpdate is required for:
testNullOutputStreamOptionswe just need comment above to explain reason of not usage of verifyXxxxxx methodPlease send PR for each test method separately, to ease review.
Example of update https://github.com/checkstyle/checkstyle/pull/16698/files
Goal: use it as much as possible, ideally in all test methods.