composite_primary_keys 13.0.1 → 14.0.1

data/test/test_optimistic.rb

@@ -1,18 +1,18 @@
-require File.expand_path('../abstract_unit', __FILE__)
-
-class TestOptimisitic < ActiveSupport::TestCase
-  fixtures :restaurants
-
-  def test_update_with_stale_error
-    restaurant_1 = Restaurant.find([1, 1])
-    restaurant_1['name'] = "McDonalds renamed"
-
-    restaurant_2 = Restaurant.find([1, 1])
-    restaurant_2['name'] = "McDonalds renamed 2"
-
-    assert(restaurant_1.save)
-    assert_raise ActiveRecord::StaleObjectError do
-      restaurant_2.save
-    end
-  end
-end
+require File.expand_path('../abstract_unit', __FILE__)
+
+class TestOptimisitic < ActiveSupport::TestCase
+  fixtures :restaurants
+
+  def test_update_with_stale_error
+    restaurant_1 = Restaurant.find([1, 1])
+    restaurant_1['name'] = "McDonalds renamed"
+
+    restaurant_2 = Restaurant.find([1, 1])
+    restaurant_2['name'] = "McDonalds renamed 2"
+
+    assert(restaurant_1.save)
+    assert_raise ActiveRecord::StaleObjectError do
+      restaurant_2.save
+    end
+  end
+end

data/test/test_pagination.rb

@@ -1,36 +1,36 @@
-#require File.expand_path('../abstract_unit', __FILE__)
-#require 'plugins/pagination'
-#
-#class TestPagination < ActiveSupport::TestCase
-#  fixtures :reference_types, :reference_codes
-#
-#  include ActionController::Pagination
-#  DEFAULT_PAGE_SIZE = 2
-#
-#  attr_accessor :params
-#
-#  CLASSES = {
-#    :single => {
-#      :class => ReferenceType,
-#      :primary_keys => :reference_type_id,
-#      :table => :reference_types,
-#    },
-#    :dual   => {
-#      :class => ReferenceCode,
-#      :primary_keys => [:reference_type_id, :reference_code],
-#      :table => :reference_codes,
-#    },
-#  }
-#
-#  def setup
-#    self.class.classes = CLASSES
-#    @params = {}
-#  end
-#
-#  def test_paginate_all
-#    testing_with do
-#      @object_pages, @objects = paginate @klass_info[:table], :per_page => DEFAULT_PAGE_SIZE
-#      assert_equal 2, @objects.length, "Each page should have #{DEFAULT_PAGE_SIZE} items"
-#    end
-#  end
+#require File.expand_path('../abstract_unit', __FILE__)
+#require 'plugins/pagination'
+#
+#class TestPagination < ActiveSupport::TestCase
+#  fixtures :reference_types, :reference_codes
+#
+#  include ActionController::Pagination
+#  DEFAULT_PAGE_SIZE = 2
+#
+#  attr_accessor :params
+#
+#  CLASSES = {
+#    :single => {
+#      :class => ReferenceType,
+#      :primary_keys => :reference_type_id,
+#      :table => :reference_types,
+#    },
+#    :dual   => {
+#      :class => ReferenceCode,
+#      :primary_keys => [:reference_type_id, :reference_code],
+#      :table => :reference_codes,
+#    },
+#  }
+#
+#  def setup
+#    self.class.classes = CLASSES
+#    @params = {}
+#  end
+#
+#  def test_paginate_all
+#    testing_with do
+#      @object_pages, @objects = paginate @klass_info[:table], :per_page => DEFAULT_PAGE_SIZE
+#      assert_equal 2, @objects.length, "Each page should have #{DEFAULT_PAGE_SIZE} items"
+#    end
+#  end
 #end

data/test/test_polymorphic.rb

@@ -1,43 +1,43 @@
-require File.expand_path('../abstract_unit', __FILE__)
-
-class TestPolymorphic < ActiveSupport::TestCase
-  fixtures :articles, :departments, :employees, :users, :comments
-
-  def test_has_many
-    user = users(:santiago)
-    comments = user.comments
-    assert_equal(user.id, comments[0].person_id)
-  end
-
-  def test_has_one
-    user = users(:santiago)
-    first_comment = user.first_comment
-    assert_equal(user.id, first_comment.person_id)
-  end
-
-  def test_has_many_through
-    department = departments(:accounting)
-    comment = comments(:employee_comment)
-
-    assert_equal(1, department.comments.size)
-    assert_equal(comment, department.comments[0])
-  end
-
-  def test_has_many_through_2
-    article = articles(:second)
-
-    user = users(:santiago)
-    assert_equal(user, article.user_commentators[0])
-
-    user = users(:drnic)
-    assert_equal(user, article.user_commentators[1])
-  end
-
-  def test_clear_has_many_through
-    article = articles(:second)
-
-    assert_equal(2, article.comments.size)
-    article.user_commentators = []
-    assert_equal(0, article.comments.size)
-  end
-end
+require File.expand_path('../abstract_unit', __FILE__)
+
+class TestPolymorphic < ActiveSupport::TestCase
+  fixtures :articles, :departments, :employees, :users, :comments
+
+  def test_has_many
+    user = users(:santiago)
+    comments = user.comments
+    assert_equal(user.id, comments[0].person_id)
+  end
+
+  def test_has_one
+    user = users(:santiago)
+    first_comment = user.first_comment
+    assert_equal(user.id, first_comment.person_id)
+  end
+
+  def test_has_many_through
+    department = departments(:accounting)
+    comment = comments(:employee_comment)
+
+    assert_equal(1, department.comments.size)
+    assert_equal(comment, department.comments[0])
+  end
+
+  def test_has_many_through_2
+    article = articles(:second)
+
+    user = users(:santiago)
+    assert_equal(user, article.user_commentators[0])
+
+    user = users(:drnic)
+    assert_equal(user, article.user_commentators[1])
+  end
+
+  def test_clear_has_many_through
+    article = articles(:second)
+
+    assert_equal(2, article.comments.size)
+    article.user_commentators = []
+    assert_equal(0, article.comments.size)
+  end
+end

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