Don't push group id to stack when encountering non-test classes (#1891)

Some tests use classes instead of modules for namespacing, such us:

```ruby
module Foo
  class Bar
    class MyTest < Minitest::Test
      def test_something
        assert true
      end
    end
  end
end
```

And when this happens, the current implementation would still push the
group id to the stack, which would cause the first test group (`MyTest`)
to already have non-nil group id. This will make the extension to think
there's another parent test group, which is not the case and will cause
no code lens to be shown.

This commit fixes this by not pushing the group id to the stack when
encountering non-test classes, and only pop the group id when the
encountered class is a test class.

Author: st0012

lib/ruby_lsp/listeners/code_lens.rb

@@ -68,17 +68,22 @@ def on_class_node_enter(node)
             command: generate_test_command(group_stack: @group_stack),
             kind: :group,
           )
-        end
 
-        @group_id_stack.push(@group_id)
-        @group_id += 1
+          @group_id_stack.push(@group_id)
+          @group_id += 1
+        end
       end
 
       sig { params(node: Prism::ClassNode).void }
       def on_class_node_leave(node)
         @visibility_stack.pop
         @group_stack.pop
-        @group_id_stack.pop
+
+        class_name = node.constant_path.slice
+
+        if @path && class_name.end_with?("Test")
+          @group_id_stack.pop
+        end
       end
 
       sig { params(node: Prism::DefNode).void }