Skip to content

Use Particular.AnalyzerTesting package#382

Merged
DavidBoike merged 3 commits intomasterfrom
testing-framework
Mar 5, 2026
Merged

Use Particular.AnalyzerTesting package#382
DavidBoike merged 3 commits intomasterfrom
testing-framework

Conversation

@DavidBoike
Copy link
Member

@DavidBoike DavidBoike commented Feb 23, 2026

This pull request updates the project to use the Particular.AnalyzerTesting NuGet package. By adopting this package, we consolidate various analyzer testing utilities into a dedicated and maintained library, reducing custom code and duplication in our own source. This helps streamline analyzer unit tests, improves maintainability, and ensures our test infrastructure benefits from ongoing upstream improvements and bug fixes. All references to internal test utilities have been replaced with usage of the Particular.AnalyzerTesting package, and existing tests have been updated as needed for compatibility.

Key changes:

  • Removes custom analyzer testing helpers.
  • Incorporates the Particular.AnalyzerTesting NuGet package.
  • Updates unit tests to use the new abstractions.
  • Improves maintainability and future-proofing for analyzer test code.
  • Runs tests on both the minimum and current Roslyn versions

This is a non-breaking change intended for developer tooling and should not affect runtime library behavior.

@DavidBoike
Copy link
Member Author

Found some stuff I want to change, please do not merge without me.

@DavidBoike DavidBoike changed the title Unified analyzer testing framework Use Particular.AnalyzerTesting package Mar 5, 2026
@DavidBoike DavidBoike marked this pull request as ready for review March 5, 2026 20:24
@DavidBoike DavidBoike requested a review from bording March 5, 2026 20:26
@DavidBoike DavidBoike merged commit 9f74a25 into master Mar 5, 2026
4 checks passed
@DavidBoike DavidBoike deleted the testing-framework branch March 5, 2026 22:02
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.

3 participants