From 165a03c4faf6dbcf5c2195c0b04c017807086691 Mon Sep 17 00:00:00 2001 From: Mitchell Hanberg Date: Thu, 7 Mar 2024 23:27:41 -0500 Subject: [PATCH] fix: use correct spelling of Next LS in logs --- lib/next_ls.ex | 22 +++++++++---------- lib/next_ls/logger.ex | 2 +- test/next_ls/dependency_test.exs | 8 +++---- test/next_ls/diagnostics_test.exs | 2 +- .../extensions/credo_extension_test.exs | 6 ++--- test/next_ls/workspaces_test.exs | 2 +- test/next_ls_test.exs | 2 +- test/support/utils.ex | 2 +- 8 files changed, 23 insertions(+), 23 deletions(-) diff --git a/lib/next_ls.ex b/lib/next_ls.ex index 8e7df0d1..b8eb0e6a 100644 --- a/lib/next_ls.ex +++ b/lib/next_ls.ex @@ -612,7 +612,7 @@ defmodule NextLS do e -> GenLSP.warning( lsp, - "[NextLS] Failed to run completion request: #{Exception.format_banner(:error, e, __STACKTRACE__)}" + "[Next LS] Failed to run completion request: #{Exception.format_banner(:error, e, __STACKTRACE__)}" ) {:reply, [], lsp} @@ -687,7 +687,7 @@ defmodule NextLS do end def handle_request(%{method: method}, lsp) do - GenLSP.warning(lsp, "[NextLS] Method Not Found: #{method}") + GenLSP.warning(lsp, "[Next LS] Method Not Found: #{method}") {:reply, %ErrorResponse{ @@ -698,7 +698,7 @@ defmodule NextLS do @impl true def handle_notification(%Initialized{}, lsp) do - GenLSP.log(lsp, "[NextLS] NextLS v#{version()} has initialized!") + GenLSP.log(lsp, "[Next LS] NextLS v#{version()} has initialized!") with opts when is_list(opts) <- lsp.assigns.auto_update do {:ok, _} = @@ -748,7 +748,7 @@ defmodule NextLS do }) end - GenLSP.log(lsp, "[NextLS] Booting runtimes...") + GenLSP.log(lsp, "[Next LS] Booting runtimes...") parent = self() @@ -775,7 +775,7 @@ defmodule NextLS do on_initialized: fn status -> if status == :ready do Progress.stop(lsp, token, "NextLS runtime for folder #{name} has initialized!") - GenLSP.log(lsp, "[NextLS] Runtime for folder #{name} is ready...") + GenLSP.log(lsp, "[Next LS] Runtime for folder #{name} is ready...") msg = {:runtime_ready, name, self()} @@ -789,7 +789,7 @@ defmodule NextLS do send(parent, {:runtime_failed, name, status}) - GenLSP.error(lsp, "[NextLS] Runtime for folder #{name} failed to initialize") + GenLSP.error(lsp, "[Next LS] Runtime for folder #{name} failed to initialize") end end, logger: lsp.assigns.logger @@ -870,7 +870,7 @@ defmodule NextLS do names = Enum.map(entries, fn {_, %{name: name}} -> name end) for %{name: name, uri: uri} <- added, name not in names do - GenLSP.log(lsp, "[NextLS] Adding workspace folder #{name}") + GenLSP.log(lsp, "[Next LS] Adding workspace folder #{name}") token = Progress.token() Progress.start(lsp, token, "Initializing NextLS runtime for folder #{name}...") parent = self() @@ -890,7 +890,7 @@ defmodule NextLS do on_initialized: fn status -> if status == :ready do Progress.stop(lsp, token, "NextLS runtime for folder #{name} has initialized!") - GenLSP.log(lsp, "[NextLS] Runtime for folder #{name} is ready...") + GenLSP.log(lsp, "[Next LS] Runtime for folder #{name} is ready...") msg = {:runtime_ready, name, self()} @@ -904,7 +904,7 @@ defmodule NextLS do send(parent, {:runtime_failed, name, status}) - GenLSP.error(lsp, "[NextLS] Runtime for folder #{name} failed to initialize") + GenLSP.error(lsp, "[Next LS] Runtime for folder #{name} failed to initialize") end end, logger: lsp.assigns.logger @@ -915,7 +915,7 @@ defmodule NextLS do names = Enum.map(removed, & &1.name) for {pid, %{name: name}} <- entries, name in names do - GenLSP.log(lsp, "[NextLS] Removing workspace folder #{name}") + GenLSP.log(lsp, "[Next LS] Removing workspace folder #{name}") NextLS.Runtime.stop(lsp.assigns.dynamic_supervisor, pid) end end) @@ -1102,7 +1102,7 @@ defmodule NextLS do end def handle_info(message, lsp) do - GenLSP.log(lsp, "[NextLS] Unhandled message: #{inspect(message)}") + GenLSP.log(lsp, "[Next LS] Unhandled message: #{inspect(message)}") {:noreply, lsp} end diff --git a/lib/next_ls/logger.ex b/lib/next_ls/logger.ex index dd7423f1..bba77e9c 100644 --- a/lib/next_ls/logger.ex +++ b/lib/next_ls/logger.ex @@ -23,7 +23,7 @@ defmodule NextLS.Logger do end def handle_cast({:log, type, msg}, state) do - apply(GenLSP, type, [state.lsp, String.trim("[NextLS] #{msg}")]) + apply(GenLSP, type, [state.lsp, String.trim("[Next LS] #{msg}")]) case type do :log -> Logger.debug(msg) diff --git a/test/next_ls/dependency_test.exs b/test/next_ls/dependency_test.exs index 32354ac8..be345faa 100644 --- a/test/next_ls/dependency_test.exs +++ b/test/next_ls/dependency_test.exs @@ -89,12 +89,12 @@ defmodule NextLS.DependencyTest do end) assert_notification "window/logMessage", %{ - "message" => "[NextLS] Running `mix deps.get` in directory" <> _, + "message" => "[Next LS] Running `mix deps.get` in directory" <> _, "type" => 3 } assert_notification "window/logMessage", %{ - "message" => "[NextLS] Restarting runtime" <> _, + "message" => "[Next LS] Restarting runtime" <> _, "type" => 3 } @@ -151,12 +151,12 @@ defmodule NextLS.DependencyTest do end) assert_notification "window/logMessage", %{ - "message" => "[NextLS] Running `mix deps.get` in directory" <> _, + "message" => "[Next LS] Running `mix deps.get` in directory" <> _, "type" => 3 } assert_notification "window/logMessage", %{ - "message" => "[NextLS] Restarting runtime" <> _, + "message" => "[Next LS] Restarting runtime" <> _, "type" => 3 } diff --git a/test/next_ls/diagnostics_test.exs b/test/next_ls/diagnostics_test.exs index 64900ba0..d602331b 100644 --- a/test/next_ls/diagnostics_test.exs +++ b/test/next_ls/diagnostics_test.exs @@ -58,7 +58,7 @@ defmodule NextLS.DiagnosticsTest do assert :ok == notify(client, %{method: "initialized", jsonrpc: "2.0", params: %{}}) assert_notification "window/logMessage", %{ - "message" => "[NextLS] NextLS v" <> _, + "message" => "[Next LS] NextLS v" <> _, "type" => 4 } diff --git a/test/next_ls/extensions/credo_extension_test.exs b/test/next_ls/extensions/credo_extension_test.exs index 928a3000..441a4c3f 100644 --- a/test/next_ls/extensions/credo_extension_test.exs +++ b/test/next_ls/extensions/credo_extension_test.exs @@ -75,7 +75,7 @@ defmodule NextLS.CredoExtensionTest do assert_compiled(context, "my_proj") assert_notification "window/logMessage", %{ - "message" => "[NextLS] [extension] Credo disabled", + "message" => "[Next LS] [extension] Credo disabled", "type" => 3 } end @@ -88,7 +88,7 @@ defmodule NextLS.CredoExtensionTest do assert_compiled(context, "my_proj") assert_notification "window/logMessage", %{ - "message" => "[NextLS] [extension] Credo initializing with options" <> _, + "message" => "[Next LS] [extension] Credo initializing with options" <> _, "type" => 3 } @@ -126,7 +126,7 @@ defmodule NextLS.CredoExtensionTest do assert_notification "$/progress", %{"value" => %{"kind" => "end", "message" => "Finished indexing!"}} assert_notification "window/logMessage", %{ - "message" => "[NextLS] [extension] Credo initializing with options" <> _, + "message" => "[Next LS] [extension] Credo initializing with options" <> _, "type" => 3 } diff --git a/test/next_ls/workspaces_test.exs b/test/next_ls/workspaces_test.exs index 08550678..54fcfaff 100644 --- a/test/next_ls/workspaces_test.exs +++ b/test/next_ls/workspaces_test.exs @@ -88,7 +88,7 @@ defmodule NextLS.WorkspacesTest do } }) - message = "[NextLS] The runtime for #{context.module}-proj_two has successfully shut down." + message = "[Next LS] The runtime for #{context.module}-proj_two has successfully shut down." assert_notification "window/logMessage", %{ "message" => ^message diff --git a/test/next_ls_test.exs b/test/next_ls_test.exs index b7ce8402..ffcb7cf6 100644 --- a/test/next_ls_test.exs +++ b/test/next_ls_test.exs @@ -84,7 +84,7 @@ defmodule NextLSTest do }) assert_notification "window/logMessage", %{ - "message" => "[NextLS] Method Not Found: textDocument/signatureHelp", + "message" => "[Next LS] Method Not Found: textDocument/signatureHelp", "type" => 2 } diff --git a/test/support/utils.ex b/test/support/utils.ex index d628c5b7..719aca7a 100644 --- a/test/support/utils.ex +++ b/test/support/utils.ex @@ -99,7 +99,7 @@ defmodule NextLS.Support.Utils do timeout \\ Application.get_env(:ex_unit, :assert_receive_timeout) ) do quote do - message = "[NextLS] Runtime for folder #{unquote(context).module}-#{unquote(name)} is ready..." + message = "[Next LS] Runtime for folder #{unquote(context).module}-#{unquote(name)} is ready..." assert_notification "window/logMessage", %{"message" => ^message}, unquote(timeout) end