declare_schema 1.4.0.colin.3 → 1.4.0.colin.5

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 8d774a80915bfd6f22349e18be176e237c901d595a871144a08d73478946147b
-  data.tar.gz: 0cc5d18ba43357d595dff24ff3a2bf3f9443b9d3f8ed9ec909d53708b2820984
+  metadata.gz: 770c10f063190aa951ca0ab0956c1988876e9a60c235e08ac9013d6a2f55dc3d
+  data.tar.gz: e0935d656bc3dd48e21c41edfbfbaf8fa416dee00bac305fcaa8b08a85bc0c02
 SHA512:
-  metadata.gz: 96b51490860259a27e1b1fcbe02bab2722ca069ec9ee314879ccf563fc4f420c44485de9ec57035165a057953a9ba4705c5575c38f6a4592088d47c8704dafc1
-  data.tar.gz: ddb153c4999b17c8c74885b4bfa9ef7488992c760797808af40840cdb579f8e162d66b0cf043b1543905ef1282252c5dd371d40881cd403bdba2429b6590f81a
+  metadata.gz: d8e261efe79f5b6fdbb86f1fec4a81445e65baf664d26dd6db2c3e85d2d94c305967a4e0f9f5dcce0106daab83480975034dd20c47a319b9365a60b031ec7d49
+  data.tar.gz: 22da299dd84f766df2668d4a81f86d3398be03cab012ac098fc1d7d1fab10d65e6ecc0b3fec126ff7c4b57ee18df6c5cab0219fc681b146891f98c06131e3f74

data/Gemfile.lock

@@ -1,7 +1,7 @@
 PATH
   remote: .
   specs:
-    declare_schema (1.4.0.colin.3)
+    declare_schema (1.4.0.colin.5)
       rails (>= 5.0)
 
 GEM

data/lib/declare_schema/model/habtm_model_shim.rb

@@ -52,23 +52,23 @@ module DeclareSchema
       end
 
       def index_definitions_with_primary_key
-        [
+        @index_definitions_with_primary_key ||= Set.new([
           IndexDefinition.new(self, foreign_keys, unique: true, name: Model::IndexDefinition::PRIMARY_KEY_NAME), # creates a primary composite key on both foreign keys
           IndexDefinition.new(self, foreign_keys.last) # not unique by itself; combines with primary key to be unique
-        ]
+        ])
       end
 
       alias_method :index_definitions, :index_definitions_with_primary_key
 
       def ignore_indexes
-        []
+        @ignore_indexes ||= Set.new
       end
 
       def constraint_specs
-        [
+        @constraint_specs ||= Set.new([
           ForeignKeyDefinition.new(self, foreign_keys.first, parent_table: foreign_key_classes.first.table_name, constraint_name: "#{join_table}_FK1", dependent: :delete),
           ForeignKeyDefinition.new(self, foreign_keys.last, parent_table: foreign_key_classes.last.table_name, constraint_name: "#{join_table}_FK2", dependent: :delete)
-        ]
+        ])
       end
     end
   end

data/lib/declare_schema/model.rb

@@ -26,9 +26,9 @@ module DeclareSchema
           inheriting_cattr_reader field_specs: HashWithIndifferentAccess.new
 
           # index_definitions holds IndexDefinition objects for all the declared indexes.
-          inheriting_cattr_reader index_definitions: []
-          inheriting_cattr_reader ignore_indexes: []
-          inheriting_cattr_reader constraint_specs: []
+          inheriting_cattr_reader index_definitions: Set.new
+          inheriting_cattr_reader ignore_indexes: Set.new
+          inheriting_cattr_reader constraint_specs: Set.new
 
           # table_options holds optional configuration for the create_table statement
           # supported options include :charset and :collation
@@ -50,11 +50,7 @@ module DeclareSchema
 
     module ClassMethods
       def index(fields, **options)
-        # make index idempotent
-        index = ::DeclareSchema::Model::IndexDefinition.new(self, fields, **options)
-        unless index_definitions.any? { |index_spec| index_spec.equivalent?(index) }
-          index_definitions << index
-        end
+        index_definitions << ::DeclareSchema::Model::IndexDefinition.new(self, fields, **options)
       end
 
       def primary_key_index(*fields)
@@ -69,7 +65,7 @@ module DeclareSchema
       end
 
       # tell the migration generator to ignore the named index. Useful for existing indexes, or for indexes
-      # that can't be automatically generated (for example: a prefix index in MySQL)
+      # that can't be automatically generated.
       def ignore_index(index_name)
         ignore_indexes << index_name.to_s
       end
@@ -124,13 +120,14 @@ module DeclareSchema
           when String
             Kernel.warn("belongs_to index: 'name' is deprecated; use index: { name: 'name' } instead (in #{name})")
             index_options[:name] = index_value
-          # when false -- impossible since we checked that above
           when true
+          when false
+            raise ArgumentError, "belongs_to index: false contradicts others options #{options.inspect} (in #{name})"
           when nil
           when Hash
             index_options = index_value
           else
-            raise ArgumentError, "belongs_to index: must be true or false or a Hash; got #{index_value.inspect}"
+            raise ArgumentError, "belongs_to index: must be true or false or a Hash; got #{index_value.inspect} (in #{name})"
           end
 
           if options.has_key?(:unique)

data/lib/declare_schema/version.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 module DeclareSchema
-  VERSION = "1.4.0.colin.3"
+  VERSION = "1.4.0.colin.5"
 end

data/spec/lib/declare_schema/model/habtm_model_shim_spec.rb

@@ -141,8 +141,8 @@ RSpec.describe DeclareSchema::Model::HabtmModelShim do
     end
 
     describe 'ignore_indexes' do
-      it 'returns empty array' do
-        expect(subject.ignore_indexes).to eq([])
+      it 'returns empty Set' do
+        expect(subject.ignore_indexes).to eq(Set.new)
       end
     end
 
@@ -156,10 +156,11 @@ RSpec.describe DeclareSchema::Model::HabtmModelShim do
         expect(result.first.parent_table_name).to be(Parent1.table_name)
         expect(result.first.on_delete_cascade).to be_truthy
 
-        expect(result.last).to be_a(::DeclareSchema::Model::ForeignKeyDefinition)
-        expect(result.last.foreign_key).to eq(foreign_keys.last)
-        expect(result.last.parent_table_name).to be(Parent2.table_name)
-        expect(result.last.on_delete_cascade).to be_truthy
+        sample = result.to_a.last
+        expect(sample).to be_a(::DeclareSchema::Model::ForeignKeyDefinition)
+        expect(sample.foreign_key).to eq(foreign_keys.last)
+        expect(sample.parent_table_name).to be(Parent2.table_name)
+        expect(sample.on_delete_cascade).to be_truthy
       end
     end
   end

data/spec/lib/declare_schema/model/index_definition_spec.rb

@@ -112,15 +112,16 @@ RSpec.describe DeclareSchema::Model::IndexDefinition do
         it do
           expect(model_class.index_definitions.size).to eq(1)
 
-          expect(model_class.index_definitions[0].name).to eq('index_index_definition_test_models_on_name')
-          expect(model_class.index_definitions[0].fields).to eq(['name'])
-          expect(model_class.index_definitions[0].unique).to eq(false)
+          sample = model_class.index_definitions.to_a.first
+          expect(sample.name).to eq('index_index_definition_test_models_on_name')
+          expect(sample.fields).to eq(['name'])
+          expect(sample.unique).to eq(false)
         end
       end
 
       describe 'has index_definitions_with_primary_key' do
         it do
-          expect(model_class.index_definitions_with_primary_key).to be_kind_of(Array)
+          expect(model_class.index_definitions_with_primary_key).to be_kind_of(Set)
           result = model_class.index_definitions_with_primary_key.sort_by(&:name)
           expect(result.size).to eq(2)
 

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: declare_schema
 version: !ruby/object:Gem::Version
-  version: 1.4.0.colin.3
+  version: 1.4.0.colin.5
 platform: ruby
 authors:
 - Invoca Development adapted from hobo_fields by Tom Locke
 autorequire:
 bindir: bin
 cert_chain: []
-date: 2024-01-13 00:00:00.000000000 Z
+date: 2024-01-14 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: rails