Skip and fix broken cross-plat tests.

These tests will be reenabled as part of dotnet/aspnetcore#26612
dotnet · Oct 5, 2020 · f7718a9 · f7718a9
1 parent 975d46b
commit f7718a9
Show file tree

Hide file tree

Showing 2 changed files with 13 additions and 6 deletions.
diff --git a/...t.AspNetCore.Razor.LanguageServer.Test/MonitorProjectConfigurationFilePathEndpointTest.cs b/...t.AspNetCore.Razor.LanguageServer.Test/MonitorProjectConfigurationFilePathEndpointTest.cs
@@ -8,6 +8,7 @@
 using System.Threading.Tasks;
 using Microsoft.AspNetCore.Razor.LanguageServer.Common;
 using Microsoft.AspNetCore.Razor.Test.Common;
+using Microsoft.AspNetCore.Testing;
 using Microsoft.CodeAnalysis.Razor;
 using Moq;
 using Xunit;
@@ -150,7 +151,9 @@ public async Task Handle_DuplicateMonitors_Noops()
             Assert.Equal(0, detector.StopCount);
         }
 
-        [Fact]
+        [ConditionalFact]
+        [OSSkipCondition(OperatingSystems.Linux)]
+        [OSSkipCondition(OperatingSystems.MacOSX)]
         public async Task Handle_ChangedConfigurationOutputPath_StartsWithNewPath()
         {
             // Arrange
@@ -181,7 +184,9 @@ public async Task Handle_ChangedConfigurationOutputPath_StartsWithNewPath()
             Assert.Equal(1, detector.StopCount);
         }
 
-        [Fact]
+        [ConditionalFact]
+        [OSSkipCondition(OperatingSystems.Linux)]
+        [OSSkipCondition(OperatingSystems.MacOSX)]
         public async Task Handle_ChangedConfigurationExternalToInternal_StopsWithoutRestarting()
         {
             // Arrange
@@ -194,7 +199,7 @@ public async Task Handle_ChangedConfigurationExternalToInternal_StopsWithoutRest
                 Enumerable.Empty<IProjectConfigurationFileChangeListener>());
             var externalRequest = new MonitorProjectConfigurationFilePathParams()
             {
-                ProjectFilePath = "C:\\dir\\project.csproj",
+                ProjectFilePath = "C:\\dir\\project.csproj",z
                 ConfigurationFilePath = "C:\\externaldir\\obj\\Debug\\project.razor.json",
             };
             var internalRequest = new MonitorProjectConfigurationFilePathParams()
@@ -212,7 +217,9 @@ public async Task Handle_ChangedConfigurationExternalToInternal_StopsWithoutRest
             Assert.Equal(1, detector.StopCount);
         }
 
-        [Fact]
+        [ConditionalFact]
+        [OSSkipCondition(OperatingSystems.Linux)]
+        [OSSkipCondition(OperatingSystems.MacOSX)]
         public async Task Handle_MultipleProjects_StartedAndStopped()
         {
             // Arrange

diff --git a/...r/test/Microsoft.AspNetCore.Razor.LanguageServer.Test/RemoteRazorProjectFileSystemTest.cs b/...r/test/Microsoft.AspNetCore.Razor.LanguageServer.Test/RemoteRazorProjectFileSystemTest.cs
@@ -49,8 +49,8 @@ public void GetItem_RootedFilePath_BelongsToProject()
         public void GetItem_RootedFilePath_DoesNotBelongToProject()
         {
             // Arrange
-            var fileSystem = new RemoteRazorProjectFileSystem("C:/path/to", FilePathNormalizer);
-            var documentFilePath = "C:/otherpath/to/file.cshtml";
+            var fileSystem = new RemoteRazorProjectFileSystem("/C:/path/to", FilePathNormalizer);
+            var documentFilePath = "/C:/otherpath/to/file.cshtml";
 
             // Act
             var item = fileSystem.GetItem(documentFilePath, fileKind: null);