Skip to content

Commit 32a31b8

Browse files
committed
Fix comments
1 parent dd50e7b commit 32a31b8

4 files changed

Lines changed: 13 additions & 3 deletions

File tree

Roslyn.Testing/Analyzer/DiagnosticVerifier.Helpers.cs

Lines changed: 8 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -3,6 +3,7 @@
33
using System.Collections.Immutable;
44
using System.Diagnostics;
55
using System.Linq;
6+
using JetBrains.Annotations;
67
using Microsoft.CodeAnalysis;
78
using Microsoft.CodeAnalysis.CSharp;
89
using Microsoft.CodeAnalysis.Diagnostics;
@@ -46,11 +47,13 @@ public abstract class DiagnosticVerifier
4647
/// </summary>
4748
/// <param name="sources"> Classes in the form of strings </param>
4849
/// <param name="language"> The language the source classes are in </param>
49-
/// <param name="analyzer"> The analyzer to be run on the sources </param>
50+
/// <param name="analyzers"> The analyzer to be run on the sources </param>
51+
/// <param name="references"></param>
5052
/// <returns>
5153
/// An IEnumerable of Diagnostics that surfaced in the source code, sorted by
5254
/// Location
5355
/// </returns>
56+
[UsedImplicitly]
5457
private static Diagnostic[] GetSortedDiagnostics(string[] sources, string language, ImmutableArray<DiagnosticAnalyzer> analyzers,
5558
IEnumerable<MetadataReference> references = null)
5659
{
@@ -62,7 +65,7 @@ private static Diagnostic[] GetSortedDiagnostics(string[] sources, string langua
6265
/// array of diagnostics found in it.
6366
/// The returned diagnostics are then ordered by location in the source document.
6467
/// </summary>
65-
/// <param name="analyzer"> The analyzer to run on the documents </param>
68+
/// <param name="analyzers"> The analyzer to run on the documents </param>
6669
/// <param name="documents"> The Documents that the analyzer will be run on </param>
6770
/// <returns>
6871
/// An IEnumerable of Diagnostics that surfaced in the source code, sorted by
@@ -131,6 +134,7 @@ private static Diagnostic[] SortDiagnostics(IEnumerable<Diagnostic> diagnostics)
131134
/// </summary>
132135
/// <param name="sources"> Classes in the form of strings </param>
133136
/// <param name="language"> The language the source code is in </param>
137+
/// <param name="references"></param>
134138
/// <returns>
135139
/// A Tuple containing the Documents produced from the sources and their TextSpans
136140
/// if relevant
@@ -158,6 +162,7 @@ private static Document[] GetDocuments(string[] sources, string language, IEnume
158162
/// </summary>
159163
/// <param name="source"> Classes in the form of a string </param>
160164
/// <param name="language"> The language the source code is in </param>
165+
/// <param name="references"></param>
161166
/// <returns> A Document created from the source string </returns>
162167
protected static Document CreateDocument(string source, string language = LanguageNames.CSharp,
163168
IEnumerable<MetadataReference> references = null)
@@ -170,6 +175,7 @@ protected static Document CreateDocument(string source, string language = Langua
170175
/// </summary>
171176
/// <param name="sources"> Classes in the form of strings </param>
172177
/// <param name="language"> The language the source code is in </param>
178+
/// <param name="references"></param>
173179
/// <returns>
174180
/// A Project created out of the Documents created from the source
175181
/// strings

Roslyn.Testing/Analyzer/DiagnosticVerifier.cs

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -47,6 +47,7 @@ internal static class DiagnosticAnalyzerTestExtensions
4747
/// <param name="sources"> Classes in the form of strings </param>
4848
/// <param name="language"> The language the source classes are in </param>
4949
/// <param name="analyzer"> The analyzer to be run on the sources </param>
50+
/// <param name="references"></param>
5051
/// <returns>
5152
/// An IEnumerable of Diagnostics that surfaced in the source code, sorted by
5253
/// Location
@@ -134,6 +135,7 @@ private static Diagnostic[] SortDiagnostics(IEnumerable<Diagnostic> diagnostics)
134135
/// </summary>
135136
/// <param name="sources"> Classes in the form of strings </param>
136137
/// <param name="language"> The language the source code is in </param>
138+
/// <param name="references"></param>
137139
/// <returns>
138140
/// A Tuple containing the Documents produced from the sources and their TextSpans
139141
/// if relevant
@@ -161,6 +163,7 @@ private static Document[] GetDocuments(string[] sources, string language, IEnume
161163
/// </summary>
162164
/// <param name="source"> Classes in the form of a string </param>
163165
/// <param name="language"> The language the source code is in </param>
166+
/// <param name="references"></param>
164167
/// <returns> A Document created from the source string </returns>
165168
public static Document CreateDocument(string source, string language, IEnumerable<MetadataReference> references = null)
166169
{

Roslyn.Testing/CodeFix/CodeFixProviderTestExtensions.cs

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -62,6 +62,7 @@ public static Document ApplyFix(this Document document, CodeAction codeAction)
6262
/// A bool controlling whether or not the test will fail if the CodeFix introduces
6363
/// other warnings after being applied
6464
/// </param>
65+
/// <param name="additionalReferences"></param>
6566
public static VerifyCodeFixProviderResult VerifyFix(this CodeFixProvider codeFixProvider,
6667
string language,
6768
DiagnosticAnalyzer analyzer,

Roslyn.Testing/Model/FileReaderTest.cs

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,4 @@
1-
using System.IO;
1+
using System.IO;
22
using JetBrains.Annotations;
33

44
namespace Roslyn.Testing.Model

0 commit comments

Comments
 (0)