From ffa46d0618884693eb08a791e730dde52e6f5eeb Mon Sep 17 00:00:00 2001 From: Manish Vasani Date: Fri, 20 Mar 2020 16:57:41 -0700 Subject: [PATCH] Port UseExplicitTupleName analyzer/fixer/tests to shared layer --- .../CSharp/Tests/CSharpAnalyzers.UnitTests.projitems | 1 + .../UseExplicitTupleName/UseExplicitTupleNameTests.cs | 7 ++++++- src/Analyzers/Core/Analyzers/Analyzers.projitems | 1 + .../UseExplicitTupleNameDiagnosticAnalyzer.cs | 8 ++++++-- src/Analyzers/Core/CodeFixes/CodeFixes.projitems | 1 + .../UseExplicitTupleNameCodeFixProvider.cs | 6 +++--- .../UseExplicitTupleName/UseExplicitTupleNameTests.vb | 0 .../Tests/VisualBasicAnalyzers.UnitTests.projitems | 1 + 8 files changed, 19 insertions(+), 6 deletions(-) rename src/{EditorFeatures/CSharpTest => Analyzers/CSharp/Tests}/UseExplicitTupleName/UseExplicitTupleNameTests.cs (98%) rename src/{Features/Core/Portable => Analyzers/Core/Analyzers}/UseExplicitTupleName/UseExplicitTupleNameDiagnosticAnalyzer.cs (90%) rename src/{Features/Core/Portable => Analyzers/Core/CodeFixes}/UseExplicitTupleName/UseExplicitTupleNameCodeFixProvider.cs (91%) rename src/{EditorFeatures/VisualBasicTest => Analyzers/VisualBasic/Tests}/UseExplicitTupleName/UseExplicitTupleNameTests.vb (100%) diff --git a/src/Analyzers/CSharp/Tests/CSharpAnalyzers.UnitTests.projitems b/src/Analyzers/CSharp/Tests/CSharpAnalyzers.UnitTests.projitems index 7fc9450e76e..382a41cc2c3 100644 --- a/src/Analyzers/CSharp/Tests/CSharpAnalyzers.UnitTests.projitems +++ b/src/Analyzers/CSharp/Tests/CSharpAnalyzers.UnitTests.projitems @@ -17,6 +17,7 @@ + diff --git a/src/EditorFeatures/CSharpTest/UseExplicitTupleName/UseExplicitTupleNameTests.cs b/src/Analyzers/CSharp/Tests/UseExplicitTupleName/UseExplicitTupleNameTests.cs similarity index 98% rename from src/EditorFeatures/CSharpTest/UseExplicitTupleName/UseExplicitTupleNameTests.cs rename to src/Analyzers/CSharp/Tests/UseExplicitTupleName/UseExplicitTupleNameTests.cs index d6b67c02772..928f2dbffce 100644 --- a/src/EditorFeatures/CSharpTest/UseExplicitTupleName/UseExplicitTupleNameTests.cs +++ b/src/Analyzers/CSharp/Tests/UseExplicitTupleName/UseExplicitTupleNameTests.cs @@ -4,13 +4,18 @@ using System.Threading.Tasks; using Microsoft.CodeAnalysis.CodeFixes; -using Microsoft.CodeAnalysis.CodeStyle; using Microsoft.CodeAnalysis.Diagnostics; using Microsoft.CodeAnalysis.Editor.CSharp.UnitTests.Diagnostics; using Microsoft.CodeAnalysis.Test.Utilities; using Microsoft.CodeAnalysis.UseExplicitTupleName; using Xunit; +#if CODE_STYLE +using Microsoft.CodeAnalysis.Internal.Options; +#else +using Microsoft.CodeAnalysis.CodeStyle; +#endif + namespace Microsoft.CodeAnalysis.Editor.CSharp.UnitTests.UseExplicitTupleName { public class UseExplicitTupleNameTests : AbstractCSharpDiagnosticProviderBasedUserDiagnosticTest diff --git a/src/Analyzers/Core/Analyzers/Analyzers.projitems b/src/Analyzers/Core/Analyzers/Analyzers.projitems index ab2962b149d..fd2d2232eb0 100644 --- a/src/Analyzers/Core/Analyzers/Analyzers.projitems +++ b/src/Analyzers/Core/Analyzers/Analyzers.projitems @@ -47,6 +47,7 @@ + diff --git a/src/Features/Core/Portable/UseExplicitTupleName/UseExplicitTupleNameDiagnosticAnalyzer.cs b/src/Analyzers/Core/Analyzers/UseExplicitTupleName/UseExplicitTupleNameDiagnosticAnalyzer.cs similarity index 90% rename from src/Features/Core/Portable/UseExplicitTupleName/UseExplicitTupleNameDiagnosticAnalyzer.cs rename to src/Analyzers/Core/Analyzers/UseExplicitTupleName/UseExplicitTupleNameDiagnosticAnalyzer.cs index da75153c5fc..ddf07b33524 100644 --- a/src/Features/Core/Portable/UseExplicitTupleName/UseExplicitTupleNameDiagnosticAnalyzer.cs +++ b/src/Analyzers/Core/Analyzers/UseExplicitTupleName/UseExplicitTupleNameDiagnosticAnalyzer.cs @@ -9,6 +9,10 @@ using Microsoft.CodeAnalysis.Diagnostics; using Microsoft.CodeAnalysis.Operations; +#if CODE_STYLE +using Microsoft.CodeAnalysis.Internal.Options; +#endif + namespace Microsoft.CodeAnalysis.UseExplicitTupleName { [DiagnosticAnalyzer(LanguageNames.CSharp, LanguageNames.VisualBasic)] @@ -19,8 +23,8 @@ internal class UseExplicitTupleNameDiagnosticAnalyzer : AbstractBuiltInCodeStyle public UseExplicitTupleNameDiagnosticAnalyzer() : base(IDEDiagnosticIds.UseExplicitTupleNameDiagnosticId, CodeStyleOptions.PreferExplicitTupleNames, - title: new LocalizableResourceString(nameof(FeaturesResources.Use_explicitly_provided_tuple_name), FeaturesResources.ResourceManager, typeof(FeaturesResources)), - messageFormat: new LocalizableResourceString(nameof(FeaturesResources.Prefer_explicitly_provided_tuple_element_name), FeaturesResources.ResourceManager, typeof(FeaturesResources))) + title: new LocalizableResourceString(nameof(AnalyzersResources.Use_explicitly_provided_tuple_name), AnalyzersResources.ResourceManager, typeof(AnalyzersResources)), + messageFormat: new LocalizableResourceString(nameof(AnalyzersResources.Prefer_explicitly_provided_tuple_element_name), AnalyzersResources.ResourceManager, typeof(AnalyzersResources))) { } diff --git a/src/Analyzers/Core/CodeFixes/CodeFixes.projitems b/src/Analyzers/Core/CodeFixes/CodeFixes.projitems index 9486be620f3..9e3cc88f1ed 100644 --- a/src/Analyzers/Core/CodeFixes/CodeFixes.projitems +++ b/src/Analyzers/Core/CodeFixes/CodeFixes.projitems @@ -29,6 +29,7 @@ + diff --git a/src/Features/Core/Portable/UseExplicitTupleName/UseExplicitTupleNameCodeFixProvider.cs b/src/Analyzers/Core/CodeFixes/UseExplicitTupleName/UseExplicitTupleNameCodeFixProvider.cs similarity index 91% rename from src/Features/Core/Portable/UseExplicitTupleName/UseExplicitTupleNameCodeFixProvider.cs rename to src/Analyzers/Core/CodeFixes/UseExplicitTupleName/UseExplicitTupleNameCodeFixProvider.cs index 23bdb873cd4..fbcece92415 100644 --- a/src/Features/Core/Portable/UseExplicitTupleName/UseExplicitTupleNameCodeFixProvider.cs +++ b/src/Analyzers/Core/CodeFixes/UseExplicitTupleName/UseExplicitTupleNameCodeFixProvider.cs @@ -57,12 +57,12 @@ public override Task RegisterCodeFixesAsync(CodeFixContext context) return Task.CompletedTask; } - private class MyCodeAction : CodeAction.DocumentChangeAction + private class MyCodeAction : CustomCodeActions.DocumentChangeAction { public MyCodeAction(Func> createChangedDocument) - : base(FeaturesResources.Use_explicitly_provided_tuple_name, + : base(AnalyzersResources.Use_explicitly_provided_tuple_name, createChangedDocument, - FeaturesResources.Use_explicitly_provided_tuple_name) + AnalyzersResources.Use_explicitly_provided_tuple_name) { } } diff --git a/src/EditorFeatures/VisualBasicTest/UseExplicitTupleName/UseExplicitTupleNameTests.vb b/src/Analyzers/VisualBasic/Tests/UseExplicitTupleName/UseExplicitTupleNameTests.vb similarity index 100% rename from src/EditorFeatures/VisualBasicTest/UseExplicitTupleName/UseExplicitTupleNameTests.vb rename to src/Analyzers/VisualBasic/Tests/UseExplicitTupleName/UseExplicitTupleNameTests.vb diff --git a/src/Analyzers/VisualBasic/Tests/VisualBasicAnalyzers.UnitTests.projitems b/src/Analyzers/VisualBasic/Tests/VisualBasicAnalyzers.UnitTests.projitems index 669fa759fd1..2ce7e494f23 100644 --- a/src/Analyzers/VisualBasic/Tests/VisualBasicAnalyzers.UnitTests.projitems +++ b/src/Analyzers/VisualBasic/Tests/VisualBasicAnalyzers.UnitTests.projitems @@ -30,6 +30,7 @@ + -- GitLab