Skip flaky Linux tests which depend on help system

Author: andyleejordan

src/PowerShellEditorServices/Utility/VersionUtils.cs

@@ -50,17 +50,17 @@ internal static class VersionUtils
         public static bool IsPS7OrGreater { get; } = PSVersion.Major >= 7;
 
         /// <summary>
-        /// True if we are running in on Windows, false otherwise.
+        /// True if we are running on Windows, false otherwise.
         /// </summary>
         public static bool IsWindows { get; } = RuntimeInformation.IsOSPlatform(OSPlatform.Windows);
 
         /// <summary>
-        /// True if we are running in on macOS, false otherwise.
+        /// True if we are running on macOS, false otherwise.
         /// </summary>
         public static bool IsMacOS { get; } = RuntimeInformation.IsOSPlatform(OSPlatform.OSX);
 
         /// <summary>
-        /// True if we are running in on Linux, false otherwise.
+        /// True if we are running on Linux, false otherwise.
         /// </summary>
         public static bool IsLinux { get; } = RuntimeInformation.IsOSPlatform(OSPlatform.Linux);
     }

test/PowerShellEditorServices.Test.E2E/LanguageServerProtocolMessageTests.cs

@@ -26,6 +26,7 @@
 using Range = OmniSharp.Extensions.LanguageServer.Protocol.Models.Range;
 using Microsoft.PowerShell.EditorServices.Logging;
 using Microsoft.PowerShell.EditorServices.Services.Configuration;
+using Microsoft.PowerShell.EditorServices.Utility;
 
 namespace PowerShellEditorServices.Test.E2E
 {
@@ -924,9 +925,10 @@ await PsesLanguageClient
         }
 
         [Trait("Category", "LSP")]
-        [Fact]
+        [SkippableFact]
         public async Task CanSendCompletionAndCompletionResolveRequestAsync()
         {
+            Skip.If(VersionUtils.IsLinux, "This depends on the help system, which is flaky on Linux.");
             string filePath = NewTestFile("Write-H");
 
             CompletionList completionItems = await PsesLanguageClient.TextDocument.RequestCompletion(
@@ -950,9 +952,10 @@ public async Task CanSendCompletionAndCompletionResolveRequestAsync()
         }
 
         [Trait("Category", "LSP")]
-        [Fact]
+        [SkippableFact]
         public async Task CanSendCompletionResolveWithModulePrefixRequestAsync()
         {
+            Skip.If(VersionUtils.IsLinux, "This depends on the help system, which is flaky on Linux.");
             await PsesLanguageClient
                 .SendRequest<EvaluateRequestArguments>(
                     "evaluate",
@@ -985,9 +988,10 @@ await PsesLanguageClient
         }
 
         [Trait("Category", "LSP")]
-        [Fact]
+        [SkippableFact]
         public async Task CanSendHoverRequestAsync()
         {
+            Skip.If(VersionUtils.IsLinux, "This depends on the help system, which is flaky on Linux.");
             string filePath = NewTestFile("Write-Host");
 
             Hover hover = await PsesLanguageClient.TextDocument.RequestHover(