Reformatted to optimize long lines in test project

bkoelman · bkoelman · commit ea0b2fd5ff5f · 2021-03-14T23:13:21.000+01:00
diff --git a/src/CSharpGuidelinesAnalyzer/CSharpGuidelinesAnalyzer.Test/CSharpGuidelinesAnalysisTestFixture.cs b/src/CSharpGuidelinesAnalyzer/CSharpGuidelinesAnalyzer.Test/CSharpGuidelinesAnalysisTestFixture.cs
@@ -5,8 +5,7 @@ namespace CSharpGuidelinesAnalyzer.Test
 {
     public abstract class CSharpGuidelinesAnalysisTestFixture : AnalysisTestFixture
     {
-        private protected void VerifyGuidelineDiagnostic([NotNull] ParsedSourceCode source,
-            [NotNull] [ItemNotNull] params string[] messages)
+        private protected void VerifyGuidelineDiagnostic([NotNull] ParsedSourceCode source, [NotNull] [ItemNotNull] params string[] messages)
         {
             Guard.NotNull(source, nameof(source));
             Guard.NotNull(messages, nameof(messages));
diff --git a/src/CSharpGuidelinesAnalyzer/CSharpGuidelinesAnalyzer.Test/CSharpGuidelinesAnalyzer.Test.csproj.DotSettings b/src/CSharpGuidelinesAnalyzer/CSharpGuidelinesAnalyzer.Test/CSharpGuidelinesAnalyzer.Test.csproj.DotSettings
@@ -1,13 +1,7 @@
 ﻿<wpf:ResourceDictionary xml:space="preserve" xmlns:x="http://schemas.microsoft.com/winfx/2006/xaml" xmlns:s="clr-namespace:System;assembly=mscorlib" xmlns:ss="urn:shemas-jetbrains-com:settings-storage-xaml" xmlns:wpf="http://schemas.microsoft.com/winfx/2006/xaml/presentation">
 	<s:Boolean x:Key="/Default/CodeStyle/CodeFormatting/CSharpFormat/KEEP_USER_LINEBREAKS/@EntryValue">True</s:Boolean>
-	<s:String x:Key="/Default/CodeStyle/CodeFormatting/CSharpFormat/PLACE_ACCESSORHOLDER_ATTRIBUTE_ON_SAME_LINE_EX/@EntryValue">NEVER</s:String>
-	<s:Boolean x:Key="/Default/Environment/SettingsMigration/IsMigratorApplied/=JetBrains_002EReSharper_002EPsi_002ECSharp_002ECodeStyle_002ECSharpAttributeForSingleLineMethodUpgrade/@EntryIndexedValue">True</s:Boolean>
 	<s:Boolean x:Key="/Default/Environment/SettingsMigration/IsMigratorApplied/=JetBrains_002EReSharper_002EPsi_002ECSharp_002ECodeStyle_002ECSharpKeepExistingMigration/@EntryIndexedValue">True</s:Boolean>
 	<s:Boolean x:Key="/Default/Environment/SettingsMigration/IsMigratorApplied/=JetBrains_002EReSharper_002EPsi_002ECSharp_002ECodeStyle_002ECSharpPlaceEmbeddedOnSameLineMigration/@EntryIndexedValue">True</s:Boolean>
-	<s:Boolean x:Key="/Default/Environment/SettingsMigration/IsMigratorApplied/=JetBrains_002EReSharper_002EPsi_002ECSharp_002ECodeStyle_002ECSharpRenamePlacementToArrangementMigration/@EntryIndexedValue">True</s:Boolean>
 	<s:Boolean x:Key="/Default/Environment/SettingsMigration/IsMigratorApplied/=JetBrains_002EReSharper_002EPsi_002ECSharp_002ECodeStyle_002ECSharpUseContinuousIndentInsideBracesMigration/@EntryIndexedValue">True</s:Boolean>
-	<s:Boolean x:Key="/Default/Environment/SettingsMigration/IsMigratorApplied/=JetBrains_002EReSharper_002EPsi_002ECSharp_002ECodeStyle_002ESettingsUpgrade_002EAddAccessorOwnerDeclarationBracesMigration/@EntryIndexedValue">True</s:Boolean>
-	<s:Boolean x:Key="/Default/Environment/SettingsMigration/IsMigratorApplied/=JetBrains_002EReSharper_002EPsi_002ECSharp_002ECodeStyle_002ESettingsUpgrade_002ECSharpPlaceAttributeOnSameLineMigration/@EntryIndexedValue">True</s:Boolean>
 	<s:Boolean x:Key="/Default/Environment/SettingsMigration/IsMigratorApplied/=JetBrains_002EReSharper_002EPsi_002ECSharp_002ECodeStyle_002ESettingsUpgrade_002EMigrateBlankLinesAroundFieldToBlankLinesAroundProperty/@EntryIndexedValue">True</s:Boolean>
-	<s:Boolean x:Key="/Default/Environment/SettingsMigration/IsMigratorApplied/=JetBrains_002EReSharper_002EPsi_002ECSharp_002ECodeStyle_002ESettingsUpgrade_002EMigrateThisQualifierSettings/@EntryIndexedValue">True</s:Boolean>
 </wpf:ResourceDictionary>
diff --git a/src/CSharpGuidelinesAnalyzer/CSharpGuidelinesAnalyzer.Test/TestDataBuilders/AnalyzerOptionsBuilder.cs b/src/CSharpGuidelinesAnalyzer/CSharpGuidelinesAnalyzer.Test/TestDataBuilders/AnalyzerOptionsBuilder.cs
@@ -22,9 +22,7 @@ public AnalyzerOptions Build()
                     xmlSettings
                 }.ToImmutableArray();
 
-            return editorConfigSettings != null
-                ? new AnalyzerOptions(textArray, editorConfigSettings)
-                : new AnalyzerOptions(textArray);
+            return editorConfigSettings != null ? new AnalyzerOptions(textArray, editorConfigSettings) : new AnalyzerOptions(textArray);
         }
 
         [NotNull]
diff --git a/src/CSharpGuidelinesAnalyzer/CSharpGuidelinesAnalyzer.Test/TestDataBuilders/EditorConfigSettingsBuilder.cs b/src/CSharpGuidelinesAnalyzer/CSharpGuidelinesAnalyzer.Test/TestDataBuilders/EditorConfigSettingsBuilder.cs
@@ -53,8 +53,7 @@ public override AnalyzerConfigOptions GetOptions([NotNull] AdditionalText textFi
             private sealed class FakeAnalyzerConfigOptions : AnalyzerConfigOptions
             {
                 [NotNull]
-                public IDictionary<string, string> Settings { get; } =
-                    new Dictionary<string, string>(StringComparer.InvariantCultureIgnoreCase);
+                public IDictionary<string, string> Settings { get; } = new Dictionary<string, string>(StringComparer.InvariantCultureIgnoreCase);
 
                 public override bool TryGetValue([NotNull] string key, [CanBeNull] out string value)
                 {
diff --git a/src/CSharpGuidelinesAnalyzer/CSharpGuidelinesAnalyzer.Test/TestDataBuilders/SourceCodeBuilder.cs b/src/CSharpGuidelinesAnalyzer/CSharpGuidelinesAnalyzer.Test/TestDataBuilders/SourceCodeBuilder.cs
@@ -16,8 +16,8 @@ namespace CSharpGuidelinesAnalyzer.Test.TestDataBuilders
     internal abstract class SourceCodeBuilder : ITestDataBuilder<ParsedSourceCode>
     {
         [NotNull]
-        private static readonly AnalyzerTestContext DefaultTestContext = new AnalyzerTestContext(string.Empty,
-            Array.Empty<TextSpan>(), LanguageNames.CSharp, new AnalyzerOptions(ImmutableArray<AdditionalText>.Empty));
+        private static readonly AnalyzerTestContext DefaultTestContext = new AnalyzerTestContext(string.Empty, Array.Empty<TextSpan>(), LanguageNames.CSharp,
+            new AnalyzerOptions(ImmutableArray<AdditionalText>.Empty));
 
         [ItemNotNull]
         protected static readonly ImmutableArray<string> DefaultNamespaceImports = new[]
@@ -87,8 +87,7 @@ protected string GetLinesOfCode([NotNull] [ItemNotNull] IEnumerable<string> code
             return builder.ToString();
         }
 
-        private static void AppendCodeBlocks([NotNull] [ItemNotNull] IEnumerable<string> codeBlocks,
-            [NotNull] StringBuilder builder)
+        private static void AppendCodeBlocks([NotNull] [ItemNotNull] IEnumerable<string> codeBlocks, [NotNull] StringBuilder builder)
         {
             bool isInFirstBlock = true;
 
diff --git a/src/CSharpGuidelinesAnalyzer/CSharpGuidelinesAnalyzer.Test/TestDataBuilders/SourceCodeBuilderExtensions.cs b/src/CSharpGuidelinesAnalyzer/CSharpGuidelinesAnalyzer.Test/TestDataBuilders/SourceCodeBuilderExtensions.cs
@@ -117,8 +117,7 @@ public static TBuilder AllowingCompileErrors<TBuilder>([NotNull] this TBuilder s
         }
 
         [NotNull]
-        public static TBuilder WithOptions<TBuilder>([NotNull] this TBuilder source,
-            [NotNull] AnalyzerOptionsBuilder builder)
+        public static TBuilder WithOptions<TBuilder>([NotNull] this TBuilder source, [NotNull] AnalyzerOptionsBuilder builder)
             where TBuilder : SourceCodeBuilder
         {
             Guard.NotNull(builder, nameof(builder));

Original file line number	Diff line number	Diff line change
`@@ -5,8 +5,7 @@ namespace CSharpGuidelinesAnalyzer.Test`
`5`	`5`	`{`
`6`	`6`	`public abstract class CSharpGuidelinesAnalysisTestFixture : AnalysisTestFixture`
`7`	`7`	`{`
`8`		`- private protected void VerifyGuidelineDiagnostic([NotNull] ParsedSourceCode source,`
`9`		`- [NotNull] [ItemNotNull] params string[] messages)`
	`8`	`+ private protected void VerifyGuidelineDiagnostic([NotNull] ParsedSourceCode source, [NotNull] [ItemNotNull] params string[] messages)`
`10`	`9`	`{`
`11`	`10`	`Guard.NotNull(source, nameof(source));`
`12`	`11`	`Guard.NotNull(messages, nameof(messages));`
Original file line number	Diff line number	Diff line change
`@@ -22,9 +22,7 @@ public AnalyzerOptions Build()`
`22`	`22`	`xmlSettings`
`23`	`23`	`}.ToImmutableArray();`
`24`	`24`
`25`		`- return editorConfigSettings != null`
`26`		`- ? new AnalyzerOptions(textArray, editorConfigSettings)`
`27`		`- : new AnalyzerOptions(textArray);`
	`25`	`+ return editorConfigSettings != null ? new AnalyzerOptions(textArray, editorConfigSettings) : new AnalyzerOptions(textArray);`
`28`	`26`	`}`
`29`	`27`
`30`	`28`	`[NotNull]`
Original file line number	Diff line number	Diff line change
`@@ -53,8 +53,7 @@ public override AnalyzerConfigOptions GetOptions([NotNull] AdditionalText textFi`
`53`	`53`	`private sealed class FakeAnalyzerConfigOptions : AnalyzerConfigOptions`
`54`	`54`	`{`
`55`	`55`	`[NotNull]`
`56`		`- public IDictionary<string, string> Settings { get; } =`
`57`		`- new Dictionary<string, string>(StringComparer.InvariantCultureIgnoreCase);`
	`56`	`+ public IDictionary<string, string> Settings { get; } = new Dictionary<string, string>(StringComparer.InvariantCultureIgnoreCase);`
`58`	`57`
`59`	`58`	`public override bool TryGetValue([NotNull] string key, [CanBeNull] out string value)`
`60`	`59`	`{`
Original file line number	Diff line number	Diff line change
`@@ -16,8 +16,8 @@ namespace CSharpGuidelinesAnalyzer.Test.TestDataBuilders`
`16`	`16`	`internal abstract class SourceCodeBuilder : ITestDataBuilder<ParsedSourceCode>`
`17`	`17`	`{`
`18`	`18`	`[NotNull]`
`19`		`- private static readonly AnalyzerTestContext DefaultTestContext = new AnalyzerTestContext(string.Empty,`
`20`		`- Array.Empty<TextSpan>(), LanguageNames.CSharp, new AnalyzerOptions(ImmutableArray<AdditionalText>.Empty));`
	`19`	`+ private static readonly AnalyzerTestContext DefaultTestContext = new AnalyzerTestContext(string.Empty, Array.Empty<TextSpan>(), LanguageNames.CSharp,`
	`20`	`+ new AnalyzerOptions(ImmutableArray<AdditionalText>.Empty));`
`21`	`21`
`22`	`22`	`[ItemNotNull]`
`23`	`23`	`protected static readonly ImmutableArray<string> DefaultNamespaceImports = new[]`
`@@ -87,8 +87,7 @@ protected string GetLinesOfCode([NotNull] [ItemNotNull] IEnumerable<string> code`
`87`	`87`	`return builder.ToString();`
`88`	`88`	`}`
`89`	`89`
`90`		`- private static void AppendCodeBlocks([NotNull] [ItemNotNull] IEnumerable<string> codeBlocks,`
`91`		`- [NotNull] StringBuilder builder)`
	`90`	`+ private static void AppendCodeBlocks([NotNull] [ItemNotNull] IEnumerable<string> codeBlocks, [NotNull] StringBuilder builder)`
`92`	`91`	`{`
`93`	`92`	`bool isInFirstBlock = true;`
`94`	`93`
Original file line number	Diff line number	Diff line change
`@@ -117,8 +117,7 @@ public static TBuilder AllowingCompileErrors<TBuilder>([NotNull] this TBuilder s`
`117`	`117`	`}`
`118`	`118`
`119`	`119`	`[NotNull]`
`120`		`- public static TBuilder WithOptions<TBuilder>([NotNull] this TBuilder source,`
`121`		`- [NotNull] AnalyzerOptionsBuilder builder)`
	`120`	`+ public static TBuilder WithOptions<TBuilder>([NotNull] this TBuilder source, [NotNull] AnalyzerOptionsBuilder builder)`
`122`	`121`	`where TBuilder : SourceCodeBuilder`
`123`	`122`	`{`
`124`	`123`	`Guard.NotNull(builder, nameof(builder));`