Skip to content

Commit 3e180ad

Browse files
sharwellsharwell
committed
Merge remote-tracking branch 'DotNetAnalyzers/master' into relational-logical-patterns
2 parents c75ee35 + e6480ae commit 3e180ad

File tree

52 files changed

+1997
-111
lines changed

Some content is hidden

Large Commits have some content hidden by default. Use the searchbox below for content that may be hidden.

52 files changed

+1997
-111
lines changed

‎StyleCop.Analyzers/StyleCop.Analyzers.Test.CSharp9/DocumentationRules/SA1611CSharp9UnitTests.cs‎

Lines changed: 19 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -37,5 +37,24 @@ public async Task TestPrimaryRecordConstructorIncludeMissingParametersAsync(stri
3737

3838
await VerifyCSharpDiagnosticAsync(testCode, DiagnosticResult.EmptyDiagnosticResults, CancellationToken.None).ConfigureAwait(false);
3939
}
40+
41+
[Fact]
42+
[WorkItem(3977, "https://github.com/DotNetAnalyzers/StyleCopAnalyzers/issues/3977")]
43+
public async Task TestLambdaDiscardParametersAsync()
44+
{
45+
var testCode = @"
46+
/// <summary>Test class.</summary>
47+
public class TestClass
48+
{
49+
/// <summary>Test method.</summary>
50+
public void TestMethod()
51+
{
52+
System.Func<int, int, int> handler = (_, _) => 0;
53+
}
54+
}
55+
";
56+
57+
await VerifyCSharpDiagnosticAsync(testCode, DiagnosticResult.EmptyDiagnosticResults, CancellationToken.None).ConfigureAwait(false);
58+
}
4059
}
4160
}

‎StyleCop.Analyzers/StyleCop.Analyzers.Test.CSharp9/DocumentationRules/SA1612CSharp9UnitTests.cs‎

Lines changed: 22 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -3,9 +3,31 @@
33

44
namespace StyleCop.Analyzers.Test.CSharp9.DocumentationRules
55
{
6+
using System.Threading;
7+
using System.Threading.Tasks;
8+
using Microsoft.CodeAnalysis.Testing;
69
using StyleCop.Analyzers.Test.CSharp8.DocumentationRules;
10+
using Xunit;
711

812
public partial class SA1612CSharp9UnitTests : SA1612CSharp8UnitTests
913
{
14+
[Fact]
15+
[WorkItem(3977, "https://github.com/DotNetAnalyzers/StyleCopAnalyzers/issues/3977")]
16+
public async Task TestLambdaDiscardParametersAsync()
17+
{
18+
var testCode = @"
19+
/// <summary>Test class.</summary>
20+
public class TestClass
21+
{
22+
/// <summary>Test method.</summary>
23+
public void TestMethod()
24+
{
25+
System.Func<int, int, int> handler = (_, _) => 0;
26+
}
27+
}
28+
";
29+
30+
await VerifyCSharpDiagnosticAsync(testCode, DiagnosticResult.EmptyDiagnosticResults, CancellationToken.None).ConfigureAwait(false);
31+
}
1032
}
1133
}

‎StyleCop.Analyzers/StyleCop.Analyzers.Test.CSharp9/DocumentationRules/SA1623CSharp9UnitTests.cs‎

Lines changed: 203 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -3,9 +3,212 @@
33

44
namespace StyleCop.Analyzers.Test.CSharp9.DocumentationRules
55
{
6+
using System.Threading;
7+
using System.Threading.Tasks;
8+
using Microsoft.CodeAnalysis.Testing;
9+
using StyleCop.Analyzers.DocumentationRules;
610
using StyleCop.Analyzers.Test.CSharp8.DocumentationRules;
11+
using Xunit;
12+
using static StyleCop.Analyzers.Test.Verifiers.StyleCopCodeFixVerifier<
13+
StyleCop.Analyzers.DocumentationRules.PropertySummaryDocumentationAnalyzer,
14+
StyleCop.Analyzers.DocumentationRules.PropertySummaryDocumentationCodeFixProvider>;
715

816
public partial class SA1623CSharp9UnitTests : SA1623CSharp8UnitTests
917
{
18+
/// <summary>
19+
/// Verifies that property documentation that does not start with the appropriate text will result in a diagnostic.
20+
/// </summary>
21+
/// <param name="accessibility">The accessibility of the property.</param>
22+
/// <param name="type">The type to use for the property.</param>
23+
/// <param name="accessors">The accessors for the property.</param>
24+
/// <param name="expectedArgument">The expected argument for the diagnostic message.</param>
25+
/// <returns>A <see cref="Task"/> representing the asynchronous unit test.</returns>
26+
[Theory]
27+
[WorkItem(3966, "https://github.com/DotNetAnalyzers/StyleCopAnalyzers/issues/3966")]
28+
[InlineData("public", "int", "{ get; init; }", "Gets or initializes")]
29+
[InlineData("public", "int", "{ get; private init; }", "Gets")]
30+
[InlineData("public", "int", "{ init { } }", "Initializes")]
31+
[InlineData("public", "int", "{ private get { return 0; } init { } }", "Initializes")]
32+
[InlineData("public", "bool", "{ get; init; }", "Gets or initializes a value indicating whether")]
33+
[InlineData("public", "bool", "{ get; private init; }", "Gets a value indicating whether")]
34+
[InlineData("public", "bool", "{ init { } }", "Initializes a value indicating whether")]
35+
[InlineData("public", "bool", "{ private get { return false; } init { } }", "Initializes a value indicating whether")]
36+
public async Task VerifyDocumentationWithWrongStartingTextWithInitAccessorWillProduceDiagnosticAsync(string accessibility, string type, string accessors, string expectedArgument)
37+
{
38+
var testCode = $@"
39+
public class TestClass
40+
{{
41+
/// <summary>
42+
/// The first test value.
43+
/// </summary>
44+
{accessibility} {type} {{|#0:TestProperty|}} {accessors}
45+
}}
46+
";
47+
48+
var fixedTestCode = $@"
49+
public class TestClass
50+
{{
51+
/// <summary>
52+
/// {expectedArgument} the first test value.
53+
/// </summary>
54+
{accessibility} {type} TestProperty {accessors}
55+
}}
56+
";
57+
58+
var expected = Diagnostic(PropertySummaryDocumentationAnalyzer.SA1623Descriptor).WithLocation(0).WithArguments(expectedArgument);
59+
await VerifyCSharpFixAsync(testCode, expected, fixedTestCode, CancellationToken.None).ConfigureAwait(false);
60+
61+
if (fixedTestCode.Contains("Gets or initializes"))
62+
{
63+
// These are allowed to be written as just 'Gets'
64+
await VerifyCSharpDiagnosticAsync(fixedTestCode.Replace("Gets or initializes", "Gets"), DiagnosticResult.EmptyDiagnosticResults, CancellationToken.None).ConfigureAwait(false);
65+
}
66+
}
67+
68+
/// <summary>
69+
/// Verifies that property documentation that does not start with the appropriate text will result in a
70+
/// diagnostic. This is a regression test for DotNetAnalyzers/StyleCopAnalyzers#1844.
71+
/// </summary>
72+
/// <param name="accessibility">The accessibility of the property.</param>
73+
/// <param name="type">The type to use for the property.</param>
74+
/// <param name="accessors">The accessors for the property.</param>
75+
/// <param name="expectedArgument">The expected argument for the diagnostic message.</param>
76+
/// <returns>A <see cref="Task"/> representing the asynchronous unit test.</returns>
77+
[Theory]
78+
[WorkItem(3966, "https://github.com/DotNetAnalyzers/StyleCopAnalyzers/issues/3966")]
79+
[InlineData("public", "int", "{ get; init; }", "Gets or initializes")]
80+
[InlineData("public", "int", "{ get; private init; }", "Gets")]
81+
[InlineData("public", "int", "{ init { } }", "Initializes")]
82+
[InlineData("public", "int", "{ private get { return 0; } init { } }", "Initializes")]
83+
[InlineData("public", "bool", "{ get; init; }", "Gets or initializes a value indicating whether")]
84+
[InlineData("public", "bool", "{ get; private init; }", "Gets a value indicating whether")]
85+
[InlineData("public", "bool", "{ init { } }", "Initializes a value indicating whether")]
86+
[InlineData("public", "bool", "{ private get { return false; } init { } }", "Initializes a value indicating whether")]
87+
public async Task VerifyDocumentationOfPublicMethodInPrivateClassWithInitAccessorWillProduceDiagnosticAsync(string accessibility, string type, string accessors, string expectedArgument)
88+
{
89+
var testCode = $@"
90+
public class TestClass
91+
{{
92+
private class PrivateTestClass
93+
{{
94+
/// <summary>
95+
/// The first test value.
96+
/// </summary>
97+
{accessibility} {type} {{|#0:TestProperty|}} {accessors}
98+
}}
99+
}}
100+
";
101+
102+
var fixedTestCode = $@"
103+
public class TestClass
104+
{{
105+
private class PrivateTestClass
106+
{{
107+
/// <summary>
108+
/// {expectedArgument} the first test value.
109+
/// </summary>
110+
{accessibility} {type} TestProperty {accessors}
111+
}}
112+
}}
113+
";
114+
115+
var expected = Diagnostic(PropertySummaryDocumentationAnalyzer.SA1623Descriptor).WithLocation(0).WithArguments(expectedArgument);
116+
await VerifyCSharpFixAsync(testCode, expected, fixedTestCode, CancellationToken.None).ConfigureAwait(false);
117+
118+
if (fixedTestCode.Contains("Gets or initializes"))
119+
{
120+
// These are allowed to be written as just 'Gets'
121+
await VerifyCSharpDiagnosticAsync(fixedTestCode.Replace("Gets or initializes", "Gets"), DiagnosticResult.EmptyDiagnosticResults, CancellationToken.None).ConfigureAwait(false);
122+
}
123+
}
124+
125+
/// <summary>
126+
/// Verifies that an incorrect summary tag with a known prefix will be fixed correctly.
127+
/// </summary>
128+
/// <param name="accessibility">The accessibility of the property.</param>
129+
/// <param name="type">The type to use for the property.</param>
130+
/// <param name="accessors">The accessors for the property.</param>
131+
/// <param name="summaryPrefix">The summary prefix used in the test code.</param>
132+
/// <param name="expectedArgument">The expected argument for the diagnostic message.</param>
133+
/// <returns>A <see cref="Task"/> representing the asynchronous unit test.</returns>
134+
[Theory]
135+
[WorkItem(3966, "https://github.com/DotNetAnalyzers/StyleCopAnalyzers/issues/3966")]
136+
[InlineData("public", "int", "{ get; init; }", "Sets", "Gets or initializes")]
137+
[InlineData("public", "int", "{ get; init; }", "Initializes", "Gets or initializes")]
138+
[InlineData("public", "int", "{ init {} }", "Gets", "Initializes")]
139+
[InlineData("public", "int", "{ init {} }", "Gets or sets", "Initializes")]
140+
[InlineData("public", "int", "{ init {} }", "Sets", "Initializes")]
141+
[InlineData("public", "int", "{ init {} }", "Gets or initializes", "Initializes")]
142+
[InlineData("public", "int", "{ get; private init; }", "Sets", "Gets")]
143+
[InlineData("public", "int", "{ get; private init; }", "Initializes", "Gets")]
144+
[InlineData("public", "int", "{ private get; init; }", "Gets", "Initializes")]
145+
[InlineData("public", "int", "{ private get; init; }", "Sets", "Initializes")]
146+
[InlineData("public", "int", "{ get; init; }", "Returns", "Gets or initializes")]
147+
[InlineData("public", "bool", "{ get; init; }", "Returns a value indicating whether", "Gets or initializes a value indicating whether")]
148+
public async Task IncorrectSummaryTagWithKnownPrefixAndInitAccessorShouldBeFixedCorrectlyAsync(string accessibility, string type, string accessors, string summaryPrefix, string expectedArgument)
149+
{
150+
var testCode = $@"
151+
public class TestClass
152+
{{
153+
/// <summary>
154+
/// {summaryPrefix} the value.
155+
/// </summary>
156+
{accessibility} {type} {{|#0:TestProperty|}} {accessors}
157+
}}
158+
";
159+
160+
var fixedTestCode = $@"
161+
public class TestClass
162+
{{
163+
/// <summary>
164+
/// {expectedArgument} the value.
165+
/// </summary>
166+
{accessibility} {type} TestProperty {accessors}
167+
}}
168+
";
169+
170+
DiagnosticResult[] expected =
171+
{
172+
Diagnostic(PropertySummaryDocumentationAnalyzer.SA1623Descriptor).WithLocation(0).WithArguments(expectedArgument),
173+
};
174+
175+
await VerifyCSharpFixAsync(testCode, expected, fixedTestCode, CancellationToken.None).ConfigureAwait(false);
176+
177+
if (fixedTestCode.Contains("Gets or initializes"))
178+
{
179+
// These are allowed to be written as just 'Gets'
180+
await VerifyCSharpDiagnosticAsync(fixedTestCode.Replace("Gets or initializes", "Gets"), DiagnosticResult.EmptyDiagnosticResults, CancellationToken.None).ConfigureAwait(false);
181+
}
182+
}
183+
184+
[Fact]
185+
[WorkItem(3966, "https://github.com/DotNetAnalyzers/StyleCopAnalyzers/issues/3966")]
186+
public async Task TestGetInitPropertyRequiresGetsOrInitializesSummaryAsync()
187+
{
188+
var testCode = @"
189+
public class TestClass
190+
{
191+
/// <summary>
192+
/// Sets the value.
193+
/// </summary>
194+
public string {|#0:Name|} { get; init; }
195+
}
196+
";
197+
var fixedCode = @"
198+
public class TestClass
199+
{
200+
/// <summary>
201+
/// Gets or initializes the value.
202+
/// </summary>
203+
public string Name { get; init; }
204+
}
205+
";
206+
207+
var expected = Diagnostic(PropertySummaryDocumentationAnalyzer.SA1623Descriptor)
208+
.WithLocation(0)
209+
.WithArguments("Gets or initializes");
210+
211+
await VerifyCSharpFixAsync(testCode, expected, fixedCode, CancellationToken.None).ConfigureAwait(false);
212+
}
10213
}
11214
}

0 commit comments

Comments
 (0)