Update to Microsoft.CodeAnalysis.CSharp.CodeFix.Testing.XUnit 1.0.0-beta1-63810-01

sharwell · sharwell · commit ac7d7c913535 · 2019-02-10T21:50:10.000-06:00
diff --git a/StyleCop.Analyzers/StyleCop.Analyzers.Test.CSharp7/MaintainabilityRules/SA1119CSharp7UnitTests.cs b/StyleCop.Analyzers/StyleCop.Analyzers.Test.CSharp7/MaintainabilityRules/SA1119CSharp7UnitTests.cs
@@ -9,9 +9,7 @@ namespace StyleCop.Analyzers.Test.CSharp7.MaintainabilityRules
     using StyleCop.Analyzers.Test.MaintainabilityRules;
     using TestHelper;
     using Xunit;
-    using static StyleCop.Analyzers.Test.Verifiers.StyleCopCodeFixVerifier<
-        StyleCop.Analyzers.MaintainabilityRules.SA1119StatementMustNotUseUnnecessaryParenthesis,
-        StyleCop.Analyzers.MaintainabilityRules.SA1119CodeFixProvider>;
+    using static StyleCop.Analyzers.Test.Verifiers.StyleCopDiagnosticVerifier<StyleCop.Analyzers.MaintainabilityRules.SA1119StatementMustNotUseUnnecessaryParenthesis>;
 
     public class SA1119CSharp7UnitTests : SA1119UnitTests
     {
diff --git a/StyleCop.Analyzers/StyleCop.Analyzers.Test/DocumentationRules/FileHeaderTestBase.cs b/StyleCop.Analyzers/StyleCop.Analyzers.Test/DocumentationRules/FileHeaderTestBase.cs
@@ -121,6 +121,7 @@ protected Task VerifyCSharpFixAsync(string source, DiagnosticResult[] expected,
                 Settings = this.GetSettings(),
             };
 
+            test.Exclusions &= ~AnalysisExclusions.Suppression;
             test.ExpectedDiagnostics.AddRange(expected);
             test.RemainingDiagnostics.AddRange(remainingDiagnostics);
             test.DisabledDiagnostics.AddRange(this.GetDisabledDiagnostics());
diff --git a/StyleCop.Analyzers/StyleCop.Analyzers.Test/DocumentationRules/NoXmlFileHeaderUnitTests.cs b/StyleCop.Analyzers/StyleCop.Analyzers.Test/DocumentationRules/NoXmlFileHeaderUnitTests.cs
@@ -410,6 +410,7 @@ private static Task VerifyCSharpFixAsync(string source, string testSettings, Dia
                 Settings = testSettings,
             };
 
+            test.Exclusions &= ~AnalysisExclusions.Suppression;
             test.ExpectedDiagnostics.AddRange(expected);
             return test.RunAsync(cancellationToken);
         }
diff --git a/StyleCop.Analyzers/StyleCop.Analyzers.Test/ExclusionTests.cs b/StyleCop.Analyzers/StyleCop.Analyzers.Test/ExclusionTests.cs
@@ -6,6 +6,7 @@ namespace StyleCop.Analyzers.Test
     using System.Collections.Generic;
     using System.Threading;
     using System.Threading.Tasks;
+    using Microsoft.CodeAnalysis.Testing;
     using Xunit;
     using static StyleCop.Analyzers.Test.Verifiers.StyleCopDiagnosticVerifier<TestHelper.ExclusionTestAnalyzer>;
 
@@ -82,7 +83,7 @@ public async Task TestIsNotExcludedAsync(string filename, string testCode)
         {
             var result = Diagnostic().WithLocation(filename, 1, 1);
 
-            await new CSharpTest
+            var test = new CSharpTest
             {
                 TestSources =
                 {
@@ -92,7 +93,10 @@ public async Task TestIsNotExcludedAsync(string filename, string testCode)
                 {
                     result,
                 },
-            }.RunAsync(CancellationToken.None).ConfigureAwait(false);
+            };
+
+            test.Exclusions &= ~AnalysisExclusions.Suppression;
+            await test.RunAsync(CancellationToken.None).ConfigureAwait(false);
         }
     }
 }
diff --git a/StyleCop.Analyzers/StyleCop.Analyzers.Test/LayoutRules/SA1503UnitTests.cs b/StyleCop.Analyzers/StyleCop.Analyzers.Test/LayoutRules/SA1503UnitTests.cs
@@ -350,8 +350,15 @@ public void Bar(int i)
     }
 }";
 
-            var expected = Diagnostic().WithLocation(8, 13);
-            await VerifyCSharpFixAsync(testCode, expected, fixedTestCode, CancellationToken.None).ConfigureAwait(false);
+            var test = new CSharpTest
+            {
+                TestCode = testCode,
+                ExpectedDiagnostics = { Diagnostic().WithLocation(8, 13) },
+                FixedCode = fixedTestCode,
+            };
+
+            test.Exclusions &= ~AnalysisExclusions.Suppression;
+            await test.RunAsync(CancellationToken.None).ConfigureAwait(false);
         }
 
         /// <summary>
diff --git a/StyleCop.Analyzers/StyleCop.Analyzers.Test/LayoutRules/SA1519UnitTests.cs b/StyleCop.Analyzers/StyleCop.Analyzers.Test/LayoutRules/SA1519UnitTests.cs
@@ -582,8 +582,15 @@ public void Bar(int i)
     }
 }";
 
-            var expected = Diagnostic().WithLocation(8, 13);
-            await VerifyCSharpFixAsync(testCode, expected, fixedTestCode, CancellationToken.None).ConfigureAwait(false);
+            var test = new CSharpTest
+            {
+                TestCode = testCode,
+                ExpectedDiagnostics = { Diagnostic().WithLocation(8, 13) },
+                FixedCode = fixedTestCode,
+            };
+
+            test.Exclusions &= ~AnalysisExclusions.Suppression;
+            await test.RunAsync(CancellationToken.None).ConfigureAwait(false);
         }
 
         /// <summary>
diff --git a/StyleCop.Analyzers/StyleCop.Analyzers.Test/LayoutRules/SA1520UnitTests.cs b/StyleCop.Analyzers/StyleCop.Analyzers.Test/LayoutRules/SA1520UnitTests.cs
@@ -395,8 +395,15 @@ public void Bar(int i)
     }
 }";
 
-            var expected = Diagnostic().WithLocation(8, 13);
-            await VerifyCSharpFixAsync(testCode, expected, fixedTestCode, CancellationToken.None).ConfigureAwait(false);
+            var test = new CSharpTest
+            {
+                TestCode = testCode,
+                ExpectedDiagnostics = { Diagnostic().WithLocation(8, 13) },
+                FixedCode = fixedTestCode,
+            };
+
+            test.Exclusions &= ~AnalysisExclusions.Suppression;
+            await test.RunAsync(CancellationToken.None).ConfigureAwait(false);
         }
 
         /// <summary>
diff --git a/StyleCop.Analyzers/StyleCop.Analyzers.Test/MaintainabilityRules/SA1119UnitTests.cs b/StyleCop.Analyzers/StyleCop.Analyzers.Test/MaintainabilityRules/SA1119UnitTests.cs
@@ -7,10 +7,9 @@ namespace StyleCop.Analyzers.Test.MaintainabilityRules
     using System.Threading.Tasks;
     using Microsoft.CodeAnalysis.Testing;
     using StyleCop.Analyzers.MaintainabilityRules;
+    using StyleCop.Analyzers.Test.Verifiers;
     using Xunit;
-    using static StyleCop.Analyzers.Test.Verifiers.StyleCopCodeFixVerifier<
-        StyleCop.Analyzers.MaintainabilityRules.SA1119StatementMustNotUseUnnecessaryParenthesis,
-        StyleCop.Analyzers.MaintainabilityRules.SA1119CodeFixProvider>;
+    using static StyleCop.Analyzers.Test.Verifiers.StyleCopDiagnosticVerifier<StyleCop.Analyzers.MaintainabilityRules.SA1119StatementMustNotUseUnnecessaryParenthesis>;
 
     public class SA1119UnitTests
     {
@@ -1039,6 +1038,14 @@ public void Bar()
         bool flag = false;
         string data = $""{(flag)}"";
     }
+}";
+            string fixedCode = @"class Foo
+{
+    public void Bar()
+    {
+        bool flag = false;
+        string data = $""{ flag}"";
+    }
 }";
             DiagnosticResult[] expected =
             {
@@ -1047,7 +1054,7 @@ public void Bar()
                 Diagnostic(ParenthesesDiagnosticId).WithLocation(6, 31),
             };
 
-            await VerifyCSharpDiagnosticAsync(testCode, expected, CancellationToken.None).ConfigureAwait(false);
+            await VerifyCSharpFixAsync(testCode, expected, fixedCode, CancellationToken.None).ConfigureAwait(false);
         }
 
         [Fact]
@@ -1429,5 +1436,20 @@ public static void Main(string[] args)
 
             await VerifyCSharpFixAsync(testCode, expected, fixedCode, CancellationToken.None).ConfigureAwait(false);
         }
+
+        protected static async Task VerifyCSharpFixAsync(string source, DiagnosticResult[] expected, string fixedSource, CancellationToken cancellationToken)
+        {
+            var test = new StyleCopCodeFixVerifier<SA1119StatementMustNotUseUnnecessaryParenthesis, SA1119CodeFixProvider>.CSharpTest
+            {
+                TestCode = source,
+                FixedCode = fixedSource,
+            };
+
+            // Workaround for https://github.com/dotnet/roslyn-sdk/pull/252
+            test.Exclusions &= ~AnalysisExclusions.Suppression;
+
+            test.ExpectedDiagnostics.AddRange(expected);
+            await test.RunAsync(cancellationToken).ConfigureAwait(false);
+        }
     }
 }
diff --git a/StyleCop.Analyzers/StyleCop.Analyzers.Test/MaintainabilityRules/SA1412UnitTests.cs b/StyleCop.Analyzers/StyleCop.Analyzers.Test/MaintainabilityRules/SA1412UnitTests.cs
@@ -8,6 +8,7 @@ namespace StyleCop.Analyzers.Test.MaintainabilityRules
     using System.Threading;
     using System.Threading.Tasks;
     using Microsoft.CodeAnalysis.CodeFixes;
+    using Microsoft.CodeAnalysis.Testing;
     using Microsoft.CodeAnalysis.Text;
     using Xunit;
     using static StyleCop.Analyzers.Test.Verifiers.StyleCopCodeFixVerifier<
@@ -38,12 +39,19 @@ public async Task TestFileWithWrongEncodingAsync(int codepage)
 
             var expected = Diagnostic().WithLocation(1, 1);
 
-            await new CSharpTest
+            var test = new CSharpTest
             {
                 TestSources = { testCode },
                 ExpectedDiagnostics = { expected },
                 FixedSources = { fixedCode },
-            }.RunAsync(CancellationToken.None).ConfigureAwait(false);
+            };
+
+            // Workaround for https://github.com/dotnet/roslyn-sdk/pull/251
+            test.FixedState.InheritanceMode = StateInheritanceMode.AutoInherit;
+            test.BatchFixedState.InheritanceMode = StateInheritanceMode.AutoInherit;
+
+            test.Exclusions &= ~AnalysisExclusions.Suppression;
+            await test.RunAsync(CancellationToken.None).ConfigureAwait(false);
         }
 
         [Fact]
@@ -54,12 +62,19 @@ public async Task TestFileWithUtf8EncodingWithoutBOMAsync()
 
             var expected = Diagnostic().WithLocation(1, 1);
 
-            await new CSharpTest
+            var test = new CSharpTest
             {
                 TestSources = { testCode },
                 ExpectedDiagnostics = { expected },
                 FixedSources = { fixedCode },
-            }.RunAsync(CancellationToken.None).ConfigureAwait(false);
+            };
+
+            // Workaround for https://github.com/dotnet/roslyn-sdk/pull/251
+            test.FixedState.InheritanceMode = StateInheritanceMode.AutoInherit;
+            test.BatchFixedState.InheritanceMode = StateInheritanceMode.AutoInherit;
+
+            test.Exclusions &= ~AnalysisExclusions.Suppression;
+            await test.RunAsync(CancellationToken.None).ConfigureAwait(false);
         }
 
         [Theory]
@@ -73,7 +88,7 @@ public async Task TestFixAllAsync(int codepage)
         [Fact]
         public async Task TestFixAllWithMultipleEncodingsAsync()
         {
-            await new CSharpTest
+            var test = new CSharpTest
             {
                 TestSources =
                 {
@@ -95,12 +110,19 @@ public async Task TestFixAllWithMultipleEncodingsAsync()
                 },
                 NumberOfFixAllIterations = 2,
                 NumberOfFixAllInDocumentIterations = 3,
-            }.RunAsync(CancellationToken.None).ConfigureAwait(false);
+            };
+
+            // Workaround for https://github.com/dotnet/roslyn-sdk/pull/251
+            test.FixedState.InheritanceMode = StateInheritanceMode.AutoInherit;
+            test.BatchFixedState.InheritanceMode = StateInheritanceMode.AutoInherit;
+
+            test.Exclusions &= ~AnalysisExclusions.Suppression;
+            await test.RunAsync(CancellationToken.None).ConfigureAwait(false);
         }
 
         private async Task TestFixAllExecuterAsync(int codepage, FixAllScope scope)
         {
-            await new CSharpTest
+            var test = new CSharpTest
             {
                 TestSources =
                 {
@@ -119,7 +141,14 @@ private async Task TestFixAllExecuterAsync(int codepage, FixAllScope scope)
                 },
                 NumberOfFixAllIterations = 1,
                 NumberOfFixAllInDocumentIterations = 2,
-            }.RunAsync(CancellationToken.None).ConfigureAwait(false);
+            };
+
+            // Workaround for https://github.com/dotnet/roslyn-sdk/pull/251
+            test.FixedState.InheritanceMode = StateInheritanceMode.AutoInherit;
+            test.BatchFixedState.InheritanceMode = StateInheritanceMode.AutoInherit;
+
+            test.Exclusions &= ~AnalysisExclusions.Suppression;
+            await test.RunAsync(CancellationToken.None).ConfigureAwait(false);
         }
     }
 }
diff --git a/StyleCop.Analyzers/StyleCop.Analyzers.Test/Settings/SettingsFileCodeFixProviderUnitTests.cs b/StyleCop.Analyzers/StyleCop.Analyzers.Test/Settings/SettingsFileCodeFixProviderUnitTests.cs
@@ -31,7 +31,7 @@ namespace NamespaceName
         [Fact]
         public async Task TestMissingFileHeaderWithLeadingTriviaAsync()
         {
-            await new CSharpTest
+            var test = new CSharpTest
             {
                 TestCode = TestCode,
                 ExpectedDiagnostics = { Diagnostic(FileHeaderAnalyzers.SA1633DescriptorMissing).WithLocation(1, 1) },
@@ -42,7 +42,10 @@ public async Task TestMissingFileHeaderWithLeadingTriviaAsync()
                     (SettingsHelper.SettingsFileName, SettingsFileCodeFixProvider.DefaultSettingsFileContent),
                 },
                 Settings = null,
-            }.RunAsync(CancellationToken.None).ConfigureAwait(false);
+            };
+
+            test.Exclusions &= ~AnalysisExclusions.Suppression;
+            await test.RunAsync(CancellationToken.None).ConfigureAwait(false);
         }
 
         /// <summary>
@@ -52,7 +55,7 @@ public async Task TestMissingFileHeaderWithLeadingTriviaAsync()
         [Fact]
         public async Task TestSettingsFileDoesNotExistAsync()
         {
-            await new CSharpTest
+            var test = new CSharpTest
             {
                 TestCode = TestCode,
                 ExpectedDiagnostics = { Diagnostic(FileHeaderAnalyzers.SA1633DescriptorMissing).WithLocation(1, 1) },
@@ -63,7 +66,10 @@ public async Task TestSettingsFileDoesNotExistAsync()
                     (SettingsHelper.SettingsFileName, SettingsFileCodeFixProvider.DefaultSettingsFileContent),
                 },
                 Settings = null,
-            }.RunAsync(CancellationToken.None).ConfigureAwait(false);
+            };
+
+            test.Exclusions &= ~AnalysisExclusions.Suppression;
+            await test.RunAsync(CancellationToken.None).ConfigureAwait(false);
         }
 
         /// <summary>
@@ -73,15 +79,18 @@ public async Task TestSettingsFileDoesNotExistAsync()
         [Fact]
         public async Task TestSettingsFileAlreadyExistsAsync()
         {
-            await new CSharpTest
+            var test = new CSharpTest
             {
                 TestCode = TestCode,
                 ExpectedDiagnostics = { Diagnostic(FileHeaderAnalyzers.SA1633DescriptorMissing).WithLocation(1, 1) },
                 FixedCode = TestCode,
                 FixedState = { InheritanceMode = StateInheritanceMode.AutoInheritAll },
                 Settings = "{}",
                 SettingsFileName = SettingsHelper.SettingsFileName,
-            }.RunAsync(CancellationToken.None).ConfigureAwait(false);
+            };
+
+            test.Exclusions &= ~AnalysisExclusions.Suppression;
+            await test.RunAsync(CancellationToken.None).ConfigureAwait(false);
         }
 
         /// <summary>
@@ -91,15 +100,18 @@ public async Task TestSettingsFileAlreadyExistsAsync()
         [Fact]
         public async Task TestDotPrefixedSettingsFileAlreadyExistsAsync()
         {
-            await new CSharpTest
+            var test = new CSharpTest
             {
                 TestCode = TestCode,
                 ExpectedDiagnostics = { Diagnostic(FileHeaderAnalyzers.SA1633DescriptorMissing).WithLocation(1, 1) },
                 FixedCode = TestCode,
                 FixedState = { InheritanceMode = StateInheritanceMode.AutoInheritAll },
                 Settings = "{}",
                 SettingsFileName = SettingsHelper.AltSettingsFileName,
-            }.RunAsync(CancellationToken.None).ConfigureAwait(false);
+            };
+
+            test.Exclusions &= ~AnalysisExclusions.Suppression;
+            await test.RunAsync(CancellationToken.None).ConfigureAwait(false);
         }
     }
 }
diff --git a/StyleCop.Analyzers/StyleCop.Analyzers.Test/StyleCop.Analyzers.Test.csproj b/StyleCop.Analyzers/StyleCop.Analyzers.Test/StyleCop.Analyzers.Test.csproj
@@ -18,7 +18,7 @@
 
   <ItemGroup>
     <PackageReference Include="Microsoft.CodeAnalysis" Version="1.2.1" />
-    <PackageReference Include="Microsoft.CodeAnalysis.CSharp.CodeFix.Testing.XUnit" Version="1.0.0-beta1-63310-01" />
+    <PackageReference Include="Microsoft.CodeAnalysis.CSharp.CodeFix.Testing.XUnit" Version="1.0.0-beta1-63810-01" />
     <PackageReference Include="Microsoft.VisualStudio.Composition" Version="15.6.36" />
     <PackageReference Include="System.ValueTuple" Version="4.5.0" />
     <PackageReference Include="xunit" Version="2.3.1" />

Original file line number	Diff line number	Diff line change
`@@ -410,6 +410,7 @@ private static Task VerifyCSharpFixAsync(string source, string testSettings, Dia`
`410`	`410`	`Settings = testSettings,`
`411`	`411`	`};`
`412`	`412`
	`413`	`+ test.Exclusions &= ~AnalysisExclusions.Suppression;`
`413`	`414`	`test.ExpectedDiagnostics.AddRange(expected);`
`414`	`415`	`return test.RunAsync(cancellationToken);`
`415`	`416`	`}`
Original file line number	Diff line number	Diff line change
`@@ -6,6 +6,7 @@ namespace StyleCop.Analyzers.Test`
`6`	`6`	`using System.Collections.Generic;`
`7`	`7`	`using System.Threading;`
`8`	`8`	`using System.Threading.Tasks;`
	`9`	`+ using Microsoft.CodeAnalysis.Testing;`
`9`	`10`	`using Xunit;`
`10`	`11`	`using static StyleCop.Analyzers.Test.Verifiers.StyleCopDiagnosticVerifier<TestHelper.ExclusionTestAnalyzer>;`
`11`	`12`
`@@ -82,7 +83,7 @@ public async Task TestIsNotExcludedAsync(string filename, string testCode)`
`82`	`83`	`{`
`83`	`84`	`var result = Diagnostic().WithLocation(filename, 1, 1);`
`84`	`85`
`85`		`- await new CSharpTest`
	`86`	`+ var test = new CSharpTest`
`86`	`87`	`{`
`87`	`88`	`TestSources =`
`88`	`89`	`{`
`@@ -92,7 +93,10 @@ public async Task TestIsNotExcludedAsync(string filename, string testCode)`
`92`	`93`	`{`
`93`	`94`	`result,`
`94`	`95`	`},`
`95`		`- }.RunAsync(CancellationToken.None).ConfigureAwait(false);`
	`96`	`+ };`
	`97`	`+`
	`98`	`+ test.Exclusions &= ~AnalysisExclusions.Suppression;`
	`99`	`+ await test.RunAsync(CancellationToken.None).ConfigureAwait(false);`
`96`	`100`	`}`
`97`	`101`	`}`
`98`	`102`	`}`