Re-apply "Some random clean up and improvements (#11095)" (#11177)

Author: jjonescz

src/Compiler/Microsoft.AspNetCore.Razor.Language/test/CodeGeneration/DesignTimeNodeWriterTest.cs

@@ -493,7 +493,7 @@ Render Children
             ignoreLineEndingDifferences: true);
     }
 
-    [OSSkipConditionTheory(new[] { "Linux", "OSX" })]
+    [ConditionalTheory(Is.Windows)]
     [InlineData(@"test.cshtml",                     @"test.cshtml")]
     [InlineData(@"pages/test.cshtml",               @"pages\test.cshtml")]
     [InlineData(@"pages\test.cshtml",               @"pages\test.cshtml")]
@@ -540,7 +540,7 @@ public void LinePragma_Is_Adjusted_On_Windows(string fileName, string expectedFi
             ignoreLineEndingDifferences: true);
     }
 
-    [OSSkipConditionTheory(new[] { "Linux", "OSX" })]
+    [ConditionalTheory(Is.Windows)]
     [InlineData(@"test.cshtml",                     @"test.cshtml")]
     [InlineData(@"pages/test.cshtml",               @"pages\test.cshtml")]
     [InlineData(@"pages\test.cshtml",               @"pages\test.cshtml")]

src/Compiler/Microsoft.AspNetCore.Razor.Language/test/DefaultRazorProjectFileSystemTest.cs

@@ -6,7 +6,6 @@
 using System;
 using System.IO;
 using System.Linq;
-using System.Runtime.InteropServices;
 using Xunit;
 
 namespace Microsoft.AspNetCore.Razor.Language;
@@ -261,15 +260,10 @@ public void GetItem_ReturnsFileFromDisk()
         Assert.Equal(Path.Combine("Views", "About", "About.cshtml"), item.RelativePhysicalPath);
     }
 
-    [Fact]
+    // "This test does not makes sense for case sensitive Operating Systems."
+    [ConditionalFact(Is.Windows)]
     public void GetItem_MismatchedCase_ReturnsFileFromDisk()
     {
-        if (!RuntimeInformation.IsOSPlatform(OSPlatform.Windows))
-        {
-            // "This test does not makes sense for case sensitive Operating Systems."
-            return;
-        }
-
         // Arrange
         var filePath = "/Views/About/About.cshtml";
         var lowerCaseTestFolder = TestFolder.ToLowerInvariant();

src/Compiler/Microsoft.AspNetCore.Razor.Language/test/Microsoft.AspNetCore.Razor.Language.Test.csproj

@@ -30,7 +30,6 @@
     <Compile Include="..\..\shared\JsonReaderExtensions.cs" LinkBase="Shared" />
     <Compile Include="..\..\shared\RazorDiagnosticJsonConverter.cs" LinkBase="Shared" />
     <Compile Include="..\..\shared\TagHelperDescriptorJsonConverter.cs" LinkBase="Shared" />
-    <Compile Include="..\..\..\Razor\test\OSSkipConditionFactAttribute.cs" LinkBase="Shared" />
   </ItemGroup>
 
   <ItemGroup>

src/Compiler/Microsoft.CodeAnalysis.Razor.Compiler/src/CSharp/FilePathComparer.cs

@@ -2,7 +2,7 @@
 // The .NET Foundation licenses this file to you under the MIT license.
 
 using System;
-using System.Runtime.InteropServices;
+using Microsoft.AspNetCore.Razor.Utilities;
 
 namespace Microsoft.CodeAnalysis.Razor;
 
@@ -14,7 +14,7 @@ public static StringComparer Instance
     {
         get
         {
-            return _instance ??= RuntimeInformation.IsOSPlatform(OSPlatform.Linux)
+            return _instance ??= PlatformInformation.IsLinux
                 ? StringComparer.Ordinal
                 : StringComparer.OrdinalIgnoreCase;
         }

src/Compiler/Microsoft.CodeAnalysis.Razor.Compiler/src/CSharp/FilePathComparison.cs

@@ -2,7 +2,7 @@
 // The .NET Foundation licenses this file to you under the MIT license.
 
 using System;
-using System.Runtime.InteropServices;
+using Microsoft.AspNetCore.Razor.Utilities;
 
 namespace Microsoft.CodeAnalysis.Razor;
 
@@ -14,7 +14,7 @@ public static StringComparison Instance
     {
         get
         {
-            return _instance ??= RuntimeInformation.IsOSPlatform(OSPlatform.Linux)
+            return _instance ??= PlatformInformation.IsLinux
                 ? StringComparison.Ordinal
                 : StringComparison.OrdinalIgnoreCase;
         }

src/Compiler/Microsoft.CodeAnalysis.Razor.Compiler/src/Language/CodeGeneration/CodeWriterExtensions.cs

@@ -8,8 +8,8 @@
 using System.Diagnostics;
 using System.Globalization;
 using System.Linq;
-using System.Runtime.InteropServices;
 using Microsoft.AspNetCore.Razor.Language.Intermediate;
+using Microsoft.AspNetCore.Razor.Utilities;
 
 namespace Microsoft.AspNetCore.Razor.Language.CodeGeneration;
 
@@ -661,7 +661,7 @@ public static IDisposable BuildEnhancedLinePragma(this CodeWriter writer, Source
 
     private static SourceSpan RemapFilePathIfNecessary(SourceSpan sourceSpan, CodeRenderingContext context)
     {
-        if (context.Options.RemapLinePragmaPathsOnWindows && RuntimeInformation.IsOSPlatform(OSPlatform.Windows))
+        if (context.Options.RemapLinePragmaPathsOnWindows && PlatformInformation.IsWindows)
         {
             // ISSUE: https://github.com/dotnet/razor/issues/9108
             // The razor tooling normalizes paths to be forward slash based, regardless of OS.

src/Razor/src/Microsoft.AspNetCore.Razor.LanguageServer/DefaultLanguageServerFeatureOptions.cs

@@ -1,7 +1,7 @@
 // Copyright (c) .NET Foundation. All rights reserved.
 // Licensed under the MIT license. See License.txt in the project root for license information.
 
-using System.Runtime.InteropServices;
+using Microsoft.AspNetCore.Razor.Utilities;
 using Microsoft.CodeAnalysis.Razor.Workspaces;
 
 namespace Microsoft.AspNetCore.Razor.LanguageServer;
@@ -25,8 +25,7 @@ internal class DefaultLanguageServerFeatureOptions : LanguageServerFeatureOption
 
     // Code action and rename paths in Windows VS Code need to be prefixed with '/':
     // https://github.com/dotnet/razor/issues/8131
-    public override bool ReturnCodeActionAndRenamePathsWithPrefixedSlash
-        => RuntimeInformation.IsOSPlatform(OSPlatform.Windows);
+    public override bool ReturnCodeActionAndRenamePathsWithPrefixedSlash => PlatformInformation.IsWindows;
 
     public override bool ShowAllCSharpCodeActions => false;
 

src/Razor/src/Microsoft.AspNetCore.Razor.LanguageServer/RemoteRazorProjectFileSystem.cs

@@ -4,7 +4,6 @@
 using System;
 using System.Collections.Generic;
 using System.IO;
-using System.Runtime.InteropServices;
 using Microsoft.AspNetCore.Razor.Language;
 using Microsoft.AspNetCore.Razor.Utilities;
 using Microsoft.CodeAnalysis.Razor;
@@ -82,7 +81,7 @@ protected override string NormalizeAndEnsureValidPath(string path)
 
         static bool IsPathRootedForPlatform(string path)
         {
-            if (RuntimeInformation.IsOSPlatform(OSPlatform.Windows) && path == "/")
+            if (PlatformInformation.IsWindows && path == "/")
             {
                 // We have to special case windows and "/" because for some reason Path.IsPathRooted returns true on windows for a single "/" path.
                 return false;

src/Razor/src/Microsoft.AspNetCore.Razor.ProjectEngineHost/Utilities/FilePathNormalizer.cs

@@ -12,7 +12,7 @@ namespace Microsoft.AspNetCore.Razor.Utilities;
 
 internal static class FilePathNormalizer
 {
-    private static readonly Func<char, char> s_charConverter = RuntimeInformation.IsOSPlatform(OSPlatform.Linux)
+    private static readonly Func<char, char> s_charConverter = PlatformInformation.IsLinux
         ? c => c
         : char.ToLowerInvariant;
 
@@ -223,7 +223,7 @@ private static (int start, int length) NormalizeCore(ReadOnlySpan<char> source,
         // Replace slashes in our normalized span.
         NormalizeAndDedupeSlashes(destination, ref charsWritten);
 
-        if (RuntimeInformation.IsOSPlatform(OSPlatform.Windows) &&
+        if (PlatformInformation.IsWindows &&
             charsWritten > 1 &&
             destination is ['/', ..] and not ['/', '/', ..])
         {

src/Razor/src/Microsoft.AspNetCore.Razor.ProjectEngineHost/Utilities/RazorProjectInfoFactory.cs

@@ -5,7 +5,6 @@
 using System.Collections.Immutable;
 using System.Diagnostics.CodeAnalysis;
 using System.IO;
-using System.Runtime.InteropServices;
 using System.Threading;
 using System.Threading.Tasks;
 using Microsoft.AspNetCore.Razor.Language;
@@ -34,7 +33,7 @@ internal readonly record struct ConversionResult(RazorProjectInfo? ProjectInfo,
 
     static RazorProjectInfoFactory()
     {
-        s_stringComparison = RuntimeInformation.IsOSPlatform(OSPlatform.Linux)
+        s_stringComparison = PlatformInformation.IsLinux
             ? StringComparison.Ordinal
             : StringComparison.OrdinalIgnoreCase;
     }