composite_primary_keys 12.0.2 → 12.0.3

data/test/test_polymorphic.rb

@@ -1,27 +1,43 @@
 require File.expand_path('../abstract_unit', __FILE__)
 
 class TestPolymorphic < ActiveSupport::TestCase
-  fixtures :users, :employees, :comments, :hacks, :articles, :readings
+  fixtures :articles, :departments, :employees, :users, :comments
 
-  def test_polymorphic_has_many
-    comments = Hack.find(7).comments
-    assert_equal 7, comments[0].person_id
+  def test_has_many
+    user = users(:santiago)
+    comments = user.comments
+    assert_equal(user.id, comments[0].person_id)
   end
 
-  def test_polymorphic_has_one
-    first_comment = Hack.find(7).first_comment
-    assert_equal 7, first_comment.person_id
+  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
-    assert_equal(2, Article.count, 'Baseline sanity check')
-    user = users(:santiago)
-    article_names = user.articles.collect { |a| a.name }.sort
-    assert_equal ['Article One', 'Article Two'], article_names
+    department = departments(:accounting)
+    comment = comments(:employee_comment)
+
+    assert_equal(1, department.comments.size)
+    assert_equal(comment, department.comments[0])
   end
 
-  def test_polymorphic_has_many_through
+  def test_has_many_through_2
+    article = articles(:second)
+
     user = users(:santiago)
-    assert_equal(['andrew'], user.hacks.collect { |a| a.name }.sort)
+    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_preload.rb

@@ -1,7 +1,7 @@
 require File.expand_path('../abstract_unit', __FILE__)
 
 class TestPreload < ActiveSupport::TestCase
-  fixtures :comments, :users, :employees, :groups, :hacks, :readings
+  fixtures :articles, :comments, :users, :employees, :groups, :readings
 
   class UserForPreload < User
     has_many :comments_with_include_condition, -> { where('person_type = ?', 'User')},
@@ -31,10 +31,11 @@ class TestPreload < ActiveSupport::TestCase
 
   def test_preload_for_conditioned_has_many_association
     # has one comment
+    article = articles(:first)
     user1 = users(:santiago)
     user2 = UserForPreload.create(name: 'TestPreload')
-    Comment.create(person: user2, person_type: 'User')
-    Comment.create(person: user2, person_type: 'User')
+    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)

data/test/test_serialize.rb

@@ -5,11 +5,11 @@ class TestSerialization < ActiveSupport::TestCase
 
   def test_json
     department = Department.first
-    assert_equal('{"department_id":1,"location_id":1}', department.to_json)
+    assert_equal('{"id":1,"location_id":1}', department.to_json)
   end
 
   def test_serializable_hash
     department = Department.first
-    assert_equal({"department_id" => 1,"location_id" => 1}, department.serializable_hash)
+    assert_equal({"id" => 1,"location_id" => 1}, department.serializable_hash)
   end
 end

data/test/test_update.rb

@@ -1,7 +1,7 @@
 require File.expand_path('../abstract_unit', __FILE__)
 
 class TestUpdate < ActiveSupport::TestCase
-  fixtures :reference_types, :reference_codes
+  fixtures :departments, :reference_types, :reference_codes, :rooms, :room_assignments
 
   CLASSES = {
     :single => {
@@ -36,6 +36,13 @@ class TestUpdate < ActiveSupport::TestCase
     end
   end
 
+  def test_update_attributes_with_id_field
+    department = departments(:accounting)
+    department.update_attribute(:location_id, 3)
+    department.reload
+    assert_equal(3, department.location_id)
+  end
+
   def test_update_primary_key
     obj = ReferenceCode.find([1,1])
     obj.reference_type_id = 2
@@ -76,4 +83,15 @@ class TestUpdate < ActiveSupport::TestCase
 
     assert_equal(2, query.count)
   end
+
+  def test_update_with_uniqueness
+    assignment = room_assignments(:jacksons_room)
+    room_1 = rooms(:branner_room_1)
+    room_2 = rooms(:branner_room_3)
+
+    assert_equal(room_1, assignment.room)
+    assignment.room = room_2
+    assignment.save!
+    assert_equal(room_2, assignment.room)
+  end
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: composite_primary_keys
 version: !ruby/object:Gem::Version
-  version: 12.0.2
+  version: 12.0.3
 platform: ruby
 authors:
 - Charlie Savage
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2020-05-19 00:00:00.000000000 Z
+date: 2020-11-12 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: activerecord
@@ -48,6 +48,7 @@ files:
 - README.rdoc
 - Rakefile
 - lib/composite_primary_keys.rb
+- lib/composite_primary_keys/active_model/attribute_assignment.rb
 - lib/composite_primary_keys/arel/sqlserver.rb
 - lib/composite_primary_keys/arel/to_sql.rb
 - lib/composite_primary_keys/associations/association.rb
@@ -70,6 +71,7 @@ files:
 - lib/composite_primary_keys/composite_relation.rb
 - lib/composite_primary_keys/connection_adapters/abstract/database_statements.rb
 - lib/composite_primary_keys/connection_adapters/abstract_adapter.rb
+- lib/composite_primary_keys/connection_adapters/mysql/database_statements.rb
 - lib/composite_primary_keys/connection_adapters/postgresql/database_statements.rb
 - lib/composite_primary_keys/connection_adapters/sqlserver/database_statements.rb
 - lib/composite_primary_keys/core.rb
@@ -126,14 +128,10 @@ files:
 - test/fixtures/employees.yml
 - test/fixtures/group.rb
 - test/fixtures/groups.yml
-- test/fixtures/hack.rb
-- test/fixtures/hacks.yml
 - test/fixtures/membership.rb
 - test/fixtures/membership_status.rb
 - test/fixtures/membership_statuses.yml
 - test/fixtures/memberships.yml
-- test/fixtures/pk_called_id.rb
-- test/fixtures/pk_called_ids.yml
 - test/fixtures/product.rb
 - test/fixtures/product_tariff.rb
 - test/fixtures/product_tariffs.yml
@@ -141,8 +139,6 @@ files:
 - test/fixtures/reading.rb
 - test/fixtures/readings.yml
 - test/fixtures/reference_code.rb
-- test/fixtures/reference_code_using_composite_key_alias.rb
-- test/fixtures/reference_code_using_simple_key_alias.rb
 - test/fixtures/reference_codes.yml
 - test/fixtures/reference_type.rb
 - test/fixtures/reference_types.yml
@@ -158,8 +154,6 @@ files:
 - test/fixtures/room_attribute_assignments.yml
 - test/fixtures/room_attributes.yml
 - test/fixtures/rooms.yml
-- test/fixtures/seat.rb
-- test/fixtures/seats.yml
 - test/fixtures/street.rb
 - test/fixtures/streets.yml
 - test/fixtures/student.rb
@@ -168,15 +162,12 @@ files:
 - test/fixtures/suburbs.yml
 - test/fixtures/tariff.rb
 - test/fixtures/tariffs.yml
-- test/fixtures/topic.rb
-- test/fixtures/topic_source.rb
 - test/fixtures/topic_sources.yml
 - test/fixtures/topics.yml
 - test/fixtures/user.rb
 - test/fixtures/users.yml
 - test/plugins/pagination.rb
 - test/plugins/pagination_helper.rb
-- test/test_aliases.rb
 - test/test_associations.rb
 - test/test_attribute_methods.rb
 - test/test_attributes.rb
@@ -188,7 +179,6 @@ files:
 - test/test_delete.rb
 - test/test_dumpable.rb
 - test/test_dup.rb
-- test/test_enum.rb
 - test/test_equal.rb
 - test/test_exists.rb
 - test/test_find.rb
@@ -203,7 +193,6 @@ files:
 - test/test_preload.rb
 - test/test_santiago.rb
 - test/test_serialize.rb
-- test/test_suite.rb
 - test/test_touch.rb
 - test/test_tutorial_example.rb
 - test/test_update.rb
@@ -227,14 +216,13 @@ required_rubygems_version: !ruby/object:Gem::Requirement
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubygems_version: 3.1.2
+rubygems_version: 3.1.4
 signing_key: 
 specification_version: 4
 summary: Composite key support for ActiveRecord
 test_files:
 - test/abstract_unit.rb
 - test/README_tests.rdoc
-- test/test_aliases.rb
 - test/test_associations.rb
 - test/test_attributes.rb
 - test/test_attribute_methods.rb
@@ -246,7 +234,6 @@ test_files:
 - test/test_delete.rb
 - test/test_dumpable.rb
 - test/test_dup.rb
-- test/test_enum.rb
 - test/test_equal.rb
 - test/test_exists.rb
 - test/test_find.rb
@@ -261,7 +248,6 @@ test_files:
 - test/test_preload.rb
 - test/test_santiago.rb
 - test/test_serialize.rb
-- test/test_suite.rb
 - test/test_touch.rb
 - test/test_tutorial_example.rb
 - test/test_update.rb

data/test/fixtures/hack.rb

@@ -1,5 +0,0 @@
-class Hack < ActiveRecord::Base
-  has_many :comments, :as => :person
-  
-  has_one :first_comment, :as => :person, :class_name => "Comment"
-end

data/test/fixtures/hacks.yml

@@ -1,3 +0,0 @@
-andrew:
-  id: 7
-  name: andrew

data/test/fixtures/pk_called_id.rb

@@ -1,5 +0,0 @@
-class PkCalledId < ActiveRecord::Base
-  self.primary_keys = :id, :reference_code
-
-  validates_presence_of :reference_code, :code_label, :abbreviation
-end

data/test/fixtures/pk_called_ids.yml

@@ -1,11 +0,0 @@
-name_prefix_mr:
-  id: 1
-  reference_code: 1
-  code_label: MR
-  abbreviation: Mr
-
-name_prefix_mrs:
-  id: 2
-  reference_code: 1
-  code_label: MRS
-  abbreviation: Mrs

data/test/fixtures/reference_code_using_composite_key_alias.rb

@@ -1,8 +0,0 @@
-class ReferenceCodeUsingCompositeKeyAlias < ActiveRecord::Base
-  self.table_name = 'reference_codes'
-  self.primary_key = [:reference_type_id, :reference_code]
-  
-  belongs_to :reference_type, :foreign_key => "reference_type_id"
-  
-  validates_presence_of :reference_code, :code_label, :abbreviation
-end

data/test/fixtures/reference_code_using_simple_key_alias.rb

@@ -1,8 +0,0 @@
-class ReferenceCodeUsingSimpleKeyAlias < ActiveRecord::Base
-  self.table_name = 'reference_codes'
-  self.primary_key = :code_label
-  
-  belongs_to :reference_type, :foreign_key => "reference_type_id"
-  
-  validates_presence_of :reference_code, :code_label, :abbreviation
-end

data/test/fixtures/seat.rb

@@ -1,5 +0,0 @@
-class Seat < ActiveRecord::Base
-  self.primary_keys = [:flight_number, :seat]
-
-  validates_uniqueness_of :customer
-end

data/test/fixtures/seats.yml

@@ -1,9 +0,0 @@
-seat1:
-  flight_number: 1
-  seat: 1
-  customer: 1
-
-seat2:
-  flight_number: 1
-  seat: 2
-  customer: 2

data/test/fixtures/topic.rb

@@ -1,6 +0,0 @@
-class Topic < ActiveRecord::Base
-	has_many :topic_sources, dependent: :destroy
-	accepts_nested_attributes_for :topic_sources
-
-	validates :name, :feed_size, presence: true
-end

data/test/fixtures/topic_source.rb

@@ -1,7 +0,0 @@
-class TopicSource < ActiveRecord::Base
-	self.primary_keys = :topic_id, :platform
-
-	belongs_to :topic, inverse_of: :topic_sources
-
-	validates :platform, presence: true
-end

data/test/test_aliases.rb

@@ -1,18 +0,0 @@
-require File.expand_path('../abstract_unit', __FILE__)
-
-class TestAliases < ActiveSupport::TestCase
-  fixtures :reference_codes
-
-  def test_primary_key_setter_alias_composite_key
-    reference_code = ReferenceCodeUsingCompositeKeyAlias.find([1, 2])
-    assert_equal 'MRS', reference_code.code_label
-    assert_equal 'Mrs', reference_code.abbreviation
-  end
-
-  def test_primary_key_setter_alias_simple_key
-    reference_code = ReferenceCodeUsingSimpleKeyAlias.find('MRS')
-    assert_equal 1, reference_code.reference_type_id
-    assert_equal 2, reference_code.reference_code
-    assert_equal 'Mrs', reference_code.abbreviation
-  end
-end

data/test/test_enum.rb

@@ -1,21 +0,0 @@
-require File.expand_path('../abstract_unit', __FILE__)
-
-class TestEnum < ActiveSupport::TestCase
-  fixtures :comments
-
-  def test_enum_was
-    comment = Comment.first
-    assert_nil comment.shown
-    assert_equal({}, comment.changed_attributes)
-    comment.shown = :true
-    assert_equal({ "shown" => nil }, comment.changed_attributes)
-    assert_equal 'true', comment.shown
-    assert_nil comment.shown_was
-
-    comment.save
-
-    comment.shown = :false
-    assert_equal 'false', comment.shown
-    assert_equal 'true', comment.shown_was
-  end
-end

data/test/test_suite.rb

@@ -1,35 +0,0 @@
-%w(
-  test_aliases
-  test_associations
-  test_attribute_methods
-  test_attributes
-  test_calculations
-  test_callbacks
-  test_composite_arrays
-  test_counter_cache
-  test_create
-  test_delete
-  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_preload
-  test_santiago
-  test_serialize
-  test_touch
-  test_tutorial_example
-  test_update
-  test_validations
-).each do |test|
-  require File.expand_path("../#{test}", __FILE__)
-end