Improve test output (#347)

* wip

* Fix compile warning

* Fix spurious output in the tests

* Update changelog

* remove note on readme

Author: axelson

CHANGELOG.md

@@ -18,6 +18,7 @@ House keeping:
 - Fix the link in the README to releases (thanks [RJ Dellecese](https://github.com/rjdellecese)) [#312](https://github.com/elixir-lsp/elixir-ls/pull/312)
 - Update the dialyzer section in the readme (thanks [Serenity597](https://github.com/Serenity597)) [#323](https://github.com/elixir-lsp/elixir-ls/pull/323)
 - Add vim-lsp to plugin list (thanks [Thanabodee Charoenpiriyakij](https://github.com/wingyplus)) [#340](https://github.com/elixir-lsp/elixir-ls/pull/340)
+- Cleanup test output (thanks [Jason Axelson](https://github.com/axelson)) [#347](https://github.com/elixir-lsp/elixir-ls/pull/347)
 
 Note: `MIX_TARGET` support was added in 0.5.0 but wasn't added to the changelog until later:
 - Support `MIX_TARGET` so the language server can have target specific contexts, like with Nerves (thanks [Jon Carstens](https://github.com/jjcarstens)) [#299](https://github.com/elixir-lsp/elixir-ls/pull/299)

apps/elixir_ls_debugger/test/debugger_test.exs

@@ -208,7 +208,7 @@ defmodule ElixirLS.Debugger.ServerTest do
     test "Evaluate expression with attempt to exit debugger process", %{server: server} do
       Server.receive_packet(
         server,
-        gen_packet("Process.exit(self)")
+        gen_packet("Process.exit(self(), :normal)")
       )
 
       assert_receive(%{"body" => %{"result" => result}}, 1000)

apps/language_server/lib/language_server/dialyzer.ex

@@ -25,7 +25,7 @@ defmodule ElixirLS.LanguageServer.Dialyzer do
   # Client API
 
   def check_support do
-    _otp_release = String.to_integer(System.otp_release())
+    _ = String.to_integer(System.otp_release())
     {_compiled_with, _} = System.build_info() |> Map.fetch!(:otp_release) |> Integer.parse()
 
     cond do