Try to fix tests for Linux

Author: rkeithhill

src/PowerShellEditorServices/Workspace/Workspace.cs

@@ -640,7 +640,6 @@ private static string UnescapeDriveColon(string fileUri)
         internal static string ConvertPathToDocumentUri(string path)
         {
             const string fileUriPrefix = "file:///";
-            int colonIndex;
 
             if (path.StartsWith("untitled:", StringComparison.Ordinal))
             {
@@ -671,8 +670,8 @@ internal static string ConvertPathToDocumentUri(string path)
 
             // VSCode file URIs on Windows need the drive letter lowercase, and the colon
             // URI encoded. System.Uri won't do that, so we manually create the URI.
-            var newUri = System.Web.HttpUtility.UrlPathEncode(path);
-            colonIndex = path.IndexOf(':');
+            string newUri = System.Web.HttpUtility.UrlPathEncode(path);
+            int colonIndex = path.IndexOf(':');
             if (colonIndex > 0)
             {
                 int driveLetterIndex = colonIndex - 1;
@@ -681,10 +680,7 @@ internal static string ConvertPathToDocumentUri(string path)
                 newUri = newUri.Insert(driveLetterIndex, driveLetter + "%3A");
             }
 
-            return newUri
-                .Replace('\\', '/')
-                .Insert(0, fileUriPrefix)
-                .ToString();
+            return newUri.Replace('\\', '/').Insert(0, fileUriPrefix);
         }
 
         #endregion

test/PowerShellEditorServices.Test/Session/ScriptFileTests.cs

@@ -547,7 +547,6 @@ public void PropertiesInitializedCorrectlyForFile()
 
             Assert.Equal(path, scriptFile.FilePath);
             Assert.Equal(path, scriptFile.ClientFilePath);
-            Assert.Equal("file:///TestFile.ps1", scriptFile.DocumentUri);
             Assert.True(scriptFile.IsAnalysisEnabled);
             Assert.False(scriptFile.IsInMemory);
             Assert.Empty(scriptFile.ReferencedFiles);
@@ -590,13 +589,19 @@ public void DocumentUriRetunsCorrectStringForAbsolutePath()
             ScriptFile scriptFile;
             var emptyStringReader = new StringReader("");
 
-            path = @"C:\Users\AmosBurton\projects\Rocinate\ProtoMolecule.ps1";
-            scriptFile = new ScriptFile(path, path, emptyStringReader, PowerShellVersion);
-            Assert.Equal("file:///c%3A/Users/AmosBurton/projects/Rocinate/ProtoMolecule.ps1", scriptFile.DocumentUri);
-
-            if (Environment.OSVersion.Platform == PlatformID.Unix)
+            if (Environment.OSVersion.Platform == PlatformID.Win32NT)
+            {
+                path = @"C:\Users\AmosBurton\projects\Rocinate\ProtoMolecule.ps1";
+                scriptFile = new ScriptFile(path, path, emptyStringReader, PowerShellVersion);
+                Assert.Equal("file:///c%3A/Users/AmosBurton/projects/Rocinate/ProtoMolecule.ps1", scriptFile.DocumentUri);
+            }
+            else
             {
                 // Test the following only on Linux and macOS.
+                path = "/home/AmosBurton/projects/Rocinate/ProtoMolecule.ps1";
+                scriptFile = new ScriptFile(path, path, emptyStringReader, PowerShellVersion);
+                Assert.Equal("file:///home/AmosBurton/projects/Rocinate/ProtoMolecule.ps1", scriptFile.DocumentUri);
+
                 path = "/home/AmosBurton/projects/Rocinate/Proto:Mole:cule.ps1";
                 scriptFile = new ScriptFile(path, path, emptyStringReader, PowerShellVersion);
                 Assert.Equal("file:///home/AmosBurton/projects/Rocinate/Proto%3AMole%3Acule.ps1", scriptFile.DocumentUri);