Refactor sibling tests into shared examples to reduce duplication

Co-authored-by: kwerle <23320+kwerle@users.noreply.github.com>

Author: Copilot

spec/lib/ruby_language_server/scope_parser_spec.rb

@@ -168,67 +168,55 @@ def some_method
       end
     end
 
-    describe 'sibling modules' do
-      let(:sibling_modules_source) do
-        <<-RUBY
-        module Foo
-          module Bar
-          end
-          module Baz
-          end
+    describe 'siblings' do
+      shared_examples 'sibling scopes' do |type|
+        it "should place sibling #{type}s at the same level" do
+          foo = scope_parser.root_scope.children.first
+          assert_equal('Foo', foo.name)
+
+          # Both Bar and Baz should be children of Foo, not Bar containing Baz
+          children = foo.children
+          assert_equal(2, children.size, "Foo should have 2 children, but has #{children.size}")
+
+          bar = children.detect { |c| c.name == 'Bar' }
+          baz = children.detect { |c| c.name == 'Baz' }
+
+          assert_not_nil(bar, "Bar should be a child of Foo")
+          assert_not_nil(baz, "Baz should be a child of Foo")
+
+          # Verify Baz is not a child of Bar
+          assert_equal(0, bar.children.size, "Bar should have no children, but has #{bar.children.size}")
         end
-        RUBY
-      end
-      let(:scope_parser) { RubyLanguageServer::ScopeParser.new(sibling_modules_source) }
-
-      it 'should place sibling modules at the same level' do
-        foo = scope_parser.root_scope.children.first
-        assert_equal('Foo', foo.name)
-        
-        # Both Bar and Baz should be children of Foo, not Bar containing Baz
-        children = foo.children
-        assert_equal(2, children.size, "Foo should have 2 children, but has #{children.size}")
-        
-        bar = children.detect { |c| c.name == 'Bar' }
-        baz = children.detect { |c| c.name == 'Baz' }
-        
-        assert_not_nil(bar, "Bar should be a child of Foo")
-        assert_not_nil(baz, "Baz should be a child of Foo")
-        
-        # Verify Baz is not a child of Bar
-        assert_equal(0, bar.children.size, "Bar should have no children, but has #{bar.children.size}")
       end
-    end
 
-    describe 'sibling classes' do
-      let(:sibling_classes_source) do
-        <<-RUBY
-        module Foo
-          class Bar
+      describe 'modules' do
+        let(:scope_parser) do
+          RubyLanguageServer::ScopeParser.new(<<-RUBY)
+          module Foo
+            module Bar
+            end
+            module Baz
+            end
           end
-          class Baz
+          RUBY
+        end
+
+        include_examples 'sibling scopes', 'module'
+      end
+
+      describe 'classes' do
+        let(:scope_parser) do
+          RubyLanguageServer::ScopeParser.new(<<-RUBY)
+          module Foo
+            class Bar
+            end
+            class Baz
+            end
           end
+          RUBY
         end
-        RUBY
-      end
-      let(:scope_parser) { RubyLanguageServer::ScopeParser.new(sibling_classes_source) }
-
-      it 'should place sibling classes at the same level' do
-        foo = scope_parser.root_scope.children.first
-        assert_equal('Foo', foo.name)
-        
-        # Both Bar and Baz should be children of Foo, not Bar containing Baz
-        children = foo.children
-        assert_equal(2, children.size, "Foo should have 2 children, but has #{children.size}")
-        
-        bar = children.detect { |c| c.name == 'Bar' }
-        baz = children.detect { |c| c.name == 'Baz' }
-        
-        assert_not_nil(bar, "Bar should be a child of Foo")
-        assert_not_nil(baz, "Baz should be a child of Foo")
-        
-        # Verify Baz is not a child of Bar
-        assert_equal(0, bar.children.size, "Bar should have no children, but has #{bar.children.size}")
+
+        include_examples 'sibling scopes', 'class'
       end
     end
   end