Remove ability to skip LF endings in tests

Author: davidwengier

src/Shared/Microsoft.AspNetCore.Razor.Test.Common/Formatting/FormattingTestCase.cs

@@ -31,16 +31,6 @@ protected override string GetDisplayName(IAttributeInfo factAttribute, string di
             (_shouldFlipLineEndings ? " (LF)" : " (CRLF)");
     }
 
-    protected override string GetSkipReason(IAttributeInfo factAttribute)
-    {
-        if (_shouldFlipLineEndings && factAttribute.GetNamedArgument<bool>(nameof(FormattingTestFactAttribute.SkipFlipLineEnding)))
-        {
-            return "Some tests fail with LF line endings";
-        }
-
-        return base.GetSkipReason(factAttribute);
-    }
-
     public override Task<RunSummary> RunAsync(IMessageSink diagnosticMessageSink, IMessageBus messageBus, object[] constructorArguments, ExceptionAggregator aggregator, CancellationTokenSource cancellationTokenSource)
     {
         Debug.Assert(constructorArguments.Length >= 1 && constructorArguments[0] is FormattingTestContext, $"{TestMethod.TestClass.Class.Name}.{TestMethod.Method.Name} uses a formatting test attribute in a class without a FormattingTestContext parameter?");

src/Shared/Microsoft.AspNetCore.Razor.Test.Common/Formatting/FormattingTestFactAttribute.cs

@@ -11,5 +11,4 @@ namespace Microsoft.AspNetCore.Razor.Test.Common;
 [XunitTestCaseDiscoverer($"Microsoft.AspNetCore.Razor.Test.Common.{nameof(FormattingFactDiscoverer)}", "Microsoft.AspNetCore.Razor.Test.Common")]
 internal sealed class FormattingTestFactAttribute : FactAttribute
 {
-    public bool SkipFlipLineEnding { get; set; }
 }

src/Shared/Microsoft.AspNetCore.Razor.Test.Common/Formatting/FormattingTestTheoryAttribute.cs

@@ -12,5 +12,4 @@ namespace Microsoft.AspNetCore.Razor.Test.Common;
 internal sealed class FormattingTestTheoryAttribute : TheoryAttribute
 {
     // NOTE: Property names need to match FormattingTestFactAttribute
-    public bool SkipFlipLineEnding { get; set; }
 }