Skip to content

Commit 66923b8

Browse files
authored
Merge pull request #765 from Shopify/fix-code-lens
Skip test code lens on methods unless its inside a test class
2 parents 2a6591e + c821e0c commit 66923b8

File tree

2 files changed

+7
-1
lines changed

2 files changed

+7
-1
lines changed

lib/ruby_lsp/requests/code_lens.rb

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -80,11 +80,13 @@ def after_class(node)
8080

8181
sig { params(node: SyntaxTree::DefNode).void }
8282
def on_def(node)
83+
class_name = @class_stack.last
84+
return unless class_name
85+
8386
visibility, _ = @visibility_stack.last
8487
if visibility == "public"
8588
method_name = node.name.value
8689
if method_name.start_with?("test_")
87-
class_name = T.must(@class_stack.last)
8890
add_test_code_lens(
8991
node,
9092
name: method_name,

test/fixtures/minitest_tests.rb

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -25,3 +25,7 @@ def test_with_q?; end
2525
class AnotherTest < Minitest::Test
2626
def test_public; end
2727
end
28+
29+
class Foo
30+
def test_public; end
31+
end

0 commit comments

Comments
 (0)