Merge pull request #1 from st0012/detect-binstub

Use rspec binstub when it's present

Author: st0012

lib/ruby_lsp/ruby_lsp_rspec/code_lens.rb

@@ -10,7 +10,6 @@ class CodeLens < ::RubyLsp::Listener
       include ::RubyLsp::Requests::Support::Common
 
       ResponseType = type_member { { fixed: T::Array[::RubyLsp::Interface::CodeLens] } }
-      BASE_COMMAND = T.let((File.exist?("Gemfile.lock") ? "bundle exec rspec" : "rspec"), String)
 
       sig { override.returns(ResponseType) }
       attr_reader :_response
@@ -21,6 +20,23 @@ def initialize(uri, emitter, message_queue)
         @path = T.let(uri.to_standardized_path, T.nilable(String))
         emitter.register(self, :on_command, :on_command_call, :on_call)
 
+        @base_command = T.let(
+          begin
+            cmd = if File.exist?(File.join(Dir.pwd, "bin", "rspec"))
+              "bin/rspec"
+            else
+              "rspec"
+            end
+
+            if File.exist?("Gemfile.lock")
+              "bundle exec #{cmd}"
+            else
+              cmd
+            end
+          end,
+          String,
+        )
+
         super(emitter, message_queue)
       end
 
@@ -103,7 +119,7 @@ def add_test_code_lens(node, name:, kind:)
         return unless @path
 
         line_number = node.location.start_line
-        command = "#{BASE_COMMAND} #{@path}:#{line_number}"
+        command = "#{@base_command} #{@path}:#{line_number}"
 
         arguments = [
           @path,

spec/ruby_lsp_rspec_spec.rb

@@ -105,4 +105,42 @@
     response = response.response
     expect(response.count).to eq(12)
   end
+
+  context "when there's a binstub" do
+    let(:binstub_path) { File.expand_path("../bin/rspec", __dir__) }
+
+    before do
+      File.write(binstub_path, <<~RUBY)
+        #!/usr/bin/env ruby
+        puts "binstub is called"
+      RUBY
+    end
+
+    after do
+      FileUtils.rm(binstub_path) if File.exist?(binstub_path)
+    end
+
+    it "uses the binstub" do
+      store.set(uri: uri, source: <<~RUBY, version: 1)
+        RSpec.describe(Foo::Bar) do
+        end
+      RUBY
+
+      response = RubyLsp::Executor.new(store, message_queue).execute(
+        {
+          method: "textDocument/codeLens",
+          params: {
+            textDocument: { uri: uri },
+            position: { line: 0, character: 0 },
+          },
+        },
+      )
+
+      expect(response.error).to(be_nil)
+
+      response = response.response
+      expect(response.count).to eq(3)
+      expect(response[0].command.arguments[2]).to eq("bundle exec bin/rspec /fake.rb:1")
+    end
+  end
 end