Skip to content

Commit c765ab7

Browse files
authored
Merge pull request #3417 from sharwell/index-ref
Use reference assemblies for index and range
2 parents 9724e3d + f84ed7f commit c765ab7

File tree

3 files changed

+21
-45
lines changed

3 files changed

+21
-45
lines changed

StyleCop.Analyzers/StyleCop.Analyzers.Test.CSharp8/SpacingRules/SA1008CSharp8UnitTests.cs

Lines changed: 10 additions & 13 deletions
Original file line numberDiff line numberDiff line change
@@ -30,7 +30,7 @@ public class SA1008CSharp8UnitTests : SA1008CSharp7UnitTests
3030
[Fact]
3131
public async Task TestAfterRangeExpressionAsync()
3232
{
33-
var testCode = SpecialTypeDefinitions.IndexAndRange + @"
33+
var testCode = @"
3434
namespace TestNamespace
3535
{
3636
using System;
@@ -40,13 +40,13 @@ public string TestMethod()
4040
{
4141
string str = ""test"";
4242
int finalLen = 4;
43-
return str[.. (finalLen - 1)];
43+
return str[.. {|#0:(|}finalLen - 1)];
4444
}
4545
}
4646
}
4747
";
4848

49-
var fixedCode = SpecialTypeDefinitions.IndexAndRange + @"
49+
var fixedCode = @"
5050
namespace TestNamespace
5151
{
5252
using System;
@@ -61,17 +61,14 @@ public string TestMethod()
6161
}
6262
}
6363
";
64-
var expectedResults = new DiagnosticResult[]
65-
{
66-
Diagnostic(DescriptorNotPreceded).WithLocation(28, 27),
67-
};
6864

69-
await VerifyCSharpFixAsync(
70-
LanguageVersion.CSharp8,
71-
testCode,
72-
expectedResults,
73-
fixedCode,
74-
CancellationToken.None).ConfigureAwait(false);
65+
await new CSharpTest(LanguageVersion.CSharp8)
66+
{
67+
ReferenceAssemblies = ReferenceAssemblies.NetCore.NetCoreApp31,
68+
TestCode = testCode,
69+
ExpectedDiagnostics = { Diagnostic(DescriptorNotPreceded).WithLocation(0) },
70+
FixedCode = fixedCode,
71+
}.RunAsync(CancellationToken.None).ConfigureAwait(false);
7572
}
7673

7774
[Fact]

StyleCop.Analyzers/StyleCop.Analyzers.Test.CSharp8/SpacingRules/SA1009CSharp8UnitTests.cs

Lines changed: 11 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -167,7 +167,7 @@ public IDisposable Service()
167167
[WorkItem(3064, "https://github.com/DotNetAnalyzers/StyleCopAnalyzers/issues/3064")]
168168
public async Task TestBeforeRangeExpressionAsync()
169169
{
170-
var testCode = SpecialTypeDefinitions.IndexAndRange + @"
170+
var testCode = @"
171171
namespace TestNamespace
172172
{
173173
using System;
@@ -177,13 +177,13 @@ public string TestMethod()
177177
{
178178
string str = ""test"";
179179
int startLen = 4;
180-
return str[(startLen - 1) ..];
180+
return str[(startLen - 1{|#0:)|} ..];
181181
}
182182
}
183183
}
184184
";
185185

186-
var fixedCode = SpecialTypeDefinitions.IndexAndRange + @"
186+
var fixedCode = @"
187187
namespace TestNamespace
188188
{
189189
using System;
@@ -198,8 +198,14 @@ public string TestMethod()
198198
}
199199
}
200200
";
201-
DiagnosticResult expected = Diagnostic(DescriptorNotFollowed).WithSpan(28, 37, 28, 38);
202-
await VerifyCSharpFixAsync(LanguageVersion.CSharp8, testCode, expected, fixedCode, CancellationToken.None).ConfigureAwait(false);
201+
202+
await new CSharpTest(LanguageVersion.CSharp8)
203+
{
204+
ReferenceAssemblies = ReferenceAssemblies.NetCore.NetCoreApp31,
205+
TestCode = testCode,
206+
ExpectedDiagnostics = { Diagnostic(DescriptorNotFollowed).WithLocation(0) },
207+
FixedCode = fixedCode,
208+
}.RunAsync(CancellationToken.None).ConfigureAwait(false);
203209
}
204210
}
205211
}

StyleCop.Analyzers/StyleCop.Analyzers.Test/Verifiers/SpecialTypeDefinitions.cs

Lines changed: 0 additions & 27 deletions
This file was deleted.

0 commit comments

Comments
 (0)