Merge pull request #2598 from ruby-grape/remove_settings_unset_functions

Remove Settings Unset Functions

Author: dblock

CHANGELOG.md

@@ -11,6 +11,7 @@
 * [#2582](https://github.com/ruby-grape/grape/pull/2582): Fix leaky slash when normalizing - [@ericproulx](https://github.com/ericproulx).
 * [#2583](https://github.com/ruby-grape/grape/pull/2583): Optimize api parameter documentation and memory usage - [@ericproulx](https://github.com/ericproulx).
 * [#2589](https://github.com/ruby-grape/grape/pull/2589): Replace `send` by `__send__` in codebase - [@ericproulx](https://github.com/ericproulx).
+* [#2598](https://github.com/ruby-grape/grape/pull/2598): Refactor settings DSL to use explicit methods instead of dynamic generation - [@ericproulx](https://github.com/ericproulx).
 * Your contribution here.
 
 #### Fixes

lib/grape/api/instance.rb

@@ -232,22 +232,19 @@ def collect_route_config_per_pattern(all_routes)
         end
       end
 
+      ROOT_PREFIX_VERSIONING_KEY = %i[version version_options root_prefix].freeze
+      private_constant :ROOT_PREFIX_VERSIONING_KEY
+
       # Allows definition of endpoints that ignore the versioning configuration
       # used by the rest of your API.
       def without_root_prefix_and_versioning
-        old_version = self.class.namespace_inheritable(:version)
-        old_version_options = self.class.namespace_inheritable(:version_options)
-        old_root_prefix = self.class.namespace_inheritable(:root_prefix)
-
-        self.class.namespace_inheritable_to_nil(:version)
-        self.class.namespace_inheritable_to_nil(:version_options)
-        self.class.namespace_inheritable_to_nil(:root_prefix)
-
+        inheritable_setting = self.class.inheritable_setting
+        deleted_values = inheritable_setting.namespace_inheritable.delete(*ROOT_PREFIX_VERSIONING_KEY)
         yield
-
-        self.class.namespace_inheritable(:version, old_version)
-        self.class.namespace_inheritable(:version_options, old_version_options)
-        self.class.namespace_inheritable(:root_prefix, old_root_prefix)
+      ensure
+        ROOT_PREFIX_VERSIONING_KEY.each_with_index do |key, index|
+          inheritable_setting.namespace_inheritable[key] = deleted_values[index]
+        end
       end
     end
   end

lib/grape/dsl/settings.rb

@@ -26,13 +26,6 @@ def inheritable_setting
         @inheritable_setting ||= Grape::Util::InheritableSetting.new.tap { |new_settings| new_settings.inherit_from top_level_setting }
       end
 
-      # @param type [Symbol]
-      # @param key [Symbol]
-      def unset(type, key)
-        setting = inheritable_setting.__send__(type)
-        setting.delete key
-      end
-
       # @param type [Symbol]
       # @param key [Symbol]
       # @param value [Object] will be stored if the value is currently empty
@@ -56,12 +49,6 @@ def get_or_set(type, key, value)
         end
       end
 
-      def unset_namespace_stackable(*keys)
-        keys.each do |key|
-          unset :namespace_stackable, key
-        end
-      end
-
       # defines the following methods:
       # - global_setting
       # - route_setting
@@ -73,11 +60,6 @@ def unset_namespace_stackable(*keys)
         end
       end
 
-      # @param key [Symbol]
-      def namespace_inheritable_to_nil(key)
-        inheritable_setting.namespace_inheritable[key] = nil
-      end
-
       def namespace_reverse_stackable(key, value = nil)
         get_or_set :namespace_reverse_stackable, key, value
       end

lib/grape/dsl/validations.rb

@@ -22,7 +22,7 @@ module Validations
       #      # whatever
       #    end
       def reset_validations!
-        unset_namespace_stackable :declared_params, :params, :validations
+        inheritable_setting.namespace_stackable.delete(:declared_params, :params, :validations)
       end
 
       # Opens a root-level ParamsScope, defining parameter coercions and

lib/grape/util/base_inheritable.rb

@@ -14,8 +14,11 @@ def initialize(inherited_values = nil)
         @new_values = {}
       end
 
-      def delete(key)
-        new_values.delete key
+      def delete(*keys)
+        keys.map do |key|
+          # since delete returns the deleted value, seems natural to `map` the result
+          new_values.delete key
+        end
       end
 
       def initialize_copy(other)

spec/grape/dsl/settings_spec.rb

@@ -15,16 +15,6 @@ def reset_validations!; end
     end
   end
 
-  describe '#unset' do
-    it 'deletes a key from settings' do
-      subject.namespace_setting :dummy, 1
-      expect(subject.inheritable_setting.namespace.keys).to include(:dummy)
-
-      subject.unset :namespace, :dummy
-      expect(subject.inheritable_setting.namespace.keys).not_to include(:dummy)
-    end
-  end
-
   describe '#get_or_set' do
     it 'sets a values' do
       subject.get_or_set :namespace, :dummy, 1
@@ -126,13 +116,6 @@ def reset_validations!; end
     end
   end
 
-  describe '#unset_namespace_stackable' do
-    it 'delegates to unset' do
-      expect(subject).to receive(:unset).with(:namespace_stackable, :dummy)
-      subject.unset_namespace_stackable(:dummy)
-    end
-  end
-
   describe 'complex scenario' do
     it 'plays well' do
       obj1 = dummy_class.new

spec/grape/dsl/validations_spec.rb

@@ -5,17 +5,23 @@
 
   let(:dummy_class) do
     Class.new do
+      extend Grape::DSL::Settings
       extend Grape::DSL::Validations
-      def self.unset_namespace_stackable(*_keys); end
     end
   end
 
   describe '.reset_validations' do
     subject { dummy_class.reset_validations! }
 
+    before do
+      %i[declared_params params validations other].each do |key|
+        dummy_class.inheritable_setting.namespace_stackable[key] = key
+      end
+    end
+
     it 'calls unset_namespace_stackable properly' do
-      expect(dummy_class).to receive(:unset_namespace_stackable).with(:declared_params, :params, :validations)
       subject
+      expect(dummy_class.inheritable_setting.namespace_stackable.to_hash).to eq(other: [:other])
     end
   end