Fix a false positive for RSpec/DescribedClass when SkipBlocks is true and numblocks are used

Author: Earlopain

Diff for: CHANGELOG.md

@@ -2,6 +2,8 @@
 
 ## Master (Unreleased)
 
+- Fix a false positive for `RSpec/DescribedClass` when `SkipBlocks` is true and numblocks are used. ([@earlopain])
+
 ## 3.5.0 (2025-02-16)
 
 - Don't let `RSpec/PredicateMatcher` replace `respond_to?` with two arguments with the RSpec `respond_to` matcher. ([@bquorning])

Diff for: lib/rubocop/cop/rspec/described_class.rb

@@ -153,7 +153,9 @@ def scope_change?(node)
         end
 
         def skippable_block?(node)
-          node.block_type? && !rspec_block?(node) && cop_config['SkipBlocks']
+          return false unless cop_config['SkipBlocks']
+
+          node.any_block_type? && !rspec_block?(node)
         end
 
         def only_static_constants?

Diff for: spec/rubocop/cop/rspec/described_class_spec.rb

@@ -10,14 +10,35 @@
       expect_offense(<<~RUBY)
         describe MyClass do
           controller(ApplicationController) do
-            bar = MyClass
+            self.bar = MyClass
+          end
+
+          before do
+            MyClass
+            ^^^^^^^ Use `described_class` instead of `MyClass`.
+
+            Foo.custom_block do
+              MyClass
+            end
+          end
+        end
+      RUBY
+    end
+
+    it 'ignores offenses within non-rspec numblocks' do
+      expect_offense(<<~RUBY)
+        describe MyClass do
+          controller(ApplicationController) do
+            do_some_work(_1)
+            self.bar = MyClass
           end
 
           before do
             MyClass
             ^^^^^^^ Use `described_class` instead of `MyClass`.
 
             Foo.custom_block do
+              do_some_work(_1)
               MyClass
             end
           end