frozen_record 0.25.3 → 0.26.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 55cfe5dc540e6f5aa6d4a188a45297e5daae39c67b6f79a029ed12a1333d2a17
-  data.tar.gz: 68ec4931e6cd4c24dc95da423b4d32f61f789ebb38e8012ad16f12db9a0d389b
+  metadata.gz: 558032391d508fe0e51074f70abb81395189376a610c694cf6b43a64bbf2b91a
+  data.tar.gz: e8bd4b3bb7e2dafb6d3854176b7c98e6f87a3148790dd5d5f0de0ec55b95ef48
 SHA512:
-  metadata.gz: a3bbf2f250846efd3679e39a7aea8628bfdea01c84a31cab342d5348e4a41bff3655a1f4e4158b461e2084866113c0a110b676664087ccb1b735c5d66e330cd3
-  data.tar.gz: e3bb79694ba6bb4b430cc378bad2c787a41bf71ec2834eb218d10243e0cb24d6573ef19661a454c613ede947dd33eaccc75a61395259533e2aaedb6a3089f565
+  metadata.gz: 725c9abc9be65de3c1db5643e0df340c34b9b72f50e4a650d0f0a51a27160f6f7db6bcdbb33cab77d7fac3d9e7c6558fe22fd04081b43b8da4c6f7ba2887a3e4
+  data.tar.gz: a1f7e30104161efb60b5a6fbb89c1270f6656557c4f8bfb549a18ec012fdc87be73223e2d13d1efc73d18c3c8e2a0c905f9b9d0bb30907dcd465b659801f41fe

data/.github/dependabot.yml

@@ -0,0 +1,6 @@
+version: 2
+updates:
+  - package-ecosystem: "github-actions"
+    directory: "/"
+    schedule:
+      interval: "weekly"

data/.github/workflows/main.yml

@@ -12,7 +12,7 @@ jobs:
         minimal: [ false, true ]
     name: Ruby ${{ matrix.ruby }} tests, minimal=${{ matrix.minimal }}
     steps:
-      - uses: actions/checkout@v2
+      - uses: actions/checkout@v3
       - name: Set up Ruby
         uses: ruby/setup-ruby@v1
         with:

data/CHANGELOG.md

@@ -1,5 +1,19 @@
 # Unreleased
 
+# v0.26.0
+
+- Drop dependency on `dedup` gem.
+
+# 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`).
+
+# v0.25.3
+
 - Also disable max_records_scan when loading `Compact` records.
 
 # v0.25.2

data/README.md

@@ -3,7 +3,7 @@
 [![Build Status](https://secure.travis-ci.org/byroot/frozen_record.svg)](http://travis-ci.org/byroot/frozen_record)
 [![Gem Version](https://badge.fury.io/rb/frozen_record.svg)](http://badge.fury.io/rb/frozen_record)
 
-Activec Record-like interface for **read only** access to static data files of reasonable size.
+Active Record-like interface for **read only** access to static data files of reasonable size.
 
 ## Installation
 
@@ -156,7 +156,7 @@ The primary key isn't indexed by default.
 
 ## Limitations
 
-Frozen Record is not meant to operate or large unindexed datasets.
+Frozen Record is not meant to operate on large unindexed datasets.
 
 To ensure that it doesn't happen by accident, you can set `FrozenRecord::Base.max_records_scan = 500` (or whatever limit makes sense to you), in your development and test environments.
 This setting will cause Frozen Record to raise an error if it has to scan more than `max_records_scan` records. This property can also be set on a per model basis.

data/frozen_record.gemspec

@@ -20,7 +20,6 @@ Gem::Specification.new do |spec|
   spec.required_ruby_version = '>= 2.5'
 
   spec.add_runtime_dependency 'activemodel'
-  spec.add_runtime_dependency 'dedup'
 
   spec.add_development_dependency 'rake'
   spec.add_development_dependency 'rspec'

data/lib/frozen_record/backends/json.rb

@@ -19,16 +19,16 @@ module FrozenRecord
 
         if supports_freeze
           def load(file_path)
-            JSON.load_file(file_path, freeze: true) || Dedup::EMPTY_ARRAY
+            JSON.load_file(file_path, freeze: true) || [].freeze
           end
         else
           def load(file_path)
-            Dedup.deep_intern!(JSON.load_file(file_path) || Dedup::EMPTY_ARRAY)
+            JSON.load_file(file_path) || [].freeze
           end
         end
       else
         def load(file_path)
-          Dedup.deep_intern!(JSON.parse(File.read(file_path)) || Dedup::EMPTY_ARRAY)
+          JSON.parse(File.read(file_path)) || [].freeze
         end
       end
     end

data/lib/frozen_record/backends/yaml.rb

@@ -56,27 +56,27 @@ module FrozenRecord
 
       if YAML.respond_to?(:unsafe_load_file)
         def load_file(path)
-          YAML.unsafe_load_file(path, freeze: true) || Dedup::EMPTY_ARRAY
+          YAML.unsafe_load_file(path, freeze: true) || [].freeze
         end
 
         def load_string(yaml)
-          YAML.unsafe_load(yaml, freeze: true) || Dedup::EMPTY_ARRAY
+          YAML.unsafe_load(yaml, freeze: true) || [].freeze
         end
       elsif supports_freeze
         def load_file(path)
-          YAML.load_file(path, freeze: true) || Dedup::EMPTY_ARRAY
+          YAML.load_file(path, freeze: true) || [].freeze
         end
 
         def load_string(yaml)
-          YAML.load(yaml, freeze: true) || Dedup::EMPTY_ARRAY
+          YAML.load(yaml, freeze: true) || [].freeze
         end
       else
         def load_file(path)
-          Dedup.deep_intern!(YAML.load_file(path) || Dedup::EMPTY_ARRAY)
+          YAML.load_file(path) || [].freeze
         end
 
         def load_string(yaml)
-          Dedup.deep_intern!(YAML.load(yaml) || Dedup::EMPTY_ARRAY)
+          YAML.load(yaml) || [].freeze
         end
       end
     end

data/lib/frozen_record/base.rb

@@ -69,7 +69,7 @@ module FrozenRecord
       alias_method :set_default_attributes, :default_attributes=
       private :set_default_attributes
       def default_attributes=(default_attributes)
-        set_default_attributes(Dedup.deep_intern!(default_attributes.transform_keys(&:to_s)))
+        set_default_attributes(default_attributes.transform_keys(&:to_s))
       end
 
       alias_method :set_primary_key, :primary_key=
@@ -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/compact.rb

@@ -62,7 +62,7 @@ module FrozenRecord
 
     def attributes=(attributes)
       self.class.attributes.each do |attr|
-        instance_variable_set(self.class._attributes_cache[attr], Dedup.deep_intern!(attributes[attr]))
+        instance_variable_set(self.class._attributes_cache[attr], attributes[attr])
       end
     end
 

data/lib/frozen_record/minimal.rb

@@ -4,8 +4,6 @@ require 'yaml'
 require 'set'
 require 'active_model'
 
-require 'dedup'
-
 require 'frozen_record/version'
 require 'frozen_record/scope'
 require 'frozen_record/index'

data/lib/frozen_record/scope.rb

@@ -242,17 +242,13 @@ module FrozenRecord
       if array_delegable?(method_name)
         to_a.public_send(method_name, *args, &block)
       elsif @klass.respond_to?(method_name)
-        delegate_to_class(method_name, *args, &block)
+        scoping { @klass.public_send(method_name, *args, &block) }
       else
         super
       end
     end
     ruby2_keywords :method_missing if respond_to?(:ruby2_keywords, true)
 
-    def delegate_to_class(*args, &block)
-      scoping { @klass.public_send(*args, &block) }
-    end
-
     def array_delegable?(method)
       Array.method_defined?(method) && !DISALLOWED_ARRAY_METHODS.include?(method)
     end

data/lib/frozen_record/version.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 module FrozenRecord
-  VERSION = '0.25.3'
+  VERSION = '0.26.0'
 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.3
+  version: 0.26.0
 platform: ruby
 authors:
 - Jean Boussier
 autorequire:
 bindir: bin
 cert_chain: []
-date: 2022-06-13 00:00:00.000000000 Z
+date: 2022-07-12 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: activemodel
@@ -24,20 +24,6 @@ dependencies:
     - - ">="
       - !ruby/object:Gem::Version
         version: '0'
-- !ruby/object:Gem::Dependency
-  name: dedup
-  requirement: !ruby/object:Gem::Requirement
-    requirements:
-    - - ">="
-      - !ruby/object:Gem::Version
-        version: '0'
-  type: :runtime
-  prerelease: false
-  version_requirements: !ruby/object:Gem::Requirement
-    requirements:
-    - - ">="
-      - !ruby/object:Gem::Version
-        version: '0'
 - !ruby/object:Gem::Dependency
   name: rake
   requirement: !ruby/object:Gem::Requirement
@@ -73,6 +59,7 @@ executables: []
 extensions: []
 extra_rdoc_files: []
 files:
+- ".github/dependabot.yml"
 - ".github/workflows/main.yml"
 - ".gitignore"
 - ".rspec"
@@ -137,7 +124,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubygems_version: 3.3.7
+rubygems_version: 3.2.20
 signing_key:
 specification_version: 4
 summary: ActiveRecord like interface to read only access and query static YAML files