wip

Author: andyw8

lib/ruby_lsp/ruby_lsp_rails/definition.rb

@@ -11,7 +11,7 @@ module Rails
     #
     # Currently supported targets:
     # - Callbacks
-    # - Routes
+    # - Named routes
     #
     # # Example
     #

lib/ruby_lsp/ruby_lsp_rails/runner_client.rb

@@ -97,9 +97,7 @@ def model(name)
 
       sig { params(name: String).returns(T.nilable(T::Hash[Symbol, T.untyped])) }
       def route_location(name)
-        $stderr.puts("Ruby LSP Rails getting route location for: #{name}")
         make_request("route_location", name: name)
-        # $stderr.puts("after")
       rescue IncompleteMessageError
         $stderr.puts("Ruby LSP Rails failed to get route location: #{@stderr.read}")
         nil
@@ -168,7 +166,7 @@ def read_response
           return
         end
 
-        $stderr.puts(response.fetch(:result))
+        $stderr.puts("***Result: " + response.fetch(:result).to_s)
         response.fetch(:result)
       end
     end

lib/ruby_lsp/ruby_lsp_rails/server.rb

@@ -106,7 +106,7 @@ def route_location(name)
 
         {
           result: {
-            location: ::Rails.root.join(route.source_location),
+            location: ::Rails.root.join(route.source_location).to_s,
           },
         }
       end

test/ruby_lsp_rails/definition_test.rb

@@ -140,6 +140,7 @@ def generate_definitions_for_source(source, position)
             method: "textDocument/definition",
             params: { textDocument: { uri: uri }, position: position },
           )
+
           server.pop_response.response
         end
       end

test/ruby_lsp_rails/server_test.rb

@@ -35,6 +35,7 @@ class ServerTest < ActiveSupport::TestCase
 
   test "route location" do
     response = @server.execute("route_location", { name: "user_path" })
-    assert_equal("config/routes.rb:5", response)
+    location = response[:result][:location]
+    assert location.ends_with?("test/dummy/config/routes.rb:5")
   end
 end