Merge pull request #764 from Shopify/vs/do_not_break_for_empty_gem_statements

Prevent breaking on empty gem statements

Author: vinistock

lib/ruby_lsp/requests/code_lens.rb

@@ -193,8 +193,10 @@ def add_test_code_lens(node, name:, command:, kind:)
 
       sig { params(node: SyntaxTree::Command).returns(T.nilable(String)) }
       def resolve_gem_remote(node)
-        gem_name = node.arguments.parts.flat_map(&:child_nodes).first.value
-        spec = Gem::Specification.stubs.find { |gem| gem.name == gem_name }&.to_spec
+        gem_statement = node.arguments.parts.flat_map(&:child_nodes).first
+        return unless gem_statement
+
+        spec = Gem::Specification.stubs.find { |gem| gem.name == gem_statement.value }&.to_spec
         return if spec.nil?
 
         [spec.homepage, spec.metadata["source_code_uri"]].compact.find do |page|

test/fixtures/Gemfile.rb

@@ -5,3 +5,6 @@
 group :development do
   gem "debug", "~> 1.7", require: false
 end
+
+# Make sure we don't break as the user is typing
+gem ""