Merge pull request #10520 from nobuyo/fix-code-length-calculator-bug

[Fix #10469] Fix code length calculation when kwargs written in single line

changelog/fix_code_length_calculation_when_kwargs_written_in_single_line.md

@@ -0,0 +1 @@
+* [#10469](https://github.com/rubocop/rubocop/issues/10469): Fix code length calculation when kwargs written in single line. ([@nobuyo][])

lib/rubocop/cop/metrics/utils/code_length_calculator.rb

@@ -31,7 +31,7 @@ def calculate
               descendant_length = code_length(descendant)
               length = length - descendant_length + 1
               # Subtract 2 length of opening and closing brace if method argument omits hash braces.
-              length -= 2 if descendant.hash_type? && !descendant.braces?
+              length -= 2 if descendant.hash_type? && !descendant.braces? && descendant.multiline?
             end
 
             length

spec/rubocop/cop/metrics/utils/code_length_calculator_spec.rb

@@ -97,6 +97,17 @@ def test
         expect(length).to eq(2)
       end
 
+      it 'counts single line correctly if asked folding' do
+        source = parse_source(<<~RUBY)
+          def test
+            foo(foo: :bar, baz: :quux)
+          end
+        RUBY
+
+        length = described_class.new(source.ast, source, foldable_types: %i[hash]).calculate
+        expect(length).to eq(1)
+      end
+
       it 'folds hashes as method kwargs if asked' do
         source = parse_source(<<~RUBY)
           def test