Fixes #2347 - Correct full path building for lateral scopes (#2469)

* add spec for renamed parameter in given block

* Fix the full_path for the lateral scope

* Update CHANGELOG.md

---------

Co-authored-by: Boris Drovnin <Boris.Drovnin@bia-tech.ru>

Author: numbata

.rubocop.yml

@@ -40,7 +40,7 @@ Metrics/BlockLength:
     - spec/**/*_spec.rb
 
 Metrics/ClassLength:
-  Max: 300
+  Max: 305
 
 Metrics/CyclomaticComplexity:
   Max: 15

CHANGELOG.md

@@ -8,6 +8,7 @@
 
 * [#2467](https://github.com/ruby-grape/grape/pull/2467): Fix repo coverage - [@ericproulx](https://github.com/ericproulx).
 * [#2468](https://github.com/ruby-grape/grape/pull/2468): Align `error!` method signatures across different places - [@numbata](https://github.com/numbata).
+* [#2469](https://github.com/ruby-grape/grape/pull/2469): Fix full path building for lateral scopes - [@numbata](https://github.com/numbata).
 * Your contribution here.
 
 ### 2.1.2 (2024-06-28)

lib/grape/validations/params_scope.rb

@@ -190,7 +190,13 @@ def push_declared_params(attrs, **opts)
       #
       # @return [Array<Symbol>] the nesting/path of the current parameter scope
       def full_path
-        nested? ? @parent.full_path + [@element] : []
+        if nested?
+          (@parent.full_path + [@element])
+        elsif lateral?
+          @parent.full_path
+        else
+          []
+        end
       end
 
       private

spec/grape/endpoint/declared_spec.rb

@@ -829,5 +829,31 @@
         expect(JSON.parse(last_response.body)).to match({})
       end
     end
+
+    context 'with a renamed field inside `given` block nested in hashes' do
+      before do
+        subject.format :json
+        subject.params do
+          requires :a, type: Hash do
+            optional :c, type: String
+            given :c do
+              requires :b, type: Hash do
+                requires :input_field, as: :output_field
+              end
+            end
+          end
+        end
+        subject.post '/test' do
+          declared(params)
+        end
+      end
+
+      it 'renames parameter input_field to output_field' do
+        post '/test', { a: { b: { input_field: 'value' }, c: 'value2' } }
+
+        expect(JSON.parse(last_response.body)).to \
+          match('a' => { 'b' => { 'output_field' => 'value' }, 'c' => 'value2' })
+      end
+    end
   end
 end