Make sure configurations work as sets

When they share the same name and the same values, they must be the
same.

Author: mamhoff

lib/alchemy/configuration.rb

@@ -70,6 +70,10 @@ def defined_configurations = []
 
       def defined_options = []
 
+      def defined_values
+        defined_options + defined_configurations
+      end
+
       def configuration(name, configuration_class)
         # The defined configurations on a class are all those defined directly on
         # that class as well as those defined on ancestors.
@@ -123,5 +127,18 @@ def option(name, type, default: nil, **args)
         end
       end
     end
+
+    def hash
+      self.class.defined_values.map do |ivar|
+        [ivar, send(ivar).hash]
+      end.hash
+    end
+
+    def ==(other)
+      self.class == other.class && self.class.defined_values.all? do |var|
+        send(var) == other.send(var)
+      end
+    end
+    alias_method :eql?, :==
   end
 end

lib/alchemy/configuration/base_option.rb

@@ -22,9 +22,10 @@ def allowed_classes
         [self.class.value_class]
       end
 
-      def eql?(other)
+      def ==(other)
         self.class == other.class && value == other.value
       end
+      alias_method :eql?, :==
 
       def hash
         [self.class, value].hash

spec/libraries/alchemy/configuration/configuration_option_spec.rb

@@ -40,4 +40,19 @@
       end
     end
   end
+
+  describe "Using within a set" do
+    let(:value) { {show_root: true, show_flag: false} }
+    let(:option_2) { described_class.new(value: value, name: :my_config, config_class: Alchemy::Configurations::Sitemap) }
+
+    it "should have the same hash value" do
+      expect(option.hash).to eq(option_2.hash)
+    end
+
+    it "should be able to add the option to a set" do
+      expect(option).to eql(option_2)
+      expect(Set.new([option, option_2])).to include(option)
+      expect(Set.new([option, option_2]).length).to eq(1)
+    end
+  end
 end