Address review: trailing-edge coverage for references/implementation/hover

Per review of #1270, the trailing-edge regression coverage previously only
exercised the definition locator. Add equivalent "cursor at end of symbol / end
of alias before `.`" cases for the References and Implementation locators and the
Hover provider, which share the same SurroundContext.Toxic resolution, so those
paths can't regress independently.

Also bump elixir_sense to 9460c606 (review fixes: lazy classification, operator
contexts preserved, qualified-alias override).

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01XRd15BX6wBpZ8ThaNSo232

Author: lukaszsamson

apps/language_server/test/providers/hover_test.exs

@@ -56,6 +56,32 @@ defmodule ElixirLS.LanguageServer.Providers.HoverTest do
            )
   end
 
+  # elixir-lsp/elixir-ls#1027: at the end of the alias `IO` (on the `.` before `inspect`) the module
+  # is hovered, not the remote function - the same trailing-edge resolution the definition locator uses.
+  test "elixir module hover at the end of an alias before `.` (#1027)" do
+    text = """
+    defmodule MyModule do
+      def hello() do
+        IO.inspect("hello world")
+      end
+    end
+    """
+
+    # `IO` is on line 3, columns 5-6; the caret sits just past it, on the `.` (0-based char 6).
+    {line, char} = {2, 6}
+    parser_context = ParserContextBuilder.from_string(text)
+
+    {line, char} =
+      SourceFile.lsp_position_to_elixir(parser_context.source_file.text, {line, char})
+
+    assert {:ok,
+            %GenLSP.Structures.Hover{
+              contents: %GenLSP.Structures.MarkupContent{kind: "markdown", value: v}
+            }} = Hover.hover(parser_context, line, char)
+
+    assert String.starts_with?(v, "```elixir\nIO\n```\n\n*module*")
+  end
+
   test "function hover" do
     text = """
     defmodule MyModule do

apps/language_server/test/providers/implementation/locator_test.exs

@@ -80,6 +80,22 @@ defmodule ElixirLS.LanguageServer.Providers.Implementation.LocatorTest do
              "ElixirSenseExample.ExampleBehaviourWithDocCallbackNoImpl"
   end
 
+  # elixir-lsp/elixir-ls#1038: implementations resolve with the cursor at the end of the behaviour
+  # alias (one column past its last character), not only when it is on a character of the name.
+  test "find implementations of behaviour module with cursor at the end of the alias (#1038)" do
+    buffer = """
+    defmodule ElixirSenseExample.ExampleBehaviourWithDoc do
+    end
+    """
+
+    # the alias starts at column 11 and is 42 characters long, so column 53 is its end (on the space).
+    mid = Locator.implementations(buffer, 1, 32)
+    at_end = Locator.implementations(buffer, 1, 53)
+
+    assert length(at_end) == 2
+    assert at_end == mid
+  end
+
   test "find protocol implementations" do
     buffer = """
     defprotocol ElixirSenseExample.ExampleProtocol do

apps/language_server/test/providers/references/locator_test.exs

@@ -158,6 +158,27 @@ defmodule ElixirLS.LanguageServer.Providers.References.LocatorTest do
              %{start: %{line: 65, column: 79}, end: %{line: 65, column: 83}}
   end
 
+  # elixir-lsp/elixir-ls#1038: references resolve with the cursor at the end of the symbol
+  # (one column past its last character), not only when it is on a character of the name.
+  test "find references with cursor at the end of a function definition name (#1038)", %{
+    trace: trace
+  } do
+    buffer = """
+    defmodule ElixirSense.Providers.ReferencesTest.Modules.Callee1 do
+      def func() do
+        IO.puts ""
+      end
+    end
+    """
+
+    # `func` is on line 2 at columns 7-10; column 11 is the end of the symbol (on the `(`).
+    mid = Locator.references(buffer, 2, 10, trace)
+    at_end = Locator.references(buffer, 2, 11, trace)
+
+    assert at_end != []
+    assert at_end == mid
+  end
+
   test "find references with cursor over a function definition with default arg", %{trace: trace} do
     buffer = """
     defmodule ElixirSenseExample.Subscription do

dep_versions.exs

@@ -1,5 +1,5 @@
 [
-  elixir_sense: "ee88a26f471cfc46a898d5a2c4c30bb7ad8beff4",
+  elixir_sense: "9460c60651c2419d81fa447d70d3e1596e30dab0",
   dialyxir_vendored: "accfec9393079abc4a82b7e79a4997f59f085b67",
   jason_v: "f1c10fa9c445cb9f300266122ef18671054b2330",
   erl2ex_vendored: "04f93e55f46d35d0aa3c149616f2c7a6a1ad9311",

mix.lock

@@ -2,7 +2,7 @@
   "benchee": {:hex, :benchee, "1.1.0", "f3a43817209a92a1fade36ef36b86e1052627fd8934a8b937ac9ab3a76c43062", [:mix], [{:deep_merge, "~> 1.0", [hex: :deep_merge, repo: "hexpm", optional: false]}, {:statistex, "~> 1.0", [hex: :statistex, repo: "hexpm", optional: false]}], "hexpm", "7da57d545003165a012b587077f6ba90b89210fd88074ce3c60ce239eb5e6d93"},
   "deep_merge": {:hex, :deep_merge, "1.0.0", "b4aa1a0d1acac393bdf38b2291af38cb1d4a52806cf7a4906f718e1feb5ee961", [:mix], [], "hexpm", "ce708e5f094b9cd4e8f2be4f00d2f4250c4095be93f8cd6d018c753894885430"},
   "dialyxir_vendored": {:git, "https://github.com/elixir-lsp/dialyxir.git", "accfec9393079abc4a82b7e79a4997f59f085b67", [ref: "accfec9393079abc4a82b7e79a4997f59f085b67"]},
-  "elixir_sense": {:git, "https://github.com/elixir-lsp/elixir_sense.git", "ee88a26f471cfc46a898d5a2c4c30bb7ad8beff4", [ref: "ee88a26f471cfc46a898d5a2c4c30bb7ad8beff4"]},
+  "elixir_sense": {:git, "https://github.com/elixir-lsp/elixir_sense.git", "9460c60651c2419d81fa447d70d3e1596e30dab0", [ref: "9460c60651c2419d81fa447d70d3e1596e30dab0"]},
   "erl2ex_vendored": {:git, "https://github.com/elixir-lsp/erl2ex.git", "04f93e55f46d35d0aa3c149616f2c7a6a1ad9311", [ref: "04f93e55f46d35d0aa3c149616f2c7a6a1ad9311"]},
   "erlex_vendored": {:git, "https://github.com/elixir-lsp/erlex.git", "50b8307f90451a5d0288fb239fb6405b5ca1f1a4", [ref: "50b8307f90451a5d0288fb239fb6405b5ca1f1a4"]},
   "jason_v": {:git, "https://github.com/elixir-lsp/jason.git", "f1c10fa9c445cb9f300266122ef18671054b2330", [ref: "f1c10fa9c445cb9f300266122ef18671054b2330"]},