another attempt at fixing tests

Author: lukaszsamson

apps/language_server/lib/language_server/json_rpc_logger_backend.ex

@@ -107,8 +107,6 @@ defmodule Logger.Backends.JsonRpc do
     format = Logger.Formatter.compile(Keyword.get(config, :format))
     metadata = Keyword.get(config, :metadata, []) |> configure_metadata()
 
-    Code.ensure_loaded!(ElixirLS.Utils.WireProtocol)
-
     %{
       state
       | format: format,
@@ -127,8 +125,11 @@ defmodule Logger.Backends.JsonRpc do
   end
 
   defp log_event(level, msg, ts, md, state) do
-    output = format_event(level, msg, ts, md, state) |> IO.chardata_to_string()
-    ElixirLS.LanguageServer.JsonRpc.log_message(elixir_log_level_to_lsp(level), output)
+    if function_exported?(ElixirLS.Utils.WireProtocol, :send, 1) do
+      output = format_event(level, msg, ts, md, state) |> IO.chardata_to_string()
+      ElixirLS.LanguageServer.JsonRpc.log_message(elixir_log_level_to_lsp(level), output)
+    end
+
     state
   end
 

apps/language_server/test/source_file/path_test.exs

@@ -145,18 +145,20 @@ defmodule ElixirLS.LanguageServer.SourceFile.PathTest do
     test "windows path" do
       if is_windows() do
         drive_letter = Path.expand("/") |> String.split(":") |> hd()
-        assert "file:///#{drive_letter}%3A/win/path" == to_uri("c:/win/path")
-        assert "file:///#{drive_letter}%3A/win/path" == to_uri("C:/win/path")
-        assert "file:///#{drive_letter}%3A/win/path" == to_uri("c:/win/path/")
+        assert "file:///c%3A/win/path" == to_uri("c:/win/path")
+        assert "file:///c%3A/win/path" == to_uri("C:/win/path")
+        assert "file:///c%3A/win/path" == to_uri("c:/win/path/")
+
+        # this path may actually expand to other drive letter than C: (on GHA runner it expands to D:)
         assert "file:///#{drive_letter}%3A/win/path" == to_uri("/c:/win/path")
 
-        assert "file:///#{drive_letter}%3A/win/path" == to_uri("c:\\win\\path")
-        assert "file:///#{drive_letter}%3A/win/path" == to_uri("c:\\win/path")
+        assert "file:///c%3A/win/path" == to_uri("c:\\win\\path")
+        assert "file:///c%3A/win/path" == to_uri("c:\\win/path")
 
-        assert "file:///#{drive_letter}%3A/test%20with%20%25/path" ==
+        assert "file:///c%3A/test%20with%20%25/path" ==
                  to_uri("c:\\test with %\\path")
 
-        assert "file:///#{drive_letter}%3A/test%20with%20%2525/c%23code" ==
+        assert "file:///c%3A/test%20with%20%2525/c%23code" ==
                  to_uri("c:\\test with %25\\c#code")
       end
     end