Skip to content

Commit

Permalink
fix: use correct spelling of Next LS in logs
Browse files Browse the repository at this point in the history
  • Loading branch information
mhanberg committed Mar 8, 2024
1 parent 92248b6 commit 165a03c
Show file tree
Hide file tree
Showing 8 changed files with 23 additions and 23 deletions.
22 changes: 11 additions & 11 deletions lib/next_ls.ex
Original file line number Diff line number Diff line change
Expand Up @@ -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}
Expand Down Expand Up @@ -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{
Expand All @@ -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, _} =
Expand Down Expand Up @@ -748,7 +748,7 @@ defmodule NextLS do
})
end

GenLSP.log(lsp, "[NextLS] Booting runtimes...")
GenLSP.log(lsp, "[Next LS] Booting runtimes...")

parent = self()

Expand All @@ -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()}

Expand All @@ -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
Expand Down Expand Up @@ -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()
Expand All @@ -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()}

Expand All @@ -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
Expand All @@ -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)
Expand Down Expand Up @@ -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

Expand Down
2 changes: 1 addition & 1 deletion lib/next_ls/logger.ex
Original file line number Diff line number Diff line change
Expand Up @@ -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)
Expand Down
8 changes: 4 additions & 4 deletions test/next_ls/dependency_test.exs
Original file line number Diff line number Diff line change
Expand Up @@ -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
}

Expand Down Expand Up @@ -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
}

Expand Down
2 changes: 1 addition & 1 deletion test/next_ls/diagnostics_test.exs
Original file line number Diff line number Diff line change
Expand Up @@ -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
}

Expand Down
6 changes: 3 additions & 3 deletions test/next_ls/extensions/credo_extension_test.exs
Original file line number Diff line number Diff line change
Expand Up @@ -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
Expand All @@ -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
}

Expand Down Expand Up @@ -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
}

Expand Down
2 changes: 1 addition & 1 deletion test/next_ls/workspaces_test.exs
Original file line number Diff line number Diff line change
Expand Up @@ -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
Expand Down
2 changes: 1 addition & 1 deletion test/next_ls_test.exs
Original file line number Diff line number Diff line change
Expand Up @@ -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
}

Expand Down
2 changes: 1 addition & 1 deletion test/support/utils.ex
Original file line number Diff line number Diff line change
Expand Up @@ -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
Expand Down

0 comments on commit 165a03c

Please sign in to comment.