Skip to content

SarifLoggerTest.java to use verifyWithInlineConfigParserAndLogger #16361

@romani

Description

@romani

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:

  • sarif, method testNullOutputStreamOptions we just need comment above to explain reason of not usage of verifyXxxxxx method
  • output

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.

Metadata

Metadata

Assignees

No one assigned

    Type

    No type
    No fields configured for issues without a type.

    Projects

    No projects

    Milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions