update

dotnet · Jan 25, 2024 · c60e221 · c60e221
1 parent 3908b3d
commit c60e221
Show file tree

Hide file tree

Showing 8 changed files with 28 additions and 28 deletions.
diff --git a/src/Analyzers.CodeFixes/CSharp/CodeFixes/InvocationExpressionCodeFixProvider.cs b/src/Analyzers.CodeFixes/CSharp/CodeFixes/InvocationExpressionCodeFixProvider.cs
@@ -34,7 +34,7 @@ public override ImmutableArray<string> FixableDiagnosticIds
                 DiagnosticIdentifiers.CombineEnumerableWhereMethodChain,
                 DiagnosticIdentifiers.CallExtensionMethodAsInstanceMethod,
                 DiagnosticIdentifiers.CallThenByInsteadOfOrderBy,
-                DiagnosticIdentifiers.ConvertStringConcatToInterpolatedString);
+                DiagnosticIdentifiers.UseStringInterpolationInsteadOfStringConcat);
         }
     }
 
@@ -119,11 +119,11 @@ public override async Task RegisterCodeFixesAsync(CodeFixContext context)
                         context.RegisterCodeFix(codeAction, diagnostic);
                         break;
                     }
-                case DiagnosticIdentifiers.ConvertStringConcatToInterpolatedString:
+                case DiagnosticIdentifiers.UseStringInterpolationInsteadOfStringConcat:
                     {
                         CodeAction codeAction = CodeAction.Create(
                             "Use string interpolation",
-                            ct => ConvertStringConcatToInterpolatedStringAsync(context.Document, invocation, ct),
+                            ct => UseStringInterpolationInsteadOfStringConcatAsync(context.Document, invocation, ct),
                             GetEquivalenceKey(diagnostic));
 
                         context.RegisterCodeFix(codeAction, diagnostic);
@@ -163,7 +163,7 @@ private static Task<Document> CallThenByInsteadOfOrderByAsync(
         return document.ReplaceNodeAsync(invocationExpression, newInvocationExpression, cancellationToken);
     }
 
-    private static Task<Document> ConvertStringConcatToInterpolatedStringAsync(
+    private static Task<Document> UseStringInterpolationInsteadOfStringConcatAsync(
         Document document,
         InvocationExpressionSyntax invocationExpression,
         CancellationToken cancellationToken)

diff --git a/src/Analyzers.xml b/src/Analyzers.xml
@@ -7693,8 +7693,8 @@ finally
   </Analyzer>
   <Analyzer>
     <Id>RCS1267</Id>
-    <Identifier>ConvertStringConcatToInterpolatedString</Identifier>
-    <Title>Convert 'string.Concat' to interpolated string</Title>
+    <Identifier>UseStringInterpolationInsteadOfStringConcat</Identifier>
+    <Title>Use string interpolation instead of 'string.Concat'</Title>
     <DefaultSeverity>Info</DefaultSeverity>
     <IsEnabledByDefault>true</IsEnabledByDefault>
     <Samples>

diff --git a/src/Analyzers/CSharp/Analysis/InvocationExpressionAnalyzer.cs b/src/Analyzers/CSharp/Analysis/InvocationExpressionAnalyzer.cs
@@ -45,7 +45,7 @@ public override ImmutableArray<DiagnosticDescriptor> SupportedDiagnostics
                     DiagnosticRules.SimplifyLogicalNegation,
                     DiagnosticRules.UseCoalesceExpression,
                     DiagnosticRules.OptimizeMethodCall,
-                    DiagnosticRules.ConvertStringConcatToInterpolatedString);
+                    DiagnosticRules.UseStringInterpolationInsteadOfStringConcat);
             }
 
             return _supportedDiagnostics;
@@ -456,10 +456,10 @@ private static void AnalyzeInvocationExpression(SyntaxNodeAnalysisContext contex
                 }
             case "Concat":
                 {
-                    if (DiagnosticRules.ConvertStringConcatToInterpolatedString.IsEffective(context)
+                    if (DiagnosticRules.UseStringInterpolationInsteadOfStringConcat.IsEffective(context)
                         && argumentCount > 1)
                     {
-                        ConvertStringConcatToInterpolatedStringAnalysis.Analyze(context, invocationInfo);
+                        UseStringInterpolationInsteadOfStringConcatAnalysis.Analyze(context, invocationInfo);
                     }
 
                     break;

diff --git a/...ringConcatToInterpolatedStringAnalysis.cs → ...rpolationInsteadOfStringConcatAnalysis.cs b/...ringConcatToInterpolatedStringAnalysis.cs → ...rpolationInsteadOfStringConcatAnalysis.cs
@@ -8,7 +8,7 @@
 
 namespace Roslynator.CSharp.Analysis;
 
-internal static class ConvertStringConcatToInterpolatedStringAnalysis
+internal static class UseStringInterpolationInsteadOfStringConcatAnalysis
 {
     internal static void Analyze(SyntaxNodeAnalysisContext context, SimpleMemberInvocationExpressionInfo invocationInfo)
     {
@@ -56,7 +56,7 @@ internal static void Analyze(SyntaxNodeAnalysisContext context, SimpleMemberInvo
             if (isVerbatim is not null
                 && invocationInfo.ArgumentList.IsSingleLine())
             {
-                DiagnosticHelpers.ReportDiagnostic(context, DiagnosticRules.ConvertStringConcatToInterpolatedString, invocationInfo.InvocationExpression);
+                DiagnosticHelpers.ReportDiagnostic(context, DiagnosticRules.UseStringInterpolationInsteadOfStringConcat, invocationInfo.InvocationExpression);
             }
         }
     }

diff --git a/src/Analyzers/CSharp/DiagnosticIdentifiers.Generated.cs b/src/Analyzers/CSharp/DiagnosticIdentifiers.Generated.cs
@@ -222,6 +222,6 @@ public static partial class DiagnosticIdentifiers
         public const string InvalidReferenceInDocumentationComment = "RCS1263";
         public const string UseVarOrExplicitType = "RCS1264";
         public const string RemoveRedundantCatchBlock = "RCS1265";
-        public const string ConvertStringConcatToInterpolatedString = "RCS1267";
+        public const string UseStringInterpolationInsteadOfStringConcat = "RCS1267";
     }
 }
diff --git a/src/Analyzers/CSharp/DiagnosticRules.Generated.cs b/src/Analyzers/CSharp/DiagnosticRules.Generated.cs
@@ -2628,15 +2628,15 @@ public static partial class DiagnosticRules
             customTags:         Array.Empty<string>());
 
         /// <summary>RCS1267</summary>
-        public static readonly DiagnosticDescriptor ConvertStringConcatToInterpolatedString = DiagnosticDescriptorFactory.Create(
-            id:                 DiagnosticIdentifiers.ConvertStringConcatToInterpolatedString, 
-            title:              "Convert 'string.Concat' to interpolated string", 
-            messageFormat:      "Convert 'string.Concat' to interpolated string", 
+        public static readonly DiagnosticDescriptor UseStringInterpolationInsteadOfStringConcat = DiagnosticDescriptorFactory.Create(
+            id:                 DiagnosticIdentifiers.UseStringInterpolationInsteadOfStringConcat, 
+            title:              "Use string interpolation instead of 'string.Concat'", 
+            messageFormat:      "Use string interpolation instead of 'string.Concat'", 
             category:           DiagnosticCategories.Roslynator, 
             defaultSeverity:    DiagnosticSeverity.Info, 
             isEnabledByDefault: true, 
             description:        null, 
-            helpLinkUri:        DiagnosticIdentifiers.ConvertStringConcatToInterpolatedString, 
+            helpLinkUri:        DiagnosticIdentifiers.UseStringInterpolationInsteadOfStringConcat, 
             customTags:         Array.Empty<string>());
 
     }

diff --git a/...tStringConcatToInterpolatedStringTests.cs → ...nterpolationInsteadOfStringConcatTests.cs b/...tStringConcatToInterpolatedStringTests.cs → ...nterpolationInsteadOfStringConcatTests.cs
@@ -8,11 +8,11 @@
 
 namespace Roslynator.CSharp.Analysis.Tests;
 
-public class RCS1267ConvertStringConcatToInterpolatedStringTests : AbstractCSharpDiagnosticVerifier<InvocationExpressionAnalyzer, InvocationExpressionCodeFixProvider>
+public class RCS1267UseStringInterpolationInsteadOfStringConcatTests : AbstractCSharpDiagnosticVerifier<InvocationExpressionAnalyzer, InvocationExpressionCodeFixProvider>
 {
-    public override DiagnosticDescriptor Descriptor { get; } = DiagnosticRules.ConvertStringConcatToInterpolatedString;
+    public override DiagnosticDescriptor Descriptor { get; } = DiagnosticRules.UseStringInterpolationInsteadOfStringConcat;
 
-    [Fact, Trait(Traits.Analyzer, DiagnosticIdentifiers.ConvertStringConcatToInterpolatedString)]
+    [Fact, Trait(Traits.Analyzer, DiagnosticIdentifiers.UseStringInterpolationInsteadOfStringConcat)]
     public async Task Test()
     {
         await VerifyDiagnosticAndFixAsync("""
@@ -39,7 +39,7 @@ void M()
         );
     }
 
-    [Fact, Trait(Traits.Analyzer, DiagnosticIdentifiers.ConvertStringConcatToInterpolatedString)]
+    [Fact, Trait(Traits.Analyzer, DiagnosticIdentifiers.UseStringInterpolationInsteadOfStringConcat)]
     public async Task Test_Verbatim()
     {
         await VerifyDiagnosticAndFixAsync("""
@@ -66,7 +66,7 @@ void M()
         );
     }
 
-    [Fact, Trait(Traits.Analyzer, DiagnosticIdentifiers.ConvertStringConcatToInterpolatedString)]
+    [Fact, Trait(Traits.Analyzer, DiagnosticIdentifiers.UseStringInterpolationInsteadOfStringConcat)]
     public async Task TestNoDiagnostic_SingleArgument()
     {
         await VerifyNoDiagnosticAsync("""
@@ -83,7 +83,7 @@ void M()
         );
     }
 
-    [Fact, Trait(Traits.Analyzer, DiagnosticIdentifiers.ConvertStringConcatToInterpolatedString)]
+    [Fact, Trait(Traits.Analyzer, DiagnosticIdentifiers.UseStringInterpolationInsteadOfStringConcat)]
     public async Task TestNoDiagnostic_NoStringLiteral()
     {
         await VerifyNoDiagnosticAsync("""
@@ -100,7 +100,7 @@ void M()
         );
     }
 
-    [Fact, Trait(Traits.Analyzer, DiagnosticIdentifiers.ConvertStringConcatToInterpolatedString)]
+    [Fact, Trait(Traits.Analyzer, DiagnosticIdentifiers.UseStringInterpolationInsteadOfStringConcat)]
     public async Task TestNoDiagnostic_ContainsInterpolatedString()
     {
         await VerifyNoDiagnosticAsync("""
@@ -117,7 +117,7 @@ void M()
         );
     }
 
-    [Fact, Trait(Traits.Analyzer, DiagnosticIdentifiers.ConvertStringConcatToInterpolatedString)]
+    [Fact, Trait(Traits.Analyzer, DiagnosticIdentifiers.UseStringInterpolationInsteadOfStringConcat)]
     public async Task TestNoDiagnostic_ContainsVerbatimAndNonVerbatimLiterals()
     {
         await VerifyNoDiagnosticAsync("""
@@ -134,7 +134,7 @@ void M()
         );
     }
 
-    [Fact, Trait(Traits.Analyzer, DiagnosticIdentifiers.ConvertStringConcatToInterpolatedString)]
+    [Fact, Trait(Traits.Analyzer, DiagnosticIdentifiers.UseStringInterpolationInsteadOfStringConcat)]
     public async Task TestNoDiagnostic_ContainsVerbatimAndNonVerbatimLiterals2()
     {
         await VerifyNoDiagnosticAsync("""
@@ -151,7 +151,7 @@ void M()
         );
     }
 
-    [Fact, Trait(Traits.Analyzer, DiagnosticIdentifiers.ConvertStringConcatToInterpolatedString)]
+    [Fact, Trait(Traits.Analyzer, DiagnosticIdentifiers.UseStringInterpolationInsteadOfStringConcat)]
     public async Task TestNoDiagnostic_SpansOverMultipleLines()
     {
         await VerifyNoDiagnosticAsync("""

diff --git a/src/VisualStudioCode/package/src/configurationFiles.generated.ts b/src/VisualStudioCode/package/src/configurationFiles.generated.ts
@@ -910,7 +910,7 @@ roslynator_analyzers.enabled_by_default = true|false
 # Remove redundant catch block
 #dotnet_diagnostic.rcs1265.severity = suggestion
 
-# Convert 'string.Concat' to interpolated string
+# Use string interpolation instead of 'string.Concat'
 #dotnet_diagnostic.rcs1267.severity = suggestion
 
 # Use pattern matching