Merge pull request #1205 from elixir-lsp/codex/replace-inspect/1-with-to_string/1

Fix version string quoting

Author: lukaszsamson

apps/elixir_ls_utils/lib/launch.ex

@@ -36,10 +36,10 @@ defmodule ElixirLS.Utils.Launch do
 
   def get_versions do
     %{
-      current_elixir_version: inspect(System.build_info()[:build]),
-      current_otp_version: inspect(System.otp_release()),
-      compile_elixir_version: inspect(@compiled_elixir_version),
-      compile_otp_version: inspect(@compiled_otp_version)
+      current_elixir_version: to_string(System.build_info()[:build]),
+      current_otp_version: to_string(System.otp_release()),
+      compile_elixir_version: to_string(@compiled_elixir_version),
+      compile_otp_version: to_string(@compiled_otp_version)
     }
   end
 

apps/elixir_ls_utils/test/launch_test.exs

@@ -0,0 +1,17 @@
+defmodule ElixirLS.Utils.LaunchTest do
+  use ExUnit.Case, async: true
+
+  test "get_versions returns plain strings" do
+    versions = ElixirLS.Utils.Launch.get_versions()
+
+    assert is_binary(versions.current_elixir_version)
+    assert is_binary(versions.current_otp_version)
+    assert is_binary(versions.compile_elixir_version)
+    assert is_binary(versions.compile_otp_version)
+
+    for value <- Map.values(versions) do
+      refute String.starts_with?(value, "\"")
+      refute String.ends_with?(value, "\"")
+    end
+  end
+end