frozen_record 0.25.4 → 0.25.5

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 45e9d1a344670234462c22c5d74da8b8e1bf4b726b14b447d1abe646632aae26
-  data.tar.gz: 96c4daa09b3b7b46f332e082ff94f91683d5da82b378bf32398bcbc88f4c4c7c
+  metadata.gz: b8fc155c1a350f49fbc79e245cc768722c9880cd0010b7829a33aea0ac55ab69
+  data.tar.gz: a16dde142d53022fb3b76b8262326d774342f4158b7d1ec018bf0a4879d41d77
 SHA512:
-  metadata.gz: a78872e27a9341f152867b4ee1d281c4512cff485e8c07fd343abab52fd71bb461e0c98c221460e3bce95bf63b7eaff49dd66d1c9166ca4f357eccd65531dcf8
-  data.tar.gz: 3b05ceacab2a957f44818338b2b71555d65baa66dd6cf1e5ee55b21cc0e6c8eb307d4c3ea8af9e7d00bc09ea0d3ce0712e8598225e9f952c0422cb2ddad1694c
+  metadata.gz: 4a315f98d4d6c79dc0ac6146715fdee499d383fc9d5e550bbdc7278655a19da2b744923ed41f7f445fc4175f575f6ea1330e77381edfe77fa5dbd4152e6446bb
+  data.tar.gz: d9e7d61a94c6504ecb0c3fc4e298487bfa30a5d9fd7b527657fc51259246aa23030567f676d2ff06595dca1b1de7a78281bb2380b19ae23145ccd04397dd0799

data/CHANGELOG.md

@@ -1,5 +1,9 @@
 # Unreleased
 
+# v0.25.5
+
+- `FrozenRecord::Base#==` now returns `false` if the primary key is `nil` (to match the `ActiveRecord::Base` implementation)
+
 # v0.25.4
 
 - Minor Ruby 3.2 compatiblity fix (regarding `ruby2_keywords`).

data/lib/frozen_record/base.rb

@@ -252,7 +252,7 @@ module FrozenRecord
     alias_method :attribute, :[]
 
     def ==(other)
-      super || other.is_a?(self.class) && other.id == id
+      super || other.is_a?(self.class) && !id.nil? && other.id == id
     end
 
     def persisted?

data/lib/frozen_record/version.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 module FrozenRecord
-  VERSION = '0.25.4'
+  VERSION = '0.25.5'
 end

data/spec/frozen_record_spec.rb

@@ -149,6 +149,13 @@ RSpec.shared_examples 'main' do
       expect(country).to be == second_country
     end
 
+    it 'returns false if both instances are from the same class and their ids are nil' do
+      country = country_model.new(id: nil)
+      second_country = country_model.new(id: nil)
+
+      expect(country).to_not be == second_country
+    end
+
     it 'returns false if both instances are not from the same class' do
       country = country_model.first
       car = car_model.new(id: country.id)

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: frozen_record
 version: !ruby/object:Gem::Version
-  version: 0.25.4
+  version: 0.25.5
 platform: ruby
 authors:
 - Jean Boussier
 autorequire:
 bindir: bin
 cert_chain: []
-date: 2022-06-13 00:00:00.000000000 Z
+date: 2022-06-17 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: activemodel
@@ -137,7 +137,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubygems_version: 3.3.7
+rubygems_version: 3.1.2
 signing_key:
 specification_version: 4
 summary: ActiveRecord like interface to read only access and query static YAML files