aboutsummaryrefslogtreecommitdiffstats
path: root/activerecord/test/cases/associations/has_one_through_associations_test.rb
diff options
context:
space:
mode:
authorSam DeCesare <sam@samdecesare.com>2018-04-09 17:35:33 -0700
committerSam DeCesare <sam@samdecesare.com>2018-04-09 18:49:52 -0700
commit99910dddf28faac31d6a3d4800460f1bc308bb83 (patch)
tree4b08bd565a1b2d901bd1e3b5af2ea5b5537b1881 /activerecord/test/cases/associations/has_one_through_associations_test.rb
parent601e31b362b2f59bed988de716f4b931b2aa0950 (diff)
downloadrails-99910dddf28faac31d6a3d4800460f1bc308bb83.tar.gz
rails-99910dddf28faac31d6a3d4800460f1bc308bb83.tar.bz2
rails-99910dddf28faac31d6a3d4800460f1bc308bb83.zip
Fix .new with multiple through associations
This fixes a bug with building an object that has multiple `has_many :through` associations through the same object. Previously, when building the object via .new, the intermediate object would be created instead of just being built. Here's an example: Given a GameBoard, that has_one Owner and Collection through Game. The following line would cause a game object to be created in the database. GameBoard.new(owner: some_owner, collection: some_collection) Whereas, if passing only one of those associations into `.new` would cause the Game object to be built and not created in the database. Now the above code will only build the Game object, and not save it.
Diffstat (limited to 'activerecord/test/cases/associations/has_one_through_associations_test.rb')
-rw-r--r--activerecord/test/cases/associations/has_one_through_associations_test.rb22
1 files changed, 22 insertions, 0 deletions
diff --git a/activerecord/test/cases/associations/has_one_through_associations_test.rb b/activerecord/test/cases/associations/has_one_through_associations_test.rb
index 9964f084ac..003b82e50e 100644
--- a/activerecord/test/cases/associations/has_one_through_associations_test.rb
+++ b/activerecord/test/cases/associations/has_one_through_associations_test.rb
@@ -22,6 +22,10 @@ require "models/customer"
require "models/carrier"
require "models/shop_account"
require "models/customer_carrier"
+require "models/game"
+require "models/game_board"
+require "models/game_collection"
+require "models/game_owner"
class HasOneThroughAssociationsTest < ActiveRecord::TestCase
fixtures :member_types, :members, :clubs, :memberships, :sponsors, :organizations, :minivans,
@@ -64,6 +68,24 @@ class HasOneThroughAssociationsTest < ActiveRecord::TestCase
assert_equal clubs(:moustache_club), new_member.club
end
+ def test_building_multiple_associations_builds_through_record
+ game_owner = GameOwner.create
+ game_collection = GameCollection.create
+ game_board_with_one_association = GameBoard.new(game_owner: game_owner)
+ assert_nil game_board_with_one_association.game.id
+ game_board_with_two_associations = GameBoard.new(game_owner: game_owner, game_collection: game_collection)
+ assert_nil game_board_with_two_associations.game.id
+ end
+
+ def test_creating_multiple_associations_creates_through_record
+ game_owner = GameOwner.create
+ game_collection = GameCollection.create
+ game_board_with_one_association = GameBoard.create(game_owner: game_owner)
+ assert_not_nil game_board_with_one_association.game.id
+ game_board_with_two_associations = GameBoard.create(game_owner: game_owner, game_collection: game_collection)
+ assert_not_nil game_board_with_two_associations.game.id
+ end
+
def test_creating_association_sets_both_parent_ids_for_new
member = Member.new(name: "Sean Griffin")
club = Club.new(name: "Da Club")