Skip to content

Commit 78447d0

Browse files
rjmurilloclaude
andcommitted
style(tests): add Async suffix to private async helper methods
Rename VerifyMock and VerifyMockIgnoringCompilerErrors to follow the project convention for private async methods (consistent with VerifyAnalyzerDynamicallyAsync, CreateProjectAsync, etc.). Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
1 parent d6c6781 commit 78447d0

1 file changed

Lines changed: 7 additions & 7 deletions

File tree

tests/Moq.Analyzers.Test/MethodSetupShouldSpecifyReturnValueAnalyzerTests.cs

Lines changed: 7 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -139,21 +139,21 @@ public static IEnumerable<object[]> OverloadResolutionFailureWithDiagnosticTestD
139139
[MemberData(nameof(TestData))]
140140
public async Task ShouldAnalyzeMethodSetupReturnValue(string referenceAssemblyGroup, string @namespace, string mock)
141141
{
142-
await VerifyMock(referenceAssemblyGroup, @namespace, mock);
142+
await VerifyMockAsync(referenceAssemblyGroup, @namespace, mock);
143143
}
144144

145145
[Theory]
146146
[MemberData(nameof(Issue849_FalsePositiveTestData))]
147147
public async Task ShouldNotFlagSetupWithReturnsAsyncOrCallbackChaining(string referenceAssemblyGroup, string @namespace, string mock)
148148
{
149-
await VerifyMock(referenceAssemblyGroup, @namespace, mock);
149+
await VerifyMockAsync(referenceAssemblyGroup, @namespace, mock);
150150
}
151151

152152
[Theory]
153153
[MemberData(nameof(CallbackOnlyNewMoqTestData))]
154154
public async Task ShouldFlagCallbackOnlySetupOnNewMoq(string referenceAssemblyGroup, string @namespace, string mock)
155155
{
156-
await VerifyMock(referenceAssemblyGroup, @namespace, mock);
156+
await VerifyMockAsync(referenceAssemblyGroup, @namespace, mock);
157157
}
158158

159159
[Theory]
@@ -169,14 +169,14 @@ await Verifier.VerifyAnalyzerAsync(
169169
[MemberData(nameof(OverloadResolutionFailureTestData))]
170170
public async Task ShouldRecognizeReturnValueMethodFromCandidateSymbols(string referenceAssemblyGroup, string @namespace, string mock)
171171
{
172-
await VerifyMockIgnoringCompilerErrors(referenceAssemblyGroup, @namespace, mock);
172+
await VerifyMockIgnoringCompilerErrorsAsync(referenceAssemblyGroup, @namespace, mock);
173173
}
174174

175175
[Theory]
176176
[MemberData(nameof(OverloadResolutionFailureWithDiagnosticTestData))]
177177
public async Task ShouldFlagSetupWhenOnlyNonReturnValueCandidatesExist(string referenceAssemblyGroup, string @namespace, string mock)
178178
{
179-
await VerifyMockIgnoringCompilerErrors(referenceAssemblyGroup, @namespace, mock);
179+
await VerifyMockIgnoringCompilerErrorsAsync(referenceAssemblyGroup, @namespace, mock);
180180
}
181181

182182
private static string BuildSource(string @namespace, string mock)
@@ -205,7 +205,7 @@ private void Test()
205205
""";
206206
}
207207

208-
private async Task VerifyMock(string referenceAssemblyGroup, string @namespace, string mock)
208+
private async Task VerifyMockAsync(string referenceAssemblyGroup, string @namespace, string mock)
209209
{
210210
string source = BuildSource(@namespace, mock);
211211
output.WriteLine(source);
@@ -215,7 +215,7 @@ await Verifier.VerifyAnalyzerAsync(
215215
referenceAssemblyGroup).ConfigureAwait(false);
216216
}
217217

218-
private async Task VerifyMockIgnoringCompilerErrors(string referenceAssemblyGroup, string @namespace, string mock)
218+
private async Task VerifyMockIgnoringCompilerErrorsAsync(string referenceAssemblyGroup, string @namespace, string mock)
219219
{
220220
string source = BuildSource(@namespace, mock);
221221
output.WriteLine(source);

0 commit comments

Comments
 (0)