composite_primary_keys 7.0.16 → 8.0.0

data/test/test_nested_attributes.rb

@@ -1,124 +1,124 @@
-require File.expand_path('../abstract_unit', __FILE__)
-
-# Testing the find action on composite ActiveRecords with two primary keys
-class TestNestedAttributes < ActiveSupport::TestCase
-  fixtures :reference_types, :reference_codes
-
-  def test_nested_atttribute_create
-    code_id = 1001
-
-    reference_type = reference_types(:name_prefix)
-    reference_type.update_attribute :reference_codes_attributes, [{
-      :reference_code => code_id,
-      :code_label => 'XX',
-      :abbreviation => 'Xx'
-    }]
-    assert_not_nil ReferenceCode.find_by_reference_code(code_id)
-  end
-
-  def test_nested_atttribute_update
-    code_id = 1002
-
-    reference_type = reference_types(:name_prefix)
-    reference_type.update_attribute :reference_codes_attributes, [{
-      :reference_code => code_id,
-      :code_label => 'XX',
-      :abbreviation => 'Xx'
-    }]
-
-    reference_code = ReferenceCode.find_by_reference_code(code_id)
-    cpk = CompositePrimaryKeys::CompositeKeys[reference_type.reference_type_id, code_id]
-    reference_type.update_attribute :reference_codes_attributes, [{
-      :id => cpk,
-      :code_label => 'AAA',
-      :abbreviation => 'Aaa'
-    }]
-
-    reference_code = ReferenceCode.find_by_reference_code(code_id)
-    assert_kind_of(ReferenceCode, reference_code)
-    assert_equal(reference_code.code_label, 'AAA')
-  end
-
-  def test_nested_atttribute_update_2
-    reference_type = reference_types(:gender)
-    reference_code = reference_codes(:gender_male)
-
-    reference_type.update_attributes(:reference_codes_attributes => [{:id => reference_code.id,
-                                                                      :code_label => 'XX',
-                                                                      :abbreviation => 'Xx'}])
-
-    reference_code.reload
-    assert_equal(reference_code.code_label, 'XX')
-    assert_equal(reference_code.abbreviation, 'Xx')
-  end
-
-  def test_nested_atttribute_update_3
-    reference_type = reference_types(:gender)
-    reference_code = reference_codes(:gender_male)
-
-    reference_type.update_attributes(:reference_codes_attributes => [{:id => reference_code.id.to_s,
-                                                                      :code_label => 'XX',
-                                                                      :abbreviation => 'Xx'}])
-
-    reference_code.reload
-    assert_equal(reference_code.code_label, 'XX')
-    assert_equal(reference_code.abbreviation, 'Xx')
-  end
-
-  fixtures :topics, :topic_sources
-
-  def test_nested_attributes_create_with_string_in_primary_key
-    platform = 'instagram'
-
-    topic = topics(:music)
-    topic.update_attribute :topic_sources_attributes, [{
-      :platform => platform,
-      :keywords => 'funk'
-    }]
-    assert_not_nil TopicSource.find_by_platform(platform)
-  end
-
-  def test_nested_attributes_update_with_string_in_primary_key
-    platform = 'instagram'
-
-    topic = topics(:music)
-    topic.update_attribute :topic_sources_attributes, [{
-      :platform => platform,
-      :keywords => 'funk'
-    }]
-    assert_not_nil TopicSource.find_by_platform(platform)
-
-    topic_source = TopicSource.find_by_platform(platform)
-    cpk = CompositePrimaryKeys::CompositeKeys[topic.id, platform]
-    topic.update_attribute :topic_sources_attributes, [{
-      :id => cpk,
-      :keywords => 'jazz'
-    }]
-
-    topic_source = TopicSource.find_by_platform(platform)
-    assert_kind_of(TopicSource, topic_source)
-    assert_equal(topic_source.keywords, 'jazz')
-  end
-
-  def test_nested_attributes_update_with_string_in_primary_key_2
-    topic = topics(:music)
-    topic_source = topic_sources(:music_source)
-
-    topic.update_attributes(:topic_sources_attributes => [{:id => topic_source.id,
-                                                           :keywords => 'classical, jazz'}])
-
-    topic_source.reload
-    assert_equal(topic_source.keywords, 'classical, jazz')
-  end
-
-  def test_nested_attributes_update_with_string_in_primary_key_3
-    topic = topics(:music)
-    topic_source = topic_sources(:music_source)
-
-    topic.update_attributes(:topic_sources_attributes => [{:id => topic_source.id.to_s,
-                                                           :keywords => 'classical, jazz'}])
-
-    topic_source.reload
-    assert_equal(topic_source.keywords, 'classical, jazz')
-  end
-end
+require File.expand_path('../abstract_unit', __FILE__)
+
+# Testing the find action on composite ActiveRecords with two primary keys
+class TestNestedAttributes < ActiveSupport::TestCase
+  fixtures :reference_types, :reference_codes
+
+  def test_nested_atttribute_create
+    code_id = 1001
+
+    reference_type = reference_types(:name_prefix)
+    reference_type.update_attribute :reference_codes_attributes, [{
+      :reference_code => code_id,
+      :code_label => 'XX',
+      :abbreviation => 'Xx'
+    }]
+    assert_not_nil ReferenceCode.find_by_reference_code(code_id)
+  end
+
+  def test_nested_atttribute_update
+    code_id = 1002
+
+    reference_type = reference_types(:name_prefix)
+    reference_type.update_attribute :reference_codes_attributes, [{
+      :reference_code => code_id,
+      :code_label => 'XX',
+      :abbreviation => 'Xx'
+    }]
+
+    reference_code = ReferenceCode.find_by_reference_code(code_id)
+    cpk = CompositePrimaryKeys::CompositeKeys[reference_type.reference_type_id, code_id]
+    reference_type.update_attribute :reference_codes_attributes, [{
+      :id => cpk,
+      :code_label => 'AAA',
+      :abbreviation => 'Aaa'
+    }]
+
+    reference_code = ReferenceCode.find_by_reference_code(code_id)
+    assert_kind_of(ReferenceCode, reference_code)
+    assert_equal(reference_code.code_label, 'AAA')
+  end
+
+  def test_nested_atttribute_update_2
+    reference_type = reference_types(:gender)
+    reference_code = reference_codes(:gender_male)
+
+    reference_type.update_attributes(:reference_codes_attributes => [{:id => reference_code.id,
+                                                                      :code_label => 'XX',
+                                                                      :abbreviation => 'Xx'}])
+
+    reference_code.reload
+    assert_equal(reference_code.code_label, 'XX')
+    assert_equal(reference_code.abbreviation, 'Xx')
+  end
+
+  def test_nested_atttribute_update_3
+    reference_type = reference_types(:gender)
+    reference_code = reference_codes(:gender_male)
+
+    reference_type.update_attributes(:reference_codes_attributes => [{:id => reference_code.id.to_s,
+                                                                      :code_label => 'XX',
+                                                                      :abbreviation => 'Xx'}])
+
+    reference_code.reload
+    assert_equal(reference_code.code_label, 'XX')
+    assert_equal(reference_code.abbreviation, 'Xx')
+  end
+
+  fixtures :topics, :topic_sources
+
+  def test_nested_attributes_create_with_string_in_primary_key
+    platform = 'instagram'
+
+    topic = topics(:music)
+    topic.update_attribute :topic_sources_attributes, [{
+      :platform => platform,
+      :keywords => 'funk'
+    }]
+    assert_not_nil TopicSource.find_by_platform(platform)
+  end
+
+  def test_nested_attributes_update_with_string_in_primary_key
+    platform = 'instagram'
+
+    topic = topics(:music)
+    topic.update_attribute :topic_sources_attributes, [{
+      :platform => platform,
+      :keywords => 'funk'
+    }]
+    assert_not_nil TopicSource.find_by_platform(platform)
+
+    topic_source = TopicSource.find_by_platform(platform)
+    cpk = CompositePrimaryKeys::CompositeKeys[topic.id, platform]
+    topic.update_attribute :topic_sources_attributes, [{
+      :id => cpk,
+      :keywords => 'jazz'
+    }]
+
+    topic_source = TopicSource.find_by_platform(platform)
+    assert_kind_of(TopicSource, topic_source)
+    assert_equal(topic_source.keywords, 'jazz')
+  end
+
+  def test_nested_attributes_update_with_string_in_primary_key_2
+    topic = topics(:music)
+    topic_source = topic_sources(:music_source)
+
+    topic.update_attributes(:topic_sources_attributes => [{:id => topic_source.id,
+                                                           :keywords => 'classical, jazz'}])
+
+    topic_source.reload
+    assert_equal(topic_source.keywords, 'classical, jazz')
+  end
+
+  def test_nested_attributes_update_with_string_in_primary_key_3
+    topic = topics(:music)
+    topic_source = topic_sources(:music_source)
+
+    topic.update_attributes(:topic_sources_attributes => [{:id => topic_source.id.to_s,
+                                                           :keywords => 'classical, jazz'}])
+
+    topic_source.reload
+    assert_equal(topic_source.keywords, 'classical, jazz')
+  end
+end

data/test/test_polymorphic.rb

@@ -1,26 +1,26 @@
-require File.expand_path('../abstract_unit', __FILE__)
-
-class TestPolymorphic < ActiveSupport::TestCase
-  fixtures :users, :employees, :comments, :hacks
-  
-  def test_polymorphic_has_many
-    comments = Hack.find(7).comments
-    assert_equal 7, comments[0].person_id
-  end
-
-  def test_polymorphic_has_one
-    first_comment = Hack.find(7).first_comment
-    assert_equal 7, first_comment.person_id
-  end
-
-  def test_has_many_through
-    user = users(:santiago)
-    article_names = user.articles.collect { |a| a.name }.sort
-    assert_equal ['Article One', 'Article Two'], article_names
-  end
-
-  def test_polymorphic_has_many_through
-    user = users(:santiago)
-    assert_equal(['andrew'], user.hacks.collect { |a| a.name }.sort)
-  end
-end
+require File.expand_path('../abstract_unit', __FILE__)
+
+class TestPolymorphic < ActiveSupport::TestCase
+  fixtures :users, :employees, :comments, :hacks
+  
+  def test_polymorphic_has_many
+    comments = Hack.find(7).comments
+    assert_equal 7, comments[0].person_id
+  end
+
+  def test_polymorphic_has_one
+    first_comment = Hack.find(7).first_comment
+    assert_equal 7, first_comment.person_id
+  end
+
+  def test_has_many_through
+    user = users(:santiago)
+    article_names = user.articles.collect { |a| a.name }.sort
+    assert_equal ['Article One', 'Article Two'], article_names
+  end
+
+  def test_polymorphic_has_many_through
+    user = users(:santiago)
+    assert_equal(['andrew'], user.hacks.collect { |a| a.name }.sort)
+  end
+end

data/test/test_predicates.rb

@@ -1,41 +1,41 @@
-require File.expand_path('../abstract_unit', __FILE__)
-
-class TestEqual < ActiveSupport::TestCase
-  fixtures :departments
-
-  include CompositePrimaryKeys::Predicates
-
-  def test_or
-    dep = Arel::Table.new(:departments)
-
-    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_and
-    dep = Arel::Table.new(:departments)
-
-    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 TestEqual < ActiveSupport::TestCase
+  fixtures :departments
+
+  include CompositePrimaryKeys::Predicates
+
+  def test_or
+    dep = Arel::Table.new(:departments)
+
+    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_and
+    dep = Arel::Table.new(:departments)
+
+    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_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_suite.rb

@@ -1,34 +1,33 @@
-%w(
-  test_aliases
-  test_associations
-  test_attribute_methods
-  test_attributes
-  test_calculations
-  test_callbacks
-  test_composite_arrays
-  test_create
-  test_delete
-  test_delete_all
-  test_dumpable
-  test_dup
-  test_enum
-  test_equal
-  test_exists
-  test_find
-  test_habtm
-  test_ids
-  test_miscellaneous
-  test_nested_attributes
-  test_optimistic
-  test_pagination
-  test_polymorphic
-  test_predicates
-  test_santiago
-  test_serialize
-  test_touch
-  test_tutorial_example
-  test_update
-  test_validations
-).each do |test|
-  require File.expand_path("../#{test}", __FILE__)
-end
+%w(
+  test_aliases
+  test_associations
+  test_attribute_methods
+  test_attributes
+  test_calculations
+  test_callbacks
+  test_composite_arrays
+  test_create
+  test_delete
+  test_delete_all
+  test_dumpable
+  test_dup
+  test_enum
+  test_equal
+  test_exists
+  test_find
+  test_habtm
+  test_ids
+  test_miscellaneous
+  test_nested_attributes
+  test_pagination
+  test_polymorphic
+  test_predicates
+  test_santiago
+  test_serialize
+  test_touch
+  test_tutorial_example
+  test_update
+  test_validations
+).each do |test|
+  require File.expand_path("../#{test}", __FILE__)
+end