composite_primary_keys 12.0.9 → 13.0.2

data/test/test_predicates.rb

@@ -1,60 +1,60 @@
-require File.expand_path('../abstract_unit', __FILE__)
-
-class TestPredicates < ActiveSupport::TestCase
-  fixtures :departments
-
-  include CompositePrimaryKeys::Predicates
-
-  def test_or
-    dep = Department.arel_table
-
-    predicates = Array.new
-
-    3.times do |i|
-      predicates << dep[:id].eq(i)
-    end
-
-    connection = ActiveRecord::Base.connection
-    quoted = "#{connection.quote_table_name('departments')}.#{connection.quote_column_name('id')}"
-    expected = "(#{quoted} = 0 OR #{quoted} = 1 OR #{quoted} = 2)"
-
-    pred = cpk_or_predicate(predicates)
-    assert_equal(with_quoted_identifiers(expected), pred.to_sql)
-  end
-
-  def test_or_with_many_values
-    dep = Arel::Table.new(:departments)
-
-    predicates = Array.new
-
-    number_of_predicates = 3000 # This should really be big
-    number_of_predicates.times do |i|
-      predicates << dep[:id].eq(i)
-    end
-
-    connection = ActiveRecord::Base.connection
-    quoted = "#{connection.quote_table_name('departments')}.#{connection.quote_column_name('id')}"
-    expected_ungrouped = ((0...number_of_predicates).map { |i| "#{quoted} = #{i}" }).join(' OR ')
-    expected = "(#{expected_ungrouped})"
-
-    pred = cpk_or_predicate(predicates)
-    assert_equal(with_quoted_identifiers(expected), pred.to_sql)
-  end
-
-  def test_and
-    dep = Department.arel_table
-
-    predicates = Array.new
-
-    3.times do |i|
-      predicates << dep[:id].eq(i)
-    end
-
-    connection = ActiveRecord::Base.connection
-    quoted = "#{connection.quote_table_name('departments')}.#{connection.quote_column_name('id')}"
-    expected = "#{quoted} = 0 AND #{quoted} = 1 AND #{quoted} = 2"
-
-    pred = cpk_and_predicate(predicates)
-    assert_equal(with_quoted_identifiers(expected), pred.to_sql)
-  end
+require File.expand_path('../abstract_unit', __FILE__)
+
+class TestPredicates < ActiveSupport::TestCase
+  fixtures :departments
+
+  include CompositePrimaryKeys::Predicates
+
+  def test_or
+    dep = Department.arel_table
+
+    predicates = Array.new
+
+    3.times do |i|
+      predicates << dep[:id].eq(i)
+    end
+
+    connection = ActiveRecord::Base.connection
+    quoted = "#{connection.quote_table_name('departments')}.#{connection.quote_column_name('id')}"
+    expected = "(#{quoted} = 0 OR #{quoted} = 1 OR #{quoted} = 2)"
+
+    pred = cpk_or_predicate(predicates)
+    assert_equal(with_quoted_identifiers(expected), pred.to_sql)
+  end
+
+  def test_or_with_many_values
+    dep = Arel::Table.new(:departments)
+
+    predicates = Array.new
+
+    number_of_predicates = 3000 # This should really be big
+    number_of_predicates.times do |i|
+      predicates << dep[:id].eq(i)
+    end
+
+    connection = ActiveRecord::Base.connection
+    quoted = "#{connection.quote_table_name('departments')}.#{connection.quote_column_name('id')}"
+    expected_ungrouped = ((0...number_of_predicates).map { |i| "#{quoted} = #{i}" }).join(' OR ')
+    expected = "(#{expected_ungrouped})"
+
+    pred = cpk_or_predicate(predicates)
+    assert_equal(with_quoted_identifiers(expected), pred.to_sql)
+  end
+
+  def test_and
+    dep = Department.arel_table
+
+    predicates = Array.new
+
+    3.times do |i|
+      predicates << dep[:id].eq(i)
+    end
+
+    connection = ActiveRecord::Base.connection
+    quoted = "#{connection.quote_table_name('departments')}.#{connection.quote_column_name('id')}"
+    expected = "#{quoted} = 0 AND #{quoted} = 1 AND #{quoted} = 2"
+
+    pred = cpk_and_predicate(predicates)
+    assert_equal(with_quoted_identifiers(expected), pred.to_sql)
+  end
 end

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