Properly compare enum attributes in Model#== (#29)

CHANGELOG.md

@@ -2,6 +2,7 @@
 
 ## master
 
+- [PR #29](https://github.com/DmitryTsepelev/store_model/pull/29) Properly compare enum attributes in `Model#==` ([@DmitryTsepelev][])
 - [PR #26](https://github.com/DmitryTsepelev/store_model/pull/26) Add YARD docs ([@DmitryTsepelev][])
 
 ## 0.5.1 (2019-09-06)

lib/store_model/model.rb

@@ -35,7 +35,7 @@ def as_json(options = {})
     def ==(other)
       return super unless other.is_a?(self.class)
 
-      attributes.all? { |name, value| value == other.send(name) }
+      attributes.all? { |name, value| value == other.attributes[name] }
     end
 
     # Allows to call :presence validation on the association itself.

spec/store_model/model_spec.rb

@@ -59,13 +59,39 @@
     context "when two instances have same attributes" do
       let(:second_setting) { Configuration.new(color: "red") }
 
-      it { is_expected.to be_truthy }
+      it { is_expected.to be true }
     end
 
     context "when two instances have different attributes" do
-      let(:second_setting) { Configuration.new(color: "user") }
+      let(:second_setting) { Configuration.new(color: "black") }
 
-      it { is_expected.to be_falsey }
+      it { is_expected.to be false }
+    end
+
+    context "when StoreModel has enum attribute" do
+      let(:config_class) do
+        Class.new do
+          include StoreModel::Model
+
+          enum :status, in: { active: 1, archived: 0 }
+        end
+      end
+
+      let(:first_setting) { config_class.new(status: :active) }
+
+      subject { first_setting == second_setting }
+
+      context "when two instances have same attributes" do
+        let(:second_setting) { config_class.new(status: :active) }
+
+        it { is_expected.to be true }
+      end
+
+      context "when two instances have different attributes" do
+        let(:second_setting) { config_class.new(status: :archived) }
+
+        it { is_expected.to be false }
+      end
     end
   end