Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Add tests for Roslyn internal rules #786

Open
4 tasks
srivatsn opened this issue Jan 13, 2016 · 1 comment
Open
4 tasks

Add tests for Roslyn internal rules #786

srivatsn opened this issue Jan 13, 2016 · 1 comment
Labels
Area-Roslyn.Diagnostics.Analyzers Feature Request help wanted The issue is up-for-grabs, and can be claimed by commenting Test
Milestone
Unknown

Comments

@srivatsn
Copy link
Contributor

Many of the Roslyn internal rules don't have unit tests. List of rules without tests:

  • RS0013 (DoNotInvokeDiagnosticDescriptor)
  • RS0005 (DoNotUseGenericCodeActionCreateToCreateCodeAction)
  • RS0019 (SymbolDeclaredEventMustBeGeneratedForSourceSymbols)
  • RS0004 (BasicInvokeTheCorrectPropertyToEnsureCorrectUseSiteDiagnostics) - VB only analyzer.
@srivatsn srivatsn added this to the Unknown milestone Jan 13, 2016
@srivatsn srivatsn mentioned this issue Jan 13, 2016
Merged
@mavasani mavasani removed their assignment Aug 8, 2018
@sharwell sharwell added help wanted The issue is up-for-grabs, and can be claimed by commenting and removed Up for Grabs labels Aug 1, 2019
@Youssef1313
Copy link
Member

RS0005 and RS0013 no longer exist.

The work needed here is for RS0019 and RS0004.

@mavasani I'm wondering if these analyzers are still relevant to dotnet/roslyn?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
Area-Roslyn.Diagnostics.Analyzers Feature Request help wanted The issue is up-for-grabs, and can be claimed by commenting Test
Projects
None yet
Milestone
Unknown
Development

No branches or pull requests
4 participants
@srivatsn @sharwell @mavasani @Youssef1313