composite_primary_keys 13.0.1 → 13.0.2

data/test/test_preload.rb

@@ -1,102 +1,102 @@
-require File.expand_path('../abstract_unit', __FILE__)
-
-class TestPreload < ActiveSupport::TestCase
-  fixtures :articles, :comments, :users, :employees, :groups, :readings
-
-  class UserForPreload < User
-    has_many :comments_with_include_condition, -> { where('person_type = ?', 'User')},
-             class_name: 'Comment', foreign_key: 'person_id', as: :person
-    has_many :comments_with_exclude_condition, -> { where('person_type <> ?', 'SomeType')},
-             class_name: 'Comment', foreign_key: 'person_id', as: :person
-  end
-
-  class EmployeeForPreload < Employee
-    # this is a rather random condition, which should not interfere with the normal test queries
-    has_and_belongs_to_many :groups_2, class_name: 'Group', foreign_key: 'employee_id', join_table: 'employees_groups'
-  end
-
-  def test_preload_of_polymorphic_association
-    comment = Comment.where(id: [1, 2, 3]).all
-    persons = comment.map(&:person)
-    persons.each do |person|
-      assert_kind_of(ActiveRecord::Base, person)
-    end
-
-    comment = Comment.where(id: [1, 2, 3]).preload(:person).all
-    persons = comment.map(&:person)
-    persons.each do |person|
-      assert_kind_of(ActiveRecord::Base, person)
-    end
-  end
-
-  def test_preload_for_conditioned_has_many_association
-    # has one comment
-    article = articles(:first)
-    user1 = users(:santiago)
-    user2 = UserForPreload.create(name: 'TestPreload')
-    Comment.create(article: article, person: user2, person_type: 'User')
-    Comment.create(article: article, person: user2, person_type: 'User')
-
-    users = UserForPreload.where(id: [user1.id, user2.id]).all
-    assert_equal(1, users.first.comments_with_include_condition.size)
-    assert_equal(2, users.second.comments_with_include_condition.size)
-
-    users = UserForPreload.where(id: [user1.id, user2.id]).preload(:comments_with_include_condition).all
-    assert_equal(1, users.first.comments_with_include_condition.size)
-    assert_equal(2, users.second.comments_with_include_condition.size)
-
-    users = UserForPreload.where(id: [user1.id, user2.id]).all
-    assert_equal(1, users.first.comments_with_exclude_condition.size)
-    assert_equal(2, users.second.comments_with_exclude_condition.size)
-
-    users = UserForPreload.where(id: [user1.id, user2.id]).preload(:comments_with_exclude_condition).all
-    assert_equal(1, users.first.comments_with_exclude_condition.size)
-    assert_equal(2, users.second.comments_with_exclude_condition.size)
-  end
-
-  def test_preload_for_unconditioned_habtm_association
-    employee1 = employees(:steve)
-    employee2 = employees(:jill)
-    employee1.groups = [groups(:cpk)]
-    employee2.groups = [groups(:cpk), groups(:group2)]
-    employee1.save!; employee2.save!
-
-    employees = Employee.where(id: [1, 2]).all
-    assert_equal(1, employees.first.groups.size)
-    assert_equal(2, employees.second.groups.size)
-
-    employees = Employee.where(id: [1, 2]).preload(:groups).all
-    assert_equal(1, employees.first.groups.size)
-    assert_equal(2, employees.second.groups.size)
-  end
-
-  def test_preload_for_conditioned_habtm_association
-    employee1 = employees(:steve)
-    employee2 = employees(:jill)
-    employee1.groups = [groups(:cpk)]
-    employee2.groups = [groups(:cpk), groups(:group2)]
-    employee1.save!; employee2.save!
-
-    employees = EmployeeForPreload.where(id: [1, 2]).all.order(:id)
-
-    # Even without preload two errors: First Employee has Group 1 loaded twice,
-    # Second Employee has only Group 2 instead of Group 1&2
-    assert_equal(1, employees.first.groups.size)
-    assert_equal(1, employees.first.groups_2.size)
-    assert_equal(2, employees.second.groups.size)
-    assert_equal(2, employees.second.groups_2.size)
-
-    employees = EmployeeForPreload.where(id: [1, 2]).preload(:groups_2).all.order(:id)
-
-    # with preloading, the first assertion is valid, but the second only gets Group 2 instead of 1&2
-    assert_equal(1, employees.first.groups_2.size)
-    assert_equal(2, employees.second.groups_2.size)
-  end
-
-  def test_preload_settings_inversion
-    user = User.preload(:readings).find_by!(:name => 'Santiago')
-    assert_equal(2, user.readings.count)
-    reading = user.readings.first
-    assert_equal(true, reading.association(:user).loaded?)
-  end
-end
+require File.expand_path('../abstract_unit', __FILE__)
+
+class TestPreload < ActiveSupport::TestCase
+  fixtures :articles, :comments, :users, :employees, :groups, :readings
+
+  class UserForPreload < User
+    has_many :comments_with_include_condition, -> { where('person_type = ?', 'User')},
+             class_name: 'Comment', foreign_key: 'person_id', as: :person
+    has_many :comments_with_exclude_condition, -> { where('person_type <> ?', 'SomeType')},
+             class_name: 'Comment', foreign_key: 'person_id', as: :person
+  end
+
+  class EmployeeForPreload < Employee
+    # this is a rather random condition, which should not interfere with the normal test queries
+    has_and_belongs_to_many :groups_2, class_name: 'Group', foreign_key: 'employee_id', join_table: 'employees_groups'
+  end
+
+  def test_preload_of_polymorphic_association
+    comment = Comment.where(id: [1, 2, 3]).all
+    persons = comment.map(&:person)
+    persons.each do |person|
+      assert_kind_of(ActiveRecord::Base, person)
+    end
+
+    comment = Comment.where(id: [1, 2, 3]).preload(:person).all
+    persons = comment.map(&:person)
+    persons.each do |person|
+      assert_kind_of(ActiveRecord::Base, person)
+    end
+  end
+
+  def test_preload_for_conditioned_has_many_association
+    # has one comment
+    article = articles(:first)
+    user1 = users(:santiago)
+    user2 = UserForPreload.create(name: 'TestPreload')
+    Comment.create(article: article, person: user2, person_type: 'User')
+    Comment.create(article: article, person: user2, person_type: 'User')
+
+    users = UserForPreload.where(id: [user1.id, user2.id]).all
+    assert_equal(1, users.first.comments_with_include_condition.size)
+    assert_equal(2, users.second.comments_with_include_condition.size)
+
+    users = UserForPreload.where(id: [user1.id, user2.id]).preload(:comments_with_include_condition).all
+    assert_equal(1, users.first.comments_with_include_condition.size)
+    assert_equal(2, users.second.comments_with_include_condition.size)
+
+    users = UserForPreload.where(id: [user1.id, user2.id]).all
+    assert_equal(1, users.first.comments_with_exclude_condition.size)
+    assert_equal(2, users.second.comments_with_exclude_condition.size)
+
+    users = UserForPreload.where(id: [user1.id, user2.id]).preload(:comments_with_exclude_condition).all
+    assert_equal(1, users.first.comments_with_exclude_condition.size)
+    assert_equal(2, users.second.comments_with_exclude_condition.size)
+  end
+
+  def test_preload_for_unconditioned_habtm_association
+    employee1 = employees(:steve)
+    employee2 = employees(:jill)
+    employee1.groups = [groups(:cpk)]
+    employee2.groups = [groups(:cpk), groups(:group2)]
+    employee1.save!; employee2.save!
+
+    employees = Employee.where(id: [1, 2]).all
+    assert_equal(1, employees.first.groups.size)
+    assert_equal(2, employees.second.groups.size)
+
+    employees = Employee.where(id: [1, 2]).preload(:groups).all
+    assert_equal(1, employees.first.groups.size)
+    assert_equal(2, employees.second.groups.size)
+  end
+
+  def test_preload_for_conditioned_habtm_association
+    employee1 = employees(:steve)
+    employee2 = employees(:jill)
+    employee1.groups = [groups(:cpk)]
+    employee2.groups = [groups(:cpk), groups(:group2)]
+    employee1.save!; employee2.save!
+
+    employees = EmployeeForPreload.where(id: [1, 2]).all.order(:id)
+
+    # Even without preload two errors: First Employee has Group 1 loaded twice,
+    # Second Employee has only Group 2 instead of Group 1&2
+    assert_equal(1, employees.first.groups.size)
+    assert_equal(1, employees.first.groups_2.size)
+    assert_equal(2, employees.second.groups.size)
+    assert_equal(2, employees.second.groups_2.size)
+
+    employees = EmployeeForPreload.where(id: [1, 2]).preload(:groups_2).all.order(:id)
+
+    # with preloading, the first assertion is valid, but the second only gets Group 2 instead of 1&2
+    assert_equal(1, employees.first.groups_2.size)
+    assert_equal(2, employees.second.groups_2.size)
+  end
+
+  def test_preload_settings_inversion
+    user = User.preload(:readings).find_by!(:name => 'Santiago')
+    assert_equal(2, user.readings.count)
+    reading = user.readings.first
+    assert_equal(true, reading.association(:user).loaded?)
+  end
+end

data/test/test_santiago.rb

@@ -1,23 +1,23 @@
-# Test cases devised by Santiago that broke the Composite Primary Keys
-# code at one point in time. But no more!!!
-require File.expand_path('../abstract_unit', __FILE__)
-
-class TestSantiago < ActiveSupport::TestCase
-  fixtures :suburbs, :streets, :users, :articles, :readings
-  
-  def test_normal_and_composite_associations
-    assert_not_nil @suburb = Suburb.find([1, 1])
-    assert_equal 1, @suburb.streets.length
-    
-    assert_not_nil @street = Street.find(1)
-    assert_not_nil @street.suburb
-  end
-  
-  def test_single_keys
-    @santiago = User.find(1)
-    assert_not_nil @santiago.articles
-    assert_equal 2, @santiago.articles.length
-    assert_not_nil @santiago.readings
-    assert_equal 2, @santiago.readings.length
-  end
-end
+# Test cases devised by Santiago that broke the Composite Primary Keys
+# code at one point in time. But no more!!!
+require File.expand_path('../abstract_unit', __FILE__)
+
+class TestSantiago < ActiveSupport::TestCase
+  fixtures :suburbs, :streets, :users, :articles, :readings
+  
+  def test_normal_and_composite_associations
+    assert_not_nil @suburb = Suburb.find([1, 1])
+    assert_equal 1, @suburb.streets.length
+    
+    assert_not_nil @street = Street.find(1)
+    assert_not_nil @street.suburb
+  end
+  
+  def test_single_keys
+    @santiago = User.find(1)
+    assert_not_nil @santiago.articles
+    assert_equal 2, @santiago.articles.length
+    assert_not_nil @santiago.readings
+    assert_equal 2, @santiago.readings.length
+  end
+end

data/test/test_touch.rb

@@ -1,23 +1,23 @@
-# Test cases devised by Santiago that broke the Composite Primary Keys
-# code at one point in time. But no more!!!
-require File.expand_path('../abstract_unit', __FILE__)
-
-class TestTouch < ActiveSupport::TestCase
-  fixtures :products, :tariffs
-
-  def test_touching_a_record_updates_its_timestamp
-    tariff                = tariffs(:flat)
-    previous_amount       = tariff.amount
-    previously_updated_at = tariff.updated_at
-
-    tariff.amount         = previous_amount + 1
-    sleep 1.0 # we need to sleep for 1 second because the times updated (on mysql, at least) are only precise to 1 second.
-    tariff.touch
-    assert_not_equal previously_updated_at, tariff.updated_at
-    assert_equal previous_amount + 1, tariff.amount
-    assert tariff.amount_changed?, 'tarif amount should have changed'
-    assert tariff.changed?, 'tarif should be marked as changed'
-    tariff.reload
-    assert_not_equal previously_updated_at, tariff.updated_at
-  end
-end
+# Test cases devised by Santiago that broke the Composite Primary Keys
+# code at one point in time. But no more!!!
+require File.expand_path('../abstract_unit', __FILE__)
+
+class TestTouch < ActiveSupport::TestCase
+  fixtures :products, :tariffs
+
+  def test_touching_a_record_updates_its_timestamp
+    tariff                = tariffs(:flat)
+    previous_amount       = tariff.amount
+    previously_updated_at = tariff.updated_at
+
+    tariff.amount         = previous_amount + 1
+    sleep 1.0 # we need to sleep for 1 second because the times updated (on mysql, at least) are only precise to 1 second.
+    tariff.touch
+    assert_not_equal previously_updated_at, tariff.updated_at
+    assert_equal previous_amount + 1, tariff.amount
+    assert tariff.amount_changed?, 'tarif amount should have changed'
+    assert tariff.changed?, 'tarif should be marked as changed'
+    tariff.reload
+    assert_not_equal previously_updated_at, tariff.updated_at
+  end
+end

data/test/test_tutorial_example.rb

@@ -1,26 +1,26 @@
-require File.expand_path('../abstract_unit', __FILE__)
-
-class TestTutorialExample < ActiveSupport::TestCase
-  fixtures :users, :groups, :memberships, :membership_statuses
-  
-  def test_membership
-    membership = Membership.find([1, 1])
-    assert(membership, "Cannot find a membership")
-    assert(membership.user)
-    assert(membership.group)
-  end
-  
-  def test_status
-    membership = Membership.find([1, 1])
-    statuses = membership.statuses
-    assert(membership, "Cannot find a membership")
-    assert(statuses, "No has_many association to status")
-    assert_equal(membership, statuses.first.membership)
-  end
-  
-  def test_count
-    membership = Membership.find([1, 1])
-    assert(membership, "Cannot find a membership")
-    assert_equal(1, membership.statuses.count)
-  end
+require File.expand_path('../abstract_unit', __FILE__)
+
+class TestTutorialExample < ActiveSupport::TestCase
+  fixtures :users, :groups, :memberships, :membership_statuses
+  
+  def test_membership
+    membership = Membership.find([1, 1])
+    assert(membership, "Cannot find a membership")
+    assert(membership.user)
+    assert(membership.group)
+  end
+  
+  def test_status
+    membership = Membership.find([1, 1])
+    statuses = membership.statuses
+    assert(membership, "Cannot find a membership")
+    assert(statuses, "No has_many association to status")
+    assert_equal(membership, statuses.first.membership)
+  end
+  
+  def test_count
+    membership = Membership.find([1, 1])
+    assert(membership, "Cannot find a membership")
+    assert_equal(1, membership.statuses.count)
+  end
 end

data/test/test_validations.rb

@@ -1,13 +1,13 @@
-require File.expand_path('../abstract_unit', __FILE__)
-
-class TestValidations < ActiveSupport::TestCase
-  fixtures :students, :dorms, :rooms, :room_assignments
-
-  def test_uniqueness_validation_persisted
-    room_assignment = RoomAssignment.find([1, 1, 1])
-    assert(room_assignment.valid?)
-
-    room_assignment = RoomAssignment.new(:student_id => 1, :dorm_id => 1, :room_id => 2)
-    assert(!room_assignment.valid?)
-  end
-end
+require File.expand_path('../abstract_unit', __FILE__)
+
+class TestValidations < ActiveSupport::TestCase
+  fixtures :students, :dorms, :rooms, :room_assignments
+
+  def test_uniqueness_validation_persisted
+    room_assignment = RoomAssignment.find([1, 1, 1])
+    assert(room_assignment.valid?)
+
+    room_assignment = RoomAssignment.new(:student_id => 1, :dorm_id => 1, :room_id => 2)
+    assert(!room_assignment.valid?)
+  end
+end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: composite_primary_keys
 version: !ruby/object:Gem::Version
-  version: 13.0.1
+  version: 13.0.2
 platform: ruby
 authors:
 - Charlie Savage
 autorequire:
 bindir: bin
 cert_chain: []
-date: 2021-11-14 00:00:00.000000000 Z
+date: 2022-01-09 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: activerecord
@@ -57,7 +57,7 @@ files:
 - lib/composite_primary_keys/associations/foreign_association.rb
 - lib/composite_primary_keys/associations/has_many_association.rb
 - lib/composite_primary_keys/associations/has_many_through_association.rb
-- lib/composite_primary_keys/associations/join_dependency.rb
+- lib/composite_primary_keys/associations/join_association.rb
 - lib/composite_primary_keys/associations/preloader/association.rb
 - lib/composite_primary_keys/associations/through_association.rb
 - lib/composite_primary_keys/attribute_methods.rb
@@ -216,7 +216,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubygems_version: 3.2.31
+rubygems_version: 3.3.4
 signing_key:
 specification_version: 4
 summary: Composite key support for ActiveRecord