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

S2094: Add Repro for #9241 #9249

Merged
merged 1 commit into from
May 6, 2024
Merged

S2094: Add Repro for #9241 #9249

Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Jump to
Jump to file
Failed to load files.
Diff view
Diff view
5 changes: 4 additions & 1 deletion analyzers/tests/SonarAnalyzer.Test/TestCases/ClassShouldNotBeEmpty.cs
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -121,7 +121,7 @@ class AttributeError { }

interface IMarker { } // Compliant - this rule only deals with classes

class ImplementsMarker : IMarker { } // Compliant - implements a marker interface
class ImplementsMarker : IMarker { } // Compliant - implements a marker interface

struct EmptyStruct { } // Compliant - this rule only deals with classes

Expand All @@ -131,13 +131,16 @@ class SomeCommand { } // Compliant, ignored because of th
class SomeEvent { } // Compliant, ignored because of the suffix
class SomeMessage { } // Compliant, ignored because of the suffix
class Some_Command { } // Compliant, ignored because of the suffix
class SomeQuery { } // Noncompliant FP, should be ignored because of the suffix, Repro for https://github.com/SonarSource/sonar-dotnet/issues/9241
class Someevent { } // Noncompliant
class SOMEMESSAGE { } // Noncompliant
class SomeCommandHandler { } // Noncompliant
class MessageHandler { } // Noncompliant
class Command { } // Compliant, ignored because of the suffix
class Event { } // Compliant, ignored because of the suffix
class Message { } // Compliant, ignored because of the suffix
class Query { } // Noncompliant FP, should be ignored because of the suffix


class AssemblyDoc { } // Compliant, used for DefaultDocumentation tool
class NamespaceDoc { } // compliant, used for DefaultDocumentation tool
Expand Down