-
-
Notifications
You must be signed in to change notification settings - Fork 260
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
Improve testing framework #1214
Conversation
ohh yes this is great. I was building my own analyzer Roslynator.Testing.CSharp and tried to use a copy of AbstractCSharpDiagnosticVerifier but it was using a lot of internal functions it looks like I can now just use CompilerDiagnosticFixVerifier |
@AlmarAubel @nathanpovo If you have any comments please add it to this PR. Otherwise, the PR is ready to be merged. |
Thanks for your hard work @josefpihrt! Managed to get some time to test this out. Had to add a reference to a local version of the Didn't have the chance to do a proper test suite using the new package but it looks to be ok from the couple of tests I tried. |
Thank you after creating a copy of AbstractCSharpDiagnosticVerifier I could use it on my own analyzers. |
No description provided.