composite_primary_keys 7.0.15 → 7.0.16

data/test/test_delete_all.rb

@@ -18,4 +18,9 @@ class TestValidations < ActiveSupport::TestCase
     }
     assert(EmployeesGroup.all.size == 3)
   end
+
+  # This test fails, requires fixin arel
+  def test_delete_all_with_joins
+    ReferenceCode.joins(:reference_type).where(:reference_type_id => 1).delete_all
+  end
 end

data/test/test_dumpable.rb

@@ -1,15 +1,15 @@
-require File.expand_path('../abstract_unit', __FILE__)
-
-class TestDumpable < ActiveSupport::TestCase
-  fixtures :articles, :readings, :users
-
-  def test_marshal_with_simple_preload
-    articles = Article.preload(:readings).where(id: 1).to_a
-    assert_equal(Marshal.load(Marshal.dump(articles)), articles)
-  end
-
-  def test_marshal_with_comples_preload
-    articles = Article.preload({ readings: :user }).where(id: 1).to_a
-    assert_equal(Marshal.load(Marshal.dump(articles)), articles)
-  end
-end
+require File.expand_path('../abstract_unit', __FILE__)
+
+class TestDumpable < ActiveSupport::TestCase
+  fixtures :articles, :readings, :users
+
+  def test_marshal_with_simple_preload
+    articles = Article.preload(:readings).where(id: 1).to_a
+    assert_equal(Marshal.load(Marshal.dump(articles)), articles)
+  end
+
+  def test_marshal_with_comples_preload
+    articles = Article.preload({ readings: :user }).where(id: 1).to_a
+    assert_equal(Marshal.load(Marshal.dump(articles)), articles)
+  end
+end

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_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_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