Remove namespace_reverse_stackable from public DSL interface

[ericproulx]

Remove namespace_reverse_stackable from public DSL interface

Summary

This PR removes the namespace_reverse_stackable method from the public DSL interface in Grape::DSL::Settings and replaces its usage with direct access to inheritable_setting.namespace_reverse_stackable.

Changes Made

DSL Interface Changes

• Removed namespace_reverse_stackable method from Grape::DSL::Settings
• Updated Grape::DSL::Desc to use direct inheritable_setting access instead of namespace_setting
• Updated Grape::DSL::RequestResponse to use direct inheritable_setting access for rescue handlers

Test Improvements

• Simplified test setup in request_response_spec.rb by extending Grape::DSL::Settings instead of manually stubbing methods
• Updated test expectations to verify actual behavior instead of mocking internal method calls
• Removed unnecessary manual stubbing of namespace methods

Benefits

1. Cleaner API: Removes an internal implementation detail from the public DSL interface
2. Better Test Coverage: Tests now verify actual behavior rather than just method calls
3. Simplified Code: Reduces the number of wrapper methods in the DSL
4. Direct Access: Allows direct manipulation of inheritable settings when needed

Files Modified

• lib/grape/dsl/desc.rb - Use direct inheritable_setting access
• lib/grape/dsl/request_response.rb - Use direct inheritable_setting access for rescue handlers
• lib/grape/dsl/settings.rb - Remove namespace_reverse_stackable method
• spec/grape/dsl/request_response_spec.rb - Update tests to verify actual behavior

Breaking Changes

This change removes a public method from the DSL interface. However, this method was primarily used internally and direct access to inheritable_setting.namespace_reverse_stackable provides the same functionality.

Testing

All existing tests pass with the updated implementation. The rescue_from functionality continues to work as expected with the new direct access pattern.