ruby-lsp 0.23.11 → 0.23.17

data/lib/ruby_lsp/listeners/code_lens.rb

@@ -6,45 +6,32 @@ require "shellwords"
 module RubyLsp
   module Listeners
     class CodeLens
-      extend T::Sig
       include Requests::Support::Common
 
-      BASE_COMMAND = T.let(
-        begin
-          Bundler.with_original_env { Bundler.default_lockfile }
-          "bundle exec ruby"
-        rescue Bundler::GemfileNotFound
-          "ruby"
-        end,
-        String,
-      )
-      ACCESS_MODIFIERS = T.let([:public, :private, :protected], T::Array[Symbol])
-      SUPPORTED_TEST_LIBRARIES = T.let(["minitest", "test-unit"], T::Array[String])
-      DESCRIBE_KEYWORD = T.let(:describe, Symbol)
-      IT_KEYWORD = T.let(:it, Symbol)
-      DYNAMIC_REFERENCE_MARKER = T.let("<dynamic_reference>", String)
-
-      sig do
-        params(
-          response_builder: ResponseBuilders::CollectionResponseBuilder[Interface::CodeLens],
-          global_state: GlobalState,
-          uri: URI::Generic,
-          dispatcher: Prism::Dispatcher,
-        ).void
-      end
+      BASE_COMMAND = begin
+        Bundler.with_original_env { Bundler.default_lockfile }
+        "bundle exec ruby"
+      rescue Bundler::GemfileNotFound
+        "ruby"
+      end #: String
+      ACCESS_MODIFIERS = [:public, :private, :protected] #: Array[Symbol]
+      SUPPORTED_TEST_LIBRARIES = ["minitest", "test-unit"] #: Array[String]
+      DYNAMIC_REFERENCE_MARKER = "<dynamic_reference>" #: String
+
+      #: (ResponseBuilders::CollectionResponseBuilder[Interface::CodeLens] response_builder, GlobalState global_state, URI::Generic uri, Prism::Dispatcher dispatcher) -> void
       def initialize(response_builder, global_state, uri, dispatcher)
         @response_builder = response_builder
         @global_state = global_state
-        @uri = T.let(uri, URI::Generic)
-        @path = T.let(uri.to_standardized_path, T.nilable(String))
+        @uri = uri #: URI::Generic
+        @path = uri.to_standardized_path #: String?
         # visibility_stack is a stack of [current_visibility, previous_visibility]
-        @visibility_stack = T.let([[:public, :public]], T::Array[T::Array[T.nilable(Symbol)]])
-        @group_stack = T.let([], T::Array[String])
-        @group_id = T.let(1, Integer)
-        @group_id_stack = T.let([], T::Array[Integer])
+        @visibility_stack = [[:public, :public]] #: Array[Array[Symbol?]]
+        @group_stack = [] #: Array[String]
+        @group_id = 1 #: Integer
+        @group_id_stack = [] #: Array[Integer]
         # We want to avoid adding code lenses for nested definitions
-        @def_depth = T.let(0, Integer)
-        @spec_id = T.let(0, Integer)
+        @def_depth = 0 #: Integer
+        @spec_id = 0 #: Integer
 
         dispatcher.register(
           self,
@@ -59,7 +46,7 @@ module RubyLsp
         )
       end
 
-      sig { params(node: Prism::ClassNode).void }
+      #: (Prism::ClassNode node) -> void
       def on_class_node_enter(node)
         @visibility_stack.push([:public, :public])
         class_name = node.constant_path.slice
@@ -79,7 +66,7 @@ module RubyLsp
         end
       end
 
-      sig { params(node: Prism::ClassNode).void }
+      #: (Prism::ClassNode node) -> void
       def on_class_node_leave(node)
         @visibility_stack.pop
         @group_stack.pop
@@ -91,7 +78,7 @@ module RubyLsp
         end
       end
 
-      sig { params(node: Prism::DefNode).void }
+      #: (Prism::DefNode node) -> void
       def on_def_node_enter(node)
         @def_depth += 1
         return if @def_depth > 1
@@ -114,12 +101,12 @@ module RubyLsp
         end
       end
 
-      sig { params(node: Prism::DefNode).void }
+      #: (Prism::DefNode node) -> void
       def on_def_node_leave(node)
         @def_depth -= 1
       end
 
-      sig { params(node: Prism::ModuleNode).void }
+      #: (Prism::ModuleNode node) -> void
       def on_module_node_enter(node)
         if (path = namespace_constant_name(node))
           @group_stack.push(path)
@@ -128,12 +115,12 @@ module RubyLsp
         end
       end
 
-      sig { params(node: Prism::ModuleNode).void }
+      #: (Prism::ModuleNode node) -> void
       def on_module_node_leave(node)
         @group_stack.pop
       end
 
-      sig { params(node: Prism::CallNode).void }
+      #: (Prism::CallNode node) -> void
       def on_call_node_enter(node)
         name = node.name
         arguments = node.arguments
@@ -151,23 +138,21 @@ module RubyLsp
           return
         end
 
-        if [DESCRIBE_KEYWORD, IT_KEYWORD].include?(name)
-          case name
-          when DESCRIBE_KEYWORD
-            add_spec_code_lens(node, kind: :group)
-            @group_id_stack.push(@group_id)
-            @group_id += 1
-          when IT_KEYWORD
-            add_spec_code_lens(node, kind: :example)
-          end
+        case name
+        when :describe
+          add_spec_code_lens(node, kind: :group)
+          @group_id_stack.push(@group_id)
+          @group_id += 1
+        when :it, :specify # `specify` is an alias for `it`
+          add_spec_code_lens(node, kind: :example)
         end
       end
 
-      sig { params(node: Prism::CallNode).void }
+      #: (Prism::CallNode node) -> void
       def on_call_node_leave(node)
         _, prev_visibility = @visibility_stack.pop
         @visibility_stack.push([prev_visibility, prev_visibility])
-        if node.name == DESCRIBE_KEYWORD
+        if node.name == :describe
           @group_id_stack.pop
           @group_stack.pop
         end
@@ -175,7 +160,7 @@ module RubyLsp
 
       private
 
-      sig { params(node: Prism::Node, name: String, command: String, kind: Symbol, id: String).void }
+      #: (Prism::Node node, name: String, command: String, kind: Symbol, ?id: String) -> void
       def add_test_code_lens(node, name:, command:, kind:, id: name)
         # don't add code lenses if the test library is not supported or unknown
         return unless SUPPORTED_TEST_LIBRARIES.include?(@global_state.test_library) && @path
@@ -221,15 +206,9 @@ module RubyLsp
         )
       end
 
-      sig do
-        params(
-          group_stack: T::Array[String],
-          spec_name: T.nilable(String),
-          method_name: T.nilable(String),
-        ).returns(String)
-      end
+      #: (?group_stack: Array[String], ?spec_name: String?, ?method_name: String?) -> String
       def generate_test_command(group_stack: [], spec_name: nil, method_name: nil)
-        path = T.must(@path)
+        path = @path #: as !nil
         command = BASE_COMMAND
         command += " -Itest" if File.fnmatch?("**/test/**/*", path, File::FNM_PATHNAME)
         command += " -Ispec" if File.fnmatch?("**/spec/**/*", path, File::FNM_PATHNAME)
@@ -237,44 +216,58 @@ module RubyLsp
 
         case @global_state.test_library
         when "minitest"
-          last_dynamic_reference_index = group_stack.rindex(DYNAMIC_REFERENCE_MARKER)
-          command += if last_dynamic_reference_index
-            # In cases where the test path looks like `foo::Bar`
-            # the best we can do is match everything to the right of it.
-            # Tests are classes, dynamic references are only a thing for modules,
-            # so there must be something to the left of the available path.
-            group_stack = T.must(group_stack[last_dynamic_reference_index + 1..])
-            if method_name
-              " --name " + "/::#{Shellwords.escape(group_stack.join("::")) + "#" + Shellwords.escape(method_name)}$/"
-            else
-              # When clicking on a CodeLens for `Test`, `(#|::)` will match all tests
-              # that are registered on the class itself (matches after `#`) and all tests
-              # that are nested inside of that class in other modules/classes (matches after `::`)
-              " --name " + "\"/::#{Shellwords.escape(group_stack.join("::"))}(#|::)/\""
-            end
-          elsif method_name
-            # We know the entire path, do an exact match
-            " --name " + Shellwords.escape(group_stack.join("::")) + "#" + Shellwords.escape(method_name)
-          elsif spec_name
-            " --name " + "\"/^#{Shellwords.escape(group_stack.join("::"))}##{Shellwords.escape(spec_name)}$/\""
-          else
-            # Execute all tests of the selected class and tests in
-            # modules/classes nested inside of that class
-            " --name " + "\"/^#{Shellwords.escape(group_stack.join("::"))}(#|::)/\""
-          end
+          command += generate_minitest_command(group_stack, method_name, spec_name)
         when "test-unit"
-          group_name = T.must(group_stack.last)
-          command += " --testcase " + "/#{Shellwords.escape(group_name)}/"
+          command += generate_test_unit_command(group_stack, method_name)
+        end
+
+        command
+      end
+
+      #: (Array[String] group_stack, String? method_name, String? spec_name) -> String
+      def generate_minitest_command(group_stack, method_name, spec_name)
+        last_dynamic_reference_index = group_stack.rindex(DYNAMIC_REFERENCE_MARKER)
+
+        if last_dynamic_reference_index
+          # In cases where the test path looks like `foo::Bar`
+          # the best we can do is match everything to the right of it.
+          # Tests are classes, dynamic references are only a thing for modules,
+          # so there must be something to the left of the available path.
+          dynamic_stack = group_stack[last_dynamic_reference_index + 1..] #: as !nil
 
           if method_name
-            command += " --name " + Shellwords.escape(method_name)
+            " --name " + "/::#{Shellwords.escape(dynamic_stack.join("::")) + "#" + Shellwords.escape(method_name)}$/"
+          else
+            # When clicking on a CodeLens for `Test`, `(#|::)` will match all tests
+            # that are registered on the class itself (matches after `#`) and all tests
+            # that are nested inside of that class in other modules/classes (matches after `::`)
+            " --name " + "\"/::#{Shellwords.escape(dynamic_stack.join("::"))}(#|::)/\""
           end
+        elsif method_name
+          # We know the entire path, do an exact match
+          " --name " + Shellwords.escape(group_stack.join("::")) + "#" + Shellwords.escape(method_name)
+        elsif spec_name
+          " --name " + "\"/^#{Shellwords.escape(group_stack.join("::"))}##{Shellwords.escape(spec_name)}$/\""
+        else
+          # Execute all tests of the selected class and tests in
+          # modules/classes nested inside of that class
+          " --name " + "\"/^#{Shellwords.escape(group_stack.join("::"))}(#|::)/\""
+        end
+      end
+
+      #: (Array[String] group_stack, String? method_name) -> String
+      def generate_test_unit_command(group_stack, method_name)
+        group_name = group_stack.last #: as !nil
+        command = " --testcase " + "/#{Shellwords.escape(group_name)}/"
+
+        if method_name
+          command += " --name " + Shellwords.escape(method_name)
         end
 
         command
       end
 
-      sig { params(node: Prism::CallNode, kind: Symbol).void }
+      #: (Prism::CallNode node, kind: Symbol) -> void
       def add_spec_code_lens(node, kind:)
         arguments = node.arguments
         return unless arguments
@@ -312,7 +305,7 @@ module RubyLsp
         end
       end
 
-      sig { params(group_stack: T::Array[String], method_name: T.nilable(String)).returns(String) }
+      #: (group_stack: Array[String], ?method_name: String?) -> String
       def generate_fully_qualified_id(group_stack:, method_name: nil)
         if method_name
           # For tests, this will be the test class and method name: `Foo::BarTest#test_baz`.