Skip to content

Commit 074135e

Browse files
authored
Remove underscore from field name to comply with naming conventions (#7319)
2 parents 8fd8816 + 38168bf commit 074135e

2 files changed

Lines changed: 5 additions & 5 deletions

File tree

src/Analyzers/MSTest.Analyzers/TestClassShouldHaveTestMethodAnalyzer.cs

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -33,12 +33,12 @@ public sealed class TestClassShouldHaveTestMethodAnalyzer : DiagnosticAnalyzer
3333
isEnabledByDefault: true);
3434

3535
/// <inheritdoc cref="Resources.TestClassShouldHaveTestMethodMessageFormat_BaseClassHasAssemblyAttributes" />
36-
public static readonly DiagnosticDescriptor TestClassShouldHaveTestMethodRule_BaseClassHasAssemblyAttributes = TestClassShouldHaveTestMethodRule
36+
public static readonly DiagnosticDescriptor TestClassShouldHaveTestMethodRuleBaseClassHasAssemblyAttributes = TestClassShouldHaveTestMethodRule
3737
.WithMessage(new LocalizableResourceString(nameof(Resources.TestClassShouldHaveTestMethodMessageFormat_BaseClassHasAssemblyAttributes), Resources.ResourceManager, typeof(Resources)));
3838

3939
/// <inheritdoc />
4040
public override ImmutableArray<DiagnosticDescriptor> SupportedDiagnostics { get; }
41-
= ImmutableArray.Create(TestClassShouldHaveTestMethodRule, TestClassShouldHaveTestMethodRule_BaseClassHasAssemblyAttributes);
41+
= ImmutableArray.Create(TestClassShouldHaveTestMethodRule, TestClassShouldHaveTestMethodRuleBaseClassHasAssemblyAttributes);
4242

4343
/// <inheritdoc />
4444
public override void Initialize(AnalysisContext context)
@@ -137,7 +137,7 @@ private static void AnalyzeSymbol(SymbolAnalysisContext context, INamedTypeSymbo
137137
// Non-static class that inherits assembly attributes from base class - suggest making it static
138138
if (!classSymbol.IsStatic && hasAssemblyAttributeInBaseClass && baseClassWithAssemblyAttribute is not null)
139139
{
140-
context.ReportDiagnostic(classSymbol.CreateDiagnostic(TestClassShouldHaveTestMethodRule_BaseClassHasAssemblyAttributes, classSymbol.Name, baseClassWithAssemblyAttribute.Name));
140+
context.ReportDiagnostic(classSymbol.CreateDiagnostic(TestClassShouldHaveTestMethodRuleBaseClassHasAssemblyAttributes, classSymbol.Name, baseClassWithAssemblyAttribute.Name));
141141

142142
return;
143143
}

test/UnitTests/MSTest.Analyzers.UnitTests/TestClassShouldHaveTestMethodAnalyzerTests.cs

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -253,7 +253,7 @@ public class {|#0:Derived|} : BaseClass
253253
""";
254254
await VerifyCS.VerifyAnalyzerAsync(
255255
code,
256-
VerifyCS.Diagnostic(TestClassShouldHaveTestMethodAnalyzer.TestClassShouldHaveTestMethodRule_BaseClassHasAssemblyAttributes)
256+
VerifyCS.Diagnostic(TestClassShouldHaveTestMethodAnalyzer.TestClassShouldHaveTestMethodRuleBaseClassHasAssemblyAttributes)
257257
.WithLocation(0)
258258
.WithArguments("Derived", "BaseClass"));
259259
}
@@ -283,7 +283,7 @@ public class {|#0:Derived|} : BaseClass
283283
""";
284284
await VerifyCS.VerifyAnalyzerAsync(
285285
code,
286-
VerifyCS.Diagnostic(TestClassShouldHaveTestMethodAnalyzer.TestClassShouldHaveTestMethodRule_BaseClassHasAssemblyAttributes)
286+
VerifyCS.Diagnostic(TestClassShouldHaveTestMethodAnalyzer.TestClassShouldHaveTestMethodRuleBaseClassHasAssemblyAttributes)
287287
.WithLocation(0)
288288
.WithArguments("Derived", "BaseBase"));
289289
}

0 commit comments

Comments
 (0)